Opened 6 years ago

Closed 4 years ago

Last modified 4 years 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 6 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 6 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 6 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 4 years ago by rjollos

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

comment:5 Changed 4 years ago by rjollos

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

comment:6 Changed 4 years 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 4 years ago by rjollos

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

comment:8 Changed 4 years 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 4 years ago by rjollos

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

comment:10 Changed 4 years 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.