Modify

Opened 2 years ago

Last modified 19 months ago

#10206 new defect

Duplicate column name "comment"

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

Description (last modified by rjollos)

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:

Error

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
tracrpc.search.searchrpc = 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
tracrpc.wiki.wikirpc = enabled
tracrpc.xml_rpc.xmlrpcprotocol = enabled
whiteboard.web_ui.whiteboardmodule = enabled

Attachments (0)

Change History (7)

comment:1 Changed 2 years ago by rjollos

  • Description modified (diff)

comment:2 follow-up: Changed 2 years ago by rjollos

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 2 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 2 years ago by rjollos

  • Owner changed from coling to rjollos
  • Status changed from new to assigned

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 2 years ago by rjollos

(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 2 years ago by rjollos

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

  • Renamed timeline_hook.py to timeline.py, 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 19 months ago by rjollos

  • Status changed from assigned to new

Add Comment

Modify Ticket

Action
as new The owner will remain rjollos.
Author


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

 
Note: See TracTickets for help on using tickets.