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

salvis’s picture

Forum 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'.

bazzly’s picture

Not 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

salvis’s picture

[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...

bazzly’s picture

"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.

salvis’s picture

[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.

salvis’s picture

Title: Chain Menu Access API for D6 and D7 — BETA1 » Chain Menu Access API for D6 and D7 — BETA2
Version: 7.x-1.0-beta1 » 7.x-1.0-beta2

BETA2 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.

emptyvoid’s picture

This 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

salvis’s picture

No, 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.

salvis’s picture

Title: Chain Menu Access API for D6 and D7 — BETA2 » Chain Menu Access API for D6 and D7 — BETA3
Version: 7.x-1.0-beta2 » 7.x-1.0-beta3

BETA3 (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.

salvis’s picture

Title: Chain Menu Access API for D6 and D7 — BETA3 » Chain Menu Access API for D6 and D7 — BETA4
Version: 7.x-1.0-beta3 » 7.x-1.0-beta4

BETA4 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.

salvis’s picture

Title: Chain Menu Access API for D6 and D7 — BETA4 » Chain Menu Access API for D6 and D7 — D7 RELEASED
Version: 7.x-1.0-beta4 » 7.x-1.0
Status: Needs review » Fixed

With 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.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.