Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.1, 2.2, 2.3
-
Component/s: Administration
-
Labels:
-
Testing Instructions:
- Edit admin/settings.php
- Change line 91 "if ($PAGE->user_allowed_editing() && false) {" to break the if statement
- Visit any admin settings page and confirm you don't get the notice.
-
Affected Branches:MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
-
Fixed Branches:MOODLE_21_STABLE, MOODLE_22_STABLE
-
Pull from Repository:
-
Pull Master Branch:wip-
MDL-30770-m23 -
Pull Master Diff URL:
Description
Undefined variable: buttons in .../admin/settings.php on line 107
Gliffy Diagrams
Attachments
Activity
- All
- Comments
- Work Log
- History
- Activity
- Links Hierarchy
Hi, Sam.
Assigning this to you as it looks like the change came from one of your edits a while back.
The line...
$PAGE->set_button($buttons);
|
...should probably be shifted up to be inside the if body above after line 100.
Putting this up for integration as its a very easy obvious fix.
Integrators please cherry-pick to 21, and 22.
Commit hash: 6be4afb
Cheers
Sam
Integrated, thanks! (21, 22 and master)
working as expected!
Thanks
Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!
Now... disconnect, relax and enjoy the next days, yay!
Closing...ciao 
Hi, Brent.
I can see why this error is occurring, but could you add more to this report? Please tell us how you came across this error message and what steps we might take to replicate the problem. That will help us develop testing instructions once the problem is resolved.