Re: T5 Error in 5.0.5 palette.js

[Joel Wiegman <Jo...@btservices.com>]

Just tacking onto the thread that a patch is available for the problem
discussed in this thread:

https://issues.apache.org/jira/browse/TAPESTRY-1745

Seems like a pretty unintrusive fix... Anxiously awaiting for a committer to
test and include...



Howard Lewis Ship wrote:
> 
> Please add this as an issue in JIRA if you have not done so already.
> 
> On 8/28/07, Daniel Jue <te...@gmail.com> wrote:
>>
>> When using the Palette component, I am getting a Javascript error in IE,
>>
>> Line: 183
>> Char: 1
>> Error: Expected identifier, string or number
>> Code: 0
>>
>> ............
>>   reorderSelected : function(movers,  before) {
>>     movers.each (function(option) { this.selected.add(option, before);
>> }.bind(this));
>>
>>     this.updateHidden();
>>     this.updateButtons();
>>   },
>> };
>>
>> Line 183 is the line after the };
>>
>> I think the extra comma after the end of the reorderSelected code block
>> is
>> to blame.
>> For me it was causing an error in IE6 which prevented me from using the
>> select button in the Palette.
>>
>> I removed the "extra" comma, but then when I highlight some items and
>> click
>> the select button, I get a Type Mismatch error on the comma at the end of
>> this function:
>>
>>   moveOption : function(option, to, atEnd) {
>>     var before = null;
>>
>>     if (!atEnd) {
>>       var optionOrder = this.valueToOrderIndex[option.value];
>>       var candidate = $A(to.options).find(function (o) {
>>             return this.valueToOrderIndex[o.value] > optionOrder;
>>             }.bind(this));
>>       if (candidate) before = candidate;
>>     }
>>
>>     to.add(option, before);
>>   },
>>
>> Is the T5 Palette working fine for anyone using IE?
>> The IE Version I am testing is
>> 6.0.2900.2180.xpsp_sp2_gdr.070227-2254
>>
>>
>> Everything works fine in Firefox, of course.
>>
> 
> 
> 
> -- 
> Howard M. Lewis Ship
> Partner and Senior Architect at Feature50
> 
> Creator Apache Tapestry and Apache HiveMind
> 
> 

-- 
View this message in context: http://www.nabble.com/T5-Error-in-5.0.5-palette.js-tf4344484.html#a13042126
Sent from the Tapestry - User mailing list archive at Nabble.com.


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tapestry.apache.org
For additional commands, e-mail: users-help@tapestry.apache.org