Closed (won't fix)
Project:
Administration menu
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Jun 2008 at 16:15 UTC
Updated:
25 Jun 2010 at 07:01 UTC
http://drupal.org/project/qamodules is pretty hard to find in d.o's modules list and its tiny but very useful feature does not really have to be in a separate module.
So let's fork that feature into admin_menu. As qamodules already exists for D6, we only need some backporting work for D5. Attached patch works already, but does not disable dependent (parent) modules after submitting the form.
| Comment | File | Size | Author |
|---|---|---|---|
| admin_menu.qamodules.patch | 2.96 KB | sun |
Comments
Comment #1
scottrigbySun, is there a patch for the D6 version of admin_menu somewhere? I followed this thread from this qamodules issue #270548: Merge into other module page helpers/enhancers, and now am wondering whether I should patch to include this in my 6.x-1.x-dev admin_menu module --- or just install qamodules in the meantime?
Thx in advance for clarification ;)
Scott
Comment #2
sunNot yet, but above patch should basically be easy to port to 6.x. I'd definitely like to see this feature in 6.x-1.2.
And, I've decided to not port this back to 5.x, because it is too complex in 5.x.
Comment #3
babbage commentedWill post here, since this seems to be where QA module is headed... I think this functionality is so useful, it really saves time. I'd *love* to see the next step—a "BIG SWITCH" somewhere that turns off *all* modules simultaneously, other than the required core modules... This is the recommended thing to do on upgrade, but it takes a lot of time. QA modules has made that bearable, but it still is much more work than it needs to be. Presumably the technology already developed in QA module should make this a pinch, but I'm not at a skill level where I'd be sure how to add this probably minor change. Anyone willing to give it a go?
Edit: I see now something along these lines is already being worked upon.
Comment #4
sun@dbabbage: Yes, what you want is either #173570: Add link to disable non-required core modules for upgrading or #299470: Add module_pattern(s) module to DAM's toolbox.
Comment #5
sunCross-referencing Drupal 7 issue: #107038: Javascript to select module dependencies
Comment #6
babbage commentedDecided to implement my "big switch" myself for Drupal 6.x:
http://drupal.org/project/contrib_toggle
Contrib Toggle module provides a simple button on the module configuration page (/admin/build/modules) to mark all contrib modules for deactivation, reducing the deactivation of contrib modules to a simple two-click exercise.
I am posting this here because this module also a) incorporates the functionality of Quick Admin Modules and b) solved the bug that has always existed in Quick Admin Modules that meant that modules that previously had dependants were never actually disabled even though they could be unchecked. In the near future I will post a patch to Quick Admin Menu / Administration menu to incorporate the now-functioning code if you like.
Comment #7
babbage commentedComment #8
babbage commentedI am having second thoughts on this. Primarily, I'm wondering about incorporating into Administration Menu something that has nothing to do with the Administration Menu or even menus at all. Secondly, I'm working on the dependencies necessary to get this into core in D7, and there is already a D6 solution in Contrib Toggle, so I'm wondering if this is necessary. Finally, because of how D6 handled the loading of active modules with dependencies (thus disabled on the modules page) the implementation is more complex and fraught than you'd think.
So all up, I'm shelving this. If someone else wants to implement it, feel free-the code is all in contrib_toggle. Just be warned-understand the effects of removing entries from the disabled_modules array before you start!
Comment #9
sunProbably you're right. After having worked on the patch for graph.inc I see that this feature could potentially lead to major issues in D6.
Comment #10
hixster commentedsubscribing