Opened 4 years ago

Closed 22 months ago

Last modified 22 months ago

#8140 closed task (fixed)

Merge codebase with TracTicketStatsPlugin?

Reported by: rjollos Owned by: rjollos
Priority: normal Component: TicketStatsMacro
Severity: normal Keywords:
Cc: hasienda Trac Release: 0.11


I'm working on fixing open issues with the TracTicketStatsPlugin, and since these two plugins share so much code, I was wondering how you'd feel about packaging them together.

Attachments (0)

Change History (10)

comment:1 Changed 4 years ago by anonymous

Yes, will be wonderfull. Most of the code was taken from that plugin. I should have done when I created the plugin but I didn't felt very comfortable with the API trac.

Please tell me if you need some help.

comment:2 Changed 4 years ago by rjollos

If the following is okay with you, I propose to:

Later on, I intend to merge the TracTicketStatsPlugin with the TracMetrixPlugin, and rename it ProjectMetricsPlugin. I also hope to add functionality and merge additional plugin functionality, pending buy-in by other plugin authors.

comment:3 Changed 4 years ago by rjollos

I've been committing to TracTicketStatsPlugin for some time, but did not fully go through the correct channels to adopt the plugin, so I've initiated that process in #7537.

comment:4 Changed 22 months ago by rjollos

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

comment:5 Changed 22 months ago by rjollos

(In [13096]) Refs #8140: Fixed PEP-0008 violations.

comment:6 Changed 22 months ago by rjollos

(In [13099]) Refs #8140: Merged the TicketStatsMacro into the TracTicketStatsPlugin. The TicketStatsMacro is now deprecated and development will continue in the TracTicketStatsPlugin codeline.

Bumped the version of the TracTicketStatsPlugin to 3.0.0dev and set the default Yahoo YUI API version to be 2.9.0, which is the version that the TicketStatsMacro used.

comment:7 Changed 22 months ago by rjollos

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:8 Changed 22 months ago by rjollos

(In [13108])

Refs #8140:

  • Changed macro option name from ticketstatsmacro to ticketstats.
  • Moved macro template to an external file. When only Trac 0.12 and later is supported, and add_script_data is available, the template can be transformed to a JS script.

comment:9 Changed 22 months ago by rjollos

(In [13110]) Refs #8140: Tag version 2.2.0 of TracTicketStatsPlugin.

comment:10 Changed 22 months ago by rjollos

(In [13197]) Refs #8140: Added requirement of TracAdvParseArgsPlugin since the code integrated from the TicketStatsMacro requires it. However, this dependency might be eliminated later on.

Add Comment

Modify Ticket

as closed The owner will remain rjollos.
The resolution will be deleted. Next status will be 'reopened'.

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

Note: See TracTickets for help on using tickets.