| Key: |
MDL-17501
|
| Type: |
Bug
|
| Status: |
Resolved
|
| Resolution: |
Fixed
|
| Priority: |
Major
|
| Assignee: |
Tim Hunt
|
| Reporter: |
Tim Hunt
|
| Votes: |
0
|
| Watchers: |
1
|
|
|
| Participants: |
Petr Skoda and Tim Hunt
|
| Security Level: |
None
|
| Resolved date: |
09/Dec/08
|
| Affected Branches: |
MOODLE_19_STABLE
|
| Fixed Branches: |
MOODLE_19_STABLE
|
|
Need to:
Give that capability to teachers by default.
Add the capability check in the settings form.
|
|
Description
|
Need to:
Give that capability to teachers by default.
Add the capability check in the settings form. |
Show » |
tjhunt committed 4 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 08/Dec/08 11:30 AM
course settings: MDL-17501 check for moodle/course:visibility in the course settings form.
1. If a user does not have that capability, don't let them change whether the course is visible.
2. Give teachers that capability by default.
3. To avoid regressions, on upgrade, give moodle/course:visibility to any role that has moodle/course:update.
|
|
|
tjhunt committed 4 files to 'Moodle CVS' - 08/Dec/08 11:31 AM
course settings: MDL-17501 check for moodle/course:visibility in the course settings form.
1. If a user does not have that capability, don't let them change whether the course is visible.
2. Give teachers that capability by default.
3. To avoid regressions, on upgrade, give moodle/course:visibility to any role that has moodle/course:update.
|
|
|
made changes - 08/Dec/08 11:33 AM
| Field |
Original Value |
New Value |
|
Status
|
Open
[ 1
]
|
Resolved
[ 5
]
|
|
Resolution
|
|
Fixed
[ 1
]
|
made changes - 09/Dec/08 05:10 AM
|
Status
|
Resolved
[ 5
]
|
Reopened
[ 4
]
|
|
Resolution
|
Fixed
[ 1
]
|
|
tjhunt committed 1 file to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 09/Dec/08 01:49 PM
course settings: MDL-17501 fix potential problem for people upgrading 1.6 -> 1.9.4.
I am amazed that Petr spotted this. Great catch.
|
|
|
made changes - 09/Dec/08 01:49 PM
|
Status
|
Reopened
[ 4
]
|
Resolved
[ 5
]
|
|
Resolution
|
|
Fixed
[ 1
]
|
|
moodle/course:visibility capability ignored on the course settings page
I think it should verify that roles are already configured
if (!empty($CFG->rolesactive)) {
becasue it might fail when upgrading from 1.6.x, right?