You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Andrew Robinson (JIRA)" <de...@myfaces.apache.org> on 2008/01/05 00:10:33 UTC

[jira] Created: (TRINIDAD-886) Add the ability to the show* components to always get their disclosed property from EL

Add the ability to the show* components to always get their disclosed property from EL
--------------------------------------------------------------------------------------

                 Key: TRINIDAD-886
                 URL: https://issues.apache.org/jira/browse/TRINIDAD-886
             Project: MyFaces Trinidad
          Issue Type: New Feature
          Components: Components
    Affects Versions: 1.0.5-core, 1.2.5-core
            Reporter: Andrew Robinson
            Assignee: Andrew Robinson
            Priority: Minor


The disclosed property of showDetail components are updated by the renderers. It would be extremely helpful to stop this behavior so that EL is always used.

Example of someone wanting to disclose only one item:
public class Bean {
  private int disclosedIndex = -1;
  public void handleDisclose(DisclosureEvent event) {
    // manually update the index
  }
  ...
}

<tr:iterator var="_item" value="#{bean.items}" varStatus="_varStatus">
  <tr:showDetail disclosedText="#{_item.title}" undisclosedText="#{_item.title}":
        disclosed="#{bean.disclosedItem eq _varStatus.index}"
                 disclosureListener="#{bean.handleDisclose}">
    <af:outputFormatted value="#{_item.desc}"/>
  </af:showDetail>
</af:iterator>

Proposal:

A new attribute to be added to showDetial* components:
public boolean isDisclosedTransient() {}

This should default to false. If set, the renderer should never set the disclosed property locally, or the faces bean property value should ignore sets. This way the disclosed property set in the UI would always be used to determine disclosed state.




-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TRINIDAD-886) Add the ability to the show* components to always get their disclosed property from EL

Posted by "Andrew Robinson (JIRA)" <de...@myfaces.apache.org>.
    [ https://issues.apache.org/jira/browse/TRINIDAD-886?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12557150#action_12557150 ] 

Andrew Robinson commented on TRINIDAD-886:
------------------------------------------

Committed revision 610278 in the 1.2 trunk

> Add the ability to the show* components to always get their disclosed property from EL
> --------------------------------------------------------------------------------------
>
>                 Key: TRINIDAD-886
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-886
>             Project: MyFaces Trinidad
>          Issue Type: New Feature
>          Components: Components
>    Affects Versions: 1.0.5-core, 1.2.5-core
>            Reporter: Andrew Robinson
>            Assignee: Andrew Robinson
>            Priority: Minor
>             Fix For: 1.0.5-core, 1.2.5-core
>
>
> The disclosed property of showDetail components are updated by the renderers. It would be extremely helpful to stop this behavior so that EL is always used.
> Example of someone wanting to disclose only one item:
> public class Bean {
>   private int disclosedIndex = -1;
>   public void handleDisclose(DisclosureEvent event) {
>     // manually update the index
>   }
>   ...
> }
> <tr:iterator var="_item" value="#{bean.items}" varStatus="_varStatus">
>   <tr:showDetail disclosedText="#{_item.title}" undisclosedText="#{_item.title}":
>         disclosed="#{bean.disclosedItem eq _varStatus.index}"
>                  disclosureListener="#{bean.handleDisclose}">
>     <af:outputFormatted value="#{_item.desc}"/>
>   </af:showDetail>
> </af:iterator>
> Proposal:
> A new attribute to be added to showDetial* components:
> public boolean isDisclosedTransient() {}
> This should default to false. If set, the renderer should never set the disclosed property locally, or the faces bean property value should ignore sets. This way the disclosed property set in the UI would always be used to determine disclosed state.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (TRINIDAD-886) Add the ability to the show* components to always get their disclosed property from EL

Posted by "Andrew Robinson (JIRA)" <de...@myfaces.apache.org>.
     [ https://issues.apache.org/jira/browse/TRINIDAD-886?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Robinson resolved TRINIDAD-886.
--------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.0.5-core

Committed revision 610275 in the trunk

> Add the ability to the show* components to always get their disclosed property from EL
> --------------------------------------------------------------------------------------
>
>                 Key: TRINIDAD-886
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-886
>             Project: MyFaces Trinidad
>          Issue Type: New Feature
>          Components: Components
>    Affects Versions: 1.0.5-core, 1.2.5-core
>            Reporter: Andrew Robinson
>            Assignee: Andrew Robinson
>            Priority: Minor
>             Fix For: 1.0.5-core
>
>
> The disclosed property of showDetail components are updated by the renderers. It would be extremely helpful to stop this behavior so that EL is always used.
> Example of someone wanting to disclose only one item:
> public class Bean {
>   private int disclosedIndex = -1;
>   public void handleDisclose(DisclosureEvent event) {
>     // manually update the index
>   }
>   ...
> }
> <tr:iterator var="_item" value="#{bean.items}" varStatus="_varStatus">
>   <tr:showDetail disclosedText="#{_item.title}" undisclosedText="#{_item.title}":
>         disclosed="#{bean.disclosedItem eq _varStatus.index}"
>                  disclosureListener="#{bean.handleDisclose}">
>     <af:outputFormatted value="#{_item.desc}"/>
>   </af:showDetail>
> </af:iterator>
> Proposal:
> A new attribute to be added to showDetial* components:
> public boolean isDisclosedTransient() {}
> This should default to false. If set, the renderer should never set the disclosed property locally, or the faces bean property value should ignore sets. This way the disclosed property set in the UI would always be used to determine disclosed state.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.