Issue Details (XML | Word | Printable)

Key: MDL-15785
Type: Sub-task Sub-task
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Petr Skoda
Reporter: Matthew Davidson
Votes: 1
Watchers: 2
Operations

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

Make Enrollment keys a required field. Security improvement.

Created: 23/Jul/08 04:04 AM   Updated: 10/Dec/08 08:27 PM
Return to search
Component/s: Course
Affects Version/s: 1.9.2
Fix Version/s: 1.9.4

Database: MySQL
Participants: Helen Foster, Matthew Davidson, Petr Skoda and Tim Hunt
Security Level: None
QA Assignee: Tim Hunt
Resolved date: 07/Dec/08
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE


 Description  « Hide
We need to be able to make sure each course is properly secured by making the enrollment key a required field. Also settings like "at least 6 characters long" would be extra nice.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Matthew Davidson added a comment - 23/Sep/08 08:51 PM
This needs to be made an admin option. An option to require an enrollment key for courses.

Petr Skoda made changes - 22/Nov/08 10:41 AM
Field Original Value New Value
Link This issue has been marked as being related by MDL-17205 [ MDL-17205 ]
Petr Skoda made changes - 01/Dec/08 06:41 AM
Assignee Eloy Lafuente (stronk7) [ stronk7 ] Petr ?koda [ skodak ]
Petr Skoda made changes - 07/Dec/08 02:49 AM
Parent MDL-17205 [ 29354 ]
Security Could be a security issue [ 10030 ]
Issue Type Improvement [ 4 ] Sub-task [ 5 ]
Petr Skoda made changes - 07/Dec/08 02:56 AM
Status Open [ 1 ] In Progress [ 3 ]
Petr Skoda made changes - 07/Dec/08 02:56 AM
Fix Version/s 1.9.4 [ 10300 ]
Petr Skoda committed 5 files to 'Moodle CVS' - 07/Dec/08 06:20 AM
MDL-15785 new options for enforcing of enrolment key commplexity and requirements
MODIFY group/group_form.php   Rev. 1.9    (+10 -2 lines)
MODIFY enrol/manual/config.html   Rev. 1.4    (+28 -0 lines)
MODIFY enrol/manual/enrol.php   Rev. 1.42    (+9 -1 lines)
MODIFY course/edit_form.php   Rev. 1.62    (+20 -2 lines)
MODIFY lang/en_utf8/enrol_manual.php   Rev. 1.5    (+3 -1 lines)
Petr Skoda committed 5 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 07/Dec/08 06:21 AM
MDL-15785 new options for enforcing of enrolment key commplexity and requirements; backported from HEAD
MODIFY course/edit_form.php   Rev. 1.37.2.14    (+21 -1 lines)
MODIFY enrol/manual/config.html   Rev. 1.2.2.2    (+28 -0 lines)
MODIFY enrol/manual/enrol.php   Rev. 1.25.2.10    (+9 -1 lines)
MODIFY group/group_form.php   Rev. 1.3.2.3    (+10 -2 lines)
MODIFY lang/en_utf8/enrol_manual.php   Rev. 1.3.2.2    (+3 -1 lines)
Petr Skoda added a comment - 07/Dec/08 06:26 AM
in cvs, thanks for the report

please test, you can find new settings in internal enrolment plugin options page


Petr Skoda made changes - 07/Dec/08 06:26 AM
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Petr Skoda made changes - 07/Dec/08 06:26 AM
Link This issue has been marked as being related by MDL-17205 [ MDL-17205 ]
martignoni committed 1 file to 'Lang CVS' - 08/Dec/08 12:42 AM
MDL-15785 New strings translated
MODIFY fr_utf8/enrol_manual.php   Rev. 1.4    (+3 -1 lines)
Mitsuhiro Yoshida committed 2 files to 'Lang CVS' - 08/Dec/08 12:56 AM
MDL-15785 Translated new strings for enforcing of enrolment key.
MODIFY ja_utf8/enrol_manual.php   Rev. 1.7    (+3 -1 lines)
MODIFY ja_utf8/README   Rev. 1.887    (+1 -1 lines)
Tim Hunt made changes - 09/Dec/08 06:15 PM
QA Assignee timhunt
Tim Hunt added a comment - 09/Dec/08 06:31 PM
Tested and reviewed code. All looks good.

Tim Hunt made changes - 09/Dec/08 06:31 PM
Status Resolved [ 5 ] Closed [ 6 ]