Issue Details (XML | Word | Printable)

Key: MDL-15876
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Petr Skoda
Reporter: Sam Marshall
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Defaults broken in admin_setting_configmulticheckbox::output_html

Created: 29/Jul/08 10:26 PM   Updated: 28/Aug/08 03:08 PM
Return to search
Component/s: Administration
Affects Version/s: 2.0
Fix Version/s: 2.0

Participants: Petr Skoda and Sam Marshall
Security Level: None
Resolved date: 28/Aug/08
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
This code (used eg for selecting roles in gradebook, and now in completion system too) had some minor logical errors. It was filling a $current array which was then never used. This caused defaults not to work/display correctly when displaying these lists.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
[ Permalink | « Hide ]
Sam Marshall added a comment - 29/Jul/08 10:27 PM
OK, fixed in HEAD only.

[ Permalink | « Hide ]
Petr Skoda added a comment - 13/Aug/08 06:13 AM
the problem was not in defaults, but in the refactored code - reverting the incorrect change of logic there and fixing...

[ Permalink | « Hide ]
Petr Skoda added a comment - 13/Aug/08 06:52 AM
hmm, this is more difficult than I expected
there is also a problem in settings and search page because it uses defaults when settings missing, but only upgradesettings should do this

testing a patch, now I see that your approach was better than what we had before, though there were still some minor problems


[ Permalink | « Hide ]
Petr Skoda added a comment - 13/Aug/08 06:53 AM
I will have to file a new bug from 1.9 defaults too

[ Permalink | « Hide ]
Petr Skoda added a comment - 18/Aug/08 06:50 AM
reclosing, thanks Sam

[ Permalink | « Hide ]
Petr Skoda added a comment - 28/Aug/08 03:05 PM
reopening, still some probs there, testing patch

[ Permalink | « Hide ]
Petr Skoda added a comment - 28/Aug/08 03:08 PM
I hope now it is on


Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.
Atlassian JIRA the Professional Issue Tracker. (Enterprise Edition, Version: 3.13.5-#360) - Bug/feature request - Atlassian news - Contact Administrators