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

[simon@…]

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.

[simon@…]

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

[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.

[Prentice Wongvibulsin]

Applied patch in [6379].

Thanks!

[anonymous]

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

[Ryan J Ollos]

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

[Ryan J Ollos]

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

[Ryan J Ollos]

This feature was implemented for the TracMetrixPlugin in #5322.

[Ryan J Ollos]

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?

[Ryan J Ollos]

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

[Ryan J Ollos]

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