michaeluno / admin-page-framework

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_Path.php

Spacing +7 added lines, -7 removed lines

@@ -28,8 +28,8 @@ 
     static public function getRelativePath( $from, $to ) {
 
         // some compatibility fixes for Windows paths
-        $from = is_dir( $from ) ? rtrim( $from, '\/') . '/' : $from;
-        $to   = is_dir( $to )   ? rtrim( $to, '\/') . '/'   : $to;
+        $from = is_dir( $from ) ? rtrim( $from, '\/' ).'/' : $from;
+        $to   = is_dir( $to ) ? rtrim( $to, '\/' ).'/' : $to;
         $from = str_replace( '\\', '/', $from );
         $to   = str_replace( '\\', '/', $to );
 
@@ -37,21 +37,21 @@ 
         $to       = explode( '/', $to );
         $relPath  = $to;
 
-        foreach( $from as $depth => $dir ) {
+        foreach ( $from as $depth => $dir ) {
             // find first non-matching dir
-            if( $dir === $to[ $depth ] ) {
+            if ( $dir === $to[ $depth ] ) {
                 // ignore this directory
                 array_shift( $relPath );
             } else {
                 // get number of remaining dirs to $from
                 $remaining = count( $from ) - $depth;
-                if( $remaining > 1 ) {
+                if ( $remaining > 1 ) {
                     // add traversals up to first matching dir
                     $padLength = ( count( $relPath ) + $remaining - 1 ) * -1;
                     $relPath = array_pad( $relPath, $padLength, '..' );
                     break;
                 } else {
-                    $relPath[ 0 ] = './' . $relPath[ 0 ];
+                    $relPath[ 0 ] = './'.$relPath[ 0 ];
                 }
             }
         }
@@ -75,7 +75,7 @@ 
             'debug_backtrace',
             self::_getDebugBacktraceArguments()
         );
-        foreach( $_aBackTrace as $_aDebugInfo )  {
+        foreach ( $_aBackTrace as $_aDebugInfo ) {
             $_aDebugInfo      = self::getAsArray( $_aDebugInfo ) + array( 'file' => '' ); // prevents an undefined index
             $_sCallerFilePath = $_aDebugInfo[ 'file' ];
             if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) {

Braces +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
             'debug_backtrace',
             self::_getDebugBacktraceArguments()
         );
-        foreach( $_aBackTrace as $_aDebugInfo )  {
+        foreach( $_aBackTrace as $_aDebugInfo ) {
             $_aDebugInfo      = self::getAsArray( $_aDebugInfo ) + array( 'file' => '' ); // prevents an undefined index
             $_sCallerFilePath = $_aDebugInfo[ 'file' ];
             if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) {

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer.php

Spacing +19 added lines, -19 removed lines

@@ -47,7 +47,7 @@ 
 
         // Do actions before rendering the page. In this order, global -> page -> in-page tab
         $this->addAndDoActions(
-            $this->oFactory,  // the caller object
+            $this->oFactory, // the caller object
             $this->getFilterArrayByPrefix( 'do_before_', $this->oFactory->oProp->sClassName, $_sPageSlug, $_sTabSlug, true ), // the action hooks
             $this->oFactory   // the argument 1
         );
@@ -72,7 +72,7 @@ 
                     </div><!-- #post-body -->
                 </div><!-- #poststuff -->
 
-            <?php echo $this->_printFormClosingTag( $_sPageSlug, $_sTabSlug, $this->oFactory->oProp->bEnableForm );  // </form> ?>
+            <?php echo $this->_printFormClosingTag( $_sPageSlug, $_sTabSlug, $this->oFactory->oProp->bEnableForm ); // </form> ?>
             </div><!-- .admin-page-framework-container -->
 
             <?php
@@ -83,7 +83,7 @@ 
         <?php
         // Do actions after rendering the page.
         $this->addAndDoActions(
-            $this->oFactory,  // the caller object
+            $this->oFactory, // the caller object
             $this->getFilterArrayByPrefix( 'do_after_', $this->oFactory->oProp->sClassName, $_sPageSlug, $_sTabSlug, true ), // the action hooks
             $this->oFactory   // the argument 1
         );
@@ -100,14 +100,14 @@ 
          */
         private function _getNumberOfColumns() {
 
-            if ( ! $this->doesMetaBoxExist( 'side' ) ) {
+            if ( !$this->doesMetaBoxExist( 'side' ) ) {
                 return 1;
             }
 
             $_iColumns = $this->getNumberOfScreenColumns();
             return $_iColumns
                 ? $_iColumns
-                : 1;    // default - this is because generic pages do not have meta boxes.
+                : 1; // default - this is because generic pages do not have meta boxes.
         }
             // @deprecated
             // Make sure if no side meta box exists, set it 1.
@@ -126,7 +126,7 @@ 
         private function _getContentTop() {
 
             // Screen icon, page heading tabs(page title), and in-page tabs.
-            $_oScreenIcon       = new AdminPageFramework_View__PageRenderer__ScreenIcon(
+            $_oScreenIcon = new AdminPageFramework_View__PageRenderer__ScreenIcon(
                 $this->oFactory,
                 $this->sPageSlug,
                 $this->sTabSlug
@@ -142,7 +142,7 @@ 
                 $this->oFactory,
                 $this->sPageSlug
             );
-            $_sContentTop      .= $_oInPageTabs->get();
+            $_sContentTop .= $_oInPageTabs->get();
 
             // Apply filters in this order, in-page tab -> page -> global.
             return $this->addAndApplyFilters(
@@ -214,7 +214,7 @@ 
              */
             private function _getFormOutput( $sPageSlug ) {
 
-                if ( ! $this->oFactory->oProp->bEnableForm ) {
+                if ( !$this->oFactory->oProp->bEnableForm ) {
                     return '';
                 }
                 return $this->oFactory->oForm->get();
@@ -244,9 +244,9 @@ 
          * @internal
          * @return      void
          */
-        private function _printFormOpeningTag( $fEnableForm=true ) {
+        private function _printFormOpeningTag( $fEnableForm = true ) {
 
-            if ( ! $fEnableForm ) {
+            if ( !$fEnableForm ) {
                 return;
             }
 
@@ -259,12 +259,12 @@ 
                             'action'    => wp_unslash( remove_query_arg( 'settings-updated', $this->oFactory->oProp->sTargetFormPage ) ),
                         )
                     )
-                . " >" . PHP_EOL;
+                . " >".PHP_EOL;
 
             // [3.5.11+] Insert a mark that indicates the framework form has started.
             // This will be checked in the validation method with the `is_admin_page_framework` input value which gets inserted at the end of the form
             // in order to determine all the fields are sent for the PHP max_input_vars limitation set in the server configuration.
-            echo "<input type='hidden' name='admin_page_framework_start' value='1' />" . PHP_EOL;
+            echo "<input type='hidden' name='admin_page_framework_start' value='1' />".PHP_EOL;
 
             // Embed the '_wp_http_referer' hidden field that is checked in the submit data processing method.
             settings_fields( $this->oFactory->oProp->sOptionKey );
@@ -280,16 +280,16 @@ 
          * @internal
          * @return      void
          */
-        private function _printFormClosingTag( $sPageSlug, $sTabSlug, $fEnableForm=true ) {
+        private function _printFormClosingTag( $sPageSlug, $sTabSlug, $fEnableForm = true ) {
 
-            if ( ! $fEnableForm ) {
+            if ( !$fEnableForm ) {
                 return;
             }
-            $_sNonce = wp_create_nonce( 'form_' . md5( $this->oFactory->oProp->sClassName . get_current_user_id() ) );
-            echo "<input type='hidden' name='page_slug' value='{$sPageSlug}' />" . PHP_EOL
-                . "<input type='hidden' name='tab_slug' value='{$sTabSlug}' />" . PHP_EOL
-                . "<input type='hidden' name='_is_admin_page_framework' value='{$_sNonce}' />" . PHP_EOL
-                . "</form><!-- End Form -->" . PHP_EOL;
+            $_sNonce = wp_create_nonce( 'form_'.md5( $this->oFactory->oProp->sClassName.get_current_user_id() ) );
+            echo "<input type='hidden' name='page_slug' value='{$sPageSlug}' />".PHP_EOL
+                . "<input type='hidden' name='tab_slug' value='{$sTabSlug}' />".PHP_EOL
+                . "<input type='hidden' name='_is_admin_page_framework' value='{$_sNonce}' />".PHP_EOL
+                . "</form><!-- End Form -->".PHP_EOL;
 
         }
 

development/factory/admin_page/_view/AdminPageFramework_View__Resource.php

Spacing +13 added lines, -13 removed lines

@@ -67,9 +67,9 @@ 
                  */
                 public function _appendInternalAssets( $sInternal, &$aContainer ) {
                     $_aInternals = array_unique( $aContainer );
-                    $sInternal   = PHP_EOL . $sInternal;
-                    foreach( $_aInternals as $_iIndex => $_sInternal ) {
-                        $sInternal .= $_sInternal . PHP_EOL;
+                    $sInternal   = PHP_EOL.$sInternal;
+                    foreach ( $_aInternals as $_iIndex => $_sInternal ) {
+                        $sInternal .= $_sInternal.PHP_EOL;
                         unset( $_aInternals[ $_iIndex ] );
                     }
                     $aContainer = $_aInternals; // update the container array.
@@ -83,23 +83,23 @@ 
         private function _parseAssets( $oFactory ) {
 
             // page
-            $_aPageStyles      = $this->getElementAsArray(
+            $_aPageStyles = $this->getElementAsArray(
                 $oFactory->oProp->aPages,
                 array( $this->sCurrentPageSlug, 'style' )
             );
             $this->_enqueuePageAssets( $_aPageStyles, 'style' );
 
-            $_aPageScripts     = $this->getElementAsArray(
+            $_aPageScripts = $this->getElementAsArray(
                 $oFactory->oProp->aPages,
                 array( $this->sCurrentPageSlug, 'script' )
             );
             $this->_enqueuePageAssets( $_aPageScripts, 'script' );
 
             // In-page tabs
-            if ( ! $this->sCurrentTabSlug ) {
+            if ( !$this->sCurrentTabSlug ) {
                 return;
             }
-            $_aInPageTabStyles  = $this->getElementAsArray(
+            $_aInPageTabStyles = $this->getElementAsArray(
                 $oFactory->oProp->aInPageTabs,
                 array( $this->sCurrentPageSlug, $this->sCurrentTabSlug, 'style' )
             );
@@ -116,10 +116,10 @@ 
              * @since       3.6.3
              * @return      void
              */
-            private function _enqueuePageAssets( array $aPageAssets, $sType='style' ) {
-                $_sMethodName = "_enqueueAsset_" . $sType;
-                foreach( $aPageAssets as $_asPageAsset ) {
-                    $this->{$_sMethodName}( $_asPageAsset);
+            private function _enqueuePageAssets( array $aPageAssets, $sType = 'style' ) {
+                $_sMethodName = "_enqueueAsset_".$sType;
+                foreach ( $aPageAssets as $_asPageAsset ) {
+                    $this->{$_sMethodName}( $_asPageAsset );
                 }
             }
 
@@ -144,7 +144,7 @@ 
                     }
 
                     // Insert the CSS rule in the head tag.
-                    $this->aCSSRules[] = $_sSRC;
+                    $this->aCSSRules[ ] = $_sSRC;
 
                 }
 
@@ -169,7 +169,7 @@ 
                     }
 
                     // Insert the scripts in the head tag.
-                    $this->aScripts[] = $_sSRC;
+                    $this->aScripts[ ] = $_sSRC;
 
                 }
 

development/factory/_common/_abstract/_controller/AdminPageFramework_Resource_Base.php

Indentation +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
     /*
      * Shared methods
      */
-    	/**
+        /**
          * Checks the src url of the enqueued script/style to determine whether or not to set up a attribute modification callback.
          *
          * If it is one of the framework added item, the method sets up a hook to modify the url to add custom attributes.

Braces +6 added lines, -3 removed lines

@@ -352,7 +352,8 @@ 
             static $_iCallCount = 0;
 
             $_sFilterName = "style_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
                 return '';
             }
             $_sStyle = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyle );
@@ -379,7 +380,8 @@ 
             static $_iCallCountIE = 1;
 
             $_sFilterName = "style_ie_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
                 return '';
             }
             $_sStyleIE = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyleIE );
@@ -407,7 +409,8 @@ 
 
         static $_iCallCount = 1;
         $_sFilterName = "script_{$this->oProp->sClassName}";
-        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
             return '';
         }
         $_sScript = $this->addAndApplyFilters( $this->oProp->oCaller, $_sFilterName, $this->oProp->sScript );

Spacing +63 added lines, -63 removed lines

@@ -34,30 +34,30 @@ 
         /* The system internal keys. */
         'sSRC'              => null,
         'sSRCRaw'           => null,
-        'aPostTypes'        => array(),     // for meta box class
+        'aPostTypes'        => array(), // for meta box class
         'sPageSlug'         => null,
         'sTabSlug'          => null,
-        'sType'             => null,        // script or style
+        'sType'             => null, // script or style
 
         /* The below keys are for users. */
         'handle_id'         => null,
         'dependencies'      => array(),
-        'version'           => false,       // although the type should be string, the wp_enqueue_...() functions want false as the default value.
-        'attributes'        => array(),     // [3.3.0+] - the attribute array @deprecated 3.9.0
-        'conditional'       => null,        // [3.9.0+] Comments for IE 6, lte IE 7 etc. @see wp_style_add_data() and wp_script_add_data()
+        'version'           => false, // although the type should be string, the wp_enqueue_...() functions want false as the default value.
+        'attributes'        => array(), // [3.3.0+] - the attribute array @deprecated 3.9.0
+        'conditional'       => null, // [3.9.0+] Comments for IE 6, lte IE 7 etc. @see wp_style_add_data() and wp_script_add_data()
 
         // script specific
-        'translation'       => array(),     // only for scripts
-        'translation_var'   => '',          // [3.9.0+] the object name of the passed translation data to JavaScript script
-        'in_footer'         => false,       // only for scripts
+        'translation'       => array(), // only for scripts
+        'translation_var'   => '', // [3.9.0+] the object name of the passed translation data to JavaScript script
+        'in_footer'         => false, // only for scripts
 
         // style specific
-        'media'             => 'all',       // only for styles
+        'media'             => 'all', // only for styles
         /// [3.9.0+] @see wp_style_add_data()
-        'rtl'               => null,        // bool|string To declare an RTL stylesheet.
-        'suffix'            => null,        // string      Optional suffix, used in combination with RTL.
-        'alt'               => null,        // bool        For rel="alternate stylesheet".
-        'title'             => null,        // string      For preferred/alternate stylesheets.
+        'rtl'               => null, // bool|string To declare an RTL stylesheet.
+        'suffix'            => null, // string      Optional suffix, used in combination with RTL.
+        'alt'               => null, // bool        For rel="alternate stylesheet".
+        'title'             => null, // string      For preferred/alternate stylesheets.
     );
 
     /**
@@ -67,7 +67,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */
-    protected $_sClassSelector_Style    = 'admin-page-framework-style';
+    protected $_sClassSelector_Style = 'admin-page-framework-style';
 
     /**
      * Stores the class selector used to the class-specific script.
@@ -76,7 +76,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */
-    protected $_sClassSelector_Script   = 'admin-page-framework-script';
+    protected $_sClassSelector_Script = 'admin-page-framework-script';
 
     /**
      * Stores hand IDs by resource url to look up handle id and add custom arguments.
@@ -127,13 +127,13 @@ 
          */
         public function _replyToSetUpHooks() {
 
-            if ( ! $this->oProp->oCaller->isInThePage() ) {
+            if ( !$this->oProp->oCaller->isInThePage() ) {
                 return;
             }
 
             // Hook the admin header to insert custom admin stylesheets and scripts.
             // add_action( 'admin_enqueue_scripts', array( $this, '_replyToEnqueueCommonScripts' ), 1 );    // @deprecated 3.9.0
-            add_action( 'admin_enqueue_scripts', array( $this, '_replyToEnqueueCommonStyles' ), 1  );
+            add_action( 'admin_enqueue_scripts', array( $this, '_replyToEnqueueCommonStyles' ), 1 );
 
             add_action( 'admin_enqueue_scripts', array( $this, '_replyToEnqueueScripts' ) );
             add_action( 'admin_enqueue_scripts', array( $this, '_replyToEnqueueStyles' ) );
@@ -228,8 +228,8 @@ 
                         return $sSanitizedURL;
                     }
 
-                    $_sAttributes   = $this->getAttributes( $_aAttributes );
-                    return $sSanitizedURL . "' " . rtrim( $_sAttributes, "'\"" );
+                    $_sAttributes = $this->getAttributes( $_aAttributes );
+                    return $sSanitizedURL."' ".rtrim( $_sAttributes, "'\"" );
 
                 }
 
@@ -249,7 +249,7 @@ 
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
 
-        if ( $this->hasBeenCalled( 'COMMON_STYLES: ' . get_class( $this ) . '::' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_STYLES: '.get_class( $this ).'::'.__METHOD__ ) ) {
             return;
         }
         $_oCaller = $this->oProp->oCaller;
@@ -265,10 +265,10 @@ 
          */
         private function ___getCommonStyleTag( $oCaller, $sIDPrefix ) {
 
-            $_sStyle     = $this->addAndApplyFilters(
+            $_sStyle = $this->addAndApplyFilters(
                 $oCaller,
                 array(
-                    "style_common_admin_page_framework",            // 3.2.1+
+                    "style_common_admin_page_framework", // 3.2.1+
                     "style_common_{$this->oProp->sClassName}",
                 ),
                 '' // AdminPageFramework_CSS::getDefaultCSS() @deprecated 3.9.0 No longer uses internal stylesheets
@@ -276,7 +276,7 @@ 
             $_sStyle     = $this->isDebugMode() ? $_sStyle : $this->getCSSMinified( $_sStyle );
             $_sStyle     = trim( $_sStyle );
             if ( $_sStyle ) {
-                return "<style type='text/css' id='" . esc_attr( strtolower( $sIDPrefix ) ) . "'>"
+                return "<style type='text/css' id='".esc_attr( strtolower( $sIDPrefix ) )."'>"
                         . $_sStyle
                     . "</style>";
             }
@@ -290,10 +290,10 @@ 
          * @return   string
          */
         private function ___getCommonIEStyleTag( $oCaller, $sIDPrefix ) {
-            $_sStyleIE   = $this->addAndApplyFilters(
+            $_sStyleIE = $this->addAndApplyFilters(
                 $oCaller,
                 array(
-                    "style_ie_common_admin_page_framework",         // 3.2.1+
+                    "style_ie_common_admin_page_framework", // 3.2.1+
                     "style_ie_common_{$this->oProp->sClassName}",
                 ),
                 AdminPageFramework_CSS::getDefaultCSSIE()
@@ -301,7 +301,7 @@ 
             $_sStyleIE  = $this->isDebugMode() ? $_sStyleIE : $this->getCSSMinified( $_sStyleIE );
             $_sStyleIE  = trim( $_sStyleIE );
             return $_sStyleIE
-                ? "<!--[if IE]><style type='text/css' id='" . esc_attr( strtolower( $sIDPrefix . "-ie" ) ) . "'>"
+                ? "<!--[if IE]><style type='text/css' id='".esc_attr( strtolower( $sIDPrefix."-ie" ) )."'>"
                         . $_sStyleIE
                     . "</style><![endif]-->"
                 : '';
@@ -319,23 +319,23 @@ 
      */
     protected function _printCommonScripts( $sIDPrefix, $sClassName ) {
 
-        if ( $this->hasBeenCalled( 'COMMON_SCRIPT: ' . get_class( $this ) . '::' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_SCRIPT: '.get_class( $this ).'::'.__METHOD__ ) ) {
             return;
         }
 
         $_sScript = $this->addAndApplyFilters(
             $this->oProp->oCaller,
             array(
-                "script_common_admin_page_framework",       // 3.2.1+
+                "script_common_admin_page_framework", // 3.2.1+
                 "script_common_{$this->oProp->sClassName}",
             ),
             AdminPageFramework_Property_Base::$_sDefaultScript
         );
         $_sScript = trim( $_sScript );
-        if ( ! $_sScript ) {
+        if ( !$_sScript ) {
             return;
         }
-        echo "<script type='text/javascript' id='" . esc_attr( strtolower( $sIDPrefix ) ) . "'>"
+        echo "<script type='text/javascript' id='".esc_attr( strtolower( $sIDPrefix ) )."'>"
                 . '/* <![CDATA[ */'
                 . $_sScript
                 . '/* ]]> */'
@@ -352,7 +352,7 @@ 
      */
     protected function _printClassSpecificStyles( $sIDPrefix ) {
 
-        $_oCaller   = $this->oProp->oCaller;
+        $_oCaller = $this->oProp->oCaller;
         echo $this->_getClassSpecificStyleTag( $_oCaller, $sIDPrefix );
         echo $this->_getClassSpecificIEStyleTag( $_oCaller, $sIDPrefix );
 
@@ -373,18 +373,18 @@ 
             static $_iCallCount = 0;
 
             $_sFilterName = "style_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
                 return '';
             }
             $_sStyle = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyle );
             $_sStyle = $this->isDebugMode() ? $_sStyle : $this->getCSSMinified( $_sStyle );
             $_sStyle = trim( $_sStyle );
-            if ( ! $_sStyle ) {
+            if ( !$_sStyle ) {
                 return '';
             }
             $_iCallCount++;
-            $_sID = strtolower( "{$sIDPrefix}-" . $this->oProp->sClassName . "_{$_iCallCount}" );
-            return "<style type='text/css' id='" . esc_attr( $_sID ) . "'>"
+            $_sID = strtolower( "{$sIDPrefix}-".$this->oProp->sClassName."_{$_iCallCount}" );
+            return "<style type='text/css' id='".esc_attr( $_sID )."'>"
                     . $_sStyle
                 . "</style>";
 
@@ -400,18 +400,18 @@ 
             static $_iCallCountIE = 1;
 
             $_sFilterName = "style_ie_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
                 return '';
             }
             $_sStyleIE = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyleIE );
             $_sStyleIE = $this->isDebugMode() ? $_sStyleIE : $this->getCSSMinified( $_sStyleIE );
             $_sStyleIE = trim( $_sStyleIE );
-            if ( ! $_sStyleIE ) {
+            if ( !$_sStyleIE ) {
                 return '';
             }
             $_iCallCountIE++;
-            $_sID  = strtolower( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" );
-            return "<!--[if IE]><style type='text/css' id='" . esc_attr( $_sID ) . "'>"
+            $_sID = strtolower( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" );
+            return "<!--[if IE]><style type='text/css' id='".esc_attr( $_sID )."'>"
                     . $_sStyleIE
                 . "</style><![endif]-->";
 
@@ -428,18 +428,18 @@ 
 
         static $_iCallCount = 1;
         $_sFilterName = "script_{$this->oProp->sClassName}";
-        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+        if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
             return '';
         }
         $_sScript = $this->addAndApplyFilters( $this->oProp->oCaller, $_sFilterName, $this->oProp->sScript );
         $_sScript = trim( $_sScript );
-        if ( ! $_sScript ) {
+        if ( !$_sScript ) {
             return '';
         }
 
         $_iCallCount++;
         $_sID = strtolower( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" );
-        echo "<script type='text/javascript' id='" . esc_attr( $_sID ) . "'>"
+        echo "<script type='text/javascript' id='".esc_attr( $_sID )."'>"
                 . '/* <![CDATA[ */'
                 . $_sScript
                 . '/* ]]> */'
@@ -462,12 +462,12 @@ 
     public function _replyToAddStyle() {
 
         $_oCaller = $this->oProp->oCaller;
-        if ( ! $_oCaller->isInThePage() ) {
+        if ( !$_oCaller->isInThePage() ) {
             return;
         }
 
         $this->_printCommonStyles( 'admin-page-framework-style-common', get_class() );
-        $this->_printClassSpecificStyles( $this->_sClassSelector_Style . '-' . $this->oProp->sStructureType );
+        $this->_printClassSpecificStyles( $this->_sClassSelector_Style.'-'.$this->oProp->sStructureType );
 
     }
     /**
@@ -482,12 +482,12 @@ 
     public function _replyToAddScript() {
 
         $_oCaller = $this->oProp->oCaller;
-        if ( ! $_oCaller->isInThePage() ) {
+        if ( !$_oCaller->isInThePage() ) {
             return;
         }
 
         $this->_printCommonScripts( 'admin-page-framework-script-common', get_class() );
-        $this->_printClassSpecificScripts( $this->_sClassSelector_Script . '-' . $this->oProp->sStructureType );
+        $this->_printClassSpecificScripts( $this->_sClassSelector_Script.'-'.$this->oProp->sStructureType );
 
     }
 
@@ -550,8 +550,8 @@ 
                 $aEnqueueItem[ 'media' ]
             );
             $_aAddData = array( 'conditional', 'rtl', 'suffix', 'alt', 'title' );
-            foreach( $_aAddData as $_sDataKey ) {
-                if ( ! isset( $aEnqueueItem[ $_sDataKey ] ) ) {
+            foreach ( $_aAddData as $_sDataKey ) {
+                if ( !isset( $aEnqueueItem[ $_sDataKey ] ) ) {
                     continue;
                 }
                 wp_style_add_data( $aEnqueueItem[ 'handle_id' ], $_sDataKey, $aEnqueueItem[ $_sDataKey ] );
@@ -567,7 +567,7 @@ 
          */
         private function ___getSRCFormatted( array $aEnqueueItem ) {
 
-            if ( ! $this->oProp->bAutoloadMinifiedResource ) {
+            if ( !$this->oProp->bAutoloadMinifiedResource ) {
                 return $aEnqueueItem[ 'sSRC' ];
             }
 
@@ -594,16 +594,16 @@ 
                 + array( 'dirname' => '', 'filename' => '', 'basename' => '', 'extension' => '' ); // avoid undefined index warnings
 
             // If there is no extension, avoid using a minified version.
-            if ( ! $_aPathParts[ 'extension' ] ) {
+            if ( !$_aPathParts[ 'extension' ] ) {
                 return $aEnqueueItem[ 'sSRC' ];
             }
 
             $_aPathPartsURL = pathinfo( $aEnqueueItem[ 'sSRC' ] )
                 + array( 'dirname' => '', 'filename' => '', 'basename' => '', 'extension' => '' ); // avoid undefined index warnings
 
-            $_sPathMinifiedVersion = $_aPathParts[ 'dirname' ] . '/' . $_aPathParts[ 'filename' ] . $_sMinPrefix . '.' . $_aPathParts[ 'extension' ];
+            $_sPathMinifiedVersion = $_aPathParts[ 'dirname' ].'/'.$_aPathParts[ 'filename' ].$_sMinPrefix.'.'.$_aPathParts[ 'extension' ];
             return file_exists( $_sPathMinifiedVersion )
-                ? $_aPathPartsURL[ 'dirname' ] . '/' . $_aPathPartsURL[ 'filename' ] . $_sMinPrefix . '.' . $_aPathPartsURL[ 'extension' ]
+                ? $_aPathPartsURL[ 'dirname' ].'/'.$_aPathPartsURL[ 'filename' ].$_sMinPrefix.'.'.$_aPathPartsURL[ 'extension' ]
                 : $aEnqueueItem[ 'sSRC' ];
 
         }
@@ -633,11 +633,11 @@ 
      * @callback action wp_enqueue_scripts
      */
     public function _replyToEnqueueCommonStyles() {
-        if ( $this->hasBeenCalled( 'COMMON_EXTERNAL_STYLES: ' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_EXTERNAL_STYLES: '.__METHOD__ ) ) {
             return;
         }
         $this->_addEnqueuingResourceByType(
-            AdminPageFramework_Registry::$sDirPath . '/factory/_common/asset/css/common.css',
+            AdminPageFramework_Registry::$sDirPath.'/factory/_common/asset/css/common.css',
             array(
                 'version' => AdminPageFramework_Registry::VERSION,
             ),
@@ -656,7 +656,7 @@ 
      * @internal
      */
     public function _replyToEnqueueStyles() {
-        foreach( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
+        foreach ( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingStyle );
             unset( $this->oProp->aEnqueuingStyles[ $_sKey ] );
         }
@@ -673,7 +673,7 @@ 
      * @internal
      */
     public function _replyToEnqueueScripts() {
-        foreach( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
+        foreach ( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingScript );
             unset( $this->oProp->aEnqueuingScripts[ $_sKey ] );
         }
@@ -687,10 +687,10 @@ 
      * @param  string   $sType          Accepts 'style' or 'script'
      * @return string[] Added resource handle IDs.
      */
-    public function _enqueueResourcesByType( $aSRCs, array $aCustomArgs=array(), $sType='style' ) {
+    public function _enqueueResourcesByType( $aSRCs, array $aCustomArgs = array(), $sType = 'style' ) {
         $_aHandleIDs = array();
-        foreach( $aSRCs as $_sSRC ) {
-            $_aHandleIDs[] = call_user_func_array( array( $this, '_addEnqueuingResourceByType' ), array( $_sSRC, $aCustomArgs, $sType ) );
+        foreach ( $aSRCs as $_sSRC ) {
+            $_aHandleIDs[ ] = call_user_func_array( array( $this, '_addEnqueuingResourceByType' ), array( $_sSRC, $aCustomArgs, $sType ) );
         }
         return $_aHandleIDs;
     }
@@ -707,9 +707,9 @@ 
      * @return      string      The script handle ID if added. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */
-    public function _addEnqueuingResourceByType( $sSRC, array $aCustomArgs=array(), $sType='style' ) {
+    public function _addEnqueuingResourceByType( $sSRC, array $aCustomArgs = array(), $sType = 'style' ) {
 
-        $sSRC       = trim( $sSRC );
+        $sSRC = trim( $sSRC );
         if ( empty( $sSRC ) ) {
             return '';
         }
@@ -725,12 +725,12 @@ 
                 'sSRCRaw'   => $_sRawSRC,
                 'sSRC'      => $_sSRC,
                 'sType'     => $sType,
-                'handle_id' => $sType . '_' . strtolower( $this->oProp->sClassName ) . '_' .  ( ++$this->oProp->{$_sEnqueuedIndexPropertyName} ),
+                'handle_id' => $sType.'_'.strtolower( $this->oProp->sClassName ).'_'.( ++$this->oProp->{$_sEnqueuedIndexPropertyName} ),
             )
             + self::$_aStructure_EnqueuingResources;
 
         // Store the attributes in another container by url.
-        $this->oProp->aResourceAttributes[ $this->oProp->{$_sContainerPropertyName}[ $_sSRC ]['handle_id'] ] = $this->oProp->{$_sContainerPropertyName}[ $_sSRC ]['attributes'];
+        $this->oProp->aResourceAttributes[ $this->oProp->{$_sContainerPropertyName}[ $_sSRC ][ 'handle_id' ] ] = $this->oProp->{$_sContainerPropertyName}[ $_sSRC ][ 'attributes' ];
 
         return $this->oProp->{$_sContainerPropertyName}[ $_sSRC ][ 'handle_id' ];
 

development/factory/_common/form/_view/fieldset/AdminPageFramework_Form_View___Fieldset_Base.php

Indentation +3 added lines, -3 removed lines

@@ -272,8 +272,8 @@ 
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
                         : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                            . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
+                            . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -285,7 +285,7 @@ 
             private function ___getRemoveButtonAttributes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
                 $_aMinusButtonAttributes = array(
                     'class'     => 'repeatable-field-remove-button button-secondary repeatable-field-button button'
-                         . $sSmallButtonSelector,
+                            . $sSmallButtonSelector,
                     'title'     => $this->oMsg->get( 'remove' ),
                     'style'     => $iFieldCount <= 1
                         ? 'visibility: hidden'

Spacing +21 added lines, -21 removed lines

@@ -67,7 +67,7 @@ 
      * @param       object      $oMsg                   An object storing the system messages.
      * @param       array       $aCallbacks             An array storing the form-field specific callbacks.     
      */
-    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks=array() ) {
+    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks = array() ) {
 
         // Set up the properties that will be accessed later in the methods.
         $this->aFieldset                = $this->_getFormatted( $aFieldset, $aFieldTypeDefinitions );
@@ -76,13 +76,13 @@ 
         $this->aErrors                  = $this->getAsArray( $aErrors );
         $this->oMsg                     = $oMsg;
         $this->aCallbacks               = $aCallbacks + array(
-            'hfID'              => null,    // the input id attribute
-            'hfTagID'           => null,    // the fieldset/field row container id attribute
-            'hfName'            => null,    // the input name attribute
-            'hfNameFlat'        => null,    // the flat input name attribute                
+            'hfID'              => null, // the input id attribute
+            'hfTagID'           => null, // the fieldset/field row container id attribute
+            'hfName'            => null, // the input name attribute
+            'hfNameFlat'        => null, // the flat input name attribute                
             'hfInputName'       => null,
             'hfInputNameFlat'   => null,
-            'hfClass'           => null,    // the class attribute
+            'hfClass'           => null, // the class attribute
         );
 
     }    
@@ -142,7 +142,7 @@ 
         $_aSettings  = $this->getAsArray( $aSettings );
         $_oFormatter = new AdminPageFramework_Form_Model___Format_RepeatableField( $_aSettings, $this->oMsg ); // @todo Move this formatting routine to the field-set formatter class.
         $_aSettings  = $_oFormatter->get();
-        return "<div class='hidden repeatable-field-buttons-model' " . $this->getDataAttributes( $_aSettings ) . ">"
+        return "<div class='hidden repeatable-field-buttons-model' ".$this->getDataAttributes( $_aSettings ).">"
                 . $this->___getRepeatableButtonHTML( $sFieldsContainerID, $_aSettings, $iFieldCount, false )
             . "</div>";
     }
@@ -156,30 +156,30 @@ 
          * @param   integer $iFieldCount
          * @param   boolean $bSmall                 @deprecated
          */
-        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall=true ) {
+        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall = true ) {
 
             $_aArguments            = $aArguments;
             $_sSmallButtonSelector  = $bSmall ? ' button-small' : '';
             $_sDisabledContent      = $this->getModalForDisabledRepeatableElement(
-                'repeatable_field_disabled_' . $sFieldsContainerID,
+                'repeatable_field_disabled_'.$sFieldsContainerID,
                 $_aArguments[ 'disabled' ]
             );
             if ( version_compare( $GLOBALS[ 'wp_version' ], '5.3', '>=' ) ) {
-                return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                        . "<a " . $this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">"
+                return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                        . "<a ".$this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">"
                             . "<span class='dashicons dashicons-minus'></span>"
                         . "</a>"
-                        . "<a " . $this->___getAddButtonAttributes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">"
+                        . "<a ".$this->___getAddButtonAttributes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">"
                             . "<span class='dashicons dashicons-plus-alt2'></span>"
                         ."</a>"
                     . "</div>"
                     . $_sDisabledContent;
             }
-            return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                    . "<a " . $this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">"
+            return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                    . "<a ".$this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">"
                         . "-"
                     . "</a>"
-                    . "<a " . $this->___getAddButtonAttributes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">"
+                    . "<a ".$this->___getAddButtonAttributes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">"
                         . "+"
                     ."</a>"
                 . "</div>"
@@ -200,9 +200,9 @@ 
                     'data-id'   => $sFieldsContainerID,
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
-                        : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                        : '#TB_inline?width='.$aArguments[ 'disabled' ][ 'box_width' ]
+                          . '&height='.$aArguments[ 'disabled' ][ 'box_height' ]
+                          . '&inlineId='.'repeatable_field_disabled_'.$sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -229,10 +229,10 @@ 
              * @return      string
              */
             private function ___getContainerAttributes( $aArguments ) {
-                $_aContainerAttributes   = array(
+                $_aContainerAttributes = array(
                     'class' => $this->getClassAttribute(
                         'admin-page-framework-repeatable-field-buttons',
-                        ! empty( $aArguments[ 'disabled' ] )
+                        !empty( $aArguments[ 'disabled' ] )
                             ? 'disabled'
                             : ''
                     ),
@@ -245,7 +245,7 @@ 
                     unset( $aArguments[ 'disabled' ] );
                 }
                 return $this->getAttributes( $_aContainerAttributes )
-                    . ' ' . $this->getDataAttributes( $aArguments );
+                    . ' '.$this->getDataAttributes( $aArguments );
             }
 
 }
\ No newline at end of file

development/factory/page_meta_box/AdminPageFramework_PageMetaBox.php

Spacing +4 added lines, -4 removed lines

@@ -76,21 +76,21 @@
      * @param       string          $sPriority      The priority, either `high`, `core`, `default` or `low`.
      * @param       string          $sCapability    The capability. See <a href="https://codex.wordpress.org/Roles_and_Capabilities" target="_blank">Roles and Capabilities</a>.
      */
-    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs = array(), $sContext = 'normal', $sPriority = 'default', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         if ( empty( $asPageSlugs ) ) {
             return;
         }
 
-        if ( ! $this->_isInstantiatable() ) {
+        if ( !$this->_isInstantiatable() ) {
             return;
         }
 
         // The property object needs to be done first before the parent constructor.
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
-        $this->oProp             = new $_sPropertyClassName(
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
+        $this->oProp = new $_sPropertyClassName(
             $this,
             get_class( $this ),
             $sCapability,

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
      * @param       string          The text domain. Default: `admin-page-framework`.
      * @return      void
      */
-    function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    function __construct( $asTaxonomySlug, $sOptionKey = '', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         if ( empty( $asTaxonomySlug ) ) {
             return;
@@ -54,8 +54,8 @@ 
         // Properties
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
-        $this->oProp        = new $_sPropertyClassName(
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
+        $this->oProp = new $_sPropertyClassName(
             $this,
             get_class( $this ),
             $sCapability,

development/factory/user_meta/AdminPageFramework_UserMeta.php

Spacing +6 added lines, -6 removed lines

@@ -38,16 +38,16 @@
      * @since       3.7.4       Changed the default capability value to `read`.
      * @todo        Examine the appropriate default capability level.
      */
-    public function __construct( $sCapability='read', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sCapability = 'read', $sTextDomain = 'admin-page-framework' ) {
 
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
         $this->oProp = new $_sPropertyClassName(
-            $this,                  // the caller object
-            get_class( $this ),     // the caller class name
-            $sCapability,           // the capability level
-            $sTextDomain,           // the text domain
+            $this, // the caller object
+            get_class( $this ), // the caller class name
+            $sCapability, // the capability level
+            $sTextDomain, // the text domain
             $this->_sStructureType  // the structure type
         );
 

development/factory/admin_page/AdminPageFramework_View_Page.php

Spacing +3 added lines, -3 removed lines

@@ -32,10 +32,10 @@ 
                 $this->oProp->aInPageTabs,
                 array( $this->oProp->getCurrentPageSlug(), $this->oProp->getCurrentTabSlug(), 'title' )
             );
-        if ( ! $_sTabTitle ) {
+        if ( !$_sTabTitle ) {
             return $sAdminTitle;
         }
-        return $_sTabTitle . ' ‹ ' . $sAdminTitle;
+        return $_sTabTitle.' ‹ '.$sAdminTitle;
     }
 
     /**
@@ -82,7 +82,7 @@ 
      * @return      void
      * @internal
      */
-    protected function _renderPage( $sPageSlug, $sTabSlug=null ) {
+    protected function _renderPage( $sPageSlug, $sTabSlug = null ) {
         $_oPageRenderer = new AdminPageFramework_View__PageRenderer( $this, $sPageSlug, $sTabSlug );
         $_oPageRenderer->render();
     }