Hi!

I understand that the requested functionality is not exactly what the module is intended to do, but...

This module lets site owners set PER CONTENT TYPE permissions for:
* sticky;
* published;
* promoted to front page;
* revision;
* authored by;
* authored on.

Why not add path to this list?

That would be great. Currently, path module lets you only enable change paths for all node types at once. And there is no module (afaik) that lets you set path permission per node type. I believe, your module is great for this purpose.

Comments

lolmaus’s picture

This also concerns menus!

hambers’s picture

@1 - Have you tried the Menu settings per Content type (http://drupal.org/project/ctm) module for controlling visiblity of menus. It met my simple case (although admittedly it doesn't do permissions per content type per role.

@0 - I'm also looking for this override_node_options functionality to extend to paths too!

tsi’s picture

+1
subscribing

dave reid’s picture

Version: 6.x-1.9 » 6.x-1.x-dev
caspercash’s picture

Was this feature request already implemented in the override_node_options? thanks for this great module!

caspercash’s picture

checked the module and also its dev but didn't find this feature..

maico de jong’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new1.37 KB

I'm also looking for this functionality so i created a patch for D7 that works for me.

Maybe someone can backport this to D6?

maico de jong’s picture

StatusFileSize
new1.37 KB

Please use this patch instead of the one in #7

opdavies’s picture

Status: Needs review » Needs work

I've applied the patch in #8, and I can see the URL alias tab when editing the node. However, once the node is saved, the update URL alias isn't applying and the original alias remains active. I'm more than happy to backport this to D6 once I've confirmed that it's working in D7.

maico de jong’s picture

Status: Needs work » Needs review

Can't reproduce the problem you are describing.

Did you try to rebuild the permissions after applying the patch?
Are you also sure not having permission: "edit create url aliases" turned on?

I'm using this patch now for a couple weeks without having the problem you mentioned.

jenlampton’s picture

Issue summary: View changes
Status: Needs review » Needs work

I can reproduce this issue. Steps are as follows:
1) create a piece of content, add an alias (it works!)
2) edit the content, attempt to change the alias (it doesn't work!)
3) Log in as a user with administer nodes permission
4) edit the content, attempt to change the alias (it works!)

It looks like in order to change an alias, users need the 'Create and edit URL aliases' permission.

jenlampton’s picture

Status: Needs work » Needs review
StatusFileSize
new1.52 KB

Here's a new version of the patch that allows people to edit existing aliases too.

opdavies’s picture

StatusFileSize
new1.52 KB

Trivial patch award candidate. :)

opdavies’s picture

StatusFileSize
new610 bytes

Interdiff for #13.

opdavies’s picture

Status: Needs review » Needs work
StatusFileSize
new35.9 KB

I've applied the patch in #12, and I can see the vertical tab whilst logged in as a non-admin user, but not the input field itself.

kiwimind’s picture

Issue tags: +Vienna2017

Patch no longer applies to head, so going to re-roll. Not looking into the outstanding issue mentioned in #15 yet.

kiwimind’s picture

StatusFileSize
new1.54 KB

Re-roll attached, haven't done an interdiff as the code being inserted is the same, it's primarily the line numbers for insertion.

kiwimind’s picture

Status: Needs work » Needs review
kiwimind’s picture

Further to the issue described in #11 - even when a node has an alias set, the alias is not showing, nor is the input field available.

As mentioned above, the permission to "Create and edit URL aliases" is needed to see this field.

kiwimind’s picture

It appears that the override_node_options_after_build function does return TRUE for the user when checking against user_access('override ' . $form['#node']->type . ' URL alias') where $form['#node']->type is page, so not quite sure it's not filtering down to form output.

opdavies’s picture

Status: Needs review » Needs work

Now #974730: permissions for "all content types" is merged, we should add a global permission for this too.

opdavies’s picture

Whilst I consider the 7.x branch to be feature complete, if there is a per content type permission for this, a global permission should be added, too, for consistency.