[Mahara-contributors] [Bug 1432641] [NEW] Changing view layout causes an error

2015-03-16 Thread Olli Puurunen
Public bug reported:

Steps to reproduce:

1. Create a new view
2. Select the layout with one column in first row and three columns in second
3. Add two textboxes to each column, 8 textboxes total
4. Change the layout to one row & one column

This results in following error:

[WAR] d1 (lib/dml.php:890) Failed to get a recordset: mysqli error: [1062: 
Duplicate entry '593-2-1-0' for key 'blocinst_vierowcolord_uk'] in 
adodb_throw(UPDATE "block_instance" SET "order" = 0  WHERE "order" = '1' AND 
"view" = '593' AND "row" = '2' AND "column" = 1, )
Command was: UPDATE "block_instance" SET "order" = ?  WHERE "order" = ? AND 
"view" = ? AND "row" = ? AND "column" = ? and values was (0,1,593,2,1)

Tested in 1.9.1 (Ubuntu 14.04.2 / MySQL 5.5.35 / Chrome 41.0.2272.89
(64-bit)) and Mahara Demo (currently 1.10).

** Affects: mahara
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1432641

Title:
  Changing view layout causes an error

Status in Mahara ePortfolio:
  New

Bug description:
  Steps to reproduce:

  1. Create a new view
  2. Select the layout with one column in first row and three columns in second
  3. Add two textboxes to each column, 8 textboxes total
  4. Change the layout to one row & one column

  This results in following error:

  [WAR] d1 (lib/dml.php:890) Failed to get a recordset: mysqli error: [1062: 
Duplicate entry '593-2-1-0' for key 'blocinst_vierowcolord_uk'] in 
adodb_throw(UPDATE "block_instance" SET "order" = 0  WHERE "order" = '1' AND 
"view" = '593' AND "row" = '2' AND "column" = 1, )
  Command was: UPDATE "block_instance" SET "order" = ?  WHERE "order" = ? AND 
"view" = ? AND "row" = ? AND "column" = ? and values was (0,1,593,2,1)

  Tested in 1.9.1 (Ubuntu 14.04.2 / MySQL 5.5.35 / Chrome 41.0.2272.89
  (64-bit)) and Mahara Demo (currently 1.10).

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1432641/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp


[Mahara-contributors] [Bug 1432635] [NEW] Changing view layout causes block removal

2015-03-16 Thread Olli Puurunen
Public bug reported:

Steps to reproduce:

1. Create a new view
2. Select layout with one column in first row and three columns in second
3. Add one textbox to each column, 4 textboxes total
4. Go to layout-tab again and change the layout to one row & one column (the 
first)

Result: The block(s) in the first column of the second row is gone.

Tested in 1.9.1 and Mahara Demo (1.10 at the moment).

** Affects: mahara
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1432635

Title:
  Changing view layout causes block removal

Status in Mahara ePortfolio:
  New

Bug description:
  Steps to reproduce:

  1. Create a new view
  2. Select layout with one column in first row and three columns in second
  3. Add one textbox to each column, 4 textboxes total
  4. Go to layout-tab again and change the layout to one row & one column (the 
first)

  Result: The block(s) in the first column of the second row is gone.

  Tested in 1.9.1 and Mahara Demo (1.10 at the moment).

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1432635/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp


[Mahara-contributors] [Bug 1429871] [NEW] Link underlining in skins doesn't work

2015-03-09 Thread Olli Puurunen
Public bug reported:

If the user edits a skin (Edit skin > Fonts and colours) and checks the
Underline link -option, the link doesn't get underlined on a page with
the skin applied. There seems to be a bug in skin/style.php#38 with the
comparison

if ($option == 'transparent')

because when a checkbox is selected, the $option value is a boolean TRUE
and when using PHP's loose comparison operator, (true == 'transparent')
= true. Therefore for example
$skin->viewskin['view_link_normal_underline'] (and a few others) value
changes to an empty string and the value assigned to template is 'none'
instead of 'underline'.

A simple fix is to use strict comparison:

if ($option === 'transparent')

---

Mahara version: 1.9.1
Browser: Chrome (Linux) 40.0.2214.115 (64-bit)

** Affects: mahara
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1429871

Title:
  Link underlining in skins doesn't work

Status in Mahara ePortfolio:
  New

Bug description:
  If the user edits a skin (Edit skin > Fonts and colours) and checks
  the Underline link -option, the link doesn't get underlined on a page
  with the skin applied. There seems to be a bug in skin/style.php#38
  with the comparison

  if ($option == 'transparent')

  because when a checkbox is selected, the $option value is a boolean
  TRUE and when using PHP's loose comparison operator, (true ==
  'transparent') = true. Therefore for example
  $skin->viewskin['view_link_normal_underline'] (and a few others) value
  changes to an empty string and the value assigned to template is
  'none' instead of 'underline'.

  A simple fix is to use strict comparison:

  if ($option === 'transparent')

  ---

  Mahara version: 1.9.1
  Browser: Chrome (Linux) 40.0.2214.115 (64-bit)

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1429871/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp


[Mahara-contributors] [Bug 1371460] [NEW] Missing translations in js/customlayout.js

2014-09-19 Thread Olli Puurunen
Public bug reported:

Version 1.9.1

The following rows contain untranslated strings:

Row 11: $(newrow).find('.customrowtitle').html('Row ' + (numrows + 1) + 
'');
Row 39: $(this).find('.customrowtitle').html('Row ' + inc + 
'');
Row 112:  rowtitlediv.html('' + numrows + ' rows');
Row 224: $('#custompreview').html('Generating preview...');

** Affects: mahara
 Importance: Undecided
 Status: New


** Tags: translations

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1371460

Title:
  Missing translations in js/customlayout.js

Status in Mahara ePortfolio:
  New

Bug description:
  Version 1.9.1

  The following rows contain untranslated strings:

  Row 11: $(newrow).find('.customrowtitle').html('Row ' + (numrows + 1) 
+ '');
  Row 39: $(this).find('.customrowtitle').html('Row ' + inc + 
'');
  Row 112:  rowtitlediv.html('' + numrows + ' rows');
  Row 224: $('#custompreview').html('Generating preview...');

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1371460/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp


[Mahara-contributors] [Bug 1314012] Re: Document local/lib.php functions

2014-04-29 Thread Olli Puurunen
Small error in the documentation. It says, that
"local_header_top_content(): Returned data is printed at top of ",
when in fact the returned data is printed (almost) at top of .

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1314012

Title:
  Document local/lib.php functions

Status in Mahara ePortfolio:
  Fix Committed

Bug description:
  See forum post:
  https://mahara.org/interaction/forum/editpost.php?parent=26592

  There are quite a few local hooks that have been added to Mahara but
  are not documented in one central location. I've located these in the
  code, and added comments to local/lib.php that document the existence
  of these methods, so that a developer looking under /local/lib.php
  will know what their options are.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1314012/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp


[Mahara-contributors] [Bug 1305481] [NEW] Adding content to page buggy on Firefox

2014-04-10 Thread Olli Puurunen
Public bug reported:

When adding a block by clicking an element from the content editor
sidebar after a menu from the sidebar was expanded, the block gets added
twice and the overlay doesn't go away.

Steps to reproduce:

1. Create a new page
2. Go to Edit content -tab
3. Expand a menu from the sidebar, for example Media
4. Click "Text box" from the sidebar
5. Click Save from the popup
6. Click Save from the "Text box: Configure" popup

Expected result:

The popup and the overlay disappears and the Text box block is added to
page.

Actual result:

The popup disappears, but the overlay remains and the block is added
twice to page.

In file js/views.js, the function ViewManager.makeNewBlocksDraggable is
called on ViewManager initialization and also when the change event of
the accordion is fired. So each div.blocktype gets a new click (and
keydown) handler when the accordion is expanded and that's why
ViewManager.startAddBlock is called multiple times and multiple overlays
are created. We have made some customizations to views.js, but they
don't seem to affect this part of the code.

Mahara version: Latest from git (1.9rc2dev)
Browser: Mozilla Firefox for Ubuntu (28.0)

** Affects: mahara
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1305481

Title:
  Adding content to page buggy on Firefox

Status in Mahara ePortfolio:
  New

Bug description:
  When adding a block by clicking an element from the content editor
  sidebar after a menu from the sidebar was expanded, the block gets
  added twice and the overlay doesn't go away.

  Steps to reproduce:

  1. Create a new page
  2. Go to Edit content -tab
  3. Expand a menu from the sidebar, for example Media
  4. Click "Text box" from the sidebar
  5. Click Save from the popup
  6. Click Save from the "Text box: Configure" popup

  Expected result:

  The popup and the overlay disappears and the Text box block is added
  to page.

  Actual result:

  The popup disappears, but the overlay remains and the block is added
  twice to page.

  In file js/views.js, the function ViewManager.makeNewBlocksDraggable
  is called on ViewManager initialization and also when the change event
  of the accordion is fired. So each div.blocktype gets a new click (and
  keydown) handler when the accordion is expanded and that's why
  ViewManager.startAddBlock is called multiple times and multiple
  overlays are created. We have made some customizations to views.js,
  but they don't seem to affect this part of the code.

  Mahara version: Latest from git (1.9rc2dev)
  Browser: Mozilla Firefox for Ubuntu (28.0)

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1305481/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp


[Mahara-contributors] [Bug 1305451] [NEW] Content editor sidebar doesn't work on Chrome

2014-04-10 Thread Olli Puurunen
Public bug reported:

Adding content by clicking an item from the sidebar fails on Chrome.

Steps to repeat:

1. Create a new page
2. Go to Edit content -tab
3. Click on Text box

Expected result:

"Add block: Text box" -popup appears.

Actual result:

Nothing happens.

In file js/views.js, in function ViewManager.makeNewBlocksDraggable the
event handler of the .blocktypelink -elements is assumed that
event.buttons -property exists and that it is a number. However, as you
can see from https://developer.mozilla.org/en-
US/docs/Web/API/MouseEvent.buttons#Browser_compatibility, Chrome doesn't
seem to support the property.

Mahara version: Latest from git (1.9rc2dev)
Browser: Chrome (Linux) 33.0.1750.152

** Affects: mahara
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1305451

Title:
  Content editor sidebar doesn't work on Chrome

Status in Mahara ePortfolio:
  New

Bug description:
  Adding content by clicking an item from the sidebar fails on Chrome.

  Steps to repeat:

  1. Create a new page
  2. Go to Edit content -tab
  3. Click on Text box

  Expected result:

  "Add block: Text box" -popup appears.

  Actual result:

  Nothing happens.

  In file js/views.js, in function ViewManager.makeNewBlocksDraggable
  the event handler of the .blocktypelink -elements is assumed that
  event.buttons -property exists and that it is a number. However, as
  you can see from https://developer.mozilla.org/en-
  US/docs/Web/API/MouseEvent.buttons#Browser_compatibility, Chrome
  doesn't seem to support the property.

  Mahara version: Latest from git (1.9rc2dev)
  Browser: Chrome (Linux) 33.0.1750.152

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1305451/+subscriptions

___
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp