Ticket #3921 (closed enhancement: fixed)

Opened 5 years ago

Last modified 3 years ago

[Patch] Configuration options for default values and YUI base URL

Reported by: simon@westpoint.ltd.uk Assigned to: echo0101
Priority: normal Component: TracTicketStatsPlugin
Severity: normal Keywords:
Cc: simon@westpoint.ltd.uk Trac Release: 0.11

Description

To get TracTicketStatsPlugin working on our intranet I needed to change the URLs to various YUI scripts to a local YUI setup. I'd like this to be configurable.

I have attached a patch attached that adds a configuration option, yui_base_url, as well as options for the default interval and days to show on the graph, under a ticketstats section.

Attachments

ticketstats-r7633.patch.tgz (1.7 kB) - added by anonymous on 02/17/10 16:51:47.

Change History

10/16/08 20:33:33 changed by simon@westpoint.ltd.uk

Attachment got rejected as spam because of too many external URLs. Find the patch at: http://bleah.co.uk/~simon/stuff/ticketstats-conf.diff

10/17/08 11:08:47 changed by anonymous

Added a '/' after yui_base_url in the template, so both http://yui.yahooapis.com/2.5.2/ and http://yui.yahooapis.com/2.5.2 work.

08/14/09 21:50:13 changed by echo0101

  • status changed from new to closed.
  • resolution set to fixed.

Applied patch in [6379].

Thanks!

02/17/10 16:50:45 changed by anonymous

  • status changed from closed to reopened.
  • resolution deleted.

The trunk seems to haven't applied the patch. I attach the patch file for the review 7633. Thanks!

02/17/10 16:51:47 changed by anonymous

  • attachment ticketstats-r7633.patch.tgz added.

08/23/10 01:31:47 changed by rjollos

  • summary changed from Configuration options for default values and YUI base URL to [Patch] Configuration options for default values and YUI base URL.

09/05/10 02:07:50 changed by rjollos

It appears that the code on the trunk defines the option yui_base_url, but then does not use it.

09/05/10 03:16:24 changed by rjollos

The syntax used in this patch is clearsilver templating syntax. The replacement <?cs var:yui_base_url ?> -> ${yui_base_url} should be made.

09/05/10 03:30:14 changed by rjollos

This feature was implemented for the TracMetrixPlugin in #5322.

09/05/10 04:05:52 changed by rjollos

This will be added in the next commit. Can someone let me know what is the typical location that the Yahoo APIs would be placed in a local installation?

09/05/10 10:25:22 changed by rjollos

(In [8638]) Added option to allow YUI base URL to be specified in trac.ini. Refs #3921.

09/05/10 10:47:18 changed by rjollos

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [8642]) Re-implementing changes from [8414] on new 0.12 branch. Fixes #3921.

Add/Change #3921 ([Patch] Configuration options for default values and YUI base URL)




Change Properties
Action