From: MatmaRex <matma.rex@gmail.com>
Date: Sun, 16 Jun 2013 13:10:28 +0000 (+0200)
Subject: jquery.makeCollapsible: fix jQuery memory leak
X-Git-Tag: 1.31.0-rc.0~19255^2
X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/membres/message.php?a=commitdiff_plain;h=bf6199135c03ac46926e3f4ba249fe924f8c606a;p=lhc%2Fweb%2Fwiklou.git

jquery.makeCollapsible: fix jQuery memory leak

jQuery saves event data in jQuery.cache and magically clears it when
relevant elements are removed from DOM. However, if an element is
created, has event handlers attached and is never added to the DOM,
it never gets a chance to clear the data, resulting in a memory leak.

Only build the default toggle link when needed to avoid this.

Bug: 49626
Change-Id: I0c92e5c28a66c6a6469e107593dc9b6d3baa8a10
---

diff --git a/resources/jquery/jquery.makeCollapsible.js b/resources/jquery/jquery.makeCollapsible.js
index 2d46bdef56..282317419c 100644
--- a/resources/jquery/jquery.makeCollapsible.js
+++ b/resources/jquery/jquery.makeCollapsible.js
@@ -227,7 +227,7 @@
 		}
 
 		return this.each( function () {
-			var $collapsible, collapseText, expandText, $toggle, clickHandler, $defaultToggleLink,
+			var $collapsible, collapseText, expandText, $toggle, clickHandler, buildDefaultToggleLink,
 				premadeToggleHandler, $toggleLink, $firstItem, collapsibleId, $customTogglers, firstval;
 
 			// Ensure class "mw-collapsible" is present in case .makeCollapsible()
@@ -254,14 +254,16 @@
 				opts = $.extend( defaultOpts, options, opts );
 				togglingHandler( $( this ), $collapsible, e, opts );
 			};
-			$defaultToggleLink =
-				$( '<a href="#"></a>' )
+			// Default toggle link. Only build it when needed to avoid jQuery memory leaks (event data).
+			buildDefaultToggleLink = function () {
+				return $( '<a href="#"></a>' )
 					.text( collapseText )
 					.wrap( '<span class="mw-collapsible-toggle"></span>' )
 						.parent()
 						.prepend( '&nbsp;[' )
 						.append( ']&nbsp;' )
 						.on( 'click.mw-collapsible', clickHandler );
+			};
 
 			// Default handler for clicking on premade toggles
 			premadeToggleHandler = function ( e, opts ) {
@@ -309,7 +311,7 @@
 
 					// If theres no toggle link, add it to the last cell
 					if ( !$toggle.length ) {
-						$toggleLink = $defaultToggleLink.prependTo( $firstItem.eq( -1 ) );
+						$toggleLink = buildDefaultToggleLink().prependTo( $firstItem.eq( -1 ) );
 					} else {
 						clickHandler = premadeToggleHandler;
 						$toggleLink = $toggle.on( 'click.mw-collapsible', clickHandler );
@@ -329,7 +331,7 @@
 						if ( firstval === undefined || !firstval || firstval === '-1' || firstval === -1 ) {
 							$firstItem.attr( 'value', '1' );
 						}
-						$toggleLink = $defaultToggleLink;
+						$toggleLink = buildDefaultToggleLink();
 						$toggleLink.wrap( '<li class="mw-collapsible-toggle-li"></li>' ).parent().prependTo( $collapsible );
 					} else {
 						clickHandler = premadeToggleHandler;
@@ -348,7 +350,7 @@
 
 					// If theres no toggle link, add it
 					if ( !$toggle.length ) {
-						$toggleLink = $defaultToggleLink.prependTo( $collapsible );
+						$toggleLink = buildDefaultToggleLink().prependTo( $collapsible );
 					} else {
 						clickHandler = premadeToggleHandler;
 						$toggleLink = $toggle.on( 'click.mw-collapsible', clickHandler );