michaeluno / admin-page-framework

development/factory/_common/_abstract/_view/AdminPageFramework_TabNavigationBar.php

Spacing +14 added lines, -14 removed lines

@@ -26,7 +26,7 @@ 
     /**
      * Stores the tab items.
      */
-    public $aTabs   = array();
+    public $aTabs = array();
 
     /**
      * Stores container attributes.
@@ -36,11 +36,11 @@ 
     );
     
     public $aTab = array(
-        'slug'       => null,  // (string) tab slug (id)
-        'title'      => null,  // (string) tab title
-        'href'       => null,  // (string) link url
-        'disabled'   => null,  // (boolean)
-        'class'      => null,  // (string) class selector to append to the class attribute
+        'slug'       => null, // (string) tab slug (id)
+        'title'      => null, // (string) tab title
+        'href'       => null, // (string) link url
+        'disabled'   => null, // (boolean)
+        'class'      => null, // (string) class selector to append to the class attribute
         'attributes' => array(),
     );
     
@@ -59,11 +59,11 @@ 
      * @param       array             $aTabs              An array holding each tab definitions
      * @param       array|string      $asActiveTabSlugs   The default tab slug.
      */
-    public function __construct( array $aTabs, $asActiveTabSlugs, $sTabTag='h2', $aAttributes=array( 'class' => 'nav-tab-wrapper', ), $aCallbacks=array() ) {
+    public function __construct( array $aTabs, $asActiveTabSlugs, $sTabTag = 'h2', $aAttributes = array( 'class' => 'nav-tab-wrapper',), $aCallbacks = array() ) {
         
-        $this->aCallbacks           = $aCallbacks + array(
+        $this->aCallbacks = $aCallbacks + array(
             'format'    => null,
-            'arguments' => null,  // custom arguments to pass to the callback functions.
+            'arguments' => null, // custom arguments to pass to the callback functions.
         );
         $this->aTabs                = $this->_getFormattedTabs( $aTabs );
         $this->aActiveSlugs         = $this->getAsArray( $asActiveTabSlugs );        
@@ -77,7 +77,7 @@ 
          * @return      array
          */
         private function _getFormattedTabs( array $aTabs ) {
-            foreach( $aTabs as $_isKey => &$_aTab ) {
+            foreach ( $aTabs as $_isKey => &$_aTab ) {
                 $_aFormattedTab = $this->_getFormattedTab( $_aTab, $aTabs );
                 if ( isset( $_aFormattedTab[ 'slug' ] ) ) {
                     $_aTab = $_aFormattedTab;
@@ -128,12 +128,12 @@ 
         private function _getTabs() {
             
             $_aOutput = array();
-            foreach( $this->aTabs as $_aTab ) {
+            foreach ( $this->aTabs as $_aTab ) {
                 $_sTab = $this->_getTab( $_aTab );
-                if ( ! $_sTab ) {
+                if ( !$_sTab ) {
                     continue;
                 }
-                $_aOutput[] = $_sTab;
+                $_aOutput[ ] = $_sTab;
             }
 
             $_aContainerAttributes = $this->aAttributes + array( 'class' => null );
@@ -144,7 +144,7 @@ 
 
             return empty( $_aOutput )
                 ? ''
-                : "<{$this->sTabTag} " . $this->getAttributes( $_aContainerAttributes ) . ">"
+                : "<{$this->sTabTag} ".$this->getAttributes( $_aContainerAttributes ).">"
                     . implode( '', $_aOutput )                    
                 . "</{$this->sTabTag}>";
             

Braces +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  * @subpackage      Tab
  * @internal
  */
-class AdminPageFramework_TabNavigationBar extends AdminPageFramework_FrameworkUtility {                
+class AdminPageFramework_TabNavigationBar extends AdminPageFramework_FrameworkUtility {
     
     /**
      * The HTML tag used for the tag.

development/factory/_common/_abstract/_view/AdminPageFramework_Factory___Script_Base.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
      * 
      * @since       3.3.0
      */
-    public function __construct( $oMsg=null ) {
+    public function __construct( $oMsg = null ) {
         
         $_sClassName = get_class( $this );
         if ( in_array( $_sClassName, self::$_aEnqueued ) ) {
@@ -74,10 +74,10 @@ 
      */
     public function _replyToPrintScript() {
         $_sScript = $this->getScript( $this->oMsg );
-        if ( ! $_sScript ) {
+        if ( !$_sScript ) {
             return;
         }
-        echo "<script type='text/javascript' class='" . strtolower( get_class( $this ) ) . "'>"
+        echo "<script type='text/javascript' class='".strtolower( get_class( $this ) )."'>"
                 . $_sScript
             . "</script>";
     }

development/factory/_common/_abstract/_view/AdminPageFramework_Factory_View__SettingNotice.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
      * @since       3.7.9       Added the second parameter to accept an action hook name.
      */
         
-    public function __construct( $oFactory, $sActionHookName='admin_notices' ) {
+    public function __construct( $oFactory, $sActionHookName = 'admin_notices' ) {
         
         $this->oFactory = $oFactory;
            
@@ -46,7 +46,7 @@ 
      */
     public function _replyToPrintSettingNotice() {
             
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }
         $this->oFactory->oForm->printSubmitNotices();
@@ -60,7 +60,7 @@ 
          */
         private function _shouldProceed() {
             
-            if ( ! $this->oFactory->_isInThePage() ) { 
+            if ( !$this->oFactory->_isInThePage() ) { 
                 return false; 
             }
                 

Braces +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
          */
         private function _shouldProceed() {
             
-            if ( ! $this->oFactory->_isInThePage() ) { 
+            if ( ! $this->oFactory->_isInThePage() ) {
                 return false; 
             }
                 

development/factory/_common/_abstract/_view/AdminPageFramework_CSS.php

Spacing +2 added lines, -2 removed lines

@@ -107,8 +107,8 @@
 }
 CSSRULES;
 
-        return $_sCSS . PHP_EOL
-            . self::_getPageLoadStatsRules() . PHP_EOL
+        return $_sCSS.PHP_EOL
+            . self::_getPageLoadStatsRules().PHP_EOL
             . self::_getVersionSpecificRules();
             
     }

development/factory/_common/_abstract/AdminPageFramework_Factory_Router.php

Spacing +12 added lines, -12 removed lines

@@ -131,7 +131,7 @@ 
         // Required sub-class objects
         $this->oProp = $oProp;
             
-        if ( $this->oProp->bIsAdmin && ! $this->oProp->bIsAdminAjax ) {
+        if ( $this->oProp->bIsAdmin && !$this->oProp->bIsAdminAjax ) {
             if ( did_action( 'current_screen' ) ) {
                 $this->_replyToLoadComponents();
             } else {                
@@ -140,8 +140,8 @@ 
         }
         
         // Call the user constructor.
-        $this->start();     // defined in the controller class.
-        $this->oUtil->addAndDoAction( $this, 'start_' . $this->oProp->sClassName, $this );
+        $this->start(); // defined in the controller class.
+        $this->oUtil->addAndDoAction( $this, 'start_'.$this->oProp->sClassName, $this );
         
     }    
         
@@ -154,15 +154,15 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( !$this->_isInThePage() ) { 
             return; 
         }
                     
-        if ( ! isset( $this->oResource ) ) {
+        if ( !isset( $this->oResource ) ) {
             $this->oResource = $this->_replyTpSetAndGetInstance_oResource();
         }
         
-        if ( ! isset(  $this->oLink ) ) {
+        if ( !isset( $this->oLink ) ) {
             $this->oLink = $this->_replyTpSetAndGetInstance_oLink();         
         }
         
@@ -210,7 +210,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( !$this->_isInThePage() ) { 
             return; 
         }
 
@@ -291,7 +291,7 @@ 
         // Set and return the sub class object instance.
         if ( in_array( $sPropertyName, $this->_aSubClassNames ) ) {                        
             return call_user_func( 
-                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}"  )
+                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}" )
             );
         }
         
@@ -385,7 +385,7 @@ 
      * 
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {    
+    public function __call( $sMethodName, $aArguments = null ) {    
          
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
         
@@ -415,7 +415,7 @@ 
             
             // Check if the method name does not contain a backslash.
             if ( false === strpos( $sMethodName, "\\" ) ) {
-                return $this->oUtil->getElement( $aArguments, 0 );  // the first element - the filter value
+                return $this->oUtil->getElement( $aArguments, 0 ); // the first element - the filter value
             }
                 
             // If the method name contains a backslash, the user may be using a name space. 
@@ -426,7 +426,7 @@ 
                     array( $this, $_sAutoCallbackMethodName ),
                     $aArguments
                 )
-                : $this->oUtil->getElement( $aArguments, 0 );   // the first argument
+                : $this->oUtil->getElement( $aArguments, 0 ); // the first argument
             
         }
         
@@ -436,7 +436,7 @@ 
          */
         private function _triggerUndefinedMethodWarning( $sMethodName ) {
             trigger_error(
-                AdminPageFramework_Registry::NAME . ': ' 
+                AdminPageFramework_Registry::NAME.': ' 
                     . sprintf( 
                         __( 'The method is not defined: %1$s', $this->oProp->sTextDomain ),
                         $sMethodName 

Braces +7 added lines, -7 removed lines

@@ -134,7 +134,7 @@ 
         if ( $this->oProp->bIsAdmin && ! $this->oProp->bIsAdminAjax ) {
             if ( did_action( 'current_screen' ) ) {
                 $this->_replyToLoadComponents();
-            } else {                
+            } else {
                 add_action( 'current_screen', array( $this, '_replyToLoadComponents' ) );
             }
         }
@@ -154,7 +154,7 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
                     
@@ -181,7 +181,7 @@ 
      * @since       3.1.0
      * @internal
      */ 
-    protected function _isInstantiatable() { 
+    protected function _isInstantiatable() {
         return true; 
     }
     
@@ -196,7 +196,7 @@ 
      * @todo        Change the visibility scope to `protected` as the public version of the method `isInThePage()` has been introduced to make the design consitent.
      * @internal
      */
-    public function _isInThePage() { 
+    public function _isInThePage() {
         return true; 
     }
          
@@ -210,7 +210,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
 
@@ -289,7 +289,7 @@ 
     public function __get( $sPropertyName ) {
             
         // Set and return the sub class object instance.
-        if ( in_array( $sPropertyName, $this->_aSubClassNames ) ) {                        
+        if ( in_array( $sPropertyName, $this->_aSubClassNames ) ) {
             return call_user_func( 
                 array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}"  )
             );
@@ -385,7 +385,7 @@ 
      * 
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {    
+    public function __call( $sMethodName, $aArguments=null ) {
          
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
         

development/factory/_common/form/AdminPageFramework_Form_Meta.php

Spacing +8 added lines, -8 removed lines

@@ -31,9 +31,9 @@ 
      * @param       string      $sStructureType    The type of object. Currently 'post_meta_box' or 'user_meta' is accepted.
      * @return      void
      */
-    public function updateMetaDataByType( $iObjectID, array $aInput, array $aSavedMeta, $sStructureType='post_meta_box' ) {
+    public function updateMetaDataByType( $iObjectID, array $aInput, array $aSavedMeta, $sStructureType = 'post_meta_box' ) {
         
-        if ( ! $iObjectID ) {
+        if ( !$iObjectID ) {
             return;
         }
            
@@ -41,7 +41,7 @@ 
             'post_meta_box'     => 'update_post_meta',
             'user_meta'         => 'update_user_meta',               
         );
-        if ( ! in_array( $sStructureType, array_keys( $_aFunctionNameMapByFieldsType ) ) ) {
+        if ( !in_array( $sStructureType, array_keys( $_aFunctionNameMapByFieldsType ) ) ) {
             return;
         }
         $_sFunctionName = $this->getElement( $_aFunctionNameMapByFieldsType, $sStructureType );
@@ -69,16 +69,16 @@ 
          */
         private function _getInputByUnset( array $aInput ) {
             
-            $_sUnsetKey = '__unset_' . $this->sStructureType;
-            if ( ! isset( $_POST[ $_sUnsetKey ] ) ) {
+            $_sUnsetKey = '__unset_'.$this->sStructureType;
+            if ( !isset( $_POST[ $_sUnsetKey ] ) ) {
                 return $aInput;
             }
             
             $_aUnsetElements = array_unique( $_POST[ $_sUnsetKey ] );
-            foreach( $_aUnsetElements as $_sFlatInputName ) {
+            foreach ( $_aUnsetElements as $_sFlatInputName ) {
                 
                 $_aDimensionalKeys = explode( '|', $_sFlatInputName );
-                if ( ! isset( $_aDimensionalKeys[ 0 ] ) ) {
+                if ( !isset( $_aDimensionalKeys[ 0 ] ) ) {
                     continue;
                 }
                 
@@ -113,7 +113,7 @@ 
 
             $_vSavedValue = $this->getElement(
                 $aSavedMeta, // subject
-                $_sSectionOrFieldID,    // dimensional keys
+                $_sSectionOrFieldID, // dimensional keys
                 null   // default value
             );
                 

Braces +2 added lines, -2 removed lines

@@ -107,7 +107,7 @@ 
          */
         private function _updateMetaDatumByFuncitonName( $iObjectID, $_vValue, array $aSavedMeta, $_sSectionOrFieldID, $_sFunctionName ) {
             
-            if ( is_null( $_vValue ) ) { 
+            if ( is_null( $_vValue ) ) {
                 return;
             }
 
@@ -119,7 +119,7 @@ 
                 
             // PHP can compare even array contents with the == operator. See http://www.php.net/manual/en/language.operators.array.php
             // if the input value and the saved meta value are the same, no need to update it.
-            if ( $_vValue == $_vSavedValue ) { 
+            if ( $_vValue == $_vSavedValue ) {
                 return; 
             }
             

Indentation +7 added lines, -7 removed lines

@@ -1,11 +1,11 @@ 
 <?php
 /**
- * Admin Page Framework
- * 
- * http://en.michaeluno.jp/admin-page-framework/
- * Copyright (c) 2013-2016 Michael Uno; Licensed MIT
- * 
- */
+         * Admin Page Framework
+         * 
+         * http://en.michaeluno.jp/admin-page-framework/
+         * Copyright (c) 2013-2016 Michael Uno; Licensed MIT
+         * 
+         */
 
 /**
  * Provides methods to build forms for the user meta and post meta structure type.
@@ -84,7 +84,7 @@ 
                 
                 // The first element is the option key; the section or field dimensional keys follow.
                 if ( '__dummy_option_key' === $_aDimensionalKeys[ 0 ] ) {
-                     array_shift( $_aDimensionalKeys );
+                        array_shift( $_aDimensionalKeys );
                 }
                 
                 $this->unsetDimensionalArrayElement( 

development/factory/_common/form/AdminPageFramework_Form_Base.php

Spacing +5 added lines, -5 removed lines

@@ -57,18 +57,18 @@ 
         }
         
         // If the section ID is not registered, return false.
-        if ( ! array_key_exists( $sID, $this->aSectionsets ) ) { 
+        if ( !array_key_exists( $sID, $this->aSectionsets ) ) { 
             return false; 
         }
         
         // the fields array's first dimension is also filled with the keys of section ids.
-        if ( ! array_key_exists( $sID, $this->aFieldsets ) ) { 
+        if ( !array_key_exists( $sID, $this->aFieldsets ) ) { 
             return false; 
         }
         
         // Since numeric IDs are denied at the beginning of the method, the elements will not be sub-sections.
         $_bIsSeciton = false;
-        foreach( $this->aFieldsets as $_sSectionID => $_aFields ) {    
+        foreach ( $this->aFieldsets as $_sSectionID => $_aFields ) {    
         
             if ( $_sSectionID == $sID ) { 
                 $_bIsSeciton = true; 
@@ -94,7 +94,7 @@ 
      * @return      boolean
      */
     public function canUserView( $sCapability ) {
-        if ( ! $sCapability  ) {
+        if ( !$sCapability ) {
             return true;
         }
         return ( boolean ) current_user_can( $sCapability );
@@ -139,7 +139,7 @@ 
      * @since       DEVVER
      */
     public function callBack( $oCallable, $asParameters ) {
-        $_aParameters   = self::getAsArray( 
+        $_aParameters = self::getAsArray( 
             $asParameters, 
             true // preserve empty
         );

Braces +5 added lines, -5 removed lines

@@ -57,25 +57,25 @@
         }
         
         // If the section ID is not registered, return false.
-        if ( ! array_key_exists( $sID, $this->aSectionsets ) ) { 
+        if ( ! array_key_exists( $sID, $this->aSectionsets ) ) {
             return false; 
         }
         
         // the fields array's first dimension is also filled with the keys of section ids.
-        if ( ! array_key_exists( $sID, $this->aFieldsets ) ) { 
+        if ( ! array_key_exists( $sID, $this->aFieldsets ) ) {
             return false; 
         }
         
         // Since numeric IDs are denied at the beginning of the method, the elements will not be sub-sections.
         $_bIsSeciton = false;
-        foreach( $this->aFieldsets as $_sSectionID => $_aFields ) {    
+        foreach( $this->aFieldsets as $_sSectionID => $_aFields ) {
         
-            if ( $_sSectionID == $sID ) { 
+            if ( $_sSectionID == $sID ) {
                 $_bIsSeciton = true; 
             }
             
             // a field using the ID is found, and it precedes a section match.     
-            if ( array_key_exists( $sID, $_aFields ) ) { 
+            if ( array_key_exists( $sID, $_aFields ) ) {
                 return false; 
             }
             

development/factory/_common/form/_model/AdminPageFramework_Form_Model___FieldTypeRegistration.php

Indentation +1 added lines, -2 removed lines

@@ -9,7 +9,6 @@ 
 
 /**
  * Provides methods to retrieve field type resources.
- 
  * @package     AdminPageFramework
  * @subpackage  Form
  * @since       DEVVER
@@ -42,7 +41,7 @@ 
                 call_user_func_array( 
                     $aFieldTypeDefinition[ 'hfFieldSetTypeSetter' ], 
                     array( $sStructureType ) 
-               );
+                );
             }
             
             if ( is_callable( $aFieldTypeDefinition[ 'hfFieldLoader' ] ) ) {

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___FormatSectionset.php

Spacing +22 added lines, -22 removed lines

@@ -35,41 +35,41 @@ 
         // Optional
         'page_slug'         => null,
         'tab_slug'          => null,
-        'section_tab_slug'  => null,    // 3.0.0+
+        'section_tab_slug'  => null, // 3.0.0+
         'title'             => null,
         'description'       => null,
         'capability'        => null,
         'if'                => true,    
-        'order'             => null,    // do not set the default number here because incremented numbers will be added when registering the sections.
+        'order'             => null, // do not set the default number here because incremented numbers will be added when registering the sections.
         'help'              => null,
         'help_aside'        => null,
-        'repeatable'        => false,   // (boolean|array) 3.0.0+
-        'sortable'          => false,   // (boolean|array) 3.6.0+
+        'repeatable'        => false, // (boolean|array) 3.0.0+
+        'sortable'          => false, // (boolean|array) 3.6.0+
         'attributes'        => array(   // 3.3.1+
-            'class'         => null,    // set null to avoid undefined index warnings.
-            'style'         => null,    // set null to avoid undefined index warnings.
+            'class'         => null, // set null to avoid undefined index warnings.
+            'style'         => null, // set null to avoid undefined index warnings.
             'tab'           => array(),
         ),
         'class'             => array(    // 3.3.1+
             'tab'           => array(),
         ),
-        'hidden'            => false,    // 3.3.1+
-        'collapsible'       => false,    // 3.4.0+ (boolean|array) For the array structure see the $aStructure_CollapsibleArguments property.
-        'save'              => true,     // 3.6.0+
+        'hidden'            => false, // 3.3.1+
+        'collapsible'       => false, // 3.4.0+ (boolean|array) For the array structure see the $aStructure_CollapsibleArguments property.
+        'save'              => true, // 3.6.0+
         
-        'content'           => null,     // 3.6.1+  (string) An overriding section-set output.
+        'content'           => null, // 3.6.1+  (string) An overriding section-set output.
         
-        'tip'               => null,     // 3.7.0  (string) Tool tip HTML strings. 
+        'tip'               => null, // 3.7.0  (string) Tool tip HTML strings. 
         
         // Internal
-        '_fields_type'      => null,     // @deprecated 3.7.0+ Use the `_structure_type` instead. 3.0.0+ - same as the one of the field definition array. Used to insert debug info at the bottom of sections.        
-        '_structure_type'   => null,     // 3.7.0+
-        '_is_first_index'   => false,    // 3.4.0+ (boolean) indicates whether it is the first item of the sub-sections (for repeatable sections).
-        '_is_last_index'    => false,    // 3.4.0+ (boolean) indicates whether it is the last item of the sub-sections (for repeatable sections).
+        '_fields_type'      => null, // @deprecated 3.7.0+ Use the `_structure_type` instead. 3.0.0+ - same as the one of the field definition array. Used to insert debug info at the bottom of sections.        
+        '_structure_type'   => null, // 3.7.0+
+        '_is_first_index'   => false, // 3.4.0+ (boolean) indicates whether it is the first item of the sub-sections (for repeatable sections).
+        '_is_last_index'    => false, // 3.4.0+ (boolean) indicates whether it is the last item of the sub-sections (for repeatable sections).
         
-        '_section_path'         => '',       // 3.7.0+ (string) e.g. my_section|nested_section       
-        '_section_path_array'   => '',       // 3.7.0+ (array) an array version of the above section_path argument. Numerically indexed.
-        '_nested_depth'         => 0,        // 3.7.0+ (integer) the nested level of the section
+        '_section_path'         => '', // 3.7.0+ (string) e.g. my_section|nested_section       
+        '_section_path_array'   => '', // 3.7.0+ (array) an array version of the above section_path argument. Numerically indexed.
+        '_nested_depth'         => 0, // 3.7.0+ (integer) the nested level of the section
         
         // 3.6.0+ - (object) the caller framework factory object. This allows the framework to access the factory property when rendering the section.
         // 3.7.0+  It no longer stores a factory object but a form object.
@@ -124,11 +124,11 @@ 
         $_aSectionPath = explode( '|', $this->sSectionPath );
         $_aSectionset  = $this->uniteArrays(
             array( 
-                '_fields_type'          => $this->sStructureType,   // @deprecated  3.7.0+
-                '_structure_type'       => $this->sStructureType,   // 3.7.0+
-                '_section_path'         => $this->sSectionPath,     // 3.7.0+
+                '_fields_type'          => $this->sStructureType, // @deprecated  3.7.0+
+                '_structure_type'       => $this->sStructureType, // 3.7.0+
+                '_section_path'         => $this->sSectionPath, // 3.7.0+
                 '_section_path_array'   => $_aSectionPath,
-                '_nested_depth'         => count( $_aSectionPath ) - 1,    // 3.7.0+ - zero base
+                '_nested_depth'         => count( $_aSectionPath ) - 1, // 3.7.0+ - zero base
             ) 
             + $this->aSectionset
             + array(