Project places a project in the URL instead of a parameter for node/add

https://www.drupal.org/node/add/project-issue/panelizer #sadtrombone

There must be a permission thing here, the problem doesn't happen for me.

Yep, view/access admin theme is a separate permission.

I'm not aware of a precedent for additional things being added to the node/add path as being particularly good or bad.

If we do change this to a query parameter, redirects must also be added.

Panelizer matches the following path for the admin path:

  $items[$path . '/panelizer*'] = TRUE;

.. where $path is the entity path with all arguments converted to stars, therefore it would be node/*/panelizer* and not node/*/*/panelizer as suggested. Would that still trigger the match?

node/*/panelizer* is indeed what it uses, and panelizer does take over node/{nid}/panelizer.

I expect this reaches drupal_match_path(), which I think secretly supports regular expressions. node/[0-9]+/panelizer* may work. That's non-ideal since it is exploiting secret functionality, and maybe Panelizer can't assume entities have numeric IDs in URLs. I can't think of another way to be more specific about which paths are admin paths.

Sounds like this needs to go back to being a Panelizer issue then.

So it boils down to adding a hook_admin_paths() callback for the PanelizerEntityNode class and using a string that drupal_match_path() will accept.

Something like this might work.

Branch didn't get switched back.

Committed. Thanks drumm for the regex suggestion.

Darnit. I should have added tests first.

Rolling back the change.

The regex doesn't work because drupal_match_path() runs preg_quote() on the string, breaking the fancy regular expression.

Darnit.

Complete rollback, with the tests.

I've committed the changes above.

Should maybe d.o add this to a custom module?

public function hook_admin_paths() {
  $items['node/add/project-issue/panelizer'] = FALSE;
  return $items;
}

Maybe it just needs to check the node_admin_theme setting?

@drumm: What is node_admin_theme set to on d.o?

node_admin_theme is 0

Ah! That explains it then.

This should cover it.

Fixed the tests.

Ok. This works. Thanks all.