Opened 13 years ago
Last modified 10 years ago
#101 closed defect (FIXED)
Edit should only show up for media owner
Reported by: | Jef van Schendel | Owned by: | Caleb Davis |
---|---|---|---|
Priority: | minor | Milestone: | 0.0.3 |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description
On any media page, the text says "Upload on X by X" and then shows an "Edit" link. This link shows up for anyone, but it should only be visible if you're the owner of that piece of media and are logged in (and thus are the only one that can actually edit it).
Change History (8)
comment:2 by , 13 years ago
Status: | New → Closed |
---|
Thanks, I merged and pushed! I also changed the check to be based on the userid which is stored in the model, not a huge change. Thanks again!
comment:3 by , 13 years ago
Status: | Closed → In Progress |
---|
admins should also see the link. user.is\_admin might be the thing to check. Which is probably okay until the magical "permissions". With "permissions" a function "may\_edit()" is probably better.
comment:5 by , 13 years ago
Milestone: | → 0.0.3 |
---|
[https://gitorious.org/\ :sub:`cfdv/mediagoblin/cfdvs-mediagoblin/commits/b388+b391\_admin\_can\_edit\_user\_profile](https://gitorious.org/`\ cfdv/mediagoblin/cfdvs-mediagoblin/commits/b388+b391\_admin\_can\_edit\_user\_profile) I changed the conditional to reflect this decision. The issue seems like it's nearly resolved, so I added the 0.0.3 tag to make the roadmap more impressive :P
comment:6 by , 13 years ago
I think this one hasn't been closed yet only because `#391 </issues/391>`_ was held up. [https://gitorious.org/\ :sub:`cfdv/mediagoblin/cfdvs-mediagoblin/commit/e192d7b7a55efdfd2f50f5d068a0dfa484e152fa](https://gitorious.org/`\ cfdv/mediagoblin/cfdvs-mediagoblin/commit/e192d7b7a55efdfd2f50f5d068a0dfa484e152fa)
comment:7 by , 13 years ago
Status: | In Progress → Closed |
---|
Well that's merged, so this is resolved here too :)
Note:
See TracTickets
for help on using tickets.