Modify

Opened 16 months ago

Last modified 15 months ago

#11308 new defect

Doesn't support any DB backend other than SQLite

Reported by: korn Owned by: bphinz
Priority: normal Component: FieldGroupsPlugin
Severity: normal Keywords:
Cc: rjollos Trac Release:

Description

Co-opting code from tagsplugin/trunk/tractags/db.py helped with getting the list of tables correctly, but I don't know how to implement the auto-incrementing id field in a backend independent way.

Attachments (0)

Change History (3)

comment:1 Changed 15 months ago by rjollos

Could you post a patch of the changes you made?

I thought the cross-database incompatibility issue would have been fixed after [13213], but I didn't take a careful look at the schema in that ticket. We need to define the schema using the classes in browser:/tags/trac-1.0.1/trac/db/schema.py. For example, see: db_default.py.

Last edited 15 months ago by rjollos (previous) (diff)

comment:2 Changed 15 months ago by korn

Unfortunately no; I gave up on this plugin and have deleted my checkout, including the modifications.

Basically I replaced stuff like

        cursor.execute("SELECT * FROM sqlite_master WHERE name='field_groups' AND type='table'")

with stuff like

        cursor.execute("SELECT tablename
                 FROM pg_tables
                 WHERE tablename='field_groups' AND schemaname = ANY (current_schemas(false))")

comment:3 Changed 15 months ago by rjollos

  • Cc rjollos added; anonymous removed

Oh, right. Yeah the check for table existence needs to be fixed as well. Thanks for pointing that out.

Add Comment

Modify Ticket

Action
as new The owner will remain bphinz.
Author


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

 
Note: See TracTickets for help on using tickets.