ExtendedDataTable - multiple onselectionchange events

[joblini]

Hello,

I am using ExtendedDataTable from nightly snapshot 3.3.0-20081014.041253

When I click on a row, it generates an onselectionchange event, as expected. However, any interaction at all with the selectedrow generates more onselectionchange events.

For example:

- Click on a row which is already selected : generates onselectionchange event.

- Double click a row : generates 2 onselectionchange events.

- Choose a value from a h:selectOneMenu which is on a selected row : generates 2 onselectionchangeevents.

Since I am rerendering a child table during the onselectionchange event (using a4j:support), all of these redundant events make the user interaction slow. Am I doing something wrong? Any help would be greatly appreciated!

<h:form id="productionsForm">

 <rich:extendedDataTable id="productionsTable"
 value="#{productionListController.resultList}" var="production"
 height="#{productionListController.tableHeight}px"
 onRowMouseOver="this.style.backgroundColor='#{a4jSkin.additionalBackgroundColor}'"
 onRowMouseOut="this.style.backgroundColor=''"
 selectionMode="single"
 selection="#{productionListController.selection}"
 selectedClass="rich-sdt-row-selected"
 rowClasses="rvgRowOne,rvgRowTwo"
 >
 <a4j:support event="onselectionchange"
 ajaxSingle="true" action="#{productionListController.onSelectionChange()}"
 reRender="commonFramesDiv" limitToList="true" />

 <rich:column width="150px">

 <f:facet name="header">#{messages['org.agritrace.ProductionType']} <span class="required">*</span></f:facet>

 <s:decorate template="/layout/editTd.xhtml">
 <h:selectOneMenu id="productionTypeSelect" value="#{production.type}" binding="#{productionInputs.getInputMap().type}"
 required="true" styleClass="tableInput" label="#{messages['org.agritrace.ProductionType']}" >
 <s:selectItems value="#{productionTypeList.resultList}"
 var="productionType"
 label="#{messages[productionType.resourceKey]}"
 noSelectionLabel="#{production.isEmpty() ? messages['org.agritrace.New']:null}" />
 </h:selectOneMenu>
 </s:decorate>

 </rich:column>

 <rich:column width="150px">

 <f:facet name="header">#{messages['org.agritrace.Species']} <span class="required">*</span></f:facet>

 <s:decorate template="/layout/editTd.xhtml">
 <h:selectOneMenu id="speciesSelect" value="#{production.species}" binding="#{productionInputs.getInputMap().species}"
 required="true" styleClass="tableInput" label="#{messages['org.agritrace.productionSpecies']}">
 <s:selectItems value="#{speciesList.resultList}" var="species"
 label="#{messages[species.resourceKey]}"
 noSelectionLabel="#{messages['org.agritrace.PleaseSelect']}" />
 </h:selectOneMenu>
 </s:decorate>

 </rich:column>

 </rich:extendedDataTable>

</h:form>

[ilya_shaikovsky]

Thanks for youur feedback!
https://jira.jboss.org/jira/browse/RF-4612

[anders.norgaard]

A somwhat related question: Shouldn't the row be unselected when clicking a selected row? ( this is with richfaces-*-3.3.0-20081008.041111-30.jar )

Best
Anders

[joblini]

"ilya_shaikovsky" wrote:
Thanks for youur feedback!
https://jira.jboss.org/jira/browse/RF-4612

Great, thanks very much! I voted on this issue, keeping my fingers crossed :-)

[piotr.buda]

"joblini" wrote:
- Click on a row which is already selected : generates onselectionchange event.

This is fixed.

"joblini" wrote:
- Double click a row : generates 2 onselectionchange events.

I couldn't doubleclick a row with onselectionchange defined.

"joblini" wrote:
- Choose a value from a h:selectOneMenu which is on a selected row : generates 2 onselectionchangeevents.

This requires a workaround:

<rich:column>
 <f:facet name="header">
 <h:outputText value="#{msg['select']}" />
 </f:facet>
 <h:selectOneMenu onclick="Event.stop(event);">
 <f:selectItem itemValue="1" itemLabel="1" />
 <f:selectItem itemValue="2" itemLabel="2" />
 <f:selectItem itemValue="3" itemLabel="3" />
 </h:selectOneMenu>
</rich:column>

"biehl" wrote:
Shouldn't the row be unselected when clicking a selected row?

We'll think about it, but honestly I'd leave it as it is at the moment.