michaeluno / admin-page-framework

development/factory/admin_page/_controller/AdminPageFramework_Link_admin_page.php

Braces +5 added lines, -4 removed lines

@@ -36,7 +36,7 @@ 
      */
     public function _replyToSetFooterInfo() {
 
-        if ( ! $this->oProp->isPageAdded() ) { 
+        if ( ! $this->oProp->isPageAdded() ) {
             return; 
         }
         parent::_replyToSetFooterInfo();
@@ -211,7 +211,7 @@ 
     public function _replyToAddSettingsLinkInPluginListingPage( $aLinks ) {
          
         // If the sub-pages are not added, do nothing.
-        if ( count( $this->oProp->aPages ) < 1 ) { 
+        if ( count( $this->oProp->aPages ) < 1 ) {
             return $aLinks; 
         }    
         
@@ -260,7 +260,7 @@ 
      */
     public function _replyToAddLinkToPluginDescription( $aLinks, $sFile ) {
 
-        if ( $sFile !== plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ) ) { 
+        if ( $sFile !== plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ) ) {
             return $aLinks; 
         }
         
@@ -271,7 +271,8 @@ 
             if ( ! $_sLLinkHTML ) {
                 continue;
             }
-            if ( is_array( $_sLLinkHTML ) ) {  // should not be an array
+            if ( is_array( $_sLLinkHTML ) ) {
+// should not be an array
                 $_aAddingLinks = array_merge( $_sLLinkHTML, $_aAddingLinks );
                 continue;
             } 

Indentation +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
             $this->oProp->aPluginTitleLinks = array_merge( 
                 $this->oProp->aPluginTitleLinks, 
                 $asLinks 
-           );
+            );
         }
                 
         $this->_addFilterHook_PluginTitleActionLink();

Spacing +29 added lines, -29 removed lines

@@ -36,7 +36,7 @@ 
      */
     public function _replyToSetFooterInfo() {
 
-        if ( ! $this->oProp->isPageAdded() ) { 
+        if ( !$this->oProp->isPageAdded() ) { 
             return; 
         }
         parent::_replyToSetFooterInfo();
@@ -56,8 +56,8 @@ 
      */
     public function _addLinkToPluginDescription( $asLinks ) {
         
-        if ( ! is_array( $asLinks ) ) {
-            $this->oProp->aPluginDescriptionLinks[] = $asLinks;
+        if ( !is_array( $asLinks ) ) {
+            $this->oProp->aPluginDescriptionLinks[ ] = $asLinks;
         } else {
             $this->oProp->aPluginDescriptionLinks = array_merge( 
                 $this->oProp->aPluginDescriptionLinks, 
@@ -84,8 +84,8 @@ 
      */    
     public function _addLinkToPluginTitle( $asLinks ) {
         
-        if ( ! is_array( $asLinks ) ) {
-            $this->oProp->aPluginTitleLinks[] = $asLinks;
+        if ( !is_array( $asLinks ) ) {
+            $this->oProp->aPluginTitleLinks[ ] = $asLinks;
         } else {
             $this->oProp->aPluginTitleLinks = array_merge( 
                 $this->oProp->aPluginTitleLinks, 
@@ -120,7 +120,7 @@ 
             }
             $_sPluginBaseName = plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] );
             add_filter( 
-                $this->_sFilterSuffix_PluginActionLinks . $_sPluginBaseName, 
+                $this->_sFilterSuffix_PluginActionLinks.$_sPluginBaseName, 
                 array( $this, '_replyToAddPluginActionLinks' ) 
             );            
         }
@@ -133,14 +133,14 @@ 
      * @return      string
      * @internal
      */ 
-    public function _replyToAddInfoInFooterLeft( $sLinkHTML='' ) {
+    public function _replyToAddInfoInFooterLeft( $sLinkHTML = '' ) {
 
-        if ( ! $this->_isPageAdded() ) {
+        if ( !$this->_isPageAdded() ) {
             return $sLinkHTML; // $sLinkHTML is given by the hook.
         }
-        $sLinkHTML  = empty( $this->oProp->aScriptInfo['sName'] )
+        $sLinkHTML = empty( $this->oProp->aScriptInfo[ 'sName' ] )
             ? $sLinkHTML
-            : $this->oProp->aFooterInfo['sLeft'];     
+            : $this->oProp->aFooterInfo[ 'sLeft' ];     
             
         $_sPageSlug = $this->oProp->getCurrentPageSlug();
         $_sTabSlug  = $this->oProp->getCurrentTabSlug();        
@@ -151,11 +151,11 @@ 
             array(                 
                 $this->getAOrB( 
                     $_sTabSlug,
-                    'footer_left_' . $_sPageSlug . '_' . $_sTabSlug,
+                    'footer_left_'.$_sPageSlug.'_'.$_sTabSlug,
                     null    // will be ignored
                 ),
-                'footer_left_' . $_sPageSlug,
-                'footer_left_' . $this->oProp->sClassName
+                'footer_left_'.$_sPageSlug,
+                'footer_left_'.$this->oProp->sClassName
             ),
             $sLinkHTML
         );        
@@ -169,9 +169,9 @@ 
      * @return      string
      * @internal
      */
-    public function _replyToAddInfoInFooterRight( $sLinkHTML='' ) {
+    public function _replyToAddInfoInFooterRight( $sLinkHTML = '' ) {
 
-        if ( ! $this->_isPageAdded() ) {
+        if ( !$this->_isPageAdded() ) {
             return $sLinkHTML; // $sLinkTHML is given by the hook.
         }
             
@@ -184,13 +184,13 @@ 
             array(                 
                 $this->getAOrB( 
                     $_sTabSlug,
-                    'footer_right_' . $_sPageSlug . '_' . $_sTabSlug,
+                    'footer_right_'.$_sPageSlug.'_'.$_sTabSlug,
                     null    // will be ignored
                 ),
-                'footer_right_' . $_sPageSlug,
-                'footer_right_' . $this->oProp->sClassName
+                'footer_right_'.$_sPageSlug,
+                'footer_right_'.$this->oProp->sClassName
             ),
-            $this->oProp->aFooterInfo['sRight']
+            $this->oProp->aFooterInfo[ 'sRight' ]
         );             
             
     }
@@ -203,10 +203,10 @@ 
          */
         private function _isPageAdded() {
             
-            if ( ! isset( $_GET['page'] ) ) {
+            if ( !isset( $_GET[ 'page' ] ) ) {
                 return false;
             }            
-            return ( bool ) $this->oProp->isPageAdded( $_GET['page'] );
+            return ( bool ) $this->oProp->isPageAdded( $_GET[ 'page' ] );
 
         }
     /**
@@ -228,7 +228,7 @@ 
             : $this->oProp->sLabelPluginSettingsLink;        
 
         // If the user disables the settings link, the label property is empty. If so, do not add it.
-        if ( ! $this->oProp->sLabelPluginSettingsLink ) {
+        if ( !$this->oProp->sLabelPluginSettingsLink ) {
             return $aLinks;
         }
 
@@ -246,13 +246,13 @@ 
         array_unshift(    
             $aLinks,
             // '<a href="' . esc_url( $_sLinkURL ) . '">' 
-            '<a ' . $this->getAttributes(
+            '<a '.$this->getAttributes(
                 array(
                     'href'      => esc_url( $_sLinkURL ),
                     // 3.5.7+ Added for acceptance testing
                     'class'     => 'apf-plugin-title-action-link apf-post-type',
                 )
-            ) . '>' 
+            ).'>' 
                 . $this->oProp->sLabelPluginSettingsLink 
             . '</a>'
         ); 
@@ -275,16 +275,16 @@ 
         
         // Backward compatibility sanitisation.
         $_aAddingLinks = array();
-        foreach( array_filter( $this->oProp->aPluginDescriptionLinks ) as $_sLLinkHTML ) {
+        foreach ( array_filter( $this->oProp->aPluginDescriptionLinks ) as $_sLLinkHTML ) {
             
-            if ( ! $_sLLinkHTML ) {
+            if ( !$_sLLinkHTML ) {
                 continue;
             }
             if ( is_array( $_sLLinkHTML ) ) {  // should not be an array
                 $_aAddingLinks = array_merge( $_sLLinkHTML, $_aAddingLinks );
                 continue;
             } 
-            $_aAddingLinks[] = ( string ) $_sLLinkHTML;
+            $_aAddingLinks[ ] = ( string ) $_sLLinkHTML;
             
         }
         return array_merge( $aLinks, $_aAddingLinks );
@@ -301,13 +301,13 @@ 
     public function _replyToAddPluginActionLinks( $aLinks ) {
 
         $_aAddingLinks = array();
-        foreach( array_filter( $this->oProp->aPluginTitleLinks ) as $_sLinkHTML ) {
+        foreach ( array_filter( $this->oProp->aPluginTitleLinks ) as $_sLinkHTML ) {
 
             if ( is_array( $_sLinkHTML ) ) {
                 $_aAddingLinks = array_merge( $_sLinkHTML, $aAddingLinks );
                 continue;
             } 
-            $_aAddingLinks[] = ( string ) $_sLinkHTML;
+            $_aAddingLinks[ ] = ( string ) $_sLinkHTML;
             
         }
         return array_merge( $aLinks, $_aAddingLinks );

development/factory/admin_page/_model/delegate/validaor/AdminPageFramework_Model__FormSubmission__Validator__ResetConfirm.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
                 
         // if the 'reset' key in the field definition array is set, this value will be set.
-        if ( ! $this->_shouldProceed( $oFactory, $aSubmits ) ) {
+        if ( !$this->_shouldProceed( $oFactory, $aSubmits ) ) {
             return;
         }
         
@@ -43,7 +43,7 @@ 
         );
                     
         // Go to the catch clause.
-        $_oException = new Exception( 'aReturn' );  // the property name to return from the catch clasue.
+        $_oException = new Exception( 'aReturn' ); // the property name to return from the catch clasue.
         $_oException->aReturn = $this->_confirmSubmitButtonAction( 
             $this->getElement( $aSubmitInformation, 'input_name' ),
             $this->getElement( $aSubmitInformation, 'section_id' ),

development/factory/admin_page/_model/delegate/validaor/AdminPageFramework_Model__FormSubmission__Validator__ContactForm.php

Spacing +9 added lines, -9 removed lines

@@ -17,7 +17,7 @@ 
  */
 class AdminPageFramework_Model__FormSubmission__Validator__ContactForm extends AdminPageFramework_Model__FormSubmission__Validator_Base {
     
-    public $sActionHookPrefix = 'try_validation_after_';    // 3.7.6 Changed it from `try_validation_before_`
+    public $sActionHookPrefix = 'try_validation_after_'; // 3.7.6 Changed it from `try_validation_before_`
     public $iHookPriority = 10;
     public $iCallbackParameters = 5;
  
@@ -34,7 +34,7 @@ 
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
                 
         // Check whether sending an email has been confirmed by the user or not.
-        if ( ! $this->_shouldProceed( $oFactory, $aSubmits ) ) {
+        if ( !$this->_shouldProceed( $oFactory, $aSubmits ) ) {
             return;
         }
         
@@ -45,13 +45,13 @@ 
             $this->getElement( $aSubmitInformation, 'section_id' )
         );
         $this->oFactory->oProp->_bDisableSavingOptions = true;
-        $this->deleteTransient( 'apf_tfd' . md5( 'temporary_form_data_' . $this->oFactory->oProp->sClassName . get_current_user_id() ) );
+        $this->deleteTransient( 'apf_tfd'.md5( 'temporary_form_data_'.$this->oFactory->oProp->sClassName.get_current_user_id() ) );
         
         // Schedule to remove the confirmation url query key.
         add_action( "setting_update_url_{$this->oFactory->oProp->sClassName}", array( $this, '_replyToRemoveConfirmationQueryKey' ) );
         
         // Go to the catch clause.
-        $_oException = new Exception( 'aReturn' );  // the property name to return from the catch clause.
+        $_oException = new Exception( 'aReturn' ); // the property name to return from the catch clause.
         $_oException->aReturn = $aInputs;
         throw $_oException;
     
@@ -86,7 +86,7 @@ 
          */
         private function _sendEmailInBackground( $aInputs, $sPressedInputNameFlat, $sSubmitSectionID ) {
             
-            $_sTranskentKey = 'apf_em_' . md5( $sPressedInputNameFlat . get_current_user_id() );
+            $_sTranskentKey = 'apf_em_'.md5( $sPressedInputNameFlat.get_current_user_id() );
             $_aEmailOptions = $this->getTransient( $_sTranskentKey );
             $this->deleteTransient( $_sTranskentKey );
 
@@ -101,13 +101,13 @@ 
                 'name'          => '',
             );
 
-            $_sTransientKey  = 'apf_emd_' . md5( $sPressedInputNameFlat . get_current_user_id() );
+            $_sTransientKey  = 'apf_emd_'.md5( $sPressedInputNameFlat.get_current_user_id() );
             $_aFormEmailData = array(
                 'email_options' => $_aEmailOptions,
                 'input'         => $aInputs,
                 'section_id'    => $sSubmitSectionID,
             );
-            $_bIsSet = $this->setTransient( $_sTransientKey,  $_aFormEmailData, 100 );
+            $_bIsSet = $this->setTransient( $_sTransientKey, $_aFormEmailData, 100 );
             
             // Send the email in the background.
             wp_remote_get( 
@@ -125,7 +125,7 @@ 
             );  
             
             // @remark      Not possible to tell whether it is sent or not at the moment because it is performed in the background.
-            $_bSent      = $_bIsSet;    
+            $_bSent = $_bIsSet;    
             $this->oFactory->setSettingNotice( 
                 $this->oFactory->oMsg->get( 
                     $this->getAOrB( 
@@ -147,7 +147,7 @@ 
          * @return      string
          */
         public function _replyToRemoveConfirmationQueryKey( $sSettingUpdateURL ) {
-            return remove_query_arg( array( 'confirmation', ), $sSettingUpdateURL );
+            return remove_query_arg( array( 'confirmation',), $sSettingUpdateURL );
         }      
         
 }

development/factory/admin_page/_model/delegate/validaor/AdminPageFramework_Model__FormSubmission__Validator__ContactFormConfirm.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
                                 
-        if ( ! $this->_shouldProceed( $oFactory, $aSubmits ) ) {
+        if ( !$this->_shouldProceed( $oFactory, $aSubmits ) ) {
             return;
         }
         
@@ -46,7 +46,7 @@ 
         );            
         
         // Go to the catch clause.
-        $_oException = new Exception( 'aReturn' );  // the property name to return from the catch clasue.
+        $_oException = new Exception( 'aReturn' ); // the property name to return from the catch clasue.
         $_oException->aReturn = $this->_confirmSubmitButtonAction( 
             $this->getElement( $aSubmitInformation, 'input_name' ), 
             $this->getElement( $aSubmitInformation, 'section_id' ), 

development/factory/admin_page/_view/AdminPageFramework_View__PageMetaboxEnabler.php

Spacing +7 added lines, -7 removed lines

@@ -51,11 +51,11 @@ 
          */
         public function _replyToEnableMetaBox() {
 
-            if ( ! $this->_isMetaBoxAdded() ) {
+            if ( !$this->_isMetaBoxAdded() ) {
                 return;
             }
 
-            $_sCurrentScreenID =  $this->getCurrentScreenID();
+            $_sCurrentScreenID = $this->getCurrentScreenID();
 
             // Trigger the add_meta_boxes hooks to allow meta boxes to be added.
             do_action( "add_meta_boxes_{$_sCurrentScreenID}", null );
@@ -85,7 +85,7 @@ 
             // the network admin adds '-network' in the screen ID and the hooks with that id won't be triggered so use the 'page_hook' global variable.
             if ( isset( $GLOBALS[ 'page_hook' ] ) ) {
                 add_action( 
-                    "admin_footer-{$GLOBALS['page_hook']}", 
+                    "admin_footer-{$GLOBALS[ 'page_hook' ]}", 
                     array( $this, '_replyToAddMetaboxScript' ) 
                 );    
             }
@@ -109,11 +109,11 @@ 
                 }
                 
                 $_sPageSlug = $this->getElement( $_GET, 'page', '' );
-                if ( ! $_sPageSlug ) {     
+                if ( !$_sPageSlug ) {     
                     return false;
                 }
                 
-                foreach( $_aPageMetaBoxClasses as $_sClassName => $_oMetaBox ) {
+                foreach ( $_aPageMetaBoxClasses as $_sClassName => $_oMetaBox ) {
                     if ( $this->_isPageOfMetaBox( $_sPageSlug, $_oMetaBox ) ) { 
                         return true; 
                     }
@@ -131,10 +131,10 @@ 
                  */
                 private function _isPageOfMetaBox( $sPageSlug, $oMetaBox ) {
                     
-                    if ( in_array( $sPageSlug , $oMetaBox->oProp->aPageSlugs ) ) {
+                    if ( in_array( $sPageSlug, $oMetaBox->oProp->aPageSlugs ) ) {
                         return true; // for numeric keys with a string value.
                     }
-                    if ( ! array_key_exists( $sPageSlug , $oMetaBox->oProp->aPageSlugs ) ) {
+                    if ( !array_key_exists( $sPageSlug, $oMetaBox->oProp->aPageSlugs ) ) {
                         return false; // for keys of page slugs, the key does not exist, it means not added.
                     }
                     

Braces +2 added lines, -2 removed lines

@@ -109,12 +109,12 @@
                 }
                 
                 $_sPageSlug = $this->getElement( $_GET, 'page', '' );
-                if ( ! $_sPageSlug ) {     
+                if ( ! $_sPageSlug ) {
                     return false;
                 }
                 
                 foreach( $_aPageMetaBoxClasses as $_sClassName => $_oMetaBox ) {
-                    if ( $this->_isPageOfMetaBox( $_sPageSlug, $_oMetaBox ) ) { 
+                    if ( $this->_isPageOfMetaBox( $_sPageSlug, $_oMetaBox ) ) {
                         return true; 
                     }
                 }

development/factory/admin_page/form/AdminPageFramework_Form_admin_page.php

Spacing +19 added lines, -19 removed lines

@@ -52,16 +52,16 @@ 
     public function getPageOnlyOptions( $aOptions, $sPageSlug ) {
 
         $_aStoredOptionsOfThePage = array();
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // Check the section
-            if ( ! $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
+            if ( !$this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
                 continue;
             }
 
             // At this point, the element belongs the given page slug as the section is of the given page slug's.
             $this->_setPageOnlyOptions( 
-                $_aStoredOptionsOfThePage,  // by reference - gets updated in the method.
+                $_aStoredOptionsOfThePage, // by reference - gets updated in the method.
                 $aOptions, 
                 $_aSubSectionsOrFields, 
                 $sPageSlug,
@@ -81,7 +81,7 @@ 
          * @internal
          */
         private function _setPageOnlyOptions( array &$_aStoredOptionsOfThePage, array $aOptions, array $_aSubSectionsOrFields, $sPageSlug, $_sSectionID ) {
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
                 
                 // If it's a sub-section array,
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -153,7 +153,7 @@ 
     public function getOtherPageOptions( $aOptions, $sPageSlug ) {
 
         $_aStoredOptionsNotOfThePage = array();
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // Check the section
             if ( $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
@@ -182,7 +182,7 @@ 
          * @internal
          */
         private function _setOtherPageOptions( array &$_aStoredOptionsNotOfThePage, array $aOptions, array $_aSubSectionsOrFields, $sPageSlug ) {
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
 
                 // It's a sub-section array. 
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -245,7 +245,7 @@ 
     public function getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug ) {
 
         $_aStoredOptionsNotOfTheTab = array();
-        foreach( $this->aFieldsets as $_sSectionPath => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionPath => $_aSubSectionsOrFields ) {
                         
             // If the section is of the given page and the given tab, skip.
             if ( $this->_isThisSectionSetToThisTab( $_sSectionPath, $sPageSlug, $sTabSlug ) ) {
@@ -276,7 +276,7 @@ 
         private function _setOtherTabOptions( array &$_aStoredOptionsNotOfTheTab, array $aOptions, array $_aSubSectionsOrFields, $sSectionPath ) {
             
             // At this point, the passed element belongs to the other tabs since the section of the given tab is skipped.
-            foreach ( $_aSubSectionsOrFields as $_isSubSectionIndexOrFieldID => $_aSubSectionOrField  ) {
+            foreach ( $_aSubSectionsOrFields as $_isSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
                 
                 // If it's a sub section
                 if ( $this->isNumericInteger( $_isSubSectionIndexOrFieldID ) ) {
@@ -334,7 +334,7 @@ 
      * @since       3.7.0      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {     
+    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug = '' ) {     
                 
         $_aOtherTabOptions = $this->getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug );
         $_aTabOptions      = $this->invertCastArrayContents( $aOptions, $_aOtherTabOptions );       
@@ -352,17 +352,17 @@ 
      * @since       3.7.0      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug='' ) {
+    public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug = '' ) {
         
         $_aStoredOptionsOfTheTab = array();
-        if ( ! $sTabSlug ) { 
+        if ( !$sTabSlug ) { 
             return $_aStoredOptionsOfTheTab; 
         }
         
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
              
             // Check the section
-            if ( ! $this->_isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) ) {
+            if ( !$this->_isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) ) {
                 continue;
             }
             
@@ -388,7 +388,7 @@ 
          */
         private function _setTabOnlyOptions( array &$_aStoredOptionsOfTheTab, array $aOptions, array $_aSubSectionsOrFields, $_sSectionID ) {
             
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
                                 
                 // if it's a sub-section array.
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -445,7 +445,7 @@ 
      */
     private function _isThisSectionSetToThisPage( $sSectionPath, $sPageSlug ) {
         
-        if ( ! isset( $this->aSectionsets[ $sSectionPath ][ 'page_slug' ] ) ) {
+        if ( !isset( $this->aSectionsets[ $sSectionPath ][ 'page_slug' ] ) ) {
             return false;
         }
         return ( 
@@ -463,10 +463,10 @@ 
      */
     private function _isThisSectionSetToThisTab( $sSectionPath, $sPageSlug, $sTabSlug ) {
         
-        if ( ! $this->_isThisSectionSetToThisPage( $sSectionPath, $sPageSlug ) ) {
+        if ( !$this->_isThisSectionSetToThisPage( $sSectionPath, $sPageSlug ) ) {
             return false;
         }
-        if ( ! isset( $this->aSectionsets[ $sSectionPath ][ 'tab_slug' ] ) ) {
+        if ( !isset( $this->aSectionsets[ $sSectionPath ][ 'tab_slug' ] ) ) {
             return false;
         }
         return (
@@ -483,9 +483,9 @@ 
      */
     private function _setOptionValue( &$aSubject, $asDimensionalPath, $aOptions ) {
         $_aDimensionalPath = $this->getAsArray( $asDimensionalPath );
-        $_mValue     = $this->getElement(
+        $_mValue = $this->getElement(
             $aOptions,
-            $_aDimensionalPath,    // as of 3.7.0, it can be an array or string
+            $_aDimensionalPath, // as of 3.7.0, it can be an array or string
             null
         );
         if ( isset( $_mValue ) ) {

Doc Comments +1 added lines

@@ -458,6 +458,7 @@
      * @internal
      * @since       3.5.3
      * @since       3.7.0      Moved from `AdminPageFramework_FormDefinition_Page`.
+     * @param string $sTabSlug
      * @return      boolean
      */
     private function _isThisSectionSetToThisTab( $sSectionPath, $sPageSlug, $sTabSlug ) {

Braces +3 added lines, -3 removed lines

@@ -106,7 +106,7 @@ 
                 );
                 // @todo Examine whether this check can be removed 
                 // as the section that hods this field is already checked above outside this loop.
-                if ( $sPageSlug !== $_aFieldset[ 'page_slug' ] ) { 
+                if ( $sPageSlug !== $_aFieldset[ 'page_slug' ] ) {
                     continue; 
                 }        
                 
@@ -333,7 +333,7 @@ 
      * @since       3.7.0      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {     
+    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {
                 
         $_aOtherTabOptions = $this->getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug );
         $_aTabOptions      = $this->invertCastArrayContents( $aOptions, $_aOtherTabOptions );       
@@ -354,7 +354,7 @@ 
     public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug='' ) {
         
         $_aStoredOptionsOfTheTab = array();
-        if ( ! $sTabSlug ) { 
+        if ( ! $sTabSlug ) {
             return $_aStoredOptionsOfTheTab; 
         }
         

development/factory/admin_page/AdminPageFramework_View_Page.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
      * @return      void
      * @internal
      */ 
-    protected function _renderPage( $sPageSlug, $sTabSlug=null ) {
+    protected function _renderPage( $sPageSlug, $sTabSlug = null ) {
         $_oPageRenderer = new AdminPageFramework_View__PageRenderer( $this, $sPageSlug, $sTabSlug );
         $_oPageRenderer->render();
     }

development/factory/_common/_abstract/AdminPageFramework_Factory_Controller.php

Braces +3 added lines, -3 removed lines

@@ -169,7 +169,7 @@ 
      */    
     public function addSettingSections( /* $aSection1, $aSection2=null, $_and_more=null */ ) {
         
-        foreach( func_get_args() as $_asSectionset ) { 
+        foreach( func_get_args() as $_asSectionset ) {
             $this->addSettingSection( $_asSectionset ); 
         }
         
@@ -238,7 +238,7 @@ 
      */
     public function addSettingSection( $aSectionset ) {
         
-        if ( ! is_array( $aSectionset ) ) { 
+        if ( ! is_array( $aSectionset ) ) {
             return; 
         }
         
@@ -271,7 +271,7 @@ 
     * @return       void
     */ 
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $_aFieldset ) { 
+        foreach( func_get_args() as $_aFieldset ) {
             $this->addSettingField( $_aFieldset ); 
         }
     }    

Doc Comments +2 added lines, -2 removed lines

@@ -218,7 +218,7 @@ 
      * @access      public
      * @remark      The actual registration will be performed in the `_replyToRegisterSettings()` method with the `admin_menu` hook.
      * @remark      The `$oForm` property should be created in each extended class.
-     * @param       array|string        $aSection       the section array. If a string is passed, it is considered as a target page slug that will be used as a page slug element from the next call so that the element can be omitted.
+     * @param       array|string        $aSectionset       the section array. If a string is passed, it is considered as a target page slug that will be used as a page slug element from the next call so that the element can be omitted.
      * <h4>Section Definition Array</h4>
      * <ul>
      *      <li>**section_id** - (string) the section ID. Avoid using non-alphabetic characters except underscore and numbers.</li>
@@ -347,7 +347,7 @@ 
      * @since        3.0.0   The scope changed to public to indicate the users will use.
      * @return       void
      * @remark       The $oForm property should be created in each extended class.
-     * @param        array|string       $asField        A field definition array or a string of the target section id.
+     * @param        array|string       $asFieldset        A field definition array or a string of the target section id.
     * <h4>Built-in Field Types</h4>
     * <ul>
     *       <li>[text](./class-AdminPageFramework_FieldType_text.html) - a normal field to enter short text input.</li>

Indentation +187 added lines, -187 removed lines

@@ -306,18 +306,18 @@ 
     }     
         
     /**
-    * Adds form fields.
-    * 
-    * It inserts the given field definition arrays into the class property and later they are parsed when fields are registered. The field definition array requires specific keys. Refer to the parameter section of this method.
-    * 
-    * @since        2.0.0
-    * @since        3.5.3       Removed the parameter declarations as they are caught with the func_get_args().
-    * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
-    * @param        array       the field definition array.
-    * @param        array       (optional) another field array.
-    * @param        array       (optional) add more field arrays to the next parameters as many as necessary.
-    * @return       void
-    */ 
+     * Adds form fields.
+     * 
+     * It inserts the given field definition arrays into the class property and later they are parsed when fields are registered. The field definition array requires specific keys. Refer to the parameter section of this method.
+     * 
+     * @since        2.0.0
+     * @since        3.5.3       Removed the parameter declarations as they are caught with the func_get_args().
+     * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
+     * @param        array       the field definition array.
+     * @param        array       (optional) another field array.
+     * @param        array       (optional) add more field arrays to the next parameters as many as necessary.
+     * @return       void
+     */ 
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
         foreach( func_get_args() as $_aFieldset ) { 
             $this->addSettingField( $_aFieldset ); 
@@ -348,57 +348,57 @@ 
      * @return       void
      * @remark       The $oForm property should be created in each extended class.
      * @param        array|string       $asField        A field definition array or a string of the target section id.
-    * <h4>Built-in Field Types</h4>
-    * <ul>
-    *       <li>[text](./class-AdminPageFramework_FieldType_text.html) - a normal field to enter short text input.</li>
-    *       <li>[password](./class-AdminPageFramework_FieldType_text.html) - a masked text input field.</li>
-    *       <li>[textarea](./class-AdminPageFramework_FieldType_textarea.html) - a text input field with multiple lines. It supports rich text editors.</li>
-    *       <li>[radio](./class-AdminPageFramework_FieldType_radio.html) - a set of radio buttons that lets the user pick an option.</li>
-    *       <li>[checkbox](./class-AdminPageFramework_FieldType_checkbox.html) - a check box that lets the user enable/disable an item.</li>
-    *       <li>[select](./class-AdminPageFramework_FieldType_select.html) - a drop-down list that lest the user pick one or more item(s) from a list.</li>
-    *       <li>[hidden](./class-AdminPageFramework_FieldType_hidden.html) - a field with invisible input values.</li>
-    *       <li>[file](./class-AdminPageFramework_FieldType_file.html) - a file uploader that lets the user upload files.</li>
-    *       <li>[image](./class-AdminPageFramework_FieldType_image.html) - a text field with an image uploader that lets the user set the image URL.</li>
-    *       <li>[media](./class-AdminPageFramework_FieldType_media.html) - a text field with a media uploader that lets the user set the file URL.</li>
-    *       <li>[color](./class-AdminPageFramework_FieldType_color.html) - a text field with a color picker.</li>
-    *       <li>[submit](./class-AdminPageFramework_FieldType_submit.html) - a submit button that lets the user send the form.</li>
-    *       <li>[export](./class-AdminPageFramework_FieldType_export.html) - a custom submit field that lets the user export stored data.</li>
-    *       <li>[import](./class-AdminPageFramework_FieldType_import.html) - a custom combination field of file and submit fields that let the user import data.</li>
-    *       <li>[posttype](./class-AdminPageFramework_FieldType_posttype.html) - a check-list of post types enabled on the site.</li>
-    *       <li>[taxonomy](./class-AdminPageFramework_FieldType_taxonomy.html) - a set of check-lists of taxonomies enabled on the site in a tabbed box.</li>
-    *       <li>[size](./class-AdminPageFramework_FieldType_size.html) - a combination field of the text and the select fields that let the user set sizes with a selectable unit.</li>
-    *       <li>[section_title](./class-AdminPageFramework_FieldType_section_title.html) - [3.0.0+] a text field type that will be placed in the section title so that it lets the user set the section title. Note that only one field with this field type is allowed per a section.</li>
-    *       <li>[system](./class-AdminPageFramework_FieldType_system.html) - [3.3.0+] a custom text area field that displays the system information including the PHP settings, the framework version, MySQL version etc.</li>
-    *       <li>[inline_mixed](./class-AdminPageFramework_FieldType_inline_mixed.html) - [3.8.0+] a field that include inner fields with different field types. </li>
-    * </ul>
-    * <h4>Field Definition Arguments</h4>
-    * <ul>
-    *       <li>**field_id** - (required, string) the field ID. Avoid using non-alphabetic characters except underscore and numbers.</li>
-    *       <li>**type** - (optional, string) the type of the field. The supported types are listed below. When creating nested fields, this argument can be omitted.</li>
-    *       <li>**section_id** - (optional, string) the section ID that the field belongs to. If not set, the internal `_default` section ID will be assigned.</li>
-    *       <li>**title** - (optional, string) the title of the section.</li>
-    *       <li>**description** - (optional, string) the description of the field which is inserted into the after the input field tag.</li>
-    *       <li>**tip** - (optional, string) the tip for the field which is displayed when the mouse is hovered over the field title.</li>
-    *       <li>**capability** - (optional, string) the <a href="http://codex.wordpress.org/Roles_and_Capabilities">access level</a> of the section. If the page visitor does not have sufficient capability, the section will be invisible to them.</li>
-    *       <li>**error_message** - (optional, string) the error message to display above the input field.</li>
-    *       <li>**before_field** - (optional, string) the HTML string to insert before the input field output.</li>
-    *       <li>**after_field** - (optional, string) the HTML string to insert after the input field output.</li>
-    *       <li>**if** - (optional, boolean) if the passed value is false, the section will not be registered.</li>
-    *       <li>**order** - (optional, integer) the order number of the section. The higher the number is, the lower the position it gets.</li>
-    *       <li>**label** - (optional, string) the text label(s) associated with and displayed along with the input field. Some input types can ignore this key.</li>
-    *       <li>**default** - (optional, string|array) the default value(s) assigned to the input tag's value attribute.</li>
-    *       <li>**value** - (optional, string|array) the value(s) assigned to the input tag's `value` attribute to override the default and the stored value.</li>
-    *       <li>**delimiter** - (optional, string) the HTML string that delimits multiple elements. This is available if the <var>label</var> key is passed as array. It will be enclosed in inline-block elements so the passed HTML string should not contain block elements.</li>
-    *       <li>**before_input** - (optional, string) the HTML string inserted right before the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
-    *       <li>**after_input** - (optional, string) the HTML string inserted right after the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
-    *       <li>**label_min_width** - (optional, string) the inline style property of the `min-width` of the label tag for the field. If the unit is not specified, 'px' is applied. Default: `120`. e.g. `100%`</li> 
-    *       <li>**help** - (optional, string) the help description added to the contextual help tab.</li>
-    *       <li>**help_aside** - (optional, string) the additional help description for the side bar of the contextual help tab.</li>
-    *       <li>**repeatable** - [3.0.0+] (optional, array|boolean) whether the fields should be repeatable. If it yields true, the plus and the minus buttons appear next to each field that lets the user add/remove the fields. Optionally an setting array can be passed.
-    *           <h5>Repeatable Fields Setting Array</h5>
-    *           <ul>
-    *                 <li>**max** - the allowed maximum number of fields to be repeated.</li>
-    *                 <li>**min** - the allowed minimum number of fields to be repeated.</li>
+     * <h4>Built-in Field Types</h4>
+     * <ul>
+     *       <li>[text](./class-AdminPageFramework_FieldType_text.html) - a normal field to enter short text input.</li>
+     *       <li>[password](./class-AdminPageFramework_FieldType_text.html) - a masked text input field.</li>
+     *       <li>[textarea](./class-AdminPageFramework_FieldType_textarea.html) - a text input field with multiple lines. It supports rich text editors.</li>
+     *       <li>[radio](./class-AdminPageFramework_FieldType_radio.html) - a set of radio buttons that lets the user pick an option.</li>
+     *       <li>[checkbox](./class-AdminPageFramework_FieldType_checkbox.html) - a check box that lets the user enable/disable an item.</li>
+     *       <li>[select](./class-AdminPageFramework_FieldType_select.html) - a drop-down list that lest the user pick one or more item(s) from a list.</li>
+     *       <li>[hidden](./class-AdminPageFramework_FieldType_hidden.html) - a field with invisible input values.</li>
+     *       <li>[file](./class-AdminPageFramework_FieldType_file.html) - a file uploader that lets the user upload files.</li>
+     *       <li>[image](./class-AdminPageFramework_FieldType_image.html) - a text field with an image uploader that lets the user set the image URL.</li>
+     *       <li>[media](./class-AdminPageFramework_FieldType_media.html) - a text field with a media uploader that lets the user set the file URL.</li>
+     *       <li>[color](./class-AdminPageFramework_FieldType_color.html) - a text field with a color picker.</li>
+     *       <li>[submit](./class-AdminPageFramework_FieldType_submit.html) - a submit button that lets the user send the form.</li>
+     *       <li>[export](./class-AdminPageFramework_FieldType_export.html) - a custom submit field that lets the user export stored data.</li>
+     *       <li>[import](./class-AdminPageFramework_FieldType_import.html) - a custom combination field of file and submit fields that let the user import data.</li>
+     *       <li>[posttype](./class-AdminPageFramework_FieldType_posttype.html) - a check-list of post types enabled on the site.</li>
+     *       <li>[taxonomy](./class-AdminPageFramework_FieldType_taxonomy.html) - a set of check-lists of taxonomies enabled on the site in a tabbed box.</li>
+     *       <li>[size](./class-AdminPageFramework_FieldType_size.html) - a combination field of the text and the select fields that let the user set sizes with a selectable unit.</li>
+     *       <li>[section_title](./class-AdminPageFramework_FieldType_section_title.html) - [3.0.0+] a text field type that will be placed in the section title so that it lets the user set the section title. Note that only one field with this field type is allowed per a section.</li>
+     *       <li>[system](./class-AdminPageFramework_FieldType_system.html) - [3.3.0+] a custom text area field that displays the system information including the PHP settings, the framework version, MySQL version etc.</li>
+     *       <li>[inline_mixed](./class-AdminPageFramework_FieldType_inline_mixed.html) - [3.8.0+] a field that include inner fields with different field types. </li>
+     * </ul>
+     * <h4>Field Definition Arguments</h4>
+     * <ul>
+     *       <li>**field_id** - (required, string) the field ID. Avoid using non-alphabetic characters except underscore and numbers.</li>
+     *       <li>**type** - (optional, string) the type of the field. The supported types are listed below. When creating nested fields, this argument can be omitted.</li>
+     *       <li>**section_id** - (optional, string) the section ID that the field belongs to. If not set, the internal `_default` section ID will be assigned.</li>
+     *       <li>**title** - (optional, string) the title of the section.</li>
+     *       <li>**description** - (optional, string) the description of the field which is inserted into the after the input field tag.</li>
+     *       <li>**tip** - (optional, string) the tip for the field which is displayed when the mouse is hovered over the field title.</li>
+     *       <li>**capability** - (optional, string) the <a href="http://codex.wordpress.org/Roles_and_Capabilities">access level</a> of the section. If the page visitor does not have sufficient capability, the section will be invisible to them.</li>
+     *       <li>**error_message** - (optional, string) the error message to display above the input field.</li>
+     *       <li>**before_field** - (optional, string) the HTML string to insert before the input field output.</li>
+     *       <li>**after_field** - (optional, string) the HTML string to insert after the input field output.</li>
+     *       <li>**if** - (optional, boolean) if the passed value is false, the section will not be registered.</li>
+     *       <li>**order** - (optional, integer) the order number of the section. The higher the number is, the lower the position it gets.</li>
+     *       <li>**label** - (optional, string) the text label(s) associated with and displayed along with the input field. Some input types can ignore this key.</li>
+     *       <li>**default** - (optional, string|array) the default value(s) assigned to the input tag's value attribute.</li>
+     *       <li>**value** - (optional, string|array) the value(s) assigned to the input tag's `value` attribute to override the default and the stored value.</li>
+     *       <li>**delimiter** - (optional, string) the HTML string that delimits multiple elements. This is available if the <var>label</var> key is passed as array. It will be enclosed in inline-block elements so the passed HTML string should not contain block elements.</li>
+     *       <li>**before_input** - (optional, string) the HTML string inserted right before the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
+     *       <li>**after_input** - (optional, string) the HTML string inserted right after the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
+     *       <li>**label_min_width** - (optional, string) the inline style property of the `min-width` of the label tag for the field. If the unit is not specified, 'px' is applied. Default: `120`. e.g. `100%`</li> 
+     *       <li>**help** - (optional, string) the help description added to the contextual help tab.</li>
+     *       <li>**help_aside** - (optional, string) the additional help description for the side bar of the contextual help tab.</li>
+     *       <li>**repeatable** - [3.0.0+] (optional, array|boolean) whether the fields should be repeatable. If it yields true, the plus and the minus buttons appear next to each field that lets the user add/remove the fields. Optionally an setting array can be passed.
+     *           <h5>Repeatable Fields Setting Array</h5>
+     *           <ul>
+     *                 <li>**max** - the allowed maximum number of fields to be repeated.</li>
+     *                 <li>**min** - the allowed minimum number of fields to be repeated.</li>
      *                <li>**disabled** - (optional, boolean|array) [3.8.13+]
      *                    <ul>
      *                        <li>**message** - (optional, string) the message to show when the user clicks on the `Add` repeatable button.</li>
@@ -407,108 +407,108 @@ 
      *                        <li>**box_width** - (optional, integer) the height of the modal window that displays the message. Default: 100.</li>
      *                    <ul>
      *                </li>
-    *           </ul>
-    *       </li>
-    *       <li>**sortable** - [3.0.0+] (optional, boolean) whether the fields should be sortable. If it yields true, the fields will be enclosed in a draggable box.
-    *       <li>**attributes** - [3.0.0+] (optional, array) holds key-value pairs representing the attribute and its property. Note that some field types have specific keys in the first dimensions. e.g. `array( 'class' => 'my_custom_class_selector', 'style' => 'background-color:#777', 'size' => 20, )` Every field holds the following nested attribute definition arrays.
-    *           <ul>
-    *               <li>**fieldrow** - the `td` tag element containing the field output.</li>
-    *               <li>**fieldset** - the `fieldset` tag element containing the field output.</li>
-    *               <li>**fields** - the `div` tag element containing the sub-fields and the main field.</li>
-    *               <li>**field** - the `div` tag element containing each field.</li>
-    *           </ul>
-    *       </li>
-    *       <li>**show_title_column** - [3.0.0+] (optional, boolean) If true, the field title column will be omitted from the output.</li>
-    *       <li>**hidden** - [3.0.0+] (optional, boolean) If true, the entire field row output will be invisible with the inline style attribute of `style="display:none"`.</li>
-    *       <li>**save** - [3.6.0+] (optional, boolean) If `false`, the field value will not be saved. Default: `true`</li>
-    *       <li>**content** - (optional, string|array) [3.6.1+] a custom section output. [3.8.0+] Supports an array to be passed for nested and inline-mixed fields. If an array holding field definitions is set, those fields will be nested.
-    *           <h4>Example</h4>
-    * <pre><code>
-    *   $this->addSettingFields(
-    *       'my_section_id', // the target section ID - pass dimensional keys of the section
-    *       array(
-    *           'field_id'      => 'Y',
-    *           'title'         => __( 'Y', 'admin-page-framework-loader' ),
-    *           'description'   => __( 'By passing an array of field definition to the <code>content</code> argument, you can nest fields.', 'admin-page-framework-loader' )
-    *               . ' ' . __( 'Also the <code>type</code> argument can be omitted.', 'admin-page-framework-loader' ),
-    *           'content'       => array(
-    *               array(
-    *                   'field_id'      => 'i',
-    *                   'title'         => __( 'i', 'admin-page-framework-loader' ),                    
-    *                   'type'          => 'textarea',
-    *               ),
-    *               array(
-    *                   'field_id'      => 'ii',
-    *                   'title'         => __( 'ii', 'admin-page-framework-loader' ),                    
-    *                   'type'          => 'color',                    
-    *               ),
-    *               array(
-    *                   'field_id'      => 'iii',
-    *                   'title'         => __( 'iii', 'admin-page-framework-loader' ),
-    *                   'repeatable'    => true,
-    *                   'sortable'      => true,
-    *                   'content'       => array(
-    *                       array(
-    *                           'field_id'      => 'a',
-    *                           'title'         => __( 'a', 'admin-page-framework-loader' ),                    
-    *                           'type'          => 'image',
-    *                           'attributes'    => array(
-    *                               'preview' => array(
-    *                                   'style' => 'max-width: 200px;',
-    *                               ),
-    *                           ),                                
-    *                       ),
-    *                       array(
-    *                           'field_id'      => 'b',
-    *                           'title'         => __( 'b', 'admin-page-framework-loader' ),
-    *                           'content'       => array(
-    *                               array(
-    *                                   'field_id'      => 'first',
-    *                                   'title'         => __( '1st', 'admin-page-framework-loader' ),                    
-    *                                   'type'          => 'color',
-    *                                   'repeatable'    => true,
-    *                                   'sortable'      => true,
-    *                               ),                                
-    *                               array(
-    *                                   'field_id'      => 'second',
-    *                                   'title'         => __( '2nd', 'admin-page-framework-loader' ),                    
-    *                                   'type'          => 'size',
-    *                               ),
-    *                               array(
-    *                                   'field_id'      => 'third',
-    *                                   'title'         => __( '3rd', 'admin-page-framework-loader' ),                    
-    *                                   'type'          => 'select',
-    *                                   'label'         => array(
-    *                                       'x' => 'X',
-    *                                       'y' => 'Y',
-    *                                       'z' => 'Z',                                        
-    *                                   ),
-    *                               ),                                    
-    *                           ),
-    *                           // 'description'   => '',
-    *                       ),                            
-    *                       array(
-    *                           'field_id'      => 'c',
-    *                           'title'         => __( 'c', 'admin-page-framework-loader' ),                    
-    *                           'type'          => 'radio',                    
-    *                           'label'         => array(
-    *                               'a' => __( 'Apple', 'admin-page-framework-loader' ),
-    *                               'b' => __( 'Banana', 'admin-page-framework-loader' ),
-    *                               'c' => __( 'Cherry', 'admin-page-framework-loader' ),
-    *                           ),
-    *                           'default'       => 'b',
-    *                       ),                        
-    *                   )
-    *               ),                    
-    *           ),
-    *       )
-    *   );
-    * </code></pre>
-    *       </li>
-    *       <li>**placement** - [3.8.0+] (optional, string) Specifies where the field is rendered. Either `normal` `section_title` or `field_title` is accepted. `field_title` can only be used by nested fields defined in the `content` argument.  Default: `normal`</li>
-    *       <li>**show_debug_info** - (optional, boolean) [3.8.8+] Whether to show debug information such as field definition tool-tips. Default: `true`.</li>
-    * </ul>
-    */
+     *           </ul>
+     *       </li>
+     *       <li>**sortable** - [3.0.0+] (optional, boolean) whether the fields should be sortable. If it yields true, the fields will be enclosed in a draggable box.
+     *       <li>**attributes** - [3.0.0+] (optional, array) holds key-value pairs representing the attribute and its property. Note that some field types have specific keys in the first dimensions. e.g. `array( 'class' => 'my_custom_class_selector', 'style' => 'background-color:#777', 'size' => 20, )` Every field holds the following nested attribute definition arrays.
+     *           <ul>
+     *               <li>**fieldrow** - the `td` tag element containing the field output.</li>
+     *               <li>**fieldset** - the `fieldset` tag element containing the field output.</li>
+     *               <li>**fields** - the `div` tag element containing the sub-fields and the main field.</li>
+     *               <li>**field** - the `div` tag element containing each field.</li>
+     *           </ul>
+     *       </li>
+     *       <li>**show_title_column** - [3.0.0+] (optional, boolean) If true, the field title column will be omitted from the output.</li>
+     *       <li>**hidden** - [3.0.0+] (optional, boolean) If true, the entire field row output will be invisible with the inline style attribute of `style="display:none"`.</li>
+     *       <li>**save** - [3.6.0+] (optional, boolean) If `false`, the field value will not be saved. Default: `true`</li>
+     *       <li>**content** - (optional, string|array) [3.6.1+] a custom section output. [3.8.0+] Supports an array to be passed for nested and inline-mixed fields. If an array holding field definitions is set, those fields will be nested.
+     *           <h4>Example</h4>
+     * <pre><code>
+     *   $this->addSettingFields(
+     *       'my_section_id', // the target section ID - pass dimensional keys of the section
+     *       array(
+     *           'field_id'      => 'Y',
+     *           'title'         => __( 'Y', 'admin-page-framework-loader' ),
+     *           'description'   => __( 'By passing an array of field definition to the <code>content</code> argument, you can nest fields.', 'admin-page-framework-loader' )
+     *               . ' ' . __( 'Also the <code>type</code> argument can be omitted.', 'admin-page-framework-loader' ),
+     *           'content'       => array(
+     *               array(
+     *                   'field_id'      => 'i',
+     *                   'title'         => __( 'i', 'admin-page-framework-loader' ),                    
+     *                   'type'          => 'textarea',
+     *               ),
+     *               array(
+     *                   'field_id'      => 'ii',
+     *                   'title'         => __( 'ii', 'admin-page-framework-loader' ),                    
+     *                   'type'          => 'color',                    
+     *               ),
+     *               array(
+     *                   'field_id'      => 'iii',
+     *                   'title'         => __( 'iii', 'admin-page-framework-loader' ),
+     *                   'repeatable'    => true,
+     *                   'sortable'      => true,
+     *                   'content'       => array(
+     *                       array(
+     *                           'field_id'      => 'a',
+     *                           'title'         => __( 'a', 'admin-page-framework-loader' ),                    
+     *                           'type'          => 'image',
+     *                           'attributes'    => array(
+     *                               'preview' => array(
+     *                                   'style' => 'max-width: 200px;',
+     *                               ),
+     *                           ),                                
+     *                       ),
+     *                       array(
+     *                           'field_id'      => 'b',
+     *                           'title'         => __( 'b', 'admin-page-framework-loader' ),
+     *                           'content'       => array(
+     *                               array(
+     *                                   'field_id'      => 'first',
+     *                                   'title'         => __( '1st', 'admin-page-framework-loader' ),                    
+     *                                   'type'          => 'color',
+     *                                   'repeatable'    => true,
+     *                                   'sortable'      => true,
+     *                               ),                                
+     *                               array(
+     *                                   'field_id'      => 'second',
+     *                                   'title'         => __( '2nd', 'admin-page-framework-loader' ),                    
+     *                                   'type'          => 'size',
+     *                               ),
+     *                               array(
+     *                                   'field_id'      => 'third',
+     *                                   'title'         => __( '3rd', 'admin-page-framework-loader' ),                    
+     *                                   'type'          => 'select',
+     *                                   'label'         => array(
+     *                                       'x' => 'X',
+     *                                       'y' => 'Y',
+     *                                       'z' => 'Z',                                        
+     *                                   ),
+     *                               ),                                    
+     *                           ),
+     *                           // 'description'   => '',
+     *                       ),                            
+     *                       array(
+     *                           'field_id'      => 'c',
+     *                           'title'         => __( 'c', 'admin-page-framework-loader' ),                    
+     *                           'type'          => 'radio',                    
+     *                           'label'         => array(
+     *                               'a' => __( 'Apple', 'admin-page-framework-loader' ),
+     *                               'b' => __( 'Banana', 'admin-page-framework-loader' ),
+     *                               'c' => __( 'Cherry', 'admin-page-framework-loader' ),
+     *                           ),
+     *                           'default'       => 'b',
+     *                       ),                        
+     *                   )
+     *               ),                    
+     *           ),
+     *       )
+     *   );
+     * </code></pre>
+     *       </li>
+     *       <li>**placement** - [3.8.0+] (optional, string) Specifies where the field is rendered. Either `normal` `section_title` or `field_title` is accepted. `field_title` can only be used by nested fields defined in the `content` argument.  Default: `normal`</li>
+     *       <li>**show_debug_info** - (optional, boolean) [3.8.8+] Whether to show debug information such as field definition tool-tips. Default: `true`.</li>
+     * </ul>
+     */
     public function addSettingField( $asFieldset ) {
         if ( method_exists( $this->oForm, 'addField' ) ) {
             $this->oForm->addField( $asFieldset );     
@@ -572,28 +572,28 @@ 
     }
     
     /**
-    * Sets the given message to be displayed in the next page load. 
-    * 
-    * This is used to inform users about the submitted input data, such as "Updated successfully." or "Problem occurred." etc. 
-    * and normally used in validation callback methods.
-    * 
-    * <h4>Example</h4>
-    * `
-    * if ( ! $bVerified ) {
-    *       $this->setFieldErrors( $aErrors );     
-    *       $this->setSettingNotice( 'There was an error in your input.' );
-    *       return $aOldPageOptions;
-    * }
-    * `
-    *
-    * @since        3.0.4     
-    * @access       public
-    * @param        string      $sMessage       the text message to be displayed.
-    * @param        string      $sType          (optional) the type of the message, either "error" or "updated"  is used.
-    * @param        array       $asAttributes   (optional) the tag attribute array applied to the message container HTML element. If a string is given, it is used as the ID attribute value.
-    * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
-    * @return       void
-    */      
+     * Sets the given message to be displayed in the next page load. 
+     * 
+     * This is used to inform users about the submitted input data, such as "Updated successfully." or "Problem occurred." etc. 
+     * and normally used in validation callback methods.
+     * 
+     * <h4>Example</h4>
+     * `
+     * if ( ! $bVerified ) {
+     *       $this->setFieldErrors( $aErrors );     
+     *       $this->setSettingNotice( 'There was an error in your input.' );
+     *       return $aOldPageOptions;
+     * }
+     * `
+     *
+     * @since        3.0.4     
+     * @access       public
+     * @param        string      $sMessage       the text message to be displayed.
+     * @param        string      $sType          (optional) the type of the message, either "error" or "updated"  is used.
+     * @param        array       $asAttributes   (optional) the tag attribute array applied to the message container HTML element. If a string is given, it is used as the ID attribute value.
+     * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
+     * @return       void
+     */      
     public function setSettingNotice( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
         $this->oForm->setSubmitNotice(
             $sMessage,

Spacing +11 added lines, -11 removed lines

@@ -53,7 +53,7 @@ 
      * @return      array|string
      * @since       3.7.0
      */
-    public function getMessage( $sKey='' ) {
+    public function getMessage( $sKey = '' ) {
         return $this->oMsg->get( $sKey );
     }
        
@@ -73,7 +73,7 @@ 
      * @return      array       The array holing the queued items.
      * @internal
      */
-    public function enqueueStyles( $aSRCs, $_vArg2=null ) {} 
+    public function enqueueStyles( $aSRCs, $_vArg2 = null ) {} 
     
     /**
      * Enqueues a style of the given source.
@@ -93,7 +93,7 @@ 
      * @return      string      The style handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */
-    public function enqueueStyle( $sSRC, $_vArg2=null ) {}
+    public function enqueueStyle( $sSRC, $_vArg2 = null ) {}
     
     /**
      * Enqueues scripts by the given sources.
@@ -114,7 +114,7 @@ 
      * @return      array        The array holding the queued items.
      * @internal
      */    
-    public function enqueueScripts( $aSRCs, $_vArg2=null ) {}
+    public function enqueueScripts( $aSRCs, $_vArg2 = null ) {}
     /**
      * Enqueues a script by the given source.
      *  
@@ -146,7 +146,7 @@ 
      * @return      string      The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */    
-    public function enqueueScript( $sSRC, $_vArg2=null ) {}    
+    public function enqueueScript( $sSRC, $_vArg2 = null ) {}    
     
     /*
      * Help Pane
@@ -164,7 +164,7 @@ 
      * @since       2.1.0
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the `replyToRegisterHelpTabTextForMetaBox()` method.
      */ 
-    public function addHelpText( $sHTMLContent, $sHTMLSidebarContent="" ) {
+    public function addHelpText( $sHTMLContent, $sHTMLSidebarContent = "" ) {
         if ( method_exists( $this->oHelpPane, '_addHelpText' ) ) {
             $this->oHelpPane->_addHelpText( $sHTMLContent, $sHTMLSidebarContent );
         }
@@ -200,7 +200,7 @@ 
      */    
     public function addSettingSections( /* $aSection1, $aSection2=null, $_and_more=null */ ) {
         
-        foreach( func_get_args() as $_asSectionset ) { 
+        foreach ( func_get_args() as $_asSectionset ) { 
             $this->addSettingSection( $_asSectionset ); 
         }
         
@@ -286,7 +286,7 @@ 
      */
     public function addSettingSection( $aSectionset ) {
         
-        if ( ! is_array( $aSectionset ) ) { 
+        if ( !is_array( $aSectionset ) ) { 
             return; 
         }
         
@@ -319,7 +319,7 @@ 
     * @return       void
     */ 
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $_aFieldset ) { 
+        foreach ( func_get_args() as $_aFieldset ) { 
             $this->addSettingField( $_aFieldset ); 
         }
     }    
@@ -594,7 +594,7 @@ 
     * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
     * @return       void
     */      
-    public function setSettingNotice( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
+    public function setSettingNotice( $sMessage, $sType = 'error', $asAttributes = array(), $bOverride = true ) {
         $this->oForm->setSubmitNotice(
             $sMessage,
             $sType,
@@ -613,7 +613,7 @@ 
      * @param       string      $sType If empty, the method will check if a message exists in all types. Otherwise, it checks the existence of a message of the specified type.
      * @return      boolean     True if a setting notice is set; otherwise, false.
      */
-    public function hasSettingNotice( $sType='' ) {
+    public function hasSettingNotice( $sType = '' ) {
         return $this->oForm->hasSubmitNotice( $sType );
     }
 

development/factory/_common/_abstract/_model/AdminPageFramework_Format_Base.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $_aParameters = func_get_args() + array( 
             $this->aSubject, 
         );
-        $this->aSubject  = $_aParameters[ 0 ];        
+        $this->aSubject = $_aParameters[ 0 ];        
         
     }