Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Duplicate
-
Affects Version/s: 3.0.1
-
Fix Version/s: 3.1
-
Component/s: ACE-Components
-
Labels:None
-
Environment:Any
Description
Working on ICE-7895, I noticed a Javascript error being thrown. It turned out not to be caused by the jQuery update. The bug was introduced with the ICE-7501 improvements. The wrong function is being called when supplying additional parameters to the ajax request function. In resizable.js we have the following lines of code:
if (behaviour) {
ice.ace.ab(ice.ace.extendAjaxArguments(
behaviour,
ice.ace.extendAjaxArguments(options))
);
} else ice.ace.AjaxRequest(options);
So, ice.ace.extendAjaxArguments() is being called again and with not enough arguments. The Javascript error is thrown because an expect argument is undefined. The correct code would look like this:
if (behaviour) {
ice.ace.ab(ice.ace.extendAjaxArguments(
behaviour,
ice.ace.removeExecuteRenderOptions(options))
);
} else ice.ace.AjaxRequest(options);
This can be reproduced with the listener demo if ace:resizable in the ICEfaces showcase app.
This problem can also be seen in progresbar.js at two different points. Possibly this problem could also be present in other .js files, so it would be necessary to survey all files.
if (behaviour) {
ice.ace.ab(ice.ace.extendAjaxArguments(
behaviour,
ice.ace.extendAjaxArguments(options))
);
} else ice.ace.AjaxRequest(options);
So, ice.ace.extendAjaxArguments() is being called again and with not enough arguments. The Javascript error is thrown because an expect argument is undefined. The correct code would look like this:
if (behaviour) {
ice.ace.ab(ice.ace.extendAjaxArguments(
behaviour,
ice.ace.removeExecuteRenderOptions(options))
);
} else ice.ace.AjaxRequest(options);
This can be reproduced with the listener demo if ace:resizable in the ICEfaces showcase app.
This problem can also be seen in progresbar.js at two different points. Possibly this problem could also be present in other .js files, so it would be necessary to survey all files.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion