Opened 14 years ago

Last modified 11 years ago

#6426 closed defect

Missing YAHOO logger widget JavaScript file — at Version 1

Reported by: olle.jonsson+trachacks@… Owned by: Prentice Wongvibulsin
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

Change History (2)

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>
Note: See TracTickets for help on using tickets.