[TAP5-1969] Excessive warnings from YUICompressor

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.3.4, 5.4
• Fix Version/s: 5.4
• Component/s: tapestry-yuicompressor
• Assignee: Howard Lewis Ship
• Reporter: Howard Lewis Ship
• Created: 06/Jul/12 17:10
• Updated: 22/Dec/12 01:10
• Resolved: 22/Dec/12 01:10

Description

In 5.3, tapestry-yuicompressor was change to emit warnings to the logger; however, one particular warning occurs so often, it drowns out useful information:

(zoneId);if(manager){ --> var <-- zoneContent=reply.zones[zoneId];manager
WARN ResourceMinimizer - Try to use a single 'var' statement per scope.
)

{var reply=transport.responseJSON; ---> var <--- newElement=new Element(this.element.tagName WARN ResourceMinimizer - Try to use a single 'var' statement per scope. :this.element.className}

); --> var <-- param={};param[this.below
WARN ResourceMinimizer - Try to use a single 'var' statement per scope.
if(Prototype.Browser.IE){ --> var <-- loaded=false;element.onreadystatechange=function(
WARN ResourceMinimizer - Try to use a single 'var' statement per scope.
)

{var _=T5._; ---> var <--- loaded=_(document.scripts).chain WARN ResourceMinimizer - Try to use a single 'var' statement per scope. this.rebuildURLIfIE).value(); ---> var <--- self=this;var topCallback=_(scripts WARN ResourceMinimizer - Try to use a single 'var' statement per scope. value();var self=this; ---> var <--- topCallback=_(scripts).chain() WARN ResourceMinimizer - Try to use a single 'var' statement per scope. ;}

var =T5.; --> var <-- loaded=_(document.styleSheets).chain
WARN ResourceMinimizer - Try to use a single 'var' statement per scope.
this.rebuildURLIfIE).value(); --> var <-- toLoad=_(stylesheets).chain()
WARN ResourceMinimizer - Try to use a single 'var' statement per scope.

This should be a special case (it is too prevalent to be useful) and suppressed.