Creating a custom layout with 6 rows throws an error

Bug #1237721 reported by Aaron Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Aaron Wells

Bug Description

To replicate:

1. Edit a page
2. Go to the Layout tab
3. Expand "Advanced options"
4. Click on "Custom layout"
5. Keep clicking "Add row" until you have 6 rows
6. Click "Create layout"
7. Click "Save"

Expected result: The page has a custom layout with 6 rows
Actual result: You see an error page with this error message:

[WAR] 56 (lib/view.php:2439) [[invalidnumrows/error]]
Call stack (most recent first):

    View->addrow(array(size 3)) at /home/aaronw/www/mahara/htdocs/view/layout.php:249
    viewlayout_submit(object(Pieform), array(size 5)) at Unknown:0
    call_user_func_array("viewlayout_submit", array(size 2)) at /home/aaronw/www/mahara/htdocs/lib/pieforms/pieform.php:528
    Pieform->__construct(array(size 6)) at /home/aaronw/www/mahara/htdocs/lib/pieforms/pieform.php:170
    Pieform::process(array(size 6)) at /home/aaronw/www/mahara/htdocs/lib/pieforms/pieform.php:71
    pieform(array(size 6)) at /home/aaronw/www/mahara/htdocs/view/layout.php:171

Revision history for this message
Aaron Wells (u-aaronw) wrote :

This was spun off from https://bugs.launchpad.net/mahara/+bug/1236767

Actually, there's a second bug here as well, which is the missing lang string "invalidnumrows/error"

Revision history for this message
Aaron Wells (u-aaronw) wrote :
Revision history for this message
Aaron Wells (u-aaronw) wrote :

A second patch for the lang string: https://reviews.mahara.org/2598

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Aaron Wells (u-aaronw)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.