Issue Details (XML | Word | Printable)

Key: MDL-16166
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Petr Skoda
Reporter: Petr Skoda
Votes: 0
Watchers: 2
Operations

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

centralise enabling of various features/subsystems on one settings page

Created: 22/Aug/08 06:32 AM   Updated: 01/Dec/08 01:13 AM
Return to search
Component/s: Administration
Affects Version/s: 1.9
Fix Version/s: 2.0

File Attachments: 1. Text File feature_enabling_7.patch (26 kB)

Image Attachments:

1. features.jpeg
(114 kB)
Issue Links:
Relates
 

Participants: Eloy Lafuente (stronk7), Martin Dougiamas and Petr Skoda
Security Level: None
Resolved date: 01/Dec/08
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
benefits:
1/ it would be much easier to review active features
2/ we could hide parts of admin settings tree (like Networking category if mnet disabled)

related improvements:
* option to disable notes

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Petr Skoda made changes - 22/Aug/08 06:36 AM
Field Original Value New Value
Attachment feature_enabling_3.patch [ 14939 ]
Petr Skoda made changes - 22/Aug/08 06:36 AM
Attachment features.jpeg [ 14940 ]
Petr Skoda made changes - 22/Aug/08 06:37 AM
Attachment features.jpeg [ 14940 ]
Petr Skoda made changes - 22/Aug/08 06:39 AM
Attachment features.jpeg [ 14941 ]
Petr Skoda made changes - 22/Aug/08 03:58 PM
Attachment feature_enabling_5.patch [ 14942 ]
Petr Skoda made changes - 25/Aug/08 01:51 PM
Attachment feature_enabling_7.patch [ 14949 ]
Petr Skoda made changes - 25/Aug/08 01:52 PM
Attachment feature_enabling_3.patch [ 14939 ]
Petr Skoda made changes - 25/Aug/08 01:52 PM
Attachment feature_enabling_5.patch [ 14942 ]
Martin Dougiamas added a comment - 25/Aug/08 05:04 PM
I really do like it.

Cons: there are lots of "features" which are not listed there, and also there are going to be settings that are not in the same place as the "big switch" to turn that thing on and off

But overall I think +1 ... perhaps we need some more votes though.


Eloy Lafuente (stronk7) added a comment - 26/Aug/08 06:33 AM
Any improvement in the organization of the admin menu has my +1 B-)

Anyway, some comments after chatting with Petr a bit (perhaps for next revision):

  • Perhaps it could be possible to have one more item under "Admin->Modules" (like activities, blocks and filters). Call it "subsystems" (or whatever you decide).
  • That item will work exactly like the others, with one main "Manage subsystems" page, used as general switcher and then, one specialised settings page for each subsystem.

IMO, that would be more "in sync" with other modules providing a better (consistent) user experience.

Also, could be interesting to "normalise" subsystem switcher names, instead of everyone having its own custom name, something like "subsystem_blog, subsystem_tags, subsystem_messaging..." or so.

Just some ideas FYC.

My +1 remains for current Petr proposal, anyway. Ciao


Petr Skoda committed 24 files to 'Moodle CVS' - 26/Aug/08 01:45 PM
MDL-16166 centralised enabling of subsystems, fixed some warnings and general cleanup
MODIFY admin/mnet/index.php   Rev. 1.21    (+1 -9 lines)
MODIFY admin/cron.php   Rev. 1.150    (+2 -2 lines)
MODIFY admin/settings/plugins.php   Rev. 1.17    (+2 -3 lines)
MODIFY notes/edit.php   Rev. 1.13    (+5 -1 lines)
MODIFY lang/en_utf8/mnet.php   Rev. 1.26    (+1 -0 lines)
MODIFY notes/index.php   Rev. 1.13    (+4 -1 lines)
MODIFY notes/delete.php   Rev. 1.11    (+5 -1 lines)
MODIFY portfolio/add.php   Rev. 1.17    (+1 -1 lines)
MODIFY user/addnote.php   Rev. 1.10    (+6 -2 lines)
MODIFY lib/weblib.php   Rev. 1.1128    (+3 -3 lines)
MODIFY user/groupaddnote.php   Rev. 1.7    (+5 -1 lines)
MODIFY user/index.php   Rev. 1.220    (+2 -2 lines)
MODIFY admin/settings/top.php   Rev. 1.24    (+9 -2 lines)
MODIFY user/tabs.php   Rev. 1.61    (+8 -7 lines)
MODIFY lang/en_utf8/admin.php   Rev. 1.223    (+1 -0 lines)
MODIFY admin/settings/server.php   Rev. 1.50    (+2 -9 lines)
MODIFY user/portfolio.php   Rev. 1.4    (+1 -1 lines)
MODIFY lib/portfoliolib.php   Rev. 1.38    (+2 -2 lines)
MODIFY lang/en_utf8/notes.php   Rev. 1.8    (+5 -2 lines)
MODIFY admin/settings/grades.php   Rev. 1.33    (+2 -5 lines)
MODIFY version.php   Rev. 1.780    (+1 -1 lines)
ADD admin/settings/subsystems.php   Rev. 1.1    (+0 -0 lines)
MODIFY admin/settings/Attic/misc.php   Rev. 1.23    (+1 -2 lines)
MODIFY admin/settings/security.php   Rev. 1.30    (+1 -9 lines)
Petr Skoda added a comment - 26/Aug/08 01:47 PM
I have committed the patch with minor changes, then name of settings page is "Optional subsystems" - feel free to tweak the lang string

Petr Skoda made changes - 26/Aug/08 01:47 PM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
Mitsuhiro Yoshida committed 15 files to 'Lang CVS' - 27/Aug/08 04:28 PM
MDL-16166 Translated new strings for admin, mnet and notes.
Brushed up a translated help file for quiz max grade.
MDL-7010 Translated new strings for quiz settings.
MDL-13766 Translated new strings for repository.
Updated translated strings for repository based on lang/en_utf8 change.
MODIFY ja_utf8/admin.php   Rev. 1.296    (+2 -1 lines)
MODIFY ja_utf8/repository_local.php   Rev. 1.2    (+5 -5 lines)
MODIFY ja_utf8/quiz_responses.php   Rev. 1.6    (+2 -1 lines)
MODIFY ja_utf8/repository_flickr.php   Rev. 1.2    (+2 -2 lines)
MODIFY ja_utf8/notes.php   Rev. 1.8    (+4 -1 lines)
MODIFY ja_utf8/repository_boxnet.php   Rev. 1.2    (+3 -3 lines)
MODIFY ja_utf8/quiz_statistics.php   Rev. 1.15    (+6 -5 lines)
MODIFY ja_utf8/help/quiz/maxgrade.html   Rev. 1.6    (+2 -2 lines)
MODIFY ja_utf8/quiz_overview.php   Rev. 1.18    (+2 -1 lines)
MODIFY ja_utf8/mnet.php   Rev. 1.42    (+2 -1 lines)
MODIFY ja_utf8/quiz.php   Rev. 1.159    (+20 -1 lines)
MODIFY ja_utf8/README   Rev. 1.796    (+1 -1 lines)
MODIFY ja_utf8/repository.php   Rev. 1.9    (+6 -4 lines)
MODIFY ja_utf8/quiz_grading.php   Rev. 1.9    (+2 -1 lines)
MODIFY ja_utf8/quiz_analysis.php   Rev. 1.8    (+2 -1 lines)
martignoni committed 3 files to 'Lang CVS' - 27/Aug/08 07:37 PM
MDL-16166 New strings translated
MODIFY fr_utf8/admin.php   Rev. 1.294    (+2 -1 lines)
MODIFY fr_utf8/notes.php   Rev. 1.11    (+4 -1 lines)
MODIFY fr_utf8/mnet.php   Rev. 1.33    (+2 -1 lines)
Nigel McNie made changes - 08/Oct/08 10:19 AM
Link This issue has been marked as being related by MDL-16821 [ MDL-16821 ]
Ray Lawrence made changes - 13/Oct/08 10:24 PM
Link This issue has been marked as being related by MDL-16867 [ MDL-16867 ]
Petr Skoda made changes - 01/Dec/08 01:10 AM
Issue Type Improvement [ 4 ] Sub-task [ 5 ]
Parent MDL-17205 [ 29354 ]
Petr Skoda added a comment - 01/Dec/08 01:10 AM
reopening, going to backport into 1.9.x

Petr Skoda made changes - 01/Dec/08 01:10 AM
Resolution Fixed [ 1 ]
Fix Version/s 1.9.4 [ 10300 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Petr Skoda made changes - 01/Dec/08 01:10 AM
Status Reopened [ 4 ] In Progress [ 3 ]
Petr Skoda added a comment - 01/Dec/08 01:11 AM
grrr, I mean going to backport thew disabling of notes only

Petr Skoda added a comment - 01/Dec/08 01:11 AM
regrrrr, going to file a separate issue for backport

Petr Skoda made changes - 01/Dec/08 01:11 AM
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Petr Skoda made changes - 01/Dec/08 01:12 AM
Issue Type Sub-task [ 5 ] Bug [ 1 ]
Parent MDL-17205 [ 29354 ]
Petr Skoda added a comment - 01/Dec/08 01:12 AM
reclosing, sorry for the noise

Petr Skoda made changes - 01/Dec/08 01:12 AM
Status Resolved [ 5 ] Closed [ 6 ]
Fix Version/s 1.9.4 [ 10300 ]
Petr Skoda made changes - 01/Dec/08 01:13 AM
Link This issue has been marked as being related by MDL-17472 [ MDL-17472 ]