Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#5366 closed defect (fixed)

Update blog views for database changes

Reported by: ayleph Owned by:
Priority: blocker Milestone: 0.9.0
Component: programming Keywords: blog, db, migrations
Cc: tsyesika Parent Tickets:

Description

Blog views crash with server errors as they haven't been updated for the latest database changes. I've got local fixes that I'll compile and upload as a patch.

Attachments (1)

0001-Fix-issue-5366-Update-blog-views.patch (8.2 KB ) - added by ayleph 8 years ago.
Fix for issue 5366

Download all attachments as: .zip

Change History (7)

comment:1 by ayleph, 8 years ago

Status: newin_progress

Claiming ticket.

by ayleph, 8 years ago

Fix for issue 5366

comment:2 by ayleph, 8 years ago

Owner: ayleph removed
Status: in_progressreview

Patch attached in previous comment. Marking ticket for review.

comment:3 by Loic Dachary, 8 years ago

Owner: set to ayleph
Status: reviewin_progress

There is no existing test for the corresponding code change.

./runtests.sh --cov-report=term-missing --cov=mediagoblin mediagoblin/tests
...
mediagoblin/media_types/blog/views.py 173 173 0% 17-370
...

comment:4 by ayleph, 8 years ago

Owner: ayleph removed
Status: in_progressreview

The patch provided here fixes the server crash issue. I won't be providing tests, so someone else will need to take that on if they want to do that.

comment:5 by Jessica Tallon, 8 years ago

Resolution: fixed
Status: reviewclosed

I have reviewed this patch and merged it into master as of beb5a4a. Thanks for reporting this and providing a patch!

comment:6 by Christopher Allan Webber, 8 years ago

Milestone: 0.8.20.9.0

All 0.8.2 tickets are being rolled over to 0.9.0

Note: See TracTickets for help on using tickets.