Opened 13 years ago

Closed 12 years ago

#264 closed defect (fixed)

Make sure delete_media deletes comments — at Version 3

Reported by: Elrond Owned by:
Priority: major Milestone: 0.3.0
Component: programming Keywords:
Cc: Parent Tickets:

Description (last modified by Christopher Allan Webber) 

Looking at the code, it seems like the comments (on a media) aren't
deleted together with the media.


-  If the code exists, point me at it, so I can possibly add
   comments, so I'll find it next time easier. AND: Maybe there should
   be unit tests for the comments being deleted?
-  If even those unit tests for comment deletion exists... Oh well,
   Reject this bug.

Change History (3)

comment:1 by Will Kahn-Greene, 12 years ago

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

comment:2 by Brett Smith, 12 years ago

Milestone: → 0.2.2

I have a branch with a fix for this and associated test:

​https://gitorious.org/~bretts/mediagoblin/brettss-mediagoblin/commits/bug264-delete-comments-with-media

comment:3 by Christopher Allan Webber, 12 years ago

Description: modified (diff)
Priority: minor → major
Resolution: → fixed
Status: accepted → closed

That was stunningly fast work! The test looks good too. Great work Brett! I've merged and pushed it.

Note: See TracTickets for help on using tickets.