← Back to team overview

mahara-contributors team mailing list archive

[Bug 645040] [NEW] Too many view columns during parallel editing

 

Public bug reported:

We have a Mahara 1.2.5 on a Linux server with MySQL Database.

During a Mahara Tutorial 25 people where editing a group view in parallel using the buttons add and remove columns. 
After a while, displaying the view ended in a system error. A look in the database revealed that the view had 7 as the number of columns and (null) as its layout.  Default layouts are only available up to 5 columns. 
The function  get_layout() in lib/view.php could therefore not determine a layout anymore. 
I believe this is a Mahara bug. There should be some transactional mechanism between the JavaScript and the PHP script that prevents users from creating more than 5 columns in a view. 

I just hacked the get_layout function to simply set the number of
columns to 5 in case it's bigger than 5, but this is not a permanent
solution.

Thanks in advance 
   Juergen Zimmer

** Affects: mahara
     Importance: Undecided
         Status: New


** Tags: columns view

-- 
Too many view columns during parallel editing
https://bugs.launchpad.net/bugs/645040
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.

Status in Mahara ePortfolio: New

Bug description:
We have a Mahara 1.2.5 on a Linux server with MySQL Database. 

During a Mahara Tutorial 25 people where editing a group view in parallel using the buttons add and remove columns. 
After a while, displaying the view ended in a system error. A look in the database revealed that the view had 7 as the number of columns and (null) as its layout.  Default layouts are only available up to 5 columns. 
The function  get_layout() in lib/view.php could therefore not determine a layout anymore. 
I believe this is a Mahara bug. There should be some transactional mechanism between the JavaScript and the PHP script that prevents users from creating more than 5 columns in a view. 

I just hacked the get_layout function to simply set the number of columns to 5 in case it's bigger than 5, but this is not a permanent solution. 

Thanks in advance 
   Juergen Zimmer





Follow ups

References