michaeluno / admin-page-framework

development/factory/_common/form/field_type/AdminPageFramework_FieldType_system.php

Spacing +59 added lines, -59 removed lines

@@ -73,7 +73,7 @@ 
      * </code>
      * @var         array
      */
-    public $aFieldTypeSlugs = array( 'system', );
+    public $aFieldTypeSlugs = array( 'system',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -93,8 +93,8 @@ 
      * @remark          `$_aDefaultKeys` holds shared default key-values defined in the base class.
      */
     protected $aDefaultKeys = array(
-        'data'          =>  array(),        // [3.2.0+] Stores the data to be displayed
-        'print_type'    =>  1,              // [3.4.6+] 1: readable representation of array. 2: print_r()
+        'data'          =>  array(), // [3.2.0+] Stores the data to be displayed
+        'print_type'    =>  1, // [3.4.6+] 1: readable representation of array. 2: print_r()
         'attributes'    =>    array(
             'rows'          => 60,
             'autofocus'     => null,
@@ -233,13 +233,13 @@ 
             . "<div class='admin-page-framework-input-label-container'>"
                 . "<label for='{$aField[ 'input_id' ]}'>"
                     . $aField[ 'before_input' ]
-                    . ( $aField[ 'label' ] && ! $aField[ 'repeatable' ]
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                 . $aField[ 'label' ] 
                             . "</span>"
                         : "" 
                     )
-                    . "<textarea " . $this->getAttributes( $_aInputAttributes ) . " >"    
+                    . "<textarea ".$this->getAttributes( $_aInputAttributes )." >"    
                         . esc_textarea( $this->_getSystemInfomation( $aField[ 'value' ], $aField[ 'data' ], $aField[ 'print_type' ] ) )
                     . "</textarea>"
                     . $aField[ 'after_input' ]
@@ -254,15 +254,15 @@ 
          * @return      string      The human readable system information.
          * @internal
          */
-        private function _getSystemInfomation( $asValue=null, $asCustomData=null, $iPrintType=1 ) {
+        private function _getSystemInfomation( $asValue = null, $asCustomData = null, $iPrintType = 1 ) {
 
             if ( isset( $asValue ) ) {
                 return $asValue;
             }
 
-            $_aOutput   = array();
-            foreach( $this->_getFormattedSystemInformation( $asCustomData ) as $_sSection => $_aInfo ) {
-                $_aOutput[] = $this->_getSystemInfoBySection( $_sSection, $_aInfo, $iPrintType );
+            $_aOutput = array();
+            foreach ( $this->_getFormattedSystemInformation( $asCustomData ) as $_sSection => $_aInfo ) {
+                $_aOutput[ ] = $this->_getSystemInfoBySection( $_sSection, $_aInfo, $iPrintType );
             }
             return implode( PHP_EOL, $_aOutput );
             
@@ -277,18 +277,18 @@ 
                 
                 $_aData = $this->getAsArray( $asCustomData );
                 $_aData = $_aData + array(
-                    'Admin Page Framework'  => isset( $_aData['Admin Page Framework'] )
+                    'Admin Page Framework'  => isset( $_aData[ 'Admin Page Framework' ] )
                         ? null
                         : AdminPageFramework_Registry::getInfo(),
-                    'WordPress'             => $this->_getSiteInfoWithCache( ! isset( $_aData['WordPress'] ) ),
-                    'PHP'                   => $this->_getPHPInfo( ! isset( $_aData['PHP'] ) ),
-                    'PHP Error Log'         => $this->_getErrorLogByType( 'php', ! isset( $_aData['PHP Error Log'] ) ),
-                    'MySQL'                 => isset( $_aData['MySQL'] )
+                    'WordPress'             => $this->_getSiteInfoWithCache( !isset( $_aData[ 'WordPress' ] ) ),
+                    'PHP'                   => $this->_getPHPInfo( !isset( $_aData[ 'PHP' ] ) ),
+                    'PHP Error Log'         => $this->_getErrorLogByType( 'php', !isset( $_aData[ 'PHP Error Log' ] ) ),
+                    'MySQL'                 => isset( $_aData[ 'MySQL' ] )
                         ? null
-                        : $this->getMySQLInfo(),    // defined in the utility class.
-                    'MySQL Error Log'       => $this->_getErrorLogByType( 'mysql', ! isset( $_aData['MySQL Error Log'] ) ),
-                    'Server'                => $this->_getWebServerInfo( ! isset( $_aData['Server'] ) ),
-                    'Browser'               => $this->_getClientInfo( ! isset( $_aData['Browser'] ) ),
+                        : $this->getMySQLInfo(), // defined in the utility class.
+                    'MySQL Error Log'       => $this->_getErrorLogByType( 'mysql', !isset( $_aData[ 'MySQL Error Log' ] ) ),
+                    'Server'                => $this->_getWebServerInfo( !isset( $_aData[ 'Server' ] ) ),
+                    'Browser'               => $this->_getClientInfo( !isset( $_aData[ 'Browser' ] ) ),
                 );
                 
                 // Dropping empty elements allows the user to remove a section by setting an empty section.
@@ -305,10 +305,10 @@ 
                 switch ( $iPrintType ) {
                     default:
                     case 1: // use the framework readable representation of arrays.
-                        return $this->getReadableArrayContents( $sSectionName, $aData, 32 ) . PHP_EOL;
+                        return $this->getReadableArrayContents( $sSectionName, $aData, 32 ).PHP_EOL;
                     case 2: // use print_r()
-                        return "[{$sSectionName}]" . PHP_EOL
-                            . print_r( $aData, true ) . PHP_EOL;
+                        return "[{$sSectionName}]".PHP_EOL
+                            . print_r( $aData, true ).PHP_EOL;
                 }                      
             }
             /**
@@ -318,17 +318,17 @@ 
              * @since       3.4.6
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              */
-            private function _getClientInfo( $bGenerateInfo=true ) {
+            private function _getClientInfo( $bGenerateInfo = true ) {
                  
-                if ( ! $bGenerateInfo ) {
+                if ( !$bGenerateInfo ) {
                     return '';
                 }
                  
                 // Check the browscap value in the ini file first to prevent warnings from being populated
                 $_aBrowser = @ini_get( 'browscap' ) 
-                    ? get_browser( $_SERVER['HTTP_USER_AGENT'], true )
+                    ? get_browser( $_SERVER[ 'HTTP_USER_AGENT' ], true )
                     : array();
-                unset( $_aBrowser['browser_name_regex'] );  // this element causes output to be blank
+                unset( $_aBrowser[ 'browser_name_regex' ] ); // this element causes output to be blank
                 return empty( $_aBrowser ) 
                     ? __( 'No browser information found.', 'admin-page-framework' )
                     : $_aBrowser;
@@ -344,9 +344,9 @@ 
              * @param       string      $sType          The error log type. Either 'php' or 'mysql' is accepted.
              * @param       boolean     $bGenerateInfo  Whether to generate a log. This is for the caller method to reduce a conditinal statement.
              */
-            private function _getErrorLogByType( $sType, $bGenerateInfo=true ) {
+            private function _getErrorLogByType( $sType, $bGenerateInfo = true ) {
 
-                if ( ! $bGenerateInfo ) {
+                if ( !$bGenerateInfo ) {
                     return '';
                 }
                 switch ( $sType ) {
@@ -378,9 +378,9 @@ 
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              * @return      array      The generated site information array.
              */
-            private function _getSiteInfoWithCache( $bGenerateInfo=true ) {
+            private function _getSiteInfoWithCache( $bGenerateInfo = true ) {
                 
-                if ( ! $bGenerateInfo || isset( self::$_aSiteInfo ) ) {
+                if ( !$bGenerateInfo || isset( self::$_aSiteInfo ) ) {
                     return self::$_aSiteInfo;
                 }
                 self::$_aSiteInfo = self::_getSiteInfo();
@@ -414,8 +414,8 @@ 
                         'wp_remote_get()'                                           => $this->_getWPRemoteGetStatus(),
                         __( 'Multibite String Extension', 'admin-page-framework' )  => $this->getAOrB( function_exists( 'mb_detect_encoding' ), $this->oMsg->get( 'enabled' ), $this->oMsg->get( 'disabled' ) ),
                         __( 'WP_CONTENT_DIR Writeable', 'admin-page-framework' )    => $this->getAOrB( is_writable( WP_CONTENT_DIR ), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ), 
-                        __( 'Active Plugins', 'admin-page-framework' )              => PHP_EOL . $this->_getActivePlugins(),
-                        __( 'Network Active Plugins', 'admin-page-framework' )      => PHP_EOL . $this->_getNetworkActivePlugins(),
+                        __( 'Active Plugins', 'admin-page-framework' )              => PHP_EOL.$this->_getActivePlugins(),
+                        __( 'Network Active Plugins', 'admin-page-framework' )      => PHP_EOL.$this->_getNetworkActivePlugins(),
                         __( 'Constants', 'admin-page-framework' )                   => $this->_getDefinedConstants( 'user' ),
                     );                        
                 }
@@ -425,9 +425,9 @@ 
                      * @return      string|array
                      * @internal
                      */
-                    private function _getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {
+                    private function _getDefinedConstants( $asCategories = null, $asRemovingCategories = null ) {
                         $_asConstants = $this->getDefinedConstants( $asCategories, $asRemovingCategories );
-                        if ( ! is_array( $_asConstants ) ) {
+                        if ( !is_array( $_asConstants ) ) {
                             return $_asConstants;
                         }
                         if ( isset( $_asConstants[ 'user' ] ) ) {
@@ -463,13 +463,13 @@ 
                 private function _getActiveThemeName() {
                     
                     // If the WordPress version is less than 3.4,
-                    if ( version_compare( $GLOBALS['wp_version'], '3.4', '<' ) ) {
-                        $_aThemeData = get_theme_data( get_stylesheet_directory() . '/style.css' );
-                        return $_aThemeData['Name'] . ' ' . $_aThemeData['Version'];
+                    if ( version_compare( $GLOBALS[ 'wp_version' ], '3.4', '<' ) ) {
+                        $_aThemeData = get_theme_data( get_stylesheet_directory().'/style.css' );
+                        return $_aThemeData[ 'Name' ].' '.$_aThemeData[ 'Version' ];
                     } 
                     
                     $_oThemeData = wp_get_theme();
-                    return $_oThemeData->Name . ' ' . $_oThemeData->Version;
+                    return $_oThemeData->Name.' '.$_oThemeData->Version;
                     
                 }   
                 /**
@@ -483,10 +483,10 @@ 
                     $_aPluginList       = array();
                     $_aActivePlugins    = get_option( 'active_plugins', array() );
                     foreach ( get_plugins() as $_sPluginPath => $_aPlugin ) {
-                        if ( ! in_array( $_sPluginPath, $_aActivePlugins ) ) {
+                        if ( !in_array( $_sPluginPath, $_aActivePlugins ) ) {
                             continue;
                         }
-                        $_aPluginList[] = '    ' . $_aPlugin['Name'] . ': ' . $_aPlugin['Version'];
+                        $_aPluginList[ ] = '    '.$_aPlugin[ 'Name' ].': '.$_aPlugin[ 'Version' ];
                     }
                     return implode( PHP_EOL, $_aPluginList );
                     
@@ -498,17 +498,17 @@ 
                  */
                 private function _getNetworkActivePlugins() {
                     
-                    if ( ! is_multisite() ) {
+                    if ( !is_multisite() ) {
                         return '';
                     }
                     $_aPluginList       = array();
                     $_aActivePlugins    = get_site_option( 'active_sitewide_plugins', array() );
                     foreach ( wp_get_active_network_plugins() as $_sPluginPath ) {
-                        if ( ! array_key_exists( plugin_basename( $_sPluginPath ), $_aActivePlugins ) ) {
+                        if ( !array_key_exists( plugin_basename( $_sPluginPath ), $_aActivePlugins ) ) {
                             continue;
                         }
                         $_aPlugin       = get_plugin_data( $_sPluginPath );
-                        $_aPluginList[] = '    ' . $_aPlugin['Name'] . ' :' . $_aPlugin['Version'];
+                        $_aPluginList[ ] = '    '.$_aPlugin[ 'Name' ].' :'.$_aPlugin[ 'Version' ];
                     }
                     return implode( PHP_EOL, $_aPluginList );
                     
@@ -526,7 +526,7 @@ 
                     $_vResponse = false === $_vResponse
                         ? wp_remote_post( 
                             // 'https://www.paypal.com/cgi-bin/webscr', 
-                            add_query_arg( $_GET, admin_url( $GLOBALS['pagenow'] ) ),
+                            add_query_arg( $_GET, admin_url( $GLOBALS[ 'pagenow' ] ) ),
                             array(
                                 'sslverify'     => false,
                                 'timeout'       => 60,
@@ -549,7 +549,7 @@ 
                     $_vResponse = $this->getTransient( 'apf_rg_check' );
                     $_vResponse = false === $_vResponse
                         ? wp_remote_get( 
-                            add_query_arg( $_GET + array( 'apf_remote_request_test' => '_testing' ), admin_url( $GLOBALS['pagenow'] ) ),
+                            add_query_arg( $_GET + array( 'apf_remote_request_test' => '_testing' ), admin_url( $GLOBALS[ 'pagenow' ] ) ),
                             array(
                                 'sslverify'     => false,
                                 'timeout'       => 60,
@@ -574,7 +574,7 @@ 
                         if ( is_wp_error( $mResponse ) ) {
                             return true;
                         }
-                        if ( $mResponse[ 'response'][ 'code' ] < 200 ) {
+                        if ( $mResponse[ 'response' ][ 'code' ] < 200 ) {
                             return true;
                         }
                         if ( $mResponse[ 'response' ][ 'code' ] >= 300 ) {
@@ -597,13 +597,13 @@ 
              * @since       3.4.6
              * @since       3.5.3       Added the $bGenerateInfo parameter. This is to reduce conditional statement in the caller method.
              */
-            private function _getPHPInfo( $bGenerateInfo=true ) {
+            private function _getPHPInfo( $bGenerateInfo = true ) {
                 
-                if ( ! $bGenerateInfo || isset( self::$_aPHPInfo ) ) {
+                if ( !$bGenerateInfo || isset( self::$_aPHPInfo ) ) {
                     return self::$_aPHPInfo;
                 }
                 
-                $_oErrorReporting   = new AdminPageFramework_ErrorReporting;
+                $_oErrorReporting = new AdminPageFramework_ErrorReporting;
                 self::$_aPHPInfo = array(
                     __( 'Version', 'admin-page-framework' )                 => phpversion(),
                     __( 'Safe Mode', 'admin-page-framework' )               => $this->getAOrB( @ini_get( 'safe_mode' ), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ), 
@@ -614,16 +614,16 @@ 
                     __( 'Max Execution Time', 'admin-page-framework' )      => @ini_get( 'max_execution_time' ),
                     __( 'Max Input Vars', 'admin-page-framework' )          => @ini_get( 'max_input_vars' ),
                     __( 'Argument Separator', 'admin-page-framework' )      => @ini_get( 'arg_separator.output' ),
-                    __( 'Allow URL File Open', 'admin-page-framework' )     => $this->getAOrB( @ini_get( 'allow_url_fopen' ),    $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ),
-                    __( 'Display Errors', 'admin-page-framework' )          => $this->getAOrB( @ini_get( 'display_errors' ),     $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
-                    __( 'Log Errors', 'admin-page-framework' )              => $this->getAOrB( @ini_get( 'log_errors' ),         $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
+                    __( 'Allow URL File Open', 'admin-page-framework' )     => $this->getAOrB( @ini_get( 'allow_url_fopen' ), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ),
+                    __( 'Display Errors', 'admin-page-framework' )          => $this->getAOrB( @ini_get( 'display_errors' ), $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
+                    __( 'Log Errors', 'admin-page-framework' )              => $this->getAOrB( @ini_get( 'log_errors' ), $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
                     __( 'Error log location', 'admin-page-framework' )      => @ini_get( 'error_log' ),
                     __( 'Error Reporting Level', 'admin-page-framework' )   => $_oErrorReporting->getErrorLevel(),
-                    __( 'FSOCKOPEN', 'admin-page-framework' )               => $this->getAOrB( function_exists( 'fsockopen' ),   $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    __( 'cURL', 'admin-page-framework' )                    => $this->getAOrB( function_exists( 'curl_init' ),   $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    __( 'SOAP', 'admin-page-framework' )                    => $this->getAOrB( class_exists( 'SoapClient' ),     $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    __( 'SUHOSIN', 'admin-page-framework' )                 => $this->getAOrB( extension_loaded( 'suhosin' ),    $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    'ini_set()'                                             => $this->getAOrB( function_exists( 'ini_set' ),     $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'FSOCKOPEN', 'admin-page-framework' )               => $this->getAOrB( function_exists( 'fsockopen' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'cURL', 'admin-page-framework' )                    => $this->getAOrB( function_exists( 'curl_init' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'SOAP', 'admin-page-framework' )                    => $this->getAOrB( class_exists( 'SoapClient' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'SUHOSIN', 'admin-page-framework' )                 => $this->getAOrB( extension_loaded( 'suhosin' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    'ini_set()'                                             => $this->getAOrB( function_exists( 'ini_set' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
                 ) 
                 + $this->getPHPInfo()
                 + array( 
@@ -642,11 +642,11 @@ 
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              * @return      array|string
              */                      
-            private function _getWebServerInfo( $bGenerateInfo=true ) {
+            private function _getWebServerInfo( $bGenerateInfo = true ) {
                         
                 return $bGenerateInfo 
                     ? array(
-                        __( 'Web Server', 'admin-page-framework' )                  => $_SERVER['SERVER_SOFTWARE'],
+                        __( 'Web Server', 'admin-page-framework' )                  => $_SERVER[ 'SERVER_SOFTWARE' ],
                         'SSL'                                                       => $this->getAOrB( is_ssl(), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ),
                         __( 'Session', 'admin-page-framework' )                     => $this->getAOrB( isset( $_SESSION ), $this->oMsg->get( 'enabled' ), $this->oMsg->get( 'disabled' ) ),
                         __( 'Session Name', 'admin-page-framework' )                => esc_html( @ini_get( 'session.name' ) ),

Doc Comments +3 added lines

@@ -297,6 +297,7 @@ 
             /**
              * Returns the system information by section.
              * @since       3.5.3
+             * @param integer $iPrintType
              * @return      string      The system information by section.
              * @internal
              */
@@ -421,6 +422,8 @@ 
                     /**
                      * 
                      * @since       3.5.12
+                     * @param string $asCategories
+                     * @param string $asRemovingCategories
                      * @return      string|array
                      * @internal
                      */

Braces +5 added lines, -5 removed lines

@@ -143,7 +143,7 @@ 
      * @internal
      * @return      array
      */
-    protected function getEnqueuingScripts() { 
+    protected function getEnqueuingScripts() {
         return array();
     }
     
@@ -161,7 +161,7 @@ 
      * @return      array
      * @internal
      */
-    protected function getEnqueuingStyles() { 
+    protected function getEnqueuingStyles() {
         return array();
     }            
 
@@ -171,7 +171,7 @@ 
      * @internal
      * @return      string
      */ 
-    protected function getScripts() { 
+    protected function getScripts() {
         return '';
     }
 
@@ -181,7 +181,7 @@ 
      * @internal
      * @return      string
      */
-    protected function getIEStyles() { 
+    protected function getIEStyles() {
         return ''; 
     }
 
@@ -222,7 +222,7 @@ 
      * @internal
      * @return      string
      */
-    protected function getField( $aField ) { 
+    protected function getField( $aField ) {
 
         $_aInputAttributes             = $aField[ 'attributes' ];
         $_aInputAttributes[ 'class' ] .= ' system';

development/factory/_common/form/field_type/AdminPageFramework_FieldType_textarea.php

Spacing +23 added lines, -23 removed lines

@@ -554,8 +554,8 @@ 
     protected function getField( $aField ) {
 
         $_aOutput = array();
-        foreach( ( array ) $aField[ 'label' ] as $_sKey => $_sLabel ) {
-            $_aOutput[] = $this->_getFieldOutputByLabel( 
+        foreach ( ( array ) $aField[ 'label' ] as $_sKey => $_sLabel ) {
+            $_aOutput[ ] = $this->_getFieldOutputByLabel( 
                 $_sKey, 
                 $_sLabel, 
                 $aField
@@ -563,7 +563,7 @@ 
         }
         
         // the repeatable field buttons will be replaced with this element.
-        $_aOutput[] = "<div class='repeatable-field-buttons'></div>";
+        $_aOutput[ ] = "<div class='repeatable-field-buttons'></div>";
         return implode( '', $_aOutput );
         
     }
@@ -584,27 +584,27 @@ 
             $aField[ 'rich' ]       = $this->getElementByLabel( $aField[ 'rich' ], $sKey, $aField[ 'label' ] );
             $aField[ 'attributes' ] = $_bIsArray
                 ? array(
-                        'name'  => $aField[ 'attributes' ][ 'name' ] . "[{$sKey}]",
-                        'id'    => $aField[ 'attributes' ][ 'id' ] . "_{$sKey}",
+                        'name'  => $aField[ 'attributes' ][ 'name' ]."[{$sKey}]",
+                        'id'    => $aField[ 'attributes' ][ 'id' ]."_{$sKey}",
                         'value' => $aField[ 'value' ],
                     ) 
                     + $aField[ 'attributes' ]
                 : $aField[ 'attributes' ];        
-            $_aOutput           = array(
-                $this->getElementByLabel( $aField['before_label'], $sKey, $aField[ 'label' ] ),
+            $_aOutput = array(
+                $this->getElementByLabel( $aField[ 'before_label' ], $sKey, $aField[ 'label' ] ),
                 "<div class='admin-page-framework-input-label-container {$_sClassSelector}'>",
-                    "<label for='" . $aField[ 'attributes' ][ 'id' ] . "'>",
-                        $this->getElementByLabel( $aField['before_input'], $sKey, $aField[ 'label' ] ),
+                    "<label for='".$aField[ 'attributes' ][ 'id' ]."'>",
+                        $this->getElementByLabel( $aField[ 'before_input' ], $sKey, $aField[ 'label' ] ),
                         $_sLabel 
-                            ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                            ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                     . $_sLabel
                                 . "</span>"
                             : '',
                         $this->_getEditor( $aField ),
-                        $this->getElementByLabel( $aField['after_input'], $sKey, $aField[ 'label' ] ),
+                        $this->getElementByLabel( $aField[ 'after_input' ], $sKey, $aField[ 'label' ] ),
                     "</label>",
                 "</div>",
-                $this->getElementByLabel( $aField['after_label'], $sKey, $aField[ 'label' ] ),
+                $this->getElementByLabel( $aField[ 'after_label' ], $sKey, $aField[ 'label' ] ),
             );
             return implode( '', $_aOutput );
   
@@ -619,31 +619,31 @@ 
          */
         private function _getEditor( $aField ) {
                         
-            unset( $aField['attributes']['value'] );
+            unset( $aField[ 'attributes' ][ 'value' ] );
             
             // For no TinyMCE
-            if ( empty( $aField['rich'] ) || ! $this->isTinyMCESupported() ) {
-                return "<textarea " . $this->getAttributes( $aField['attributes'] ) . " >" // this method is defined in the base class
-                            . esc_textarea( $aField['value'] )
+            if ( empty( $aField[ 'rich' ] ) || !$this->isTinyMCESupported() ) {
+                return "<textarea ".$this->getAttributes( $aField[ 'attributes' ] )." >" // this method is defined in the base class
+                            . esc_textarea( $aField[ 'value' ] )
                         . "</textarea>";
             }
 
             // Rich editor
             ob_start();
             wp_editor( 
-                $aField['value'],
-                $aField['attributes']['id'],  
+                $aField[ 'value' ],
+                $aField[ 'attributes' ][ 'id' ],  
                 $this->uniteArrays( 
-                    ( array ) $aField['rich'],
+                    ( array ) $aField[ 'rich' ],
                     array(
                         'wpautop'           => true, // use wpautop?
                         'media_buttons'     => true, // show insert/upload button(s)
-                        'textarea_name'     => $aField['attributes']['name'],
-                        'textarea_rows'     => $aField['attributes']['rows'],
+                        'textarea_name'     => $aField[ 'attributes' ][ 'name' ],
+                        'textarea_rows'     => $aField[ 'attributes' ][ 'rows' ],
                         'tabindex'          => '',
                         'tabfocus_elements' => ':prev,:next', // the previous and next element ID to move the focus to when pressing the Tab key in TinyMCE
                         'editor_css'        => '', // intended for extra styles for both visual and Text editors buttons, needs to include the <style> tags, can use "scoped".
-                        'editor_class'      => $aField['attributes']['class'], // add extra class(es) to the editor textarea
+                        'editor_class'      => $aField[ 'attributes' ][ 'class' ], // add extra class(es) to the editor textarea
                         'teeny'             => false, // output the minimal editor config used in Press This
                         'dfw'               => false, // replace the default fullscreen with DFW (needs specific DOM elements and css)
                         'tinymce'           => true, // load TinyMCE, can be used to pass settings directly to TinyMCE using an array()
@@ -655,7 +655,7 @@ 
             ob_end_clean();
             
             return $_sContent
-                . $this->_getScriptForRichEditor( $aField['attributes']['id'] );
+                . $this->_getScriptForRichEditor( $aField[ 'attributes' ][ 'id' ] );
             
         }
     

development/factory/_common/form/field_type/AdminPageFramework_FieldType_default.php

Spacing +8 added lines, -8 removed lines

@@ -37,21 +37,21 @@
      */
     public function _replyToGetField( $aField ) {
         return 
-            $aField['before_label']
+            $aField[ 'before_label' ]
             . "<div class='admin-page-framework-input-label-container'>"
-                . "<label for='{$aField['input_id']}'>"
-                    . $aField['before_input']
-                    . ( $aField['label'] && ! $aField['repeatable']
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                . "<label for='{$aField[ 'input_id' ]}'>"
+                    . $aField[ 'before_input' ]
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                 . $aField[ 'label' ]
                             . "</span>"
                         : "" 
                     )
-                    . $aField['value']
-                    . $aField['after_input']
+                    . $aField[ 'value' ]
+                    . $aField[ 'after_input' ]
                 . "</label>"
             . "</div>"
-            . $aField['after_label']
+            . $aField[ 'after_label' ]
         ;     
     }
 

development/factory/admin_page/_view/AdminPageFramework_View__Resource.php

Spacing +13 added lines, -13 removed lines

@@ -68,9 +68,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.
@@ -84,23 +84,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' )
             );              
@@ -117,10 +117,10 @@ 
              * @since       3.6.3
              * @return      void
              */    
-            private function _enqueuePageAssets( array $aPageAssets, $sType='style' ) {
-                $_sMathodName = "_enqueueAsset_" . $sType;
-                foreach( $aPageAssets as $_asPageAsset ) {
-                    $this->{$_sMathodName}( $_asPageAsset);
+            private function _enqueuePageAssets( array $aPageAssets, $sType = 'style' ) {
+                $_sMathodName = "_enqueueAsset_".$sType;
+                foreach ( $aPageAssets as $_asPageAsset ) {
+                    $this->{$_sMathodName}( $_asPageAsset );
                 }
             }
 
@@ -145,7 +145,7 @@ 
                     }
                     
                     // Insert the CSS rule in the head tag.
-                    $this->aCSSRules[] = $_sSRC;
+                    $this->aCSSRules[ ] = $_sSRC;
                     
                 }                    
      
@@ -170,7 +170,7 @@ 
                     }                
                     
                     // Insert the scripts in the head tag.                
-                    $this->aScripts[] = $_sSRC;
+                    $this->aScripts[ ] = $_sSRC;
                     
                 }        
                 

development/factory/admin_page/_model/delegate/AdminPageFramework_Model_Menu__RegisterMenu.php

Braces +5 added lines, -5 removed lines

@@ -167,7 +167,7 @@ 
         private function _getDefaultPageSlug( array $aPages ) {
 
             foreach( $aPages as $_aPage ) {
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) { 
+                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
                     continue; 
                 }
                 return $_aPage[ 'page_slug' ];                
@@ -379,7 +379,7 @@ 
                     
                     foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
                       
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) { 
+                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
                             continue; 
                         }
                                                
@@ -394,7 +394,7 @@ 
                             $sPageTitle,
                             $sPageSlug,
                         );
-                        if ( $_aA !== $_aB ) { 
+                        if ( $_aA !== $_aB ) {
                             continue;
                         }
                         return $_iIndex;
@@ -411,7 +411,7 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
                  * @return      array       removed menu item.
                  */
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
  
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex( 
                         $nSubMenuPageIndex,
@@ -517,7 +517,7 @@ 
      * @callback    action      admin_menu
      * @internal
      */    
-    public function _replyToRemoveRootMenuPage() {      
+    public function _replyToRemoveRootMenuPage() {
         
         // After adding the sub menus, if the root menu is created, remove the page that is automatically created when registering the root menu.
         if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {

Doc Comments +5 added lines, -1 removed lines

@@ -26,6 +26,7 @@ 
      * Sets up hooks
      * 
      * @internal
+     * @param AdminPageFramework_Router $oFactory
      */
     public function __construct( $oFactory, $sActionHook='admin_menu' ) {
         
@@ -253,6 +254,7 @@ 
              * @since       3.3.0
              * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
              * @since       3.7.4       Added the `$nOrder` parameter.
+             * @param string $sMenuSlug
              * @return      string      The page hook of the added page.
              */
             private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu, $nOrder ) {
@@ -354,6 +356,7 @@ 
                 /**
                  * Sets up hooks for the page.
                  * @since       3.7.4
+                 * @param false|string $sPageHook
                  */
                 private function _setPageHooks( $sPageHook, $sPageSlug ) {
                                     
@@ -405,7 +408,7 @@ 
                 /**
                  * @since       3.7.4
                  * @return      void
-                 * @param       numeric     $$nOrder            A user set order (menu position, index).
+                 * @param       numeric     $nOrder            A user set order (menu position, index).
                  * @param       array       $aSubMenuItem       The sub menu item array set in the global `$submenu` array.
                  */
                 private function _setSubMenuPageByIndex( $nOrder, $aSubMenuItem, $sMenuSlug ) {
@@ -529,6 +532,7 @@ 
              * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
              * @since       3.5.3       Added the `$bShowInMenu` parameter.
              * @since       3.7.4       Added the `$nOrder` parameter.
+             * @param string $sMenuSlug
              * @return      void
              */
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {

Spacing +47 added lines, -47 removed lines

@@ -27,7 +27,7 @@ 
      * 
      * @internal
      */
-    public function __construct( $oFactory, $sActionHook='admin_menu' ) {
+    public function __construct( $oFactory, $sActionHook = 'admin_menu' ) {
         
         $this->oFactory = $oFactory;
         
@@ -75,7 +75,7 @@ 
     public function _replyToRegisterRootMenu() {
         
         // If the root menu label is not set but the slug is set, 
-        if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
+        if ( !$this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
             return;
         }
         $this->_registerRootMenuPage();
@@ -97,12 +97,12 @@ 
             }
 
             $this->oFactory->oProp->aRootMenu[ '_page_hook' ] = add_menu_page(  
-                $this->oFactory->oProp->sClassName,                 // Page title - will be invisible anyway
-                $this->oFactory->oProp->aRootMenu[ 'sTitle' ],      // Menu title - should be the root page title.
-                $this->oFactory->oProp->sCapability,                // Capability - access right
-                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ],   // Menu ID 
-                '',                                       // Callback function for the page content output - the root page will be removed so no need to register a function.
-                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ],  // icon path
+                $this->oFactory->oProp->sClassName, // Page title - will be invisible anyway
+                $this->oFactory->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title.
+                $this->oFactory->oProp->sCapability, // Capability - access right
+                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID 
+                '', // Callback function for the page content output - the root page will be removed so no need to register a function.
+                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ], // icon path
                 $this->getElement( 
                     $this->oFactory->oProp->aRootMenu, 
                     'iPosition', 
@@ -124,8 +124,8 @@ 
 
         // Let external scripts add sub-menu pages.
         $_aPages = $this->addAndApplyFilter( 
-            $this->oFactory,    // caller object
-            "pages_{$this->oFactory->oProp->sClassName}",   // filter
+            $this->oFactory, // caller object
+            "pages_{$this->oFactory->oProp->sClassName}", // filter
             $this->oFactory->oProp->aPages  // arguments
         );
         
@@ -135,7 +135,7 @@ 
         // Format the `$aPages` property and register the pages.
         $_iParsedIndex    = 0;
         $_aFormattedPages = array();
-        foreach( $_aPages as $_aSubMenuItem ) {
+        foreach ( $_aPages as $_aSubMenuItem ) {
             
             // needs to be sanitized because there are hook filters applied to this array.
             $_oFormatter = new AdminPageFramework_Format_SubMenuItem( 
@@ -170,8 +170,8 @@ 
          */
         private function _getDefaultPageSlug( array $aPages ) {
 
-            foreach( $aPages as $_aPage ) {
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) { 
+            foreach ( $aPages as $_aPage ) {
+                if ( !isset( $_aPage[ 'page_slug' ] ) ) { 
                     continue; 
                 }
                 return $_aPage[ 'page_slug' ];                
@@ -193,7 +193,7 @@ 
          */ 
         private function _registerSubMenuItem( array $aArgs ) {
 
-            if ( ! current_user_can( $aArgs[ 'capability' ] ) ) {
+            if ( !current_user_can( $aArgs[ 'capability' ] ) ) {
                 return '';
             }
                                     
@@ -257,14 +257,14 @@ 
              */
             private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu, $nOrder ) {
                 
-                if ( ! $sPageSlug ) {
+                if ( !$sPageSlug ) {
                     return '';
                 }
                 $_sPageHook = $this->___addSubMenuPage(
-                    $sRootPageSlug,         // the root (parent) page slug
-                    $sPageTitle,            // page title
-                    $sMenuTitle,            // menu title
-                    $sCapability,           // capability
+                    $sRootPageSlug, // the root (parent) page slug
+                    $sPageTitle, // page title
+                    $sMenuTitle, // menu title
+                    $sCapability, // capability
                     $sPageSlug              // menu slug
                 );
 
@@ -293,14 +293,14 @@ 
                 $_aRemovedMenuItem = $this->_removePageSubmenuItem( $_nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle );
                 
                 // If the visibility option is `false`, remove the one just added from the sub-menu global array
-                if ( ! $bShowInMenu && ! $this->_isCurrentPage( $sPageSlug ) ) {
+                if ( !$bShowInMenu && !$this->_isCurrentPage( $sPageSlug ) ) {
                     return $_sPageHook;
                 }                
                 
                 // Set the order index in the element of the `submenu` global array.
                 $this->_setSubMenuPageByIndex( 
-                    $nOrder,                // user-set order
-                    $_aRemovedMenuItem,     // will be reassign with a new index
+                    $nOrder, // user-set order
+                    $_aRemovedMenuItem, // will be reassign with a new index
                     $sMenuSlug
                 );
 
@@ -321,21 +321,21 @@ 
                  * @uses        add_submenu_page
                  */
                 private function ___addSubMenuPage(
-                    $sRootPageSlug,         // the root (parent) page slug
-                    $sPageTitle,            // page title
-                    $sMenuTitle,            // menu title
-                    $sCapability,           // capability
+                    $sRootPageSlug, // the root (parent) page slug
+                    $sPageTitle, // page title
+                    $sMenuTitle, // menu title
+                    $sCapability, // capability
                     $sPageSlug              // menu slug
                 ) {
                     if ( $this->oFactory->oProp->bIsAdminAjax ) {
                         return $sPageSlug;
                     }
                     return add_submenu_page(
-                        $sRootPageSlug,         // the root (parent) page slug
-                        $sPageTitle,            // page title
-                        $sMenuTitle,            // menu title
-                        $sCapability,           // capability
-                        $sPageSlug,             // menu slug
+                        $sRootPageSlug, // the root (parent) page slug
+                        $sPageTitle, // page title
+                        $sMenuTitle, // menu title
+                        $sCapability, // capability
+                        $sPageSlug, // menu slug
                         array( $this->oFactory, '_replyToRenderPage' )  // callback 3.7.10+
                     );
                 }
@@ -369,7 +369,7 @@ 
                      */
                     add_action( 
                         'current_screen', 
-                        array( $this->oFactory, "load_pre_" . $sPageSlug ), 
+                        array( $this->oFactory, "load_pre_".$sPageSlug ), 
                         20 
                     );
                     /**
@@ -378,7 +378,7 @@ 
                      */
                     add_action(
                         'pseudo_current_screen',
-                        array( $this->oFactory, "load_pre_" . $sPageSlug ),
+                        array( $this->oFactory, "load_pre_".$sPageSlug ),
                         20
                     );
                     
@@ -387,15 +387,15 @@ 
                      * Set a low priority because the user may add in-page tabs in their callback method of this action hook.
                      * @since       3.6.3
                      */
-                    add_action( "load_" . $sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
+                    add_action( "load_".$sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
                     
                     // 3.6.3+
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) );  // 3.7.10+
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) ); // 3.7.10+
                           
                     $this->oFactory->oProp->aPageHooks[ $sPageSlug ] = $this->getAOrB(
                         is_network_admin(),
-                        $sPageHook . '-network',
+                        $sPageHook.'-network',
                         $sPageHook
                     );                          
 
@@ -412,7 +412,7 @@ 
 
                     $_nNewIndex = $this->getUnusedNumericIndex( 
                         $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ), // subject array to parser
-                        $nOrder,    // a desired menu position
+                        $nOrder, // a desired menu position
                         5           // offset 
                     );
 
@@ -426,9 +426,9 @@ 
                  */
                 private function _getSubMenuPageIndex( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) {
                     
-                    foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
+                    foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
                       
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) { 
+                        if ( !isset( $_aSubMenu[ 3 ] ) ) { 
                             continue; 
                         }
                                                
@@ -460,7 +460,7 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
                  * @return      array       removed menu item.
                  */
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
  
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex( 
                         $nSubMenuPageIndex,
@@ -533,7 +533,7 @@ 
              */
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {
                 
-                if ( ! $bShowInMenu ) {
+                if ( !$bShowInMenu ) {
                     return;
                 }
     
@@ -548,9 +548,9 @@ 
                     5   // offset
                 );
                 $_aSubMenuItems[ $_nIndex ] = array(
-                    $sTitle,        // 0
-                    $sCapability,   // 1
-                    $sHref,         // 2   
+                    $sTitle, // 0
+                    $sCapability, // 1
+                    $sHref, // 2   
                 );
                 $GLOBALS[ 'submenu' ][ $sMenuSlug ] = $_aSubMenuItems;                
                 
@@ -569,7 +569,7 @@ 
     public function _replyToRemoveRootMenuPage() {      
         
         // After adding the sub menus, if the root menu is created, remove the page that is automatically created when registering the root menu.
-        if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
+        if ( !$this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
             return;
         }
         if ( $this->oFactory->oProp->bIsAdminAjax ) {
@@ -580,7 +580,7 @@ 
          * Namespace slugs have backslashes but the registered slugs backslashes are all converted to forward-slashes.
          * @since       3.5.16
          */
-        $_sMenuSlug =             str_replace(
+        $_sMenuSlug = str_replace(
             '\\',
             '/',
             $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ]

development/factory/_common/_abstract/AdminPageFramework_Factory_Model.php

Doc Comments +1 added lines, -2 removed lines

@@ -203,6 +203,7 @@ 
          * Constructs a WordPress filter hook name.
          * @internal
          * @since       3.8.4
+         * @param string $sPrefix
          * @return      string
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
@@ -339,8 +340,6 @@ 
      * @since       3.7.0      Changed back the visibility scope to protected as there is the `getFieldErrors()` public method.
      * @access      protected
      * @internal
-     * @param       string      $sID        deprecated
-     * @param       boolean     $bDelete    whether or not the transient should be deleted after retrieving it. 
      * @return      array
      * @deprecated  3.7.0      Use `getFieldErrors()` instead. Kept for backward compatibility.
      */

Braces +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
      * @since       3.7.0
      * @return      array       The modified section-sets definition array.
      */    
-    public function _replyToModifySectionsets( $aSectionsets ) {    
+    public function _replyToModifySectionsets( $aSectionsets ) {
         
         return $this->oUtil->addAndApplyFilter( 
             $this,  // caller factory object
@@ -364,7 +364,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

Spacing +13 added lines, -13 removed lines

@@ -31,7 +31,7 @@ 
         
         add_filter(
             // 'field_types_admin_page_framework',
-            'field_types_' . $oProp->sClassName,
+            'field_types_'.$oProp->sClassName,
             array( $this, '_replyToFilterFieldTypeDefinitions' )
         );
         
@@ -53,7 +53,7 @@ 
             'title'      => null,
             'help_aside' => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
         $this->oHelpPane->_addHelpTextForFormFields( 
@@ -73,9 +73,9 @@ 
     public function _replyToFilterFieldTypeDefinitions( $aFieldTypeDefinitions ) {
         
         // Not triggering `__call()` as the filter is fired manually in the form class.
-        if ( method_exists( $this, 'field_types_' . $this->oProp->sClassName ) ) {
+        if ( method_exists( $this, 'field_types_'.$this->oProp->sClassName ) ) {
             return call_user_func_array(
-                array( $this, 'field_types_' . $this->oProp->sClassName ),
+                array( $this, 'field_types_'.$this->oProp->sClassName ),
                 array( $aFieldTypeDefinitions )
             );
         }
@@ -96,7 +96,7 @@ 
     public function _replyToModifySectionsets( $aSectionsets ) {    
         
         return $this->oUtil->addAndApplyFilter( 
-            $this,  // caller factory object
+            $this, // caller factory object
             "sections_{$this->oProp->sClassName}", 
             $aSectionsets
         );
@@ -116,7 +116,7 @@ 
     public function _replyToModifyFieldsets( $aFieldsets, $aSectionsets ) {
 
         // Apply filters to added field-sets
-        foreach( $aFieldsets as $_sSectionPath => $_aFields ) {
+        foreach ( $aFieldsets as $_sSectionPath => $_aFields ) {
             $_aSectionPath  = explode( '|', $_sSectionPath );
             $_sFilterSuffix = implode( '_', $_aSectionPath );
             $aFieldsets[ $_sSectionPath ] = $this->oUtil->addAndApplyFilter(
@@ -125,7 +125,7 @@ 
                 $_aFields
             ); 
         }
-        $aFieldsets =  $this->oUtil->addAndApplyFilter( 
+        $aFieldsets = $this->oUtil->addAndApplyFilter( 
             $this,
             "fields_{$this->oProp->sClassName}",
             $aFieldsets
@@ -195,14 +195,14 @@ 
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
             
-            $_sFieldPart    = '_' . implode( '_', $aFieldset[ '_field_path_array' ] );
+            $_sFieldPart    = '_'.implode( '_', $aFieldset[ '_field_path_array' ] );
             $_sSectionPart  = implode( '_', $aFieldset[ '_section_path_array' ] );
             $_sSectionPart  = $this->oUtil->getAOrB(
                 '_default' === $_sSectionPart,
                 '',
-                '_' . $_sSectionPart
+                '_'.$_sSectionPart
             );
-            return $sPrefix . $this->oProp->sClassName . $_sSectionPart . $_sFieldPart;
+            return $sPrefix.$this->oProp->sClassName.$_sSectionPart.$_sFieldPart;
             
         }
                
@@ -272,9 +272,9 @@ 
     public function _replyToGetSavedFormData() {
         
         // Must update the property with the filtered value.
-        $this->oProp->aOptions =  $this->oUtil->addAndApplyFilter(
+        $this->oProp->aOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->oProp->aOptions      // subject value to be filtered
         );
         return $this->oProp->aOptions;
@@ -364,7 +364,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

development/factory/_common/form/_view/AdminPageFramework_Form_View___ToolTip.php

Doc Comments +1 added lines, -2 removed lines

@@ -42,8 +42,6 @@ 
      * Sets up properties.
      * @since       3.6.0
      * @since       3.7.0           Changed the parameter structure.
-     * @param       array|string    $asArguments        The content output or the tooltip argument array.
-     * @param       string          $sTitleElementID    Not used at the moment.
      */
     public function __construct( /* $aArguments, $sTitleElementID */ ) {
 
@@ -170,6 +168,7 @@ 
 
     /**
      * Generates HTML attributes of the specified element.
+     * @param string $sElementKey
      * @return      string
      * @since       3.8.5
      */

Spacing +14 added lines, -14 removed lines

@@ -21,7 +21,7 @@ 
     /**
      * Stores the default argument values.
      */
-    public $aArguments      = array(
+    public $aArguments = array(
         'attributes'    => array(
             'container'   => array(),   
             'title'       => array(),
@@ -29,11 +29,11 @@ 
             'description' => array(),
             'icon'        => array(),
         ), 
-        'icon'          => null,    // the icon output to override
-        'dash-icon'     => 'dashicons-editor-help',   // the dash-icon class selector for the icon
-        'icon_alt_text' => '[?]',   // [3.8.5+] alternative text when icon is not available. For WP v3.7.x or below.
+        'icon'          => null, // the icon output to override
+        'dash-icon'     => 'dashicons-editor-help', // the dash-icon class selector for the icon
+        'icon_alt_text' => '[?]', // [3.8.5+] alternative text when icon is not available. For WP v3.7.x or below.
         'title'         => null,  
-        'content'       => null,    // will be assigned in the constructor
+        'content'       => null, // will be assigned in the constructor
     );
     
     public $sTitleElementID;
@@ -90,7 +90,7 @@ 
                 if ( is_string( $asContent ) ) {
                     return true;
                 }
-                if ( is_array( $asContent ) && ! $this->isAssociative( $asContent ) ) {
+                if ( is_array( $asContent ) && !$this->isAssociative( $asContent ) ) {
                     return true;
                 }
                 return false;
@@ -104,12 +104,12 @@ 
      * @return      string      The output.
      */
     public function get() {
-        if ( ! $this->aArguments[ 'content' ] ) {
+        if ( !$this->aArguments[ 'content' ] ) {
             return '';
         }
-        return "<a " . $this->_getElementAttributes( 'container', 'admin-page-framework-form-tooltip' ) . ">"
+        return "<a ".$this->_getElementAttributes( 'container', 'admin-page-framework-form-tooltip' ).">"
                 . $this->_getTipLinkIcon()
-                . "<span " . $this->_getElementAttributes( 'content', 'admin-page-framework-form-tooltip-content' ) . ">"
+                . "<span ".$this->_getElementAttributes( 'content', 'admin-page-framework-form-tooltip-content' ).">"
                     . $this->_getTipTitle()
                     . $this->_getDescriptions()
                 . "</span>"
@@ -127,7 +127,7 @@ 
                 return $this->aArguments[ 'icon' ];
             }
             if ( version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' ) ) {
-                return "<span " . $this->_getElementAttributes( 
+                return "<span ".$this->_getElementAttributes( 
                         'icon', 
                         array(
                             'dashicons', 
@@ -145,7 +145,7 @@ 
          */
         private function _getTipTitle() {
             if ( isset( $this->aArguments[ 'title' ] ) ) {
-                return "<span " . $this->_getElementAttributes( 'title', 'admin-page-framework-form-tooltip-title' ) . ">"
+                return "<span ".$this->_getElementAttributes( 'title', 'admin-page-framework-form-tooltip-title' ).">"
                     . $this->aArguments[ 'title' ]
                     . "</span>";
             }
@@ -157,9 +157,9 @@ 
          */
         private function _getDescriptions() {         
             if ( isset( $this->aArguments[ 'content' ] ) ) {
-                return "<span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">"
+                return "<span ".$this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ).">"
                         . implode( 
-                            "</span><span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">", 
+                            "</span><span ".$this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ).">", 
                             $this->getAsArray( $this->aArguments[ 'content' ] )
                         )
                     . "</span>"
@@ -178,7 +178,7 @@ 
         $_aContainerAttributes = $this->getElementAsArray(
             $this->aArguments,
             array( 'attributes', $sElementKey )
-        ) + array( 'class' => '' ) ;
+        ) + array( 'class' => '' );
         $_aContainerAttributes[ 'class' ] = $this->getClassAttribute( 
             $_aContainerAttributes[ 'class' ], 
             $asClassSelectors 

Braces +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
  * @extends     AdminPageFramework_Form_View___Section_Base
  * @internal
  */
-class AdminPageFramework_Form_View___ToolTip extends AdminPageFramework_Form_View___Section_Base {            
+class AdminPageFramework_Form_View___ToolTip extends AdminPageFramework_Form_View___Section_Base {
   
     /**
      * Stores the default argument values.
@@ -154,7 +154,7 @@ 
          * @since       3.7.0
          * @return      string
          */
-        private function _getDescriptions() {         
+        private function _getDescriptions() {
             if ( isset( $this->aArguments[ 'content' ] ) ) {
                 return "<span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">"
                         . implode( 

development/factory/_common/form/_view/sectionset/AdminPageFramework_Form_View___FieldsetTableRow.php

Spacing +8 added lines, -8 removed lines

@@ -33,7 +33,7 @@ 
 
         $_aParameters = func_get_args() + array( 
             $this->aFieldset, 
-            $this->aSavedData,    // passed by reference. @todo: examine why it needs to be passed by reference.
+            $this->aSavedData, // passed by reference. @todo: examine why it needs to be passed by reference.
             $this->aFieldErrors, 
             $this->aFieldTypeDefinitions, 
             $this->aCallbacks, // field output element callables.        
@@ -61,14 +61,14 @@ 
         
         $aFieldset = $this->aFieldset;
 
-        if ( ! $this->isNormalPlacement( $aFieldset ) ) {
+        if ( !$this->isNormalPlacement( $aFieldset ) ) {
             return '';
         }
 
-        $_oFieldrowAttribute   = new AdminPageFramework_Form_View___Attribute_Fieldrow( 
+        $_oFieldrowAttribute = new AdminPageFramework_Form_View___Attribute_Fieldrow( 
             $aFieldset,
             array( 
-                'id'        => 'fieldrow-' . $aFieldset[ 'tag_id' ],
+                'id'        => 'fieldrow-'.$aFieldset[ 'tag_id' ],
                 'valign'    => 'top',
                 'class'     => 'admin-page-framework-fieldrow',
             )                
@@ -77,7 +77,7 @@ 
         return $this->_getFieldByContainer( 
             $aFieldset, 
             array(
-                'open_container'    => "<tr " . $_oFieldrowAttribute->get() . ">",
+                'open_container'    => "<tr ".$_oFieldrowAttribute->get().">",
                 'close_container'   => "</tr>",
                 'open_title'        => "<th>",
                 'close_title'       => "</th>",
@@ -126,13 +126,13 @@ 
                 $this->aCallbacks,
                 $this->oMsg
             );
-            $_aOutput   = array();
+            $_aOutput = array();
             if ( $aFieldset[ 'show_title_column' ] ) {
-                $_aOutput[] = $aOpenCloseTags[ 'open_title' ]
+                $_aOutput[ ] = $aOpenCloseTags[ 'open_title' ]
                         . $_oFieldTitle->get()
                     . $aOpenCloseTags[ 'close_title' ];
             }
-            $_aOutput[] = $aOpenCloseTags[ 'open_main' ]
+            $_aOutput[ ] = $aOpenCloseTags[ 'open_main' ]
                     // . call_user_func_array( $hfCallback, array( $aFieldset ) )
                     . $this->getFieldsetOutput( $aFieldset )
                 . $aOpenCloseTags[ 'close_main' ];

development/factory/_common/form/_view/AdminPageFramework_Form_View___DebugInfo.php

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
  */
 class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {            
 
-    public $sStructureType   = '';
+    public $sStructureType = '';
     
     /**
      * @since       3.8.5
@@ -57,12 +57,12 @@ 
      */
     public function get() {
         
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return '';
         }
         
         return "<div class='admin-page-framework-info'>" 
-                . $this->oMsg->get( 'debug_info' ) . ': '
+                . $this->oMsg->get( 'debug_info' ).': '
                     . $this->getFrameworkNameVersion()
             . "</div>";
         
@@ -73,7 +73,7 @@ 
          */
         private function _shouldProceed() {
 
-            if ( ! $this->callBack( $this->aCallbacks[ 'show_debug_info' ], true ) ) {
+            if ( !$this->callBack( $this->aCallbacks[ 'show_debug_info' ], true ) ) {
                 return false;
             }        
             // For the generic admin pages, do no show debug information for each section.

Braces +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  * @extends     AdminPageFramework_FrameworkUtility
  * @internal
  */
-class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {            
+class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {
 
     public $sStructureType   = '';