Reviewed: https://reviews.mahara.org/10717 Committed: https://git.mahara.org/mahara/mahara/commit/ecfa80e383a495bb1623c3e5f4ccd94688c20dd2 Submitter: Robert Lyon ([email protected]) Branch: master
commit ecfa80e383a495bb1623c3e5f4ccd94688c20dd2 Author: Cecilia Vela Gurovic <[email protected]> Date: Wed Jan 15 11:26:39 2020 +1300 Bug 1856506: gridstack: adding extra checks for non numbers behatnotneeded Change-Id: I5096809eee056275d45c295dd6a7b37e9ca23f68 -- 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/1856506 Title: gridstack - too much recursion error when translating from old layout Status in Mahara: Fix Committed Status in Mahara 19.10 series: In Progress Status in Mahara 20.04 series: Fix Committed Bug description: If there is an old layout page with empty rows (no blocks in that row), when translating to new gridstack layout there will be a javascript error "too much recursion". This error will make the script fail and the process of translating will never be completed. To test: - have a Mahara site version 19.04 or older - make a page and set the layout to one that has more than 3 rows. It should have at least one block in row 1 and row 3. No blocks in row 2 - upgrade the site to 19.10 or newer - go to the page and edit it expected result: the page is translated and there a message to show the process was successful actual result: there is a "too much recursion" error in the browsers console and the translation process is interrupted To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1856506/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp

