Details
-
Type: Improvement
-
Status: Closed
-
Priority: Major
-
Resolution: Cannot Reproduce
-
Affects Version/s: 4.0.BETA
-
Fix Version/s: 4.0
-
Component/s: ACE-Components
-
Labels:None
-
Environment:Any
-
Assignee Priority:P1
-
Affects:Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration
Description
A number of ACE components have ace:ajax events whose default execute value is @all. In practice, this is inconvenient and unnecessary. In most cases, the default value should simply be @this. The purpose of this JIRA is to survey the ACE components to identify those using a default value of @all in their ace:ajax events and change it to @this, if there are no other special circumstances that would conflict with this change.
Activity
Arturo Zambrano
created issue -
Arturo Zambrano
made changes -
Field | Original Value | New Value |
---|---|---|
Assignee | Arturo Zambrano [ artzambrano ] |
Arturo Zambrano
made changes -
Fix Version/s | 4.0 [ 11382 ] |
Arturo Zambrano
made changes -
Link | This issue blocks ICE-10127 [ ICE-10127 ] |
Arturo Zambrano
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Ken Fyten
made changes -
Affects | Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #41822 | Thu Jul 17 18:31:32 MDT 2014 | art.zambrano | |
Files Changed | ||||
MODIFY
/icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/autocompleteentry/autocompleteentry.js
|
Ken Fyten
made changes -
Resolution | Fixed [ 1 ] | |
Status | Resolved [ 5 ] | Reopened [ 4 ] |
Assignee Priority | P1 [ 10010 ] |
Ken Fyten
made changes -
Comment |
[ In addition:
{quote}ButtonGroup > Overview Test fails manually, Checkbox buttons can not be de-selected. CheckBox button > Overview, Custom Styling Test fails manually, Checkbox buttons can not be de-selected.{quote} ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #41824 | Mon Jul 21 14:32:58 MDT 2014 | art.zambrano | |
Files Changed | ||||
MODIFY
/icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/radiobutton/RadioButtonRenderer.java
MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/radiobutton/radiobutton.js |
Arturo Zambrano
made changes -
Status | Reopened [ 4 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #41827 | Mon Jul 21 15:22:16 MDT 2014 | art.zambrano | |
Files Changed | ||||
MODIFY
/icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/radiobutton/RadioButtonRenderer.java
|
Deryk Sinotte
made changes -
Resolution | Fixed [ 1 ] | |
Status | Resolved [ 5 ] | Reopened [ 4 ] |
Assignee | Arturo Zambrano [ artzambrano ] | Mircea Toma [ mircea.toma ] |
Mircea Toma
made changes -
Status | Reopened [ 4 ] | Resolved [ 5 ] |
Resolution | Cannot Reproduce [ 5 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Committed fix to 4.0 trunk at revision 41818.
After surveying Meta classes, Renderer classes and Javascript resources, this is what was found:
The default execute values of the ajax events events above were changed from @all to @this, and the ice.s() calls were changed to ice.se() calls in renderers and in Javascript resources. However, there were a few exceptions.
The ace:tabSet component wasn't modified, because it requires to send execute=@all for the tab set proxy feature to work. This is the way this component was designed. Likewise, the ace:dataExporter component wasn't modified either because of the use case where a table is editable (or simply contains input components) and the user edits some cells and then activates the export function; this way the changes just made will be reflected in the exported document.