The module attaches, through hook_menu_alter, two additional parameters to page arguments: the original page callback and url components. These arguments are consumed by me_handler() through the following code:

$parts = array_pop($args);
$callback = array_pop($args);

This is usually ok. However, what I think we're not considering is the cases when url provides additional arguments. For example, if we try to access user/%/edit/subscriptions, and this is not a registered path, 'subscriptions' is dynamically added as an additional argument for the page. In that case, the code above obviously fails.

Comments

cambraca’s picture

Status: Active » Needs review
StatusFileSize
new1.29 KB

This file was given to me by the OP, who is having problems with the site.

cpierce42’s picture

Status: Needs review » Reviewed & tested by the community

I encountered this same issue.

I have tested this on my site and seems to fix my warnings and white page of death when navigating to the aliased path of:
/user/2789/track/navigation

My QA person is testing the rest of the site as I type, so far this patch works.

  • nohup committed 69d96e4 on 7.x-1.x authored by cambraca
    Issue #2539462 by cambraca: Additional URL parameters break me page...

  • nohup committed d0d26fd on 7.x-1.x authored by cambraca
    Issue #2539462 by cambraca: Additional URL parameters break me page...
nohup’s picture

Status: Reviewed & tested by the community » Fixed

Patch committed. Thanks

Status: Fixed » Closed (fixed)

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