Modify ↓

Opened 15 years ago

Closed 9 years 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 15 years ago.

Download all attachments as: .zip

Change History (4)

Changed 15 years ago by anonymous

Attachment: peerreview_quoting_naming.diff​ added

comment:1 Changed 14 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 13 years ago by Marc

Owner: changed from mikechml to Marc

comment:3 Changed 9 years ago by Cinc-th

Resolution: → fixed
Status: new → 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



Modify Ticket

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

Add Comment

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

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.