Opened 5 years ago

Last modified 4 years ago

#10206 new defect

Duplicate column name "comment"

Reported by: roberts.noah@… Owned by: Ryan J Ollos
Priority: normal Component: WorkLogPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description (last modified by Ryan J Ollos)

After I installed your plugin at the system level, and enabled it for one of my projects, I get an error from trac upon looking at any trac page linked to that project:


TracError: OperationalError: duplicate column name: comment

Editing trac.ini to remove the worklog plugin enable statements fixes the project. Other enabled plugins are:

autoupgrade.autoupgrade.autoupgrade = enabled
batchmod.web_ui.batchmodifymodule = enabled
childtickets.admin.childticketsadminpanel = enabled
childtickets.childtickets.tracchildticketsmodule = enabled
coderev.api.codereviewersystem = enabled
coderev.web_ui.changesetticketmapper = enabled
coderev.web_ui.codereviewermodule = enabled
coderev.web_ui.committicketreferencemacro = enabled
mastertickets.api.masterticketssystem = enabled
mastertickets.web_ui.masterticketsmodule = enabled
simplemultiproject.admin.smpadminpanel = enabled
simplemultiproject.admin_component.smpcomponentadminpanel = enabled
simplemultiproject.environmentsetup.smpenvironmentsetupparticipant = enabled
simplemultiproject.milestone.smpmilestoneproject = enabled
simplemultiproject.model.smpmodel = enabled
simplemultiproject.roadmap.smproadmapproject = enabled
simplemultiproject.roadmap.smproadmapprojectfilter = enabled
simplemultiproject.ticket.smpticketproject = enabled
simplemultiproject.timeline.smptimelineprojectfilter = enabled
simplemultiproject.version.smpversionproject = enabled
tracbzr.backend.bzrconnector = enabled
tracbzr.backend.bzrpropertyrenderer = enabled
tracbzr.backend.bzrwikimacros = enabled
tracopt.ticket.deleter.ticketdeleter = enabled
tracrpc.api.xmlrpcsystem = enabled
tracrpc.json_rpc.jsonrpcprotocol = enabled = enabled
tracrpc.ticket.componentrpc = enabled
tracrpc.ticket.milestonerpc = enabled
tracrpc.ticket.priorityrpc = enabled
tracrpc.ticket.resolutionrpc = enabled
tracrpc.ticket.severityrpc = enabled
tracrpc.ticket.statusrpc = enabled
tracrpc.ticket.ticketrpc = enabled
tracrpc.ticket.typerpc = enabled
tracrpc.ticket.versionrpc = enabled
tracrpc.web_ui.rpcweb = enabled = enabled
tracrpc.xml_rpc.xmlrpcprotocol = enabled
whiteboard.web_ui.whiteboardmodule = enabled

Attachments (0)

Change History (7)

comment:1 Changed 5 years ago by Ryan J Ollos

Description: modified (diff)

comment:2 Changed 5 years ago by Ryan J Ollos

I can reproduce. The plugin looks like it needs a bit of work to make it compatible with Trac 0.12.

comment:3 in reply to:  2 Changed 4 years ago by anonymous

Replying to rjollos:

I can reproduce. The plugin looks like it needs a bit of work to make it compatible with Trac 0.12.

me too

comment:4 Changed 4 years ago by Ryan J Ollos

Owner: changed from Colin Guthrie to Ryan J Ollos
Status: newassigned

I posted an explanation in comment:3:ticket:10251 of some recent errors on my part. Please test the latest check-in and report back, if you can.

The upgrade was failing without being entirely rolled back, and I'm not sure I know how to fix this issue. If you continue to have trouble, and you have not used the plugin yet and therefore don't have any data in your tables, I suggest just dropping the table.

DROP TABLE "main"."work_log"

comment:5 Changed 4 years ago by Ryan J Ollos

(In [12089]) Refs #10206, #10251:

  • Made the trunk compatible with Trac 0.11, by rolling out changes introduced in [9480] and later. There is nothing forcing us to adapt to the new Trac DB API at the moment, and it is easier to maintain a single code-base that is compatible back to Trac 0.11. The Trac DB API changed again in 1.0, so we are better off waiting and adapting to that API down the road anyway.
  • Attempted to cleanup the mess of import statements in every module. Modified the entry_points to adapt to these changes.
  • Made the XmlRpc lib an optional component. There is more work to do here still.

comment:6 Changed 4 years ago by Ryan J Ollos

(In [12090]) Refs #10206, #10251:

  • Renamed to, with the goal of making the source code layout match the standard for Trac plugins.
  • Fixed an error introduced in [12089] due to a missing import.

As in [12088], if your trac.ini file specifies the individual components to enable, you'll have to make the following change:

worklog.timeline_hook.worklogtimelineaddon = enabled


worklog.timeline.worklogtimelineeventprovider = enabled

comment:7 Changed 4 years ago by Ryan J Ollos

Status: assignednew

Modify Ticket

as new The owner will remain Ryan J Ollos.

Add Comment

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

Note: See TracTickets for help on using tickets.