Closed (fixed)
Project:
Admin Menu Hider
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
24 Mar 2008 at 16:55 UTC
Updated:
2 Jul 2008 at 22:25 UTC
Jump to comment: Most recent file
Comments
Comment #1
sunThis module depends on Drupal Administration Menu. For further information about admin_menu's port to D6 and how you can help, see http://drupal.org/node/132524
Comment #2
deciphered commentedPatch rolled against HEAD.
Works perfectly on my config, can someone else confirm.
On a side note, I also cleaned up the code so it conforms to the Drupal coding standards.
Comment #3
sunLooks good, code-wise.
Heh... if you want to get nitpicky on coding standards, you need to
- add a comma after last values in arrays spanning multiple lines.
- add a trailing dot/punctuation for the description in the .info file.
- remove double-quotes from .info file values as long as they're not required.
- replace multiple calls to drupal_get_path() with one $path variable.
- remove hook_menu() and add AMD's settings to DAM's settings form.
- add a newline to the end of file, so diff stops complaining.
...however, all of these points may or may not come later ;)
Comment #4
shawn conn commentedI've gone ahead and applied the patch for the new 6.x-1.x branch after some testing. Apologies for taking so long to get this done. My life has been consumed by too many projects as of late. Deciphered is now a co-maintainer of this module so hopefully future updates for this module will be applied more quickly :)
Comment #5
deciphered commentedComment #6
rszrama commentedThis patch is actually incomplete. I can't roll a patch atm, but the .info needs the dependency to be an array:
i.e.:
Further, this seems to preserve the bug in the .css where you need more specificity on the second selector... it should be body.admin-menu, not just body.
Other than that, I have it running on my D6 test site.
Comment #7
deciphered commentedThe patch is irrelevant, the port has been complete and is available as 6.x-1.x-dev.
While what you have stated is correct and is still an issue in the dev build, if you would like to report it, start a new issue, if not, it will be fixed as soon as I get home from work.
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.