[JBoss JIRA] (RF-13371) Clarify and fix Push destroy logic
by Lukáš Fryč (JIRA)
[ https://issues.jboss.org/browse/RF-13371?page=com.atlassian.jira.plugin.s... ]
Lukáš Fryč resolved RF-13371.
-----------------------------
Resolution: Done
> Clarify and fix Push destroy logic
> ----------------------------------
>
> Key: RF-13371
> URL: https://issues.jboss.org/browse/RF-13371
> Project: RichFaces
> Issue Type: Bug
> Security Level: Public(Everyone can see)
> Reporter: Lukáš Fryč
> Assignee: Lukáš Fryč
> Priority: Critical
> Fix For: 5.0.0.Alpha2
>
>
> I need to commend on the approach I take for destroying Push component which follows approximately what Widget Factory does.
> I have also found that the widget _destro method isn't called at all, because there is no such event as {{cleanDom}}.
> richfaces.js throws {{beforeDomClean}} and {{afterDomClean}}. The foremost is the right event to listen for for component cleanup.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
10 years, 7 months
[JBoss JIRA] (RF-13371) Clarify and fix Push destroy logic
by Lukáš Fryč (JIRA)
[ https://issues.jboss.org/browse/RF-13371?page=com.atlassian.jira.plugin.s... ]
Lukáš Fryč updated RF-13371:
----------------------------
Summary: Clarify and fix Push destroy logic (was: Refactored Push destroy logic is too aggresive and causes ProgressBar to fail)
Description:
I need to commend on the approach I take for destroying Push component which follows approximately what Widget Factory does.
I have also found that the widget _destro method isn't called at all, because there is no such event as {{cleanDom}}.
richfaces.js throws {{beforeDomClean}} and {{afterDomClean}}. The foremost is the right event to listen for for component cleanup.
was:
>From RF-13359:
{quote}
It took a while, but I isolated the single offending line:
https://github.com/richfaces/richfaces/blob/master/framework/src/main/res...
{code}
$(document).on('cleanDom.RICH', function(event, ui) {
if ($.contains(ui.target, widget.element)) {
widget._destroy();
}
});
{code}
[~lfryc] It's not clear to me what you are trying to achieve here, but the end result seems to be that the destroy call is too aggressive. Can you re-visit this please?
{panel:title=Note:|borderStyle=dashed|borderColor=#ccc|titleBGColor=#F7D6C1|bgColor=#FFFFCE}
The isolation was difficult, as the file in question should have nothing to do with the progressBar component. Hence why the effect was observed only had an effect when included via resource packaging.
{panel}
{quote}
> Clarify and fix Push destroy logic
> ----------------------------------
>
> Key: RF-13371
> URL: https://issues.jboss.org/browse/RF-13371
> Project: RichFaces
> Issue Type: Bug
> Security Level: Public(Everyone can see)
> Reporter: Lukáš Fryč
> Assignee: Lukáš Fryč
> Priority: Critical
> Fix For: 5.0.0.Alpha2
>
>
> I need to commend on the approach I take for destroying Push component which follows approximately what Widget Factory does.
> I have also found that the widget _destro method isn't called at all, because there is no such event as {{cleanDom}}.
> richfaces.js throws {{beforeDomClean}} and {{afterDomClean}}. The foremost is the right event to listen for for component cleanup.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
10 years, 7 months
[JBoss JIRA] (RF-10131) Improve SelectItem(s) handling compatibility with JSF 2.0
by J W (JIRA)
[ https://issues.jboss.org/browse/RF-10131?page=com.atlassian.jira.plugin.s... ]
J W edited comment on RF-10131 at 11/22/13 4:49 AM:
----------------------------------------------------
Is this already finished?
Whenever i try to add an Item with f:selectItem with itemDisabled="true", the item will be always treated like every other item.
I am using RF 4.3.1
was (Author: jigg4):
Is this already finished?
Whenever i try to add an Item with f:selectItem with itemDisabled="true", the item will be always treated like every other item.
> Improve SelectItem(s) handling compatibility with JSF 2.0
> ---------------------------------------------------------
>
> Key: RF-10131
> URL: https://issues.jboss.org/browse/RF-10131
> Project: RichFaces
> Issue Type: Enhancement
> Security Level: Public(Everyone can see)
> Components: compatibility
> Environment: myfaces 2.0.3, richfaces 4.0 trunk rev. 20883
> Reporter: Martin Kočí
> Assignee: Gleb Galkin
> Fix For: 4.0.0.Milestone6
>
>
> 1) JSF 2.0 specifies new attributes itemLabel, itemDescription and itemDisabled for f:selectItem(s). Unfortunately those attributes are not part of public API but every renderkit must reimplement it again in own code base. RichFaces org.richfaces.component.util.SelectUtils.getSelectItems(FacesContext, UIComponent) currently support only old JSF 1.2 style (SelectItem instances are obtained from value="#{}").
> Implement support for f:selectItems value="#{}" var="variable"
> itemLabel="#{...}" itemValue="#{...}"/>
> 2) null or empty SelectItems handling: o.j.rf.SelectUtils throws an exception if no SelectItem(s) are available. If think this is too agressive and not very useful for user. I suggest to use myfaces-like solution: a warning in development stage like:
> WARNING: ValueExpression #{de.bier.humpen} of UISelectItems with component-path {Component-Path : [Class: javax.faces.component.UIViewRoot,ViewId: /X500Form.xhtml][Class: javax.faces.component.html.HtmlBody, ... ][Class: javax.faces.component.UISelectItems,Id: selectItemsId} does not reference an Object of type SelectItem, array, Iterable or Map, but of type: null
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
10 years, 7 months
[JBoss JIRA] (RF-10131) Improve SelectItem(s) handling compatibility with JSF 2.0
by J W (JIRA)
[ https://issues.jboss.org/browse/RF-10131?page=com.atlassian.jira.plugin.s... ]
J W commented on RF-10131:
--------------------------
Is this already finished?
Whenever i try to add an Item with f:selectItem with itemDisabled="true", the item will be always treated like every other item.
> Improve SelectItem(s) handling compatibility with JSF 2.0
> ---------------------------------------------------------
>
> Key: RF-10131
> URL: https://issues.jboss.org/browse/RF-10131
> Project: RichFaces
> Issue Type: Enhancement
> Security Level: Public(Everyone can see)
> Components: compatibility
> Environment: myfaces 2.0.3, richfaces 4.0 trunk rev. 20883
> Reporter: Martin Kočí
> Assignee: Gleb Galkin
> Fix For: 4.0.0.Milestone6
>
>
> 1) JSF 2.0 specifies new attributes itemLabel, itemDescription and itemDisabled for f:selectItem(s). Unfortunately those attributes are not part of public API but every renderkit must reimplement it again in own code base. RichFaces org.richfaces.component.util.SelectUtils.getSelectItems(FacesContext, UIComponent) currently support only old JSF 1.2 style (SelectItem instances are obtained from value="#{}").
> Implement support for f:selectItems value="#{}" var="variable"
> itemLabel="#{...}" itemValue="#{...}"/>
> 2) null or empty SelectItems handling: o.j.rf.SelectUtils throws an exception if no SelectItem(s) are available. If think this is too agressive and not very useful for user. I suggest to use myfaces-like solution: a warning in development stage like:
> WARNING: ValueExpression #{de.bier.humpen} of UISelectItems with component-path {Component-Path : [Class: javax.faces.component.UIViewRoot,ViewId: /X500Form.xhtml][Class: javax.faces.component.html.HtmlBody, ... ][Class: javax.faces.component.UISelectItems,Id: selectItemsId} does not reference an Object of type SelectItem, array, Iterable or Map, but of type: null
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
10 years, 7 months