Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 2.0.2
-
Fix Version/s: 2.1-Beta, 3.0, EE-2.0.0.GA_P01
-
Component/s: ACE-Components, Framework
-
Labels:None
-
Environment:ICEfaces 2, ACE, Portlet, Portal
Description
When running the ACE Calendar component as a portlet, the popup does not display when you click the calendar icon. Switching to non pop-up mode displays the calendar and it's fully functional. Switching back to pop-up mode and it still does not display.
Activity
| Field | Original Value | New Value |
|---|---|---|
| Salesforce Case | [] | |
| Fix Version/s | EE-2.0.0.GA_P01 [ 10271 ] | |
| Fix Version/s | 2.1 [ 10241 ] | |
| Affects Version/s | EE-2.0.0.GA [ 10263 ] | |
| Assignee | Arturo Zambrano [ artzambrano ] |
| Summary | ACE Calendar doesn't display in pop-up mode when running as a portlet | ACE Calendar doesn't display in pop-up mode when running as a portlet and ace:animation portlet is present on the page |
| Salesforce Case | [] |
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #24571 | Fri May 13 14:07:41 MDT 2011 | art.zambrano | |
| Files Changed | ||||
MODIFY
/icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/ace/component/resources/org.icefaces.component.util/component.js
MODIFY
/icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/ace/component/resources/org.icefaces.component.animation/animation.js
|
The calendar code tries to animate the opening/closing of the popup. There was a problem when trying to access the nodes to be animated, since the Y.one() function is used in the animation code. This function also accepts selectors, so when one passes an ID string that contains colons, it could be interpreted as a selector and not as an ID. This is solved by escaping all colons in the string. The problem in this case was that only one occurrence of a colon in the string was being escaped, so it wasn't possible to find nodes that had two or more colons in their IDs.
Committed fix to 2.0 maintenance branch to revision 24571.
(this fix along with other related fixes and refactoring will be committed to the 2.0 trunk as part of ICE-6860)
Show
Arturo Zambrano
added a comment - The calendar code tries to animate the opening/closing of the popup. There was a problem when trying to access the nodes to be animated, since the Y.one() function is used in the animation code. This function also accepts selectors, so when one passes an ID string that contains colons, it could be interpreted as a selector and not as an ID. This is solved by escaping all colons in the string. The problem in this case was that only one occurrence of a colon in the string was being escaped, so it wasn't possible to find nodes that had two or more colons in their IDs.
Committed fix to 2.0 maintenance branch to revision 24571.
(this fix along with other related fixes and refactoring will be committed to the 2.0 trunk as part of ICE-6860 )
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Fix Version/s | 2.1-Beta [ 10291 ] | |
| Fix Version/s | 2.1 [ 10241 ] |
| Fix Version/s | 3.0 [ 10241 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
The DateTime failure only seems to occur when DateTime is on the same portal page as the Animation portlet. When that happens, Chrome console shows the following JS error:
ace1:1Uncaught TypeError: Cannot call method 'cloneNode' of null
eace1:1
gace1:1
(anonymous function)ace1:1
ice.yui3.useace1:1
ice.animation.registerace1:1
ice.animation.getAnimationace1:1
(anonymous function)ace1:1
E.Subscriber._notifyace1:8
E.Subscriber.notifyace1:9
E.CustomEvent._notifyace1:8
I.bindace1:8