Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This code is new. It's short and sweet, and it has been examined very carefully. However, we have no automated tests yet.
The D6 and D7 versions are identical except for the .info file.
Please test this version and provide feedback, both good and bad:
IF it works for you, let us know by adding a note to THIS issue here.
IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
Do NOT add bug reports to this issue here.
Comments
Comment #1
salvisForum Access has started to use Chain Menu Access API for D7 and this works very nicely (see #809404-6: Forum Access for D7). Three clean calls to chain_menu_access_chain() have replaced some tricky code to do the same thing 'manually'.
Comment #2
bazzly CreditAttribution: bazzly commentedNot sure if I should post this here but....I'm getting
* Warning: array_splice() expects parameter 3 to be long, string given in _chain_menu_access_callback() (line 59 of /var/www/drupal/sites/all/modules/chain_menu_access/chain_menu_access.module).
I'm running ACL 7.x-1.0-beta2
and Forum Access 7.x-1.0-alpha4
Comment #3
salvis[EDIT: removed off-topic reply to bug report in #2 above, duplicate of #809404-14: Forum Access for D7]
Pursued and fixed in #1036478: Warning: array_splice() expects parameter 3 to be long, string given in _chain_menu_access_callback() (line 59
Skip to #6 below...
Comment #4
bazzly CreditAttribution: bazzly commented"What's the path/URL of the page where you get that warning?"
I get it
Warning: array_splice() expects parameter 3 to be long, string given in _chain_menu_access_callback() (line 59 of /var/www/drupal/sites/all/modules/chain_menu_access/chain_menu_access.module).
When I go from my home screen directly to the forum. Below is the URL. Could it be because its not a name, but an IP?
http://10.8.193.159/forum
Sorry I was wrong, it is repeatable.
Comment #5
salvis[EDIT: removed off-topic stuff]
Revised my boilerplate text after discussion with bazzly:
IF this module works for you, let us know by adding a note to THIS issue here.
IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
PLEASE do NOT add bug reports to this issue here.
Comment #6
salvisBETA2 fixes all known issues for D6 and D7, and adds the testing infrastructure plus a first test.
IF this module works for you, let us know by adding a note to THIS issue here.
IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
PLEASE do NOT add bug reports to this issue here.
Comment #7
emptyvoid CreditAttribution: emptyvoid commentedThis is perfect, it's a feature set I was planning to implement in the new version of menu access. I may make this module a dependency for the new version.
Do you provide a UI to manage priority preference of module calls for a given menu or menu item?
If not, then perhaps that is something my module could provide.
http://drupal.org/node/1067692
Comment #8
salvisNo, CMA is an API module. There's no intention to expand it beyond the raw functionality. We hope to eventually get it into D8 core.
You are welcome to build upon and expand on CMA.
If there's more to discuss, then please open a new thread.
Comment #9
salvisBETA3 (for D7) fixes a possible issue for some special paths and provides guidance to not try to chain MENU_DEFAULT_LOCAL_TASKs. See #1186208: Unable to chain in front of some system paths for details.
The D6 version is still at BETA2 — an update for D6 will follow shortly.
IF this module works for you, let us know by adding a note to THIS issue here.
IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
PLEASE do NOT add bug reports to this issue here.
Comment #10
salvisBETA4 fixes another possible issue for some special paths. See #1186208: Unable to chain in front of some system paths for details.
IF this module works for you, let us know by adding a note to THIS issue here.
IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
PLEASE do NOT add bug reports to this issue here.
Comment #11
salvisWith well over 500 known sites using BETA4, I'm happy to announce the release of 7.x-1.0, which is unchanged from BETA4.
At the same time, we've started the Drupal 8 branch — see #1243342: Chain Menu Access API for D8 — BETA5.
The D6 version has received little testing and even less feedback so far. I've created a new separate issue for it, just in case there should ever be some interest — see #1243362: Chain Menu Access API for D6 — BETA4.