jquery.makeCollapsible: Simplify code
authorCblair91 <admin@betadev.co.uk>
Mon, 2 May 2016 13:46:34 +0000 (13:46 +0000)
committerBartosz Dziewoński <matma.rex@gmail.com>
Mon, 2 May 2016 14:48:35 +0000 (14:48 +0000)
Instead of using if checks, use a piping system (like in the first function).

Change-Id: Ib96f9db9c56965c336ac85d9bdcf21bdbb4e9cce

resources/src/jquery/jquery.makeCollapsible.js

index 79e8731..bdb5ce8 100644 (file)
         */
        function togglingHandler( $toggle, $collapsible, e, options ) {
                var wasCollapsed, $textContainer, collapseText, expandText;
-
-               if ( options === undefined ) {
-                       options = {};
-               }
+               options = options || {};
 
                if ( e ) {
                        if (
         * @chainable
         */
        $.fn.makeCollapsible = function ( options ) {
-               if ( options === undefined ) {
-                       options = {};
-               }
+               options = options || {};
 
                this.each( function () {
                        var $collapsible, collapseText, expandText, $caption, $toggle, actionHandler, buildDefaultToggleLink,