564561 - Separate Edit User links for the user / admin

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect, P3)

Description

[Justin Scott [:fligtar]]

There's a prettier Edit user link on user profile pages now, but when I go to it as an admin and click, it tries to go to the admin tools to edit the user.

Let's make this two links in that section:

Edit Profile - the logged in user sees this if it's their profile and goes to the edit profile page.

Manage User - users with the right bit (Admin:EditAnyUser in remora) should see this and go to the admin tool for the user.

If I'm an admin viewing my own profile I should see both links.

Comment 1

[Wil Clouser [:clouserw]]

"Edit user" is a 404 right now too:


 File "/data/virtualenvs_preview/zamboni_a/lib/python2.6/site-packages/django/contrib/admin/widgets.py", line 235, in render
   output = [self.widget.render(name, value, *args, **kwargs)]

 File "/data/amo_python/www/preview/zamboni/apps/translations/widgets.py", line 44, in render
   languages=languages)

 File "/usr/lib/python2.6/site-packages/jinja2/environment.py", line 705, in render
   return self.environment.handle_exception(exc_info, True)

 File "/data/amo_python/www/preview/zamboni/apps/translations/templates/translations/transbox.html", line 1, in top-level template code
   <div id="{{ id }}" class="trans tab-wrapper" data-field="{{ name }}">

KeyError: 'f'

Comment 2

[Fred Wenzel [:wenzel]]

(In reply to comment #0)
> There's a prettier Edit user link on user profile pages now, but when I go to
> it as an admin and click, it tries to go to the admin tools to edit the user.

Yea that's the only link there was so far. I implemented (and tested) both buttons now according to comment 0, though.

http://github.com/jbalogh/zamboni/commits/d6025a8

Wil: I can't confirm this, wfm. If you can reproduce it somehow though, you should prolly file another bug.

Comment 3

[krupa raj[:krupa]]

For an admin user,there are two links on "View Profile" page.Calling this fixed.See post-fix screenshot.

Comment 4

[krupa raj[:krupa]]

Attachment: post-fix screenshot