[richfaces-issues] [JBoss JIRA] (RF-13117) ExtendedDataTable Sorting resets $(window).resize Events

[Brian Leathem (JIRA)]

    [ https://issues.jboss.org/browse/RF-13117?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12796655#comment-12796655 ] 

Brian Leathem commented on RF-13117:
------------------------------------

The reason this is null is because of the sequence of the pair of lines in the init function:
{code}
jQuery(document).ready(jQuery.proxy(this.initialize, this));
this.resizeEventName = "resize.rf.edt." + this.id;
{code}

When the EDT is first intialized, the _initialize_ callback isn't invoked until the ready event is fired - after the _resizeEventName_ property has been set.  However after an AJAX update, the _ready_ callback's are invoked immediately, hence the empty resizeEventName property.

In resolving this issue we should *both* re-order the LoCs, as well as implement the null check recommended by [~moosetraveller].
                
> ExtendedDataTable Sorting resets $(window).resize Events
> --------------------------------------------------------
>
>                 Key: RF-13117
>                 URL: https://issues.jboss.org/browse/RF-13117
>             Project: RichFaces
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: component-tables
>    Affects Versions: 4.3.3
>         Environment: Tomcat 7
> JBoss 7.1
>            Reporter: Thomas Zuberbuehler
>            Assignee: Brian Leathem
>              Labels: regression
>             Fix For: 4.3.4
>
>   Original Estimate: 15 minutes
>  Remaining Estimate: 15 minutes
>
> Sorting removes other resize events.
> At some point following code is called:
> {code}
> // extendedDataTable.js
> deActivateResizeListener: function() {
>   $(window).off(this.resizeEventName);
> }
> {code}
> Notice that this.resizeEventName is undefined.
> Before code is invoked (Chrome JavaScript console)
> {code}
> $._data(window,'events').resize
> [Object, Object]
> {code}
> After invoking
> {code}
> $._data(window,'events').resize
> TypeError: Cannot read property 'resize' of undefined
> {code}
> http://stackoverflow.com/questions/18045891/how-to-check-if-onresize-method-is-set

--
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