Hello,

Only the twitter module is enabled and the permission "Add authenticated Twitter accounts" is set for authenticated users. What I want is to let each authenticated user to configure his own auth twitter account.

If I sign in as uid = 3 and navigate to /user/3/edit/twitter it gives me the right twitter settings page. There is another auth user with uid = 2 on the system. If I navigate to /user/2/edit/twitter I can access and add authenticated twitter accounts for this user too! I naturally don't have access to /user/2/edit.

I think this is a bug. Any ideas?

Thank you for this great module!

Chris

Comments

Adirael’s picture

I'm trying to fix the same issue.

Adirael’s picture

Version: 7.x-5.4 » 7.x-5.6
StatusFileSize
new1.33 KB

I created a patch that fixed this for me. It adds a 'administer own twitter accounts' permission. It may be good to change a bit the granularity of the permissions but since this is the first time I add a patch here I prefer to keep it as simple as possible .

damienmckenna’s picture

damienmckenna’s picture

Version: 7.x-5.6 » 6.x-5.x-dev
Status: Active » Needs review
StatusFileSize
new1.43 KB

I think this is a better approach - use the permissions already there but only allow you to view someone else's account if you have the 'administer users' permission.

damienmckenna’s picture

Version: 6.x-5.x-dev » 7.x-5.x-dev
StatusFileSize
new1.82 KB

For 7.x-5.x

damienmckenna’s picture

Version: 7.x-5.x-dev » 7.x-6.x-dev
StatusFileSize
new1.82 KB

For 7.x-6.x.

Status: Needs review » Needs work

The last submitted patch, 6: twitter-n1895472-6-7.x-6.x.patch, failed testing.

damienmckenna’s picture

Version: 7.x-6.x-dev » 6.x-5.x-dev
Status: Needs work » Needs review
StatusFileSize
new1.78 KB

Forgot to add some code to the 6.x-5.x patch to rebuild the menu cache.

damienmckenna’s picture

Status: Needs review » Fixed

Committed.

  • DamienMcKenna committed f1a9ef5 on 6.x-5.x
    Issue #1895472 by DamienMcKenna, Adirael: Don't allow people to edit...

  • DamienMcKenna committed 2ca73d6 on
    Issue #1895472 by DamienMcKenna, Adirael: Don't allow people to edit...
damienmckenna’s picture

  • DamienMcKenna committed 8dd4d36 on 7.x-6.x
    Issue #1895472 by DamienMcKenna, Adirael: Don't allow people to edit...

Status: Fixed » Closed (fixed)

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