Modify

Opened 15 years ago

Closed 12 years ago

#6426 closed defect (fixed)

Missing YAHOO logger widget JavaScript file

Reported by: olle.jonsson+trachacks@… Owned by: Ryan J Ollos
Priority: normal Component: TracTicketStatsPlugin
Severity: normal Keywords: javascript
Cc: Trac Release: 0.11

Description (last modified by Ryan J Ollos)

I'm using the stats plugin with Python 2.5, but it kind of works.

There is one thing, in the JavaScript, that bothers me: a YAHOO widget for logging is not included, and this fires a JS error.

<script type="text/javascript"> 
var myLogReader = new YAHOO.widget.LogReader("myLogger");
</script> 

To fix this, include this http://yui.yahooapis.com/2.5.2/build/logger/logger-min.js where you include the scripts.

Version used: /usr/lib/python2.5/site-packages/Tracticketstats-2.1-py2.5.egg

Attachments (1)

LoggerEnabled.png (44.8 KB) - added by Ryan J Ollos 14 years ago.

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by Ryan J Ollos

Attachment: LoggerEnabled.png added

comment:1 Changed 14 years ago by Ryan J Ollos

Description: modified (diff)

I can confirm the error on Ubuntu 10.04, Trac 0.11.7, TracTicketStatsPlugin 2.1, and Firefox 3.6.8.

The error I see in the Firefox Error Console is:

Error: YAHOO.widget.LogReader is not a constructor
Source File: http://localhost:8000/tracdev/ticketstats
Line: 164

I patched as you suggested:

  • greensauce.html

     
    1616    <script type="text/javascript" src="http://yui.yahooapis.com/2.5.2/build/element/element-beta-min.js"></script>
    1717    <script type="text/javascript" src="http://yui.yahooapis.com/2.5.2/build/datasource/datasource-beta-min.js"></script>
    1818    <script type="text/javascript" src="http://yui.yahooapis.com/2.5.2/build/json/json-min.js"></script>
     19        <script type="text/javascript" src="http://yui.yahooapis.com/2.5.2/build/logger/logger-min.js"></script>
    1920    <!-- OPTIONAL: Connection (enables XHR) -->
    2021    <script type="text/javascript" src="http://yui.yahooapis.com/2.5.2/build/connection/connection-min.js"></script>
    2122    <!-- Source files -->

However, now there is an error console at the bottom of the page:

I know nothing about javascript, so I need some guidance on the best way to fix this issue. Should we:

  • Add some sort of debug flag that causes the logger to be included?
  • Remove the logger code from the production version?

The logger code is form lines 145:149,

</script>
<div id="myLogger"></div>
<script type="text/javascript">
var myLogReader = new YAHOO.widget.LogReader("myLogger");
</script>

comment:2 Changed 14 years ago by Ryan J Ollos

Owner: changed from Prentice Wongvibulsin to Ryan J Ollos

Reassigning ticket to new maintainer.

comment:3 Changed 12 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

debug flag was added in [13191].

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.