Closed (fixed)
Project:
Workflow
Version:
8.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
23 Apr 2009 at 18:31 UTC
Updated:
20 Nov 2016 at 18:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
bendev commentedsame question...
Comment #2
Bastlynn commentedNot by default within the module as far as I know. It would take additional code.
Since this request is over a year old, I'm going to assume a solution was found or you've moved on. If not, please get updated to the latest versions of all modules and make a request for it against Drupal 7 and I'll be glad to take a look at it.
Comment #3
tomyinhauser commentedFaced the same situation on 7.x-1.1. Managed to open workflow tab on admin theme using the following custom code:
Would be nice to have this by default on workflow module, since the workflow is an administrative feature, not something to show to your end users.
Comment #4
nancydruOn my big site, it is, indeed, something the end users access. And they don't want it (or edit) in the admin theme.
Comment #5
nancydruWe could, perhaps do something like this:
Comment #6
nancydruFor better or worse, this got committed.
Comment #7
tomyinhauser commentedMakes sense. Thanks!
Comment #9
johnvThe proposed patch in #5 is committed as such, but that cannot work due to a typo in the argument.
Comment #10
johnvComment #11
johnvThis function also needs to be adapted for all entity types, just like workflow_menu_alter already does.
Comment #13
johnvI encountered a test situation, and fixed the typo.
Comment #15
johnvIn D8, the hook workflow_admin_paths_alter(&$paths) will not workfr any more, so this functionality is lost in D8, until it is repaired.
The hook will be removed from workflow.module.
See #1316692: Convert hook_admin_paths() into declarative properties on routes for the relevant D8-change. I could not find a change notice.
The current code is :
Comment #17
jamiehollernIn Drupal 8, routes must be declared as admin routes. Please see the attached patch.
Comment #18
jamiehollernComment #21
jamiehollernCI failed the last two times for some reason.
Comment #23
johnvCommitted. Thanks.