Modify

Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#5322 closed defect (fixed)

Load yahooapis.com locally

Reported by: Martin Owned by: Ryan J Ollos
Priority: normal Component: TracMetrixPlugin
Severity: major Keywords:
Cc: Trac Release: 0.11

Description

I just tried both 0.1.5, 0.1.6 and rev. 5935. The first one generates useful graphs, the later versions not. I assume, that two issues are involved:

  1. I do not, by default, allow my web browser to load JS code from other sites (here: yahooapis.com). As far as I know, the yui library is free software, so it could be installed locally on the trac server.
  2. My web browser does not have a flash plugin installed to display the charts.swf.

The plugin is really cool, but the flash stuff in newer versions makes it unusable for me.

Attachments (0)

Change History (6)

comment:1 Changed 8 years ago by Laurent DORE

A side-effect of the remote Yahoo charts:
The plugin does not generate graphs when used on our intranet, as it does not have access to yahooapis.com.

Installing the yui library locally would probably solve that.

comment:2 Changed 7 years ago by Ryan J Ollos

Owner: changed from Bhuricha Deen Sethanandha to Ryan J Ollos
Summary: Regression: 0.1.5 works fine, later versions not (yahoo/flash stuff?)Load yahooapis.com locally

I'll implement load yahooapis.com locally. That makes sense.

As for the original reporter's refusal to use flash, there is not much I can do about that.

comment:3 Changed 7 years ago by 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?

comment:4 Changed 7 years ago by Ryan J Ollos

(In [8627]) Added option yui_base_url that allows the location of the Yahoo APIs to be specified. Refs #5322.

comment:5 Changed 7 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

(In [8628]) Merged [8627] into 0.12 branch. Fixes #5322.

comment:6 Changed 7 years ago by Ryan J Ollos

See also #3921 for the same issue with another plugin.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.