Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#9346 closed (fixed)

Comments upgrading page has wrong SQL in it

Reported by: Seemant Kulleen Owned by: nobody
Component: Documentation Version: dev
Severity: Keywords: comments, documentation
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

http://docs.djangoproject.com/en/dev/ref/contrib/comments/upgrade/ has some SQL to help you go to the new comments model in 1.0. However, the mapping is from 'is_removed' to 'approved'. Surely, the mapping shouldn't be direct, but rather inverted? In other words, if it's removed, it should be NOT approved. Or, is it that 'approved' is misnamed as a field?

Attachments (1)

upgrade_freecomments.txt (580 bytes ) - added by Michael Thornhill 16 years ago.
patch to fix freecomments upgrade

Download all attachments as: .zip

Change History (5)

by Michael Thornhill, 16 years ago

Attachment: upgrade_freecomments.txt added

patch to fix freecomments upgrade

comment:1 by anonymous, 16 years ago

Has patch: set

comment:2 by Thejaswi Puthraya, 16 years ago

Component: UncategorizedDocumentation
Keywords: comments documentation added
Triage Stage: UnreviewedAccepted
Version: 1.0SVN

Well, as you pointed out it should be an inverted mapping...my bad.

comment:3 by Jacob, 16 years ago

Resolution: fixed
Status: newclosed

(In [9732]) Fixed #9346: fixed comment upgrade SQL in documentation. Thanks, mt.

comment:4 by Jacob, 16 years ago

(In [9738]) A handful of fixes to django.contrib.comments:

Note: See TracTickets for help on using tickets.
Back to Top