Issue Details (XML | Word | Printable)

Key: MDL-19610
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Tim Hunt
Reporter: Tim Hunt
Votes: 0
Watchers: 1
Operations

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

We shoudl encapsulate $CFG->rolesactive in a function

Created: 24/Jun/09 01:48 PM   Updated: 24/Jun/09 05:18 PM
Return to search
Component/s: Lib
Affects Version/s: 2.0
Fix Version/s: 2.0

File Attachments: 1. Text File during_initial_install.patch (9 kB)


Participants: Petr Skoda and Tim Hunt
Security Level: None
Resolved date: 24/Jun/09
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
The fact that empty($CFG->rolesactive) is the right way to find out whethet Moodle is fully installed is somewhat obscure.

I think it makes code easier to follow if we create a during_initial_install() function.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Tim Hunt added a comment - 24/Jun/09 01:51 PM
Petr, are you happy for this patch to go in?

(This all started because I noticed that user_login_string uses a different way to check this. I'll fix that in a minutes.)


Petr Skoda added a comment - 24/Jun/09 04:56 PM
+1, makes sense to me

Tim Hunt added a comment - 24/Jun/09 05:18 PM
Thanks Petr.