Modify

Opened 7 years ago

Closed 6 months ago

#5808 closed defect (fixed)

PeerReview plugin is broken for case sensitive databases

Reported by: trac-peerreview@… Owned by: andersm
Priority: normal Component: PeerReviewPlugin
Severity: major Keywords:
Cc: Trac Release: 0.11

Description

PeerReview plugin is broken for case sensitive databases. (tested with postgresql)

Following patch fixes this. (for postgresql)

Attachments (1)

peerreview_quoting_naming.diff (8.3 KB) - added by anonymous 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by anonymous

comment:1 Changed 6 years ago by rjollos

Even better fix would be proper use of the Trac API, which this plugin does not do.

comment:2 Changed 5 years ago by andersm

  • Owner changed from mikechml to andersm

comment:3 Changed 6 months ago by Cinc-th

  • Resolution set to fixed
  • Status changed from new to closed

In 15192:

Upgrade for database schema to version 2.

  • all table and column names are lower case now
  • table names are prepended by peer_ to prevent name clashes
  • added some columns for future features
  • uses Trac upgrade framework
  • table contents is properly migrated

Note that the upgrade is only tested with SQLite.

Refs #10812
Refs #5395
Fixes #5808
Fixes #5401



Add Comment

Modify Ticket

Action
as closed The owner will remain andersm.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.