Modify

Opened 8 years ago

Closed 20 months ago

#5808 closed defect (fixed)

PeerReview plugin is broken for case sensitive databases

Reported by: trac-peerreview@… Owned by: Marc
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 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by anonymous

comment:1 Changed 7 years ago by Ryan J Ollos

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

comment:2 Changed 6 years ago by Marc

Owner: changed from mikechml to Marc

comment:3 Changed 20 months ago by Cinc-th

Resolution: fixed
Status: newclosed

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



Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Marc.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.