edit media details

[Elrond]

It should be possible to edit the title, description, slug and
maybe other things.

In the long run, it might be needed to even replace the media. But
this is probably another item.

[Elrond]

I have started a devel branch here:

[https://gitorious.org/\ :sub:`elrond/mediagoblin/elronds-mediagoblin/commits/dev/media\_edit](https://gitorious.org/`\ elrond/mediagoblin/elronds-mediagoblin/commits/dev/media\_edit)

Comments, suggestions, merge request welcome.

Current issues:


-  Any logged in user can edit any media
-  No checks on the entered values
-  exception from .save() not handled (what could happen? How to
   handle?)

[Elrond]

-  located as "/edit/{media\_id}/". Maybe something like
   /u/{user}/m/{media\_id}/edit/ would be better.

Note that I would suggest using media\_id instead of the slug for
the edit things, as the slug itself can be edited and we might want
a static URL for at least the edit page.

cwebber has volunteered to fixup the url/routing. So I leave that
part to him.

[Elrond]

-  permission check is now there (admins and the uploader can
   change things)
-  Preparation for moving under /u/ also there. What's left?
-  permission check probably should be moved in some lib.py (will
   do soon)
-  No checks on the entered values
-  exception from .save() not handled. Especially a duped slug will
   give an exception. How to handle that?

[Christopher Allan Webber]

Looked at your latest changes, and I think this is near mergable.
Only one thing left, and that's that I think the way the duplicate
slug error happens is wrong.


-  try: catch Exception: is gross because it catches way too many
   things. See also:
   `http://exilejedi.livejournal.com/285779.html <http://exilejedi.livejournal.com/285779.html>`_
-  I would like it if we could actually display the error inside
   the form. See the register() view for more info on how to do this.
-  I really honestly think the cleanest way to handle things is to
   simply do a query to see if such an uploader, slug pair exists. Yes
   I know, it's "kind of" duplicated functionality, but there's no
   nice way to extract what exactly broke during validation. I think
   validation should be a last resort to make sure our models have
   kept their right form and to prevent very, very rare race
   conditions (the chance of a user actually updating two MediaEntries
   at once to the same slug this way is super low but worth preventing
   from a data integrity standpoint).

[Elrond]

I would prefer it, if someone else could handle the remaining
issues. To put it simple, I'm currently exhausted on this.

You might consider to merge it in the current state and leave the
bug open. That way anyone interested to continue this can start
from the main repository. This includes the UI design team (read
"schendje"). IMHO the whole thing works okay and all the remaining
issues can be fixed "in the normal tree" (like btrfs was merged
early).

[Christopher Allan Webber]

That's fine, I can handle the rest. Thanks for your work on this,
Elrond. :)

[Christopher Allan Webber]

Merged Elrond's work and made the changes discussed. Thanks for
your work on this Elrond!

[Elrond]

Thanks Chris!

Reopening as discussed a while back with schendje and reassigning
to him.
I would like the UI team to take a quick look at the "edit media
details" page. I put a thumbnail of the picture there, so that one
sees the media in question. But I have no plan on a good position
for it.

Another simple question: Where to put the "Edit"-link on the main
media page?

This is low priority, as things work and don't look too bad. So if
the UI team has a few spare minutes, it would be nice, but
certainly not needed.

[Elrond]

I think this iteration is done. All looks fine.

If you don't agree, complain here or open a new bug.

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/353 .