#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)
Change History (5)
by , 16 years ago
| Attachment: | upgrade_freecomments.txt added |
|---|
comment:1 by , 16 years ago
| Has patch: | set |
|---|
comment:2 by , 16 years ago
| Component: | Uncategorized → Documentation |
|---|---|
| Keywords: | comments documentation added |
| Triage Stage: | Unreviewed → Accepted |
| Version: | 1.0 → SVN |
Well, as you pointed out it should be an inverted mapping...my bad.
comment:3 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
patch to fix freecomments upgrade