michaeluno / admin-page-framework

development/factory/_abstract/view/field_type/AdminPageFramework_FieldType_system.php

Doc Comments +3 added lines

@@ -231,6 +231,7 @@ 
             /**
              * Returns the system information by section.
              * @since       3.5.3
+             * @param integer $iPrintType
              * @return      string      The system information by section.
              */
             private function _getSystemInfoBySection( $sSectionName, $aData, $iPrintType ) {
@@ -350,6 +351,8 @@ 
                     /**
                      * 
                      * @since       3.5.12
+                     * @param string $asCategories
+                     * @param string $asRemovingCategories
                      * @return      stirng|array
                      */
                     private function _getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {

Spacing +69 added lines, -69 removed lines

@@ -30,7 +30,7 @@ 
      *      )
      *  );
      */
-    public $aFieldTypeSlugs = array( 'system', );
+    public $aFieldTypeSlugs = array( 'system',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -47,8 +47,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,
@@ -161,25 +161,25 @@ 
      */
     protected function getField( $aField ) { 
 
-        $_aInputAttributes           = $aField['attributes'];
-        $_aInputAttributes['class'] .= ' system';
-        unset( $_aInputAttributes['value'] );
+        $_aInputAttributes           = $aField[ 'attributes' ];
+        $_aInputAttributes[ 'class' ] .= ' system';
+        unset( $_aInputAttributes[ 'value' ] );
         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 class='admin-page-framework-input-label-string' style='min-width:" . $this->sanitizeLength( $aField['label_min_width'] ) . ";'>" . $aField['label'] . "</span>"
+                . "<label for='{$aField[ 'input_id' ]}'>"
+                    . $aField[ 'before_input' ]
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span class='admin-page-framework-input-label-string' style='min-width:".$this->sanitizeLength( $aField[ 'label_min_width' ] ).";'>".$aField[ 'label' ]."</span>"
                         : "" 
                     )
-                    . "<textarea " . $this->getAttributes( $_aInputAttributes ) . " >"    
-                        . esc_textarea( $this->_getSystemInfomation( $aField['value'], $aField['data'], $aField['print_type'] ) )
+                    . "<textarea ".$this->getAttributes( $_aInputAttributes )." >"    
+                        . esc_textarea( $this->_getSystemInfomation( $aField[ 'value' ], $aField[ 'data' ], $aField[ 'print_type' ] ) )
                     . "</textarea>"
-                    . $aField['after_input']
+                    . $aField[ 'after_input' ]
                 . "</label>"
             . "</div>"
-            . $aField['after_label'];
+            . $aField[ 'after_label' ];
         
     }    
         /**
@@ -187,15 +187,15 @@ 
          * 
          * @return      string      The human readable system information.
          */
-        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 );
             
@@ -210,18 +210,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.
@@ -237,10 +237,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;
                 }                      
             }
             /**
@@ -249,17 +249,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;
@@ -274,9 +274,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 ) {
@@ -307,9 +307,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();
@@ -342,8 +342,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' ),
                     );                        
                 }
@@ -352,9 +352,9 @@ 
                      * @since       3.5.12
                      * @return      stirng|array
                      */
-                    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' ] ) ) {
@@ -388,13 +388,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;
                     
                 }   
                 /**
@@ -407,10 +407,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 );
                     
@@ -421,17 +421,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 );
                     
@@ -448,7 +448,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,
@@ -470,7 +470,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,
@@ -494,10 +494,10 @@ 
                         if ( is_wp_error( $mResponse ) ) {
                             return true;
                         }
-                        if ( $mResponse['response']['code'] < 200 ) {
+                        if ( $mResponse[ 'response' ][ 'code' ] < 200 ) {
                             return true;
                         }
-                        if ( $mResponse['response']['code'] >= 300 ) {
+                        if ( $mResponse[ 'response' ][ 'code' ] >= 300 ) {
                             return true;
                         }
                         return false;
@@ -514,13 +514,13 @@ 
              * @since       3.4.6
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment 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' ) ), 
@@ -531,16 +531,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( 
@@ -557,11 +557,11 @@ 
              * @since       3.4.6
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              */                      
-            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' ) ),

Braces +4 added lines, -4 removed lines

@@ -94,7 +94,7 @@ 
      *     <li>**attributes** - (optional, array) [3.3.0+] attribute argument array. `array( 'async' => '', 'data-id' => '...' )`</li>
      * </ul>     
      */
-    protected function getEnqueuingScripts() { 
+    protected function getEnqueuingScripts() {
         return array();
     }
     
@@ -110,7 +110,7 @@ 
      *     <li><strong>media</strong> - ( optional, string ) the description of the field which is inserted into the after the input field tag.</li>
      * </ul>
      */
-    protected function getEnqueuingStyles() { 
+    protected function getEnqueuingStyles() {
         return array();
     }            
 
@@ -118,7 +118,7 @@ 
     /**
      * Returns the field type specific JavaScript script.
      */ 
-    protected function getScripts() { 
+    protected function getScripts() {
         return '';
     }
 
@@ -159,7 +159,7 @@ 
     /**
      * Returns the output of the field type.
      */
-    protected function getField( $aField ) { 
+    protected function getField( $aField ) {
 
         $_aInputAttributes           = $aField['attributes'];
         $_aInputAttributes['class'] .= ' system';

development/factory/_abstract/view/field_type/AdminPageFramework_FieldType_taxonomy.php

Doc Comments +1 added lines, -1 removed lines

@@ -426,7 +426,7 @@
              * 
              * @since   2.0.0
              * @param   array   $vValue This can be either an one-dimensional array ( for single field ) or a two-dimensional array ( for multiple fields ).
-             * @param   string  $sKey     
+             * @param string $sTaxonomySlug
              * @return  array   Returns an numerically indexed array holding the keys that yield true as the value.
              */ 
             private function _getSelectedKeyArray( $vValue, $sTaxonomySlug ) {

Spacing +44 added lines, -44 removed lines

@@ -21,7 +21,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'taxonomy', );
+    public $aFieldTypeSlugs = array( 'taxonomy',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -29,40 +29,40 @@ 
      * @remark  $_aDefaultKeys holds shared default key-values defined in the base class.
      */
     protected $aDefaultKeys = array(
-        'taxonomy_slugs'        => 'category',      // (array|string) This is for the taxonomy field type.
-        'height'                => '250px',         // the tab box height
-        'width'                 => null,            // 3.5.7.2+ the tab box width
-        'max_width'             => '100%',          // for the taxonomy checklist field type, since 2.1.1.     
-        'show_post_count'       => true,            // (boolean) 3.2.0+ whether or not the post count associated with the term should be displayed or not.
+        'taxonomy_slugs'        => 'category', // (array|string) This is for the taxonomy field type.
+        'height'                => '250px', // the tab box height
+        'width'                 => null, // 3.5.7.2+ the tab box width
+        'max_width'             => '100%', // for the taxonomy checklist field type, since 2.1.1.     
+        'show_post_count'       => true, // (boolean) 3.2.0+ whether or not the post count associated with the term should be displayed or not.
         'attributes'            => array(),    
-        'select_all_button'     => true,            // (boolean|string) 3.3.0+ to change the label, set the label here
-        'select_none_button'    => true,            // (boolean|string) 3.3.0+ to change the label, set the label here                
-        'label_no_term_found'   => null,            // (string) 3.3.2+  The label to display when no term is found. null needs to be set here as the default value will be assigned in the field output method.
-        'label_list_title'      => '',              // (string) 3.3.2+ The heading title string for a term list. Default: `''`. Insert an HTML custom string right before the list starts.
+        'select_all_button'     => true, // (boolean|string) 3.3.0+ to change the label, set the label here
+        'select_none_button'    => true, // (boolean|string) 3.3.0+ to change the label, set the label here                
+        'label_no_term_found'   => null, // (string) 3.3.2+  The label to display when no term is found. null needs to be set here as the default value will be assigned in the field output method.
+        'label_list_title'      => '', // (string) 3.3.2+ The heading title string for a term list. Default: `''`. Insert an HTML custom string right before the list starts.
         'query'                 => array(       // (array)  3.3.2+ Defines the default query argument.
             // see the arguments of the get_category() function: http://codex.wordpress.org/Function_Reference/get_categories
             // see the argument of the get_terms() function: http://codex.wordpress.org/Function_Reference/get_terms        
             'child_of'          => 0,
             'parent'            => '',
-            'orderby'           => 'name',      // (string) 'ID' or 'term_id' or 'name'
-            'order'             => 'ASC',       // (string) 'ASC' or 'DESC'
-            'hide_empty'        => false,       // (boolean) whether to show the terms with no post associated. Default: false
-            'hierarchical'      => true,        // (boolean) whether to show the terms as a hierarchical tree.
-            'number'            => '',          // (integer) The maximum number of the terms to show.
-            'pad_counts'        => false,       // (boolean) whether to sum up the post counts with the child post counts.
-            'exclude'           => array(),     // (string) Comma separated term IDs to exclude from the list. for example `1` will remove the 'Uncategorized' category from the list.
+            'orderby'           => 'name', // (string) 'ID' or 'term_id' or 'name'
+            'order'             => 'ASC', // (string) 'ASC' or 'DESC'
+            'hide_empty'        => false, // (boolean) whether to show the terms with no post associated. Default: false
+            'hierarchical'      => true, // (boolean) whether to show the terms as a hierarchical tree.
+            'number'            => '', // (integer) The maximum number of the terms to show.
+            'pad_counts'        => false, // (boolean) whether to sum up the post counts with the child post counts.
+            'exclude'           => array(), // (string) Comma separated term IDs to exclude from the list. for example `1` will remove the 'Uncategorized' category from the list.
             'exclude_tree'      => array(), 
-            'include'           => array(),     // (string) Comma separated term IDs to include in the list.
+            'include'           => array(), // (string) Comma separated term IDs to include in the list.
             'fields'            => 'all', 
             'slug'              => '', 
             'get'               => '', 
             'name__like'        => '',
             'description__like' => '',
             'offset'            => '', 
-            'search'            => '',          // (string) The search keyword to get the term with.
+            'search'            => '', // (string) The search keyword to get the term with.
             'cache_domain'      => 'core',
         ),
-        'queries'   => array(),         // (optional, array) 3.3.2+  Allows to set a query argument for each taxonomy. The array key must be the taxonomy slug and the value is the query argument array.
+        'queries'   => array(), // (optional, array) 3.3.2+  Allows to set a query argument for each taxonomy. The array key must be the taxonomy slug and the value is the query argument array.
     );
     
     /**
@@ -88,7 +88,7 @@ 
 
         $_aJSArray = json_encode( $this->aFieldTypeSlugs );
         
-        return parent::getScripts() . 
+        return parent::getScripts(). 
         // return 
 <<<JAVASCRIPTS
 /* For tabs */
@@ -321,7 +321,7 @@ 
     protected function getField( $aField ) {
         
         // Format
-        $aField['label_no_term_found'] = $this->getElement( 
+        $aField[ 'label_no_term_found' ] = $this->getElement( 
             $aField, 
             'label_no_term_found', 
             $this->oMsg->get( 'no_term_found' ) 
@@ -330,18 +330,18 @@ 
         // Parse
         $_aTabs         = array();
         $_aCheckboxes   = array();      
-        foreach( $this->getAsArray( $aField['taxonomy_slugs'] ) as $sKey => $sTaxonomySlug ) {
-            $_aTabs[]        = $this->_getTaxonomyTab( $aField, $sKey, $sTaxonomySlug );    
-            $_aCheckboxes[]  = $this->_getTaxonomyCheckboxes( $aField, $sKey, $sTaxonomySlug );
+        foreach ( $this->getAsArray( $aField[ 'taxonomy_slugs' ] ) as $sKey => $sTaxonomySlug ) {
+            $_aTabs[ ]        = $this->_getTaxonomyTab( $aField, $sKey, $sTaxonomySlug );    
+            $_aCheckboxes[ ]  = $this->_getTaxonomyCheckboxes( $aField, $sKey, $sTaxonomySlug );
         }
 
         // Output
-        return "<div id='tabbox-{$aField['field_id']}' class='tab-box-container categorydiv' style='max-width:{$aField['max_width']};'>"
+        return "<div id='tabbox-{$aField[ 'field_id' ]}' class='tab-box-container categorydiv' style='max-width:{$aField[ 'max_width' ]};'>"
                 . "<ul class='tab-box-tabs category-tabs'>" 
                     . implode( PHP_EOL, $_aTabs ) 
                 . "</ul>" 
                 . "<div class='tab-box-contents-container'>"
-                    . "<div class='tab-box-contents' style='height: {$aField['height']};'>"
+                    . "<div class='tab-box-contents' style='height: {$aField[ 'height' ]};'>"
                         . implode( PHP_EOL, $_aCheckboxes )
                     . "</div>"
                 . "</div>" 
@@ -358,7 +358,7 @@ 
         private function _getTaxonomyCheckboxes( array $aField, $sKey, $sTaxonomySlug ) {
             
             $_aTabBoxContainerArguments = array(
-                'id'    => "tab_{$aField['input_id']}_{$sKey}",
+                'id'    => "tab_{$aField[ 'input_id' ]}_{$sKey}",
                 'class' => 'tab-box-content',
                 'style' => $this->generateInlineCSS(
                     array(
@@ -367,9 +367,9 @@ 
                     )
                 ),
             );
-            return "<div " . $this->getAttributes( $_aTabBoxContainerArguments ) . ">"
+            return "<div ".$this->getAttributes( $_aTabBoxContainerArguments ).">"
                     . $this->getElement( $aField, array( 'before_label', $sKey ) )
-                    . "<div " . $this->getAttributes( $this->_getCheckboxContainerAttributes( $aField ) ) . "></div>"
+                    . "<div ".$this->getAttributes( $this->_getCheckboxContainerAttributes( $aField ) )."></div>"
                     . "<ul class='list:category taxonomychecklist form-no-clear'>"
                         . $this->_getTaxonomyChecklist( $aField, $sKey, $sTaxonomySlug )
                     . "</ul>"     
@@ -389,23 +389,23 @@ 
                     array(
                         'walker'                => new AdminPageFramework_WalkerTaxonomyChecklist, // the walker class instance
                         'taxonomy'              => $sTaxonomySlug, 
-                        '_name_prefix'          => is_array( $aField['taxonomy_slugs'] ) 
-                            ? "{$aField['_input_name']}[{$sTaxonomySlug}]" 
-                            : $aField['_input_name'],   // name prefix of the input
-                        '_input_id_prefix'      => $aField['input_id'],
+                        '_name_prefix'          => is_array( $aField[ 'taxonomy_slugs' ] ) 
+                            ? "{$aField[ '_input_name' ]}[{$sTaxonomySlug}]" 
+                            : $aField[ '_input_name' ], // name prefix of the input
+                        '_input_id_prefix'      => $aField[ 'input_id' ],
                         '_attributes'           => $this->getElement( 
                             $aField, 
                             array( 'attributes', $sKey ), 
                             array() 
-                        ) + $aField['attributes'],                 
+                        ) + $aField[ 'attributes' ],                 
                         
                         // checked items ( term IDs ) e.g.  array( 6, 10, 7, 15 ), 
-                        '_selected_items'       => $this->_getSelectedKeyArray( $aField['value'], $sTaxonomySlug ),
+                        '_selected_items'       => $this->_getSelectedKeyArray( $aField[ 'value' ], $sTaxonomySlug ),
                         
-                        'echo'                  => false,  // returns the output
-                        'show_post_count'       => $aField['show_post_count'],      // 3.2.0+
-                        'show_option_none'      => $aField['label_no_term_found'],  // 3.3.2+ 
-                        'title_li'              => $aField['label_list_title'],     // 3.3.2+
+                        'echo'                  => false, // returns the output
+                        'show_post_count'       => $aField[ 'show_post_count' ], // 3.2.0+
+                        'show_option_none'      => $aField[ 'label_no_term_found' ], // 3.3.2+ 
+                        'title_li'              => $aField[ 'label_list_title' ], // 3.3.2+
                     ) 
                     + $this->getAsArray( 
                         $this->getElement( 
@@ -415,7 +415,7 @@ 
                         ), 
                         true 
                     )
-                    + $aField['query']                             
+                    + $aField[ 'query' ]                             
                 );
             }
      
@@ -433,10 +433,10 @@ 
 
                 $vValue = ( array ) $vValue; // cast array because the initial value (null) may not be an array.
                 
-                if ( ! isset( $vValue[ $sTaxonomySlug ] ) ) { 
+                if ( !isset( $vValue[ $sTaxonomySlug ] ) ) { 
                     return array(); 
                 }
-                if ( ! is_array( $vValue[ $sTaxonomySlug ] ) ) { 
+                if ( !is_array( $vValue[ $sTaxonomySlug ] ) ) { 
                     return array(); 
                 }
                 
@@ -452,7 +452,7 @@ 
          */
         private function _getTaxonomyTab( array $aField, $sKey, $sTaxonomySlug ) {
             return "<li class='tab-box-tab'>"
-                    . "<a href='#tab_{$aField['input_id']}_{$sKey}'>"
+                    . "<a href='#tab_{$aField[ 'input_id' ]}_{$sKey}'>"
                         . "<span class='tab-box-tab-text'>" 
                             . $this->_getLabelFromTaxonomySlug( $sTaxonomySlug )
                         . "</span>"

Braces +2 added lines, -2 removed lines

@@ -433,10 +433,10 @@
 
                 $vValue = ( array ) $vValue; // cast array because the initial value (null) may not be an array.
                 
-                if ( ! isset( $vValue[ $sTaxonomySlug ] ) ) { 
+                if ( ! isset( $vValue[ $sTaxonomySlug ] ) ) {
                     return array(); 
                 }
-                if ( ! is_array( $vValue[ $sTaxonomySlug ] ) ) { 
+                if ( ! is_array( $vValue[ $sTaxonomySlug ] ) ) {
                     return array(); 
                 }
                 

development/factory/_abstract/view/form_part/AdminPageFramework_FormPart_SectionTitle.php

Doc Comments +2 added lines

@@ -69,6 +69,8 @@
          * @since       3.4.0       Moved from `AdminPageFramework_FormPart_Table`.
          * @since       3.6.0       Added the `$iSectionIndex` and `$aFieldTypeDefinitions` parameters.
          * @since       3.6.0       Moved from `AdminPageFramework_FormTable_Base`. Changed the name from `_getSectionTitle()`. 
+         * @param string $sTitle
+         * @param string $sTag
          * @return      string      The section title output. 
          */
         protected function _getSectionTitle( $sTitle, $sTag, $aFields, $hfFieldCallback, $iSectionIndex=null, $aFieldTypeDefinitions=array() ) {

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
          * @since       3.6.0       Moved from `AdminPageFramework_FormTable_Base`. Changed the name from `_getSectionTitle()`. 
          * @return      string      The section title output. 
          */
-        protected function _getSectionTitle( $sTitle, $sTag, $aFields, $hfFieldCallback, $iSectionIndex=null, $aFieldTypeDefinitions=array() ) {
+        protected function _getSectionTitle( $sTitle, $sTag, $aFields, $hfFieldCallback, $iSectionIndex = null, $aFieldTypeDefinitions = array() ) {
             
             $_aSectionTitleField = $this->_getSectionTitleField( $aFields, $iSectionIndex, $aFieldTypeDefinitions );
             return $_aSectionTitleField
@@ -89,7 +89,7 @@ 
              * @since       3.6.0       Added the `$iSectionIndex` parameter. Added the `$aFieldTypeDefinitions` parameter.
              */
             private function _getSectionTitleField( array $aFieldsets, $iSectionIndex, $aFieldTypeDefinitions ) {   
-                foreach( $aFieldsets as $_aFieldset ) {
+                foreach ( $aFieldsets as $_aFieldset ) {
                     // Return the first found one.
                     if ( 'section_title' === $_aFieldset[ 'type' ] ) {
                         

Braces +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
  * @since       3.6.0
  * @internal
  */
-class AdminPageFramework_FormPart_SectionTitle extends AdminPageFramework_FormPart_Base {            
+class AdminPageFramework_FormPart_SectionTitle extends AdminPageFramework_FormPart_Base {
 
     public $sTitle                  = '';
     public $sTag                    = '';
@@ -88,7 +88,7 @@ 
              * @since       3.4.0       Moved from `AdminPageFramework_FormPart_Table`.
              * @since       3.6.0       Added the `$iSectionIndex` parameter. Added the `$aFieldTypeDefinitions` parameter.
              */
-            private function _getSectionTitleField( array $aFieldsets, $iSectionIndex, $aFieldTypeDefinitions ) {   
+            private function _getSectionTitleField( array $aFieldsets, $iSectionIndex, $aFieldTypeDefinitions ) {
                 foreach( $aFieldsets as $_aFieldset ) {
                     // Return the first found one.
                     if ( 'section_title' === $_aFieldset[ 'type' ] ) {

development/factory/_abstract/view/form_part/AdminPageFramework_FormPart_Table.php

Doc Comments +6 added lines, -2 removed lines

@@ -323,6 +323,9 @@ 
              * Returns an updated sections table output array.
              * @since       3.5.3
              * @since       3.6.0       Removed the `$_sSectionID` parameter.
+             * @param string $_sSectionsID
+             * @param callable $hfSectionCallback
+             * @param callable $hfFieldCallback
              * @return      array       The updated sections table output array.
              */
             private function _getSectionsTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections, $hfSectionCallback, $hfFieldCallback  ) {
@@ -513,6 +516,7 @@ 
              * @internal
              * @since       3.5.3
              * @since       3.6.0       Removed the `$sSectionID` parameter. Added the `$aSectionset` parameter.
+             * @param string $sSectionsID
              * @return      string      The formatted sections table HTML output.
              */
             private function _getFormattedSectionsTablesOutput( array $aOutputs, $aSectionset, $sSectionsID, array $aCollapsible, $sSectionTabSlug ) {
@@ -611,9 +615,8 @@ 
      * @since       3.3.1       Now the first parameter is for the section ID not the tag ID.
      * @since       3.4.0       Changed the name from `getFormTable()`.
      * @since       3.6.0       Removed the `$sSectionID` and `$iSectionIndex` parameters.
-     * @param       integer     $iSectionIndex       The section index. Zero based.
      * @param       array       $aSection            The section definition array,
-     * @param       array       $sFields             The array holding field definition arrays.
+     * @param       array       $aFields             The array holding field definition arrays.
      * @param       callable    $hfSectionCallback   The callback for the section header output.
      * @param       callable    $hfFieldCallback     The callback for the field output.
      * @return      string
@@ -670,6 +673,7 @@ 
      * @since       3.0.0
      * @since       3.6.0       Added the `$iSectionIndex` parameter. Changed the name from `getFIeldRows`.
      * @since       3.6.0       Moved from `AdminPageFramework_FormTable_Row`.
+     * @param callable $hfCallback
      * @return      string
      */
     public function getFieldsetRows( array $aFieldsets, $hfCallback, $iSectionIndex=null ) {

Indentation +1 added lines, -1 removed lines

@@ -531,7 +531,7 @@
                  * @return      string      The generated section tab list as HTML.
                  */
                 private function _getSectionTabList( $sSectionTabSlug, array $aSectionTabList ) {
-                   return $sSectionTabSlug 
+                    return $sSectionTabSlug 
                         ? "<ul class='admin-page-framework-section-tabs nav-tab-wrapper'>" 
                             . implode( PHP_EOL, $aSectionTabList ) 
                             . "</ul>"

Spacing +62 added lines, -62 removed lines

@@ -24,7 +24,7 @@ 
      * @since 3.0.0
      * @since 3.0.4     The `$aFieldErrors` parameter was added.
      */
-    public function __construct( $aFieldTypeDefinitions, array $aFieldErrors, $oMsg=null ) {
+    public function __construct( $aFieldTypeDefinitions, array $aFieldErrors, $oMsg = null ) {
         
         $this->aFieldTypeDefinitions    = $aFieldTypeDefinitions; // used to merge the field definition array with the default field type definition. 
         $this->aFieldErrors             = $aFieldErrors;
@@ -100,9 +100,9 @@ 
         // Update the array structure by tab slug (passed by reference).
         $this->_divideElementsBySectionTabs( $aSections, $aFieldsInSections );
 
-        $_aOutput     = array();
-        foreach( $aSections as $_sSectionTabSlug => $_aSectionsBySectionTab ) {        
-            $_aOutput[] = $this->_getFormTable(
+        $_aOutput = array();
+        foreach ( $aSections as $_sSectionTabSlug => $_aSectionsBySectionTab ) {        
+            $_aOutput[ ] = $this->_getFormTable(
                 $aFieldsInSections,
                 $_sSectionTabSlug,
                 $_aSectionsBySectionTab,
@@ -126,8 +126,8 @@ 
          * @return      string
          * @remark      The first iteration item of the given array will be returned,
          */
-        private function _getSectionsFieldsType( array $aSections=array() ) {
-            foreach( $aSections as $_aSection ) {
+        private function _getSectionsFieldsType( array $aSections = array() ) {
+            foreach ( $aSections as $_aSection ) {
                 return $_aSection[ '_fields_type' ];
             }
         }    
@@ -139,18 +139,18 @@ 
         private function _getFormTable( array $aFieldsInSections, $sSectionTabSlug, array $aSectionsBySectionTab, $hfSectionCallback, $hfFieldCallback ) {
                       
             $_sSectionSet = $this->_getSectionsTables( 
-                $aSectionsBySectionTab,     // sectionset definition
+                $aSectionsBySectionTab, // sectionset definition
                 $aFieldsInSections[ $sSectionTabSlug ], // fieldset definitions
                 $hfSectionCallback, 
                 $hfFieldCallback 
             );
             return $_sSectionSet
-                ? "<div " . $this->getAttributes(
+                ? "<div ".$this->getAttributes(
                         array(
                             'class' => 'admin-page-framework-sectionset',
-                            'id'    => "sectionset-{$sSectionTabSlug}_" . md5( serialize( $aSectionsBySectionTab ) ),
+                            'id'    => "sectionset-{$sSectionTabSlug}_".md5( serialize( $aSectionsBySectionTab ) ),
                         ) 
-                    ) . ">"
+                    ).">"
                         . $_sSectionSet
                     . "</div>"
                 : '';
@@ -192,18 +192,18 @@ 
             $_aFieldsBySectionTab   = array();
             $_iIndex                = 0;
 
-            foreach( $aSections as $_sSectionID => $_aSection ) {
+            foreach ( $aSections as $_sSectionID => $_aSection ) {
 
                 // If no fields for the section, no need to add the section 
                 // unless the cusedom sectionset output is defined.
-                if ( ! isset( $aFields[ $_sSectionID ] ) && ! $this->_isCustomContentSet( $_aSection ) ) {
+                if ( !isset( $aFields[ $_sSectionID ] ) && !$this->_isCustomContentSet( $_aSection ) ) {
                     continue;
                 }
                                          
                 $_sSectionTaqbSlug = $this->getAOrB(
                     $_aSection[ 'section_tab_slug' ],
                     $_aSection[ 'section_tab_slug' ],
-                    '_default_' . ( ++$_iIndex )
+                    '_default_'.( ++$_iIndex )
                 );
                 $_aSectionsBySectionTab[ $_sSectionTaqbSlug ][ $_sSectionID ] = $_aSection;
                 $_aFieldsBySectionTab[ $_sSectionTaqbSlug ][ $_sSectionID ]   = $this->getElement(
@@ -222,9 +222,9 @@ 
              * @since       3.6.1
              * @return      boolean     True if a custom content value is set.
              */
-            private function _isCustomContentSet( array $aSection, array $aKeys=array( 'content' ) ) {
-                foreach( $aKeys as $_sKey ) {
-                    if ( ! isset( $aSection[ $_sKey ] ) ) {
+            private function _isCustomContentSet( array $aSection, array $aKeys = array( 'content' ) ) {
+                foreach ( $aKeys as $_sKey ) {
+                    if ( !isset( $aSection[ $_sKey ] ) ) {
                         continue;
                     }
                     if ( is_scalar( $aSection[ $_sKey ] ) ) {
@@ -253,9 +253,9 @@ 
                 return ''; 
             } 
             
-            $_aFirstSectionset  = $this->getFirstElement( $aSections );
+            $_aFirstSectionset = $this->getFirstElement( $aSections );
                         
-            if ( ! count( $aFieldsInSections ) ) {
+            if ( !count( $aFieldsInSections ) ) {
                 return ''; 
             }            
             
@@ -267,11 +267,11 @@ 
             );
             
             $_sThisSectionID    = $_aFirstSectionset[ 'section_id' ];
-            $_sSectionsID       = 'sections-' . $_sThisSectionID;
+            $_sSectionsID       = 'sections-'.$_sThisSectionID;
             
             $_aCollapsible      = $this->_getCollapsibleArgumentForSections( $_aFirstSectionset );
                         
-            foreach( $aSections as $_aSection ) {
+            foreach ( $aSections as $_aSection ) {
                 
                 // Need to be referred outside the loop.
                 $_sSectionID        = $_aSection[ 'section_id' ];
@@ -306,7 +306,7 @@ 
              * @since       3.6.0       Changed the first parameter to `$aSection` from `$aSections`.
              * @return      array
              */
-            protected function _getCollapsibleArgumentForSections( array $aSection=array() ) {  
+            protected function _getCollapsibleArgumentForSections( array $aSection = array() ) {  
                 // $_aCollapsible      = $this->_getCollapsibleArgument( $aSections );
                 $_oArgumentFormater = new AdminPageFramework_Format_CollapsibleSection(
                     $aSection[ 'collapsible' ],
@@ -314,7 +314,7 @@ 
                     $aSection    
                 );
                 $_aCollapsible = $this->getAsArray( $_oArgumentFormater->get() );
-                return isset( $_aCollapsible['container'] ) && 'sections' === $_aCollapsible['container'] 
+                return isset( $_aCollapsible[ 'container' ] ) && 'sections' === $_aCollapsible[ 'container' ] 
                     ? $_aCollapsible 
                     : array();            
             }
@@ -325,47 +325,47 @@ 
              * @since       3.6.0       Removed the `$_sSectionID` parameter.
              * @return      array       The updated sections table output array.
              */
-            private function _getSectionsTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections, $hfSectionCallback, $hfFieldCallback  ) {
+            private function _getSectionsTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections, $hfSectionCallback, $hfFieldCallback ) {
                                 
                 // For repeatable sections - note that sub-sections are divided field definition arrays by sub-section index, not section definition arrays.
-                $_aSubSections      = $this->getIntegerKeyElements( 
+                $_aSubSections = $this->getIntegerKeyElements( 
                     $this->getElementAsArray(
                         $aFieldsInSections, // subject
-                        $_aSection[ 'section_id' ],   // dimensional key
+                        $_aSection[ 'section_id' ], // dimensional key
                         array() // default
                     )
                 );
 
                 // If the 'save' argument is false, insert a flag that disables saving the section inputs.
-                $_aOutputs[ 'section_contents' ][] = $this->_getUnsetFlagSectionInputTag( $_aSection );
+                $_aOutputs[ 'section_contents' ][ ] = $this->_getUnsetFlagSectionInputTag( $_aSection );
                 
                 // Check sub-sections.
                 $_aOutputs[ 'count_subsections' ] = count( $_aSubSections );
                 if ( $_aOutputs[ 'count_subsections' ] ) {
 
                     // Add the repeatable sections enabler script.
-                    if ( ! empty( $_aSection[ 'repeatable' ] ) ) {
-                        $_aOutputs[ 'section_contents' ][] = AdminPageFramework_Script_RepeatableSection::getEnabler( 
+                    if ( !empty( $_aSection[ 'repeatable' ] ) ) {
+                        $_aOutputs[ 'section_contents' ][ ] = AdminPageFramework_Script_RepeatableSection::getEnabler( 
                             $_sSectionsID, 
                             $_aOutputs[ 'count_subsections' ], 
                             $_aSection[ 'repeatable' ],
                             $this->oMsg
                         );
-                        $_aOutputs[ 'section_contents' ][] = $this->_getRepeatableSectionFlagTag( $_aSection );
+                        $_aOutputs[ 'section_contents' ][ ] = $this->_getRepeatableSectionFlagTag( $_aSection );
                     }
                     // Add the sortable sections enabler script. 3.6.0+
-                    if ( ! empty( $_aSection[ 'sortable' ] ) ) {
-                        $_aOutputs[ 'section_contents' ][] = AdminPageFramework_Script_SortableSection::getEnabler( 
+                    if ( !empty( $_aSection[ 'sortable' ] ) ) {
+                        $_aOutputs[ 'section_contents' ][ ] = AdminPageFramework_Script_SortableSection::getEnabler( 
                             $_sSectionsID, 
                             $_aSection[ 'sortable' ],
                             $this->oMsg
                         );
-                        $_aOutputs[ 'section_contents' ][] = $this->_getSortableSectionFlagTag( $_aSection );
+                        $_aOutputs[ 'section_contents' ][ ] = $this->_getSortableSectionFlagTag( $_aSection );
                     }
                     
                     // Get the section tables.
                     $_aSubSections = $this->numerizeElements( $_aSubSections ); // will include the main section as well.
-                    foreach( $_aSubSections as $_iIndex => $_aFields ) { 
+                    foreach ( $_aSubSections as $_iIndex => $_aFields ) { 
 
                         $_oEachSectionArguments = new AdminPageFramework_Format_EachSection(
                             $_aSection,
@@ -433,8 +433,8 @@ 
                         array(
                             'class'                     => 'element-address',
                             'type'                      => 'hidden',
-                            'name'                      => '__repeatable_elements_' . $aSection[ '_fields_type' ] 
-                                . '[' . $aSection[ 'section_id' ] . ']',
+                            'name'                      => '__repeatable_elements_'.$aSection[ '_fields_type' ] 
+                                . '['.$aSection[ 'section_id' ].']',
                             // @todo examine whether this value should include a section index.
                             'value' => $aSection[ 'section_id' ],                            
                         )
@@ -451,8 +451,8 @@ 
                         array(
                             'class'                     => 'element-address',
                             'type'                      => 'hidden',
-                            'name'                      => '__sortable_elements_' . $aSection[ '_fields_type' ] 
-                                . '[' . $aSection[ 'section_id' ] . ']',
+                            'name'                      => '__sortable_elements_'.$aSection[ '_fields_type' ] 
+                                . '['.$aSection[ 'section_id' ].']',
                             // @todo examine whether this value should include a section index.
                             'value' => $aSection[ 'section_id' ],                            
                         )
@@ -473,8 +473,8 @@ 
                         'input',
                         array(
                             'type'  => 'hidden',
-                            'name'  => '__unset_' .  $aSection[ '_fields_type' ] . '[' . $aSection[ 'section_id' ] . ']',
-                            'value' => "__dummy_option_key|" . $aSection[ 'section_id' ],
+                            'name'  => '__unset_'.$aSection[ '_fields_type' ].'['.$aSection[ 'section_id' ].']',
+                            'value' => "__dummy_option_key|".$aSection[ 'section_id' ],
                             'class' => 'unset-element-names element-address',
                         )
                     );            
@@ -489,14 +489,14 @@ 
                 private function _getSectionTableWithTabList( array $_aOutputs, array $_aSection, $_aFields, $hfSectionCallback, $hfFieldCallback ) {
                                         
                     // Tab list
-                    $_aOutputs[ 'section_tab_list' ][] = $this->_getTabList( 
+                    $_aOutputs[ 'section_tab_list' ][ ] = $this->_getTabList( 
                         $_aSection, 
                         $_aFields, 
                         $hfFieldCallback 
                     );                
                     
                     // Section container
-                    $_aOutputs[ 'section_contents' ][] = $this->_getSectionTable( 
+                    $_aOutputs[ 'section_contents' ][ ] = $this->_getSectionTable( 
                         $_aSection, 
                         $_aFields, 
                         $hfSectionCallback, 
@@ -526,9 +526,9 @@ 
                         ? $aCollapsible[ 'title' ]
                         : '',
                     'h3',
-                    array(),  // fields
-                    null,  // field callback
-                    null,  // section index
+                    array(), // fields
+                    null, // field callback
+                    null, // section index
                     $this->aFieldTypeDefinitions,                
                     $aCollapsible, 
                     'sections',
@@ -543,7 +543,7 @@ 
                     $aOutputs[ 'count_subsections' ]
                 );
                 return $_oCollapsibleSectionTitle->get()
-                    . "<div " . $_oSectionsTablesContainerAttributes->get() . ">"
+                    . "<div ".$_oSectionsTablesContainerAttributes->get().">"
                         . $this->_getSectionTabList( $sSectionTabSlug, $aOutputs[ 'section_tab_list' ] )
                         . implode( PHP_EOL, $aOutputs[ 'section_contents' ] )
                     . "</div>";
@@ -571,21 +571,21 @@ 
              */
             private function _getTabList( array $aSection, array $aFields, $hfFieldCallback ) {
                                 
-                if ( ! $aSection['section_tab_slug'] ) {
+                if ( !$aSection[ 'section_tab_slug' ] ) {
                     return '';
                 }
                 
                 $iSectionIndex      = $aSection[ '_index' ];
 
-                $_sSectionTagID     = 'section-' . $aSection[ 'section_id' ] . '__' . $iSectionIndex;
-                $_aTabAttributes    = $aSection['attributes']['tab']
+                $_sSectionTagID     = 'section-'.$aSection[ 'section_id' ].'__'.$iSectionIndex;
+                $_aTabAttributes    = $aSection[ 'attributes' ][ 'tab' ]
                     + array(
                         'class' => 'admin-page-framework-section-tab nav-tab',
                         'id'    => "section_tab-{$_sSectionTagID}",
                         'style' => null
                     );
-                $_aTabAttributes['class'] = $this->getClassAttribute( $_aTabAttributes['class'], $aSection['class']['tab'] );  // 3.3.1+
-                $_aTabAttributes['style'] = $this->getStyleAttribute( $_aTabAttributes['style'], $aSection['hidden'] ? 'display:none' : null );  // 3.3.1+
+                $_aTabAttributes[ 'class' ] = $this->getClassAttribute( $_aTabAttributes[ 'class' ], $aSection[ 'class' ][ 'tab' ] ); // 3.3.1+
+                $_aTabAttributes[ 'style' ] = $this->getStyleAttribute( $_aTabAttributes[ 'style' ], $aSection[ 'hidden' ] ? 'display:none' : null ); // 3.3.1+
                 
                 $_oSectionTitle = new AdminPageFramework_FormPart_SectionTitle(
                     $aSection[ 'title' ],
@@ -596,7 +596,7 @@ 
                     $this->aFieldTypeDefinitions
                 );                        
                 
-                return "<li " . $this->getAttributes( $_aTabAttributes ) . ">"
+                return "<li ".$this->getAttributes( $_aTabAttributes ).">"
                     . "<a href='#{$_sSectionTagID}'>"
                         . $_oSectionTitle->get()
                     ."</a>"
@@ -642,9 +642,9 @@ 
         $_oSectionTableBodyAttributes = new AdminPageFramework_Attribute_SectionTableBody( $aSection );
         
         $_aOutput       = array();
-        $_aOutput[]     = "<table " . $_oSectionTableAttributes->get() . ">"
+        $_aOutput[ ]     = "<table ".$_oSectionTableAttributes->get().">"
                 . $_oTableCaption->get()
-                . "<tbody " . $_oSectionTableBodyAttributes->get() . ">"
+                . "<tbody ".$_oSectionTableBodyAttributes->get().">"
                     . (
                         $aSection[ 'content' ]
                             ? "<tr>"
@@ -657,8 +657,8 @@ 
                 . "</tbody>"
             . "</table>";
         
-        $_oSectionTableContainerAttributes  = new AdminPageFramework_Attribute_SectionTableContainer( $aSection );
-        return "<div " . $_oSectionTableContainerAttributes->get() . ">"
+        $_oSectionTableContainerAttributes = new AdminPageFramework_Attribute_SectionTableContainer( $aSection );
+        return "<div ".$_oSectionTableContainerAttributes->get().">"
                 . implode( PHP_EOL, $_aOutput )
             . "</div>";
         
@@ -672,14 +672,14 @@ 
      * @since       3.6.0       Moved from `AdminPageFramework_FormTable_Row`.
      * @return      string
      */
-    public function getFieldsetRows( array $aFieldsets, $hfCallback, $iSectionIndex=null ) {
+    public function getFieldsetRows( array $aFieldsets, $hfCallback, $iSectionIndex = null ) {
         
-        if ( ! is_callable( $hfCallback ) ) { 
+        if ( !is_callable( $hfCallback ) ) { 
             return ''; 
         }
         
         $_aOutput = array();
-        foreach( $aFieldsets as $_aFieldset ) {
+        foreach ( $aFieldsets as $_aFieldset ) {
             
             $_oFieldsetOutputFormatter = new AdminPageFramework_Format_FieldsetOutput( 
                 $_aFieldset, 
@@ -690,7 +690,7 @@ 
                 $_oFieldsetOutputFormatter->get(), 
                 $hfCallback             
             );
-            $_aOutput[] = $_oFieldsetRow->get();
+            $_aOutput[ ] = $_oFieldsetRow->get();
 
         } 
         return implode( PHP_EOL, $_aOutput );
@@ -708,12 +708,12 @@ 
      */
     public function getFieldsets( array $aFieldsets, $hfCallback ) {
         
-        if ( ! is_callable( $hfCallback ) ) { 
+        if ( !is_callable( $hfCallback ) ) { 
             return ''; 
         }
         
         $_aOutput = array();
-        foreach( $aFieldsets as $_aFieldset ) {
+        foreach ( $aFieldsets as $_aFieldset ) {
             $_oFieldsetOutputFormatter = new AdminPageFramework_Format_FieldsetOutput( 
                 $_aFieldset, 
                 null, // section index
@@ -723,7 +723,7 @@ 
                 $_oFieldsetOutputFormatter->get(),
                 $hfCallback             
             );
-            $_aOutput[]    = $_oFieldsetRow->get();
+            $_aOutput[ ] = $_oFieldsetRow->get();
 
         }
         return implode( PHP_EOL, $_aOutput );

Braces +7 added lines, -7 removed lines

@@ -101,7 +101,7 @@ 
         $this->_divideElementsBySectionTabs( $aSections, $aFieldsInSections );
 
         $_aOutput     = array();
-        foreach( $aSections as $_sSectionTabSlug => $_aSectionsBySectionTab ) {        
+        foreach( $aSections as $_sSectionTabSlug => $_aSectionsBySectionTab ) {
             $_aOutput[] = $this->_getFormTable(
                 $aFieldsInSections,
                 $_sSectionTabSlug,
@@ -249,7 +249,7 @@ 
          */
         private function _getSectionsTables( $aSections, $aFieldsInSections, $hfSectionCallback, $hfFieldCallback ) {
 
-            if ( empty( $aSections ) ) { 
+            if ( empty( $aSections ) ) {
                 return ''; 
             } 
             
@@ -306,7 +306,7 @@ 
              * @since       3.6.0       Changed the first parameter to `$aSection` from `$aSections`.
              * @return      array
              */
-            protected function _getCollapsibleArgumentForSections( array $aSection=array() ) {  
+            protected function _getCollapsibleArgumentForSections( array $aSection=array() ) {
                 // $_aCollapsible      = $this->_getCollapsibleArgument( $aSections );
                 $_oArgumentFormater = new AdminPageFramework_Format_CollapsibleSection(
                     $aSection[ 'collapsible' ],
@@ -365,7 +365,7 @@ 
                     
                     // Get the section tables.
                     $_aSubSections = $this->numerizeElements( $_aSubSections ); // will include the main section as well.
-                    foreach( $_aSubSections as $_iIndex => $_aFields ) { 
+                    foreach( $_aSubSections as $_iIndex => $_aFields ) {
 
                         $_oEachSectionArguments = new AdminPageFramework_Format_EachSection(
                             $_aSection,
@@ -466,7 +466,7 @@ 
                  */
                 private function _getUnsetFlagSectionInputTag( array $aSection ) {
                     
-                    if ( false !== $aSection[ 'save' ] ) {                
+                    if ( false !== $aSection[ 'save' ] ) {
                         return '';
                     }
                     return $this->getHTMLTag( 
@@ -674,7 +674,7 @@ 
      */
     public function getFieldsetRows( array $aFieldsets, $hfCallback, $iSectionIndex=null ) {
         
-        if ( ! is_callable( $hfCallback ) ) { 
+        if ( ! is_callable( $hfCallback ) ) {
             return ''; 
         }
         
@@ -708,7 +708,7 @@ 
      */
     public function getFieldsets( array $aFieldsets, $hfCallback ) {
         
-        if ( ! is_callable( $hfCallback ) ) { 
+        if ( ! is_callable( $hfCallback ) ) {
             return ''; 
         }
         

development/factory/_abstract/view/form_part/fieldset/AdminPageFramework_FormFieldset_Base.php

Doc Comments +2 added lines

@@ -176,6 +176,7 @@ 
      * Returns the repeatable fields script.
      * 
      * @since 2.1.3
+     * @param string $sFieldsContainerID
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
 
@@ -222,6 +223,7 @@ 
      * Returns the sortable fields script.
      * 
      * @since 3.0.0
+     * @param string $sFieldsContainerID
      */    
     protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
     

Indentation +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
         $_sButtons              = 
             "<div class='admin-page-framework-repeatable-field-buttons' {$_sSettingsAttributes} >"
                 . "<a class='repeatable-field-remove-button button-secondary repeatable-field-button button button-small {$_sDashiconMinus}' href='#' title='{$_sRemove}' {$_sVisibility} data-id='{$sFieldsContainerID}'>"
-                  . ( $_bDashiconSupported ? '' : '-' )
+                    . ( $_bDashiconSupported ? '' : '-' )
                 . "</a>"
                 . "<a class='repeatable-field-add-button button-secondary repeatable-field-button button button-small {$_sDashiconPlus}' href='#' title='{$_sAdd}' data-id='{$sFieldsContainerID}'>" 
                     . ( $_bDashiconSupported ? '' : '+' )

Spacing +11 added lines, -11 removed lines

@@ -68,7 +68,7 @@ 
      * @param       object      $oMsg                   An object storing the system messages.
      * @param       array       $aCallbacks             An array storing the form-field specific callbacks.     
      */
-    public function __construct( &$aField, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks=array() ) {
+    public function __construct( &$aField, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks = array() ) {
 
         // @todo Investigate why the first parameter is passed by reference. 
 
@@ -79,13 +79,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
         );        
         
         // 2. Load necessary JavaScript scripts.
@@ -152,9 +152,9 @@ 
          * @since   3.2.0   Added the $sFieldsType parameter.
          * @internal
          */
-        private function _loadScripts( $sFieldsType='' ) {
+        private function _loadScripts( $sFieldsType = '' ) {
 
-            if ( 'widget' === $sFieldsType && ! self::$_bIsLoadedSScripts_Widget ) {
+            if ( 'widget' === $sFieldsType && !self::$_bIsLoadedSScripts_Widget ) {
                 new AdminPageFramework_Script_Widget;
                 self::$_bIsLoadedSScripts_Widget = true;
             }
@@ -184,7 +184,7 @@ 
         $_sRemove               = $this->oMsg->get( 'remove' );
         $_sVisibility           = $iFieldCount <= 1 ? " style='visibility: hidden;'" : "";
         $_sSettingsAttributes   = $this->generateDataAttributes( ( array ) $aSettings );
-        $_bDashiconSupported    = false;     // version_compare( $GLOBALS['wp_version'], '3.8', '>=' );
+        $_bDashiconSupported    = false; // version_compare( $GLOBALS['wp_version'], '3.8', '>=' );
         $_sDashiconPlus         = $_bDashiconSupported ? 'dashicons dashicons-plus' : '';
         $_sDashiconMinus        = $_bDashiconSupported ? 'dashicons dashicons-minus' : '';
         $_sButtons              = 
@@ -197,7 +197,7 @@ 
                 . "</a>"                
             . "</div>";
         $_aJSArray              = json_encode( $aSettings );
-        $_sButtonsHTML          = '"' . $_sButtons . '"';
+        $_sButtonsHTML          = '"'.$_sButtons.'"';
         $_sScript               = <<<JAVASCRIPTS
 jQuery( document ).ready( function() {
     var _nodePositionIndicators = jQuery( '#{$sFieldsContainerID} .admin-page-framework-field .repeatable-field-buttons' );
@@ -215,7 +215,7 @@ 
     jQuery( '#{$sFieldsContainerID}' ).updateAdminPageFrameworkRepeatableFields( $_aJSArray ); // Update the fields     
 });
 JAVASCRIPTS;
-        return "<script type='text/javascript'>" . $_sScript . "</script>";
+        return "<script type='text/javascript'>".$_sScript."</script>";
         
     }
     

Braces +2 added lines, -2 removed lines

@@ -159,7 +159,7 @@ 
                 self::$_bIsLoadedSScripts_Widget = true;
             }
             
-            if ( self::$_bIsLoadedSScripts ) { 
+            if ( self::$_bIsLoadedSScripts ) {
                 return; 
             }
             self::$_bIsLoadedSScripts = true;
@@ -223,7 +223,7 @@ 
      * 
      * @since 3.0.0
      */    
-    protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
+    protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {
     
         $_sScript = <<<JAVASCRIPTS
     jQuery( document ).ready( function() {

development/factory/_abstract/view/form_part/input/AdminPageFramework_Input_Base.php

Doc Comments +1 added lines, -3 removed lines

@@ -105,10 +105,8 @@
      * If a parameter is not passed, it returns the entire attribute array.
      * 
      * @since       3.5.3
-     * @return      string|array|null        The specified attribute value or the entire attribute array if not specified. 
+     * @return      string        The specified attribute value or the entire attribute array if not specified. 
      * If not set, null will be returned as the `getAttributes()` method will not list an attribute with the null value.
-     * @param       string      $sName      The attribute name.
-     * @param       string      $sDefault   The defaqult value if the value is not set.
      */
     public function getAttribute( /* $sName=null, $sDefault=null */ ) {
         $_aParams = func_get_args() + array(

Spacing +6 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
      * @param       array       $aAttributes    The attribute array. A field definition array is deprecated.
      * @param       array       $aOptions       options that allows to set custom container tags and class selctors.
      */
-    public function __construct( array $aAttributes, array $aOptions=array() ) {
+    public function __construct( array $aAttributes, array $aOptions = array() ) {
 
         $this->aAttributes  = $this->getElementAsArray( $aAttributes, 'attributes', $aAttributes );
         $this->aOptions     = $aOptions + $this->aStructureOptions;
@@ -122,13 +122,13 @@ 
      * @return      string      The set class selector(s).
      */
     public function addClass( /* $asSelectors1, $asSelectors2 */ ) {
-        foreach( func_get_args() as $_asSelectors ) {            
-            $this->aAttributes['class'] = $this->getClassAttribute( 
-                $this->aAttributes['class'],
+        foreach ( func_get_args() as $_asSelectors ) {            
+            $this->aAttributes[ 'class' ] = $this->getClassAttribute( 
+                $this->aAttributes[ 'class' ],
                 $_asSelectors
             );
         }
-        return $this->aAttributes['class'];
+        return $this->aAttributes[ 'class' ];
     }
     
     /**
@@ -153,7 +153,7 @@ 
         // $this->aAttributes[ $_aParams[ 0 ] ] = $_aParams[ 1 ];
         $this->setMultiDimensionalArray(
             $this->aAttributes, 
-            $this->getElementAsArray( $_aParams, 0 ),   // $asAttributeName
+            $this->getElementAsArray( $_aParams, 0 ), // $asAttributeName
             $_aParams[ 1 ]  // $mValue
         );
     }

Braces +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
      * @return      string      The set class selector(s).
      */
     public function addClass( /* $asSelectors1, $asSelectors2 */ ) {
-        foreach( func_get_args() as $_asSelectors ) {            
+        foreach( func_get_args() as $_asSelectors ) {
             $this->aAttributes['class'] = $this->getClassAttribute( 
                 $this->aAttributes['class'],
                 $_asSelectors

development/factory/_abstract/view/form_part/input/AdminPageFramework_Input_checkbox.php

Doc Comments -2 removed lines

@@ -21,8 +21,6 @@
      * Returns the output of the input element.
      * 
      * @since       3.4.0    
-     * @param       string      $sLabel         The label text.
-     * @param       array       $aAttributes    (optional) The attribute array. If set, it will be merged with the attribute set in the constructor.
      */    
     public function get( /* $sLabel, $aAttributes=array() */ ) {
         

Indentation +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         
         // Output
         return 
-           "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
+            "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
                 // the unchecked value must be set prior to the checkbox input field.
                 . "<input " . $this->getAttributes( 
                     array(

Spacing +16 added lines, -16 removed lines

@@ -28,7 +28,7 @@ 
         
         // Parameters
         $_aParams       = func_get_args() + array( 0 => '', 1 => array() );
-        $_sLabel        = $_aParams[ 0 ];       // first parameter
+        $_sLabel        = $_aParams[ 0 ]; // first parameter
         $_aAttributes   = $this->uniteArrays(   // second parameter
             $this->getElementAsArray( $_aParams, 1, array() ),
             $this->aAttributes
@@ -36,22 +36,22 @@ 
         
         // Output
         return 
-           "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
+           "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
                 // the unchecked value must be set prior to the checkbox input field.
-                . "<input " . $this->getAttributes( 
+                . "<input ".$this->getAttributes( 
                     array(
                         'type'      => 'hidden',
-                        'class'     => $_aAttributes['class'],
-                        'name'      => $_aAttributes['name'],
+                        'class'     => $_aAttributes[ 'class' ],
+                        'name'      => $_aAttributes[ 'name' ],
                         'value'     => '0',
                     ) 
                 ) 
                 . " />"
-                . "<input " . $this->getAttributes( $_aAttributes ) . " />" 
-            . "</{$this->aOptions['input_container_tag']}>"
-            . "<{$this->aOptions['label_container_tag']} " . $this->getAttributes( $this->aOptions['label_container_attributes'] ) . ">"
+                . "<input ".$this->getAttributes( $_aAttributes )." />" 
+            . "</{$this->aOptions[ 'input_container_tag' ]}>"
+            . "<{$this->aOptions[ 'label_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'label_container_attributes' ] ).">"
                 . $_sLabel
-            . "</{$this->aOptions['label_container_tag']}>"
+            . "</{$this->aOptions[ 'label_container_tag' ]}>"
         ;
                         
     }        
@@ -65,7 +65,7 @@ 
     public function getAttributesByKey( /* $sKey */ ) {
         
         // Parameters
-        $_aParams       = func_get_args() + array( 0 => '', );
+        $_aParams       = func_get_args() + array( 0 => '',);
         $_sKey          = $_aParams[ 0 ];        
         $_bIsMultiple   = '' !== $_sKey;
         
@@ -77,15 +77,15 @@ 
             // The type needs to be specified since the postytpe field type extends this class. If not set, the 'posttype' will be passed to the type attribute.
             + array(
                 'type'      => 'checkbox', 
-                'id'        => $this->aAttributes['id'] . '_' . $_sKey,
+                'id'        => $this->aAttributes[ 'id' ].'_'.$_sKey,
                 'checked'   => $this->getElement( $this->aAttributes, array( 'value', $_sKey ), null )
                     ? 'checked' 
-                    : null,    // to not to set, pass null. An empty value '' will still set the attribute.
-                'value'     => 1,   // this must be always 1 because the key value can be zero. In that case, the value always will be false and unchecked.
+                    : null, // to not to set, pass null. An empty value '' will still set the attribute.
+                'value'     => 1, // this must be always 1 because the key value can be zero. In that case, the value always will be false and unchecked.
                 'name'      => $_bIsMultiple 
-                    ? "{$this->aAttributes['name']}[{$_sKey}]" 
-                    : $this->aAttributes['name'],
-                'data-id'   => $this->aAttributes['id'],       // referenced by the JavaScript scripts such as the revealer script.
+                    ? "{$this->aAttributes[ 'name' ]}[{$_sKey}]" 
+                    : $this->aAttributes[ 'name' ],
+                'data-id'   => $this->aAttributes[ 'id' ], // referenced by the JavaScript scripts such as the revealer script.
             )
             + $this->aAttributes
             ;

development/factory/_abstract/view/form_part/input/AdminPageFramework_Input_radio.php

Doc Comments -4 removed lines

@@ -21,8 +21,6 @@ 
      * Returns the output of the input element.
      * 
      * @since       3.4.0    
-     * @param       string      $sLabel         The label text.
-     * @param       array       $aAttributes    (optional) The attribute array. If set, it will be merged with the attribute set in the constructor.
      */    
     public function get( /* $sLabel, $aAttributes=array() */ ) {
         
@@ -52,8 +50,6 @@ 
      * 
      * @return      array       The updated attribute array. 
      * @since       3.5.3
-     * @param       string      $sKey       The array element key of the radio button. 
-     * It is assumed that there is an array that holds multiple radio buttons and each of them has an array key.
      */
     public function getAttributesByKey( /* $sKey */ ) {
         

Spacing +10 added lines, -10 removed lines

@@ -36,12 +36,12 @@ 
         
         // Output
         return 
-            "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
-                . "<input " . $this->getAttributes( $_aAttributes ) . " />" 
-            . "</{$this->aOptions['input_container_tag']}>"
-            . "<{$this->aOptions['label_container_tag']} " . $this->getAttributes( $this->aOptions['label_container_attributes'] ) . ">"
+            "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
+                . "<input ".$this->getAttributes( $_aAttributes )." />" 
+            . "</{$this->aOptions[ 'input_container_tag' ]}>"
+            . "<{$this->aOptions[ 'label_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'label_container_attributes' ] ).">"
                 . $_sLabel
-            . "</{$this->aOptions['label_container_tag']}>"
+            . "</{$this->aOptions[ 'label_container_tag' ]}>"
             ;
         
     }    
@@ -58,22 +58,22 @@ 
     public function getAttributesByKey( /* $sKey */ ) {
         
         // Parameters
-        $_aParams       = func_get_args() + array( 0 => '', );
-        $_sKey           = $_aParams[ 0 ];        
+        $_aParams = func_get_args() + array( 0 => '',);
+        $_sKey = $_aParams[ 0 ];        
         
         // Result
         return $this->getElementAsArray( $this->aAttributes, $_sKey, array() )
             + array(
                 'type'          => 'radio',
-                'checked'       => isset( $this->aAttributes['value'] ) && $this->aAttributes['value'] == $_sKey 
+                'checked'       => isset( $this->aAttributes[ 'value' ] ) && $this->aAttributes[ 'value' ] == $_sKey 
                     ? 'checked' 
                     : null,
                 'value'         => $_sKey,
                 // 'id'            => $this->aField['input_id'] . '_' . $_sKey,
-                'id'            => $this->getAttribute( 'id' ) . '_' . $_sKey,
+                'id'            => $this->getAttribute( 'id' ).'_'.$_sKey,
                 // 'data-default'  => $this->aField['default'],        // refered by the repeater script
                 // 'data-id'       => $this->aField['input_id'],       // refered by the JavaScript scripts such as the revealer script.
-                'data-id'       => $this->getAttribute( 'id' ),       // refered by the JavaScript scripts such as the revealer script.
+                'data-id'       => $this->getAttribute( 'id' ), // refered by the JavaScript scripts such as the revealer script.
             ) 
             + $this->aAttributes; 
             

development/factory/_abstract/view/form_part/input/AdminPageFramework_Input_select.php

Doc Comments +3 added lines, -1 removed lines

@@ -122,7 +122,7 @@ 
          * @internal
          * @param       string      $sInputID           The input ID that will be the base of each generated option tag ID.
          * @param       array       $aLabels            The array holding labels.
-         * @param       array       $aAttributes        The attribute arrays. Accepts the following arguments.
+         * @param       array       $aBaseAttributes        The attribute arrays. Accepts the following arguments.
          * - optgroup
          * - option
          */     
@@ -160,6 +160,7 @@ 
             /**
              * Returns an HTML output of optgroup tag.
              * @since       3.5.3
+             * @param string $sInputID
              * @return      string      an HTML output of optgroup tag.
              */
             private function _getOptGroup( array $aBaseAttributes, $sInputID, $sKey, $asLabel ) {
@@ -179,6 +180,7 @@ 
             /**
              * 
              * @since        3.5.3
+             * @param string $sInputID
              */
             private function _getOptionTagAttributes( array $aBaseAttributes, $sInputID, $sKey, $aValues ) {
             

Spacing +24 added lines, -24 removed lines

@@ -44,8 +44,8 @@ 
         // For backward compatibility.
         
         // If the $aField property is set, extract certain elements from it and set them to the attribute array.
-        if ( isset( $this->aField['is_multiple'] ) ) {
-            $this->aAttributes['select']['multiple'] = $this->aField['is_multiple']
+        if ( isset( $this->aField[ 'is_multiple' ] ) ) {
+            $this->aAttributes[ 'select' ][ 'multiple' ] = $this->aField[ 'is_multiple' ]
                 ? 'multiple'
                 : $this->getElement( $this->aAttributes, array( 'select', 'multiple' ) );
         }
@@ -69,20 +69,20 @@ 
         );    
 
         return  
-            "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
-                . "<select " . $this->getAttributes( $this->_getSelectAttributes( $_aAttributes ) ) . " >"
+            "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
+                . "<select ".$this->getAttributes( $this->_getSelectAttributes( $_aAttributes ) )." >"
                     . $this->_getDropDownList( 
                         $this->getAttribute( 'id' ),
                         $this->getAsArray(
                             isset( $_aLabels ) 
                                 ? $_aLabels
-                                : $this->aField['label'],    // backward compatibility
+                                : $this->aField[ 'label' ], // backward compatibility
                             true
                         ),
                         $_aAttributes
                     )
                 . "</select>"
-            . "</{$this->aOptions['input_container_tag']}>"
+            . "</{$this->aOptions[ 'input_container_tag' ]}>"
             ;
             
     }
@@ -104,9 +104,9 @@ 
                         ? 'multiple' 
                         : null,
                     'name'      => $_bIsMultiple 
-                        ? $this->getAttribute( 'name' ) . '[]'
+                        ? $this->getAttribute( 'name' ).'[]'
                         : $this->getAttribute( 'name' ),
-                    'data-id'   => $this->getAttribute( 'id' ),       // referenced by the JavaScript scripts such as the revealer script.
+                    'data-id'   => $this->getAttribute( 'id' ), // referenced by the JavaScript scripts such as the revealer script.
                 )
             );            
             
@@ -128,12 +128,12 @@ 
          */     
         private function _getDropDownList( $sInputID, array $aLabels, array $aBaseAttributes ) {
             
-            $_aOutput   = array();
-            foreach( $aLabels as $__sKey => $__asLabel ) {
+            $_aOutput = array();
+            foreach ( $aLabels as $__sKey => $__asLabel ) {
                 
                 // For an optgroup tag,
                 if ( is_array( $__asLabel ) ) {
-                    $_aOutput[] = $this->_getOptGroup(
+                    $_aOutput[ ] = $this->_getOptGroup(
                         $aBaseAttributes, 
                         $sInputID, 
                         $__sKey, 
@@ -143,13 +143,13 @@ 
                 }
                 
                 // A normal option tag,
-                $_aOutput[] = $this->_getOptionTag( 
-                    $__asLabel,   // the text label the user sees to be selected
+                $_aOutput[ ] = $this->_getOptionTag( 
+                    $__asLabel, // the text label the user sees to be selected
                     $this->_getOptionTagAttributes( 
                         $aBaseAttributes, 
                         $sInputID, 
                         $__sKey,
-                        $this->getAsArray( $aBaseAttributes['value'], true )
+                        $this->getAsArray( $aBaseAttributes[ 'value' ], true )
                     ) 
                 );
                     
@@ -164,13 +164,13 @@ 
              */
             private function _getOptGroup( array $aBaseAttributes, $sInputID, $sKey, $asLabel ) {
              
-                $_aOptGroupAttributes = isset( $aBaseAttributes['optgroup'][ $sKey ] ) && is_array( $aBaseAttributes['optgroup'][ $sKey ] )
-                    ? $aBaseAttributes['optgroup'][ $sKey ] + $aBaseAttributes['optgroup']
-                    : $aBaseAttributes['optgroup'];
+                $_aOptGroupAttributes = isset( $aBaseAttributes[ 'optgroup' ][ $sKey ] ) && is_array( $aBaseAttributes[ 'optgroup' ][ $sKey ] )
+                    ? $aBaseAttributes[ 'optgroup' ][ $sKey ] + $aBaseAttributes[ 'optgroup' ]
+                    : $aBaseAttributes[ 'optgroup' ];
                 $_aOptGroupAttributes = array(
                     'label' => $sKey,
                 ) + $_aOptGroupAttributes;
-                return "<optgroup " . $this->getAttributes( $_aOptGroupAttributes ) . ">"
+                return "<optgroup ".$this->getAttributes( $_aOptGroupAttributes ).">"
                         . $this->_getDropDownList( $sInputID, $asLabel, $aBaseAttributes )
                     . "</optgroup>";
              
@@ -188,14 +188,14 @@ 
                     $aValues 
                 );
                 return array(      
-                        'id'        => $sInputID . '_' . $sKey,
+                        'id'        => $sInputID.'_'.$sKey,
                         'value'     => $sKey,
                         'selected'  => in_array( ( string ) $sKey, $aValues ) 
                             ? 'selected' 
                             : null,                                        
-                    ) + ( isset( $aBaseAttributes['option'][ $sKey ] ) && is_array( $aBaseAttributes['option'][ $sKey ] )
-                        ? $aBaseAttributes['option'][ $sKey ] + $aBaseAttributes['option']
-                        : $aBaseAttributes['option'] );
+                    ) + ( isset( $aBaseAttributes[ 'option' ][ $sKey ] ) && is_array( $aBaseAttributes[ 'option' ][ $sKey ] )
+                        ? $aBaseAttributes[ 'option' ][ $sKey ] + $aBaseAttributes[ 'option' ]
+                        : $aBaseAttributes[ 'option' ] );
             
             }
         
@@ -204,8 +204,8 @@ 
              * @sicne       3.4.0
              * @return      string      The generated option tag HTML output.
              */
-            private function _getOptionTag( $sLabel, array $aOptionTagAttributes=array() ) {
-                return "<option " . $this->getAttributes( $aOptionTagAttributes ) . " >"    
+            private function _getOptionTag( $sLabel, array $aOptionTagAttributes = array() ) {
+                return "<option ".$this->getAttributes( $aOptionTagAttributes )." >"    
                         . $sLabel
                     . "</option>";
             }