Repeatable: Always

Steps to repeat:

  1. Begin create new user process (go to admin/user/user/create).
  2. Check any role checkbox.

Expected results:

Checking a role checkbox will cause the associated role expire text field to appear.

Actual results:

The role expire text field does not appear.

Patch coming.

CommentFileSizeAuthor
#1 role_expire-647866-1.patch694 byteskentr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kentr’s picture

Component: Code » User interface
FileSize
694 bytes
kentr’s picture

Status: Active » Needs review
stewsnooze’s picture

Thanks for this. I am on holiday until 21 dec and I will take a good look then

stewsnooze’s picture

Status: Needs review » Postponed (maintainer needs more info)

kentr, on my local copy this patch applies but actually stops the expiry time popout textbox appearing. i.e. without this patch the module works.

So is there anything else that could be affecting this?

kirie’s picture

Status: Postponed (maintainer needs more info) » Active

First, thanks for a nice module - it's just what I was looking for!

I just encountered this same situation and was able to track it down to the module's (lack of) integration with the Role Delegation module (mind you - it could be a different issue altogether):

- I've enabled both Role Expire (6.x-1.9) and Role Delegation (6.x-1.3)
- I've configured Role Delegation to allow my 'editor' role to assign the 'editor' role to other users (but not the 'admin' role)
- For Role Delegation to be of any use, I've disabled the 'administer permissions' permission for 'editor'
- The 'editor' can now give other users the 'editor' role - but can not do any changes to permissions in general
- The 'role expire' text field does not appear if the user doesn't have the 'administer permissions'

Would it be possible make the two modules work together - or am I overlooking something here?

(Sorry if this is a totally unrelated issue - in that case I can create a new issue for it)

PS: #543716: "Administer Permissions" requirement looks related

stewsnooze’s picture

Would it hurt to give the editor the 'administer role expire' permission? Have you tried that?

stewsnooze’s picture

Status: Active » Fixed

The original bug listed here is fixed. You should ideally move this to a new issue.

Status: Fixed » Closed (fixed)

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