michaeluno / admin-page-framework

development/factory/admin_page/AdminPageFramework_Controller.php

Indentation +35 added lines, -35 removed lines

@@ -219,24 +219,24 @@ 
     }
     
     /**
-    * Adds the given link(s) into the description cell of the plugin listing table.
-    * 
-    * <h4>Example</h4>
-    * <code>$this->addLinkToPluginDescription( 
-    *       "<a href='http://www.google.com'>Google</a>",
-    *       "<a href='http://www.yahoo.com'>Yahoo!</a>"
-    * );</code>
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public from protected.
-    * @since        3.3.1       Moved from `AdminPageFramework`.
-    * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
-    * @param        string      the tagged HTML link text.
-    * @param        string      (optional) another tagged HTML link text.
-    * @param        string      (optional) add more as many as want by adding items to the next parameters.
-    * @access       public
-    * @return       void
-    */     
+     * Adds the given link(s) into the description cell of the plugin listing table.
+     * 
+     * <h4>Example</h4>
+     * <code>$this->addLinkToPluginDescription( 
+     *       "<a href='http://www.google.com'>Google</a>",
+     *       "<a href='http://www.yahoo.com'>Yahoo!</a>"
+     * );</code>
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public from protected.
+     * @since        3.3.1       Moved from `AdminPageFramework`.
+     * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
+     * @param        string      the tagged HTML link text.
+     * @param        string      (optional) another tagged HTML link text.
+     * @param        string      (optional) add more as many as want by adding items to the next parameters.
+     * @access       public
+     * @return       void
+     */     
     public function addLinkToPluginDescription( $sTaggedLinkHTML1, $sTaggedLinkHTML2=null, $_and_more=null ) {
         if ( method_exists( $this->oLink, '_addLinkToPluginDescription' ) ) {
             $this->oLink->_addLinkToPluginDescription( func_get_args() );     
@@ -244,23 +244,23 @@ 
     }
 
     /**
-    * Adds the given link(s) into the title cell of the plugin listing table.
-    * 
-    * <h4>Example</h4>
-    * <code>$this->addLinkToPluginTitle( 
-    *       "<a href='http://www.wordpress.org'>WordPress</a>"
-    * );</code>
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public from protected.
-    * @since        3.3.1       Moved from `AdminPageFramework`.
-    * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
-    * @param        string      the tagged HTML link text.
-    * @param        string      (optional) another tagged HTML link text.
-    * @param        string      (optional) add more as many as want by adding items to the next parameters.
-    * @access       public
-    * @return       void
-    */    
+     * Adds the given link(s) into the title cell of the plugin listing table.
+     * 
+     * <h4>Example</h4>
+     * <code>$this->addLinkToPluginTitle( 
+     *       "<a href='http://www.wordpress.org'>WordPress</a>"
+     * );</code>
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public from protected.
+     * @since        3.3.1       Moved from `AdminPageFramework`.
+     * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
+     * @param        string      the tagged HTML link text.
+     * @param        string      (optional) another tagged HTML link text.
+     * @param        string      (optional) add more as many as want by adding items to the next parameters.
+     * @access       public
+     * @return       void
+     */    
     public function addLinkToPluginTitle( $sTaggedLinkHTML1, $sTaggedLinkHTML2=null, $_and_more=null ) {    
         if ( method_exists( $this->oLink, '_addLinkToPluginTitle' ) ) {
             $this->oLink->_addLinkToPluginTitle( func_get_args() );     

Spacing +10 added lines, -10 removed lines

@@ -107,7 +107,7 @@ 
      * @param       array       $aCustomArgs    (optional) The argument array for more advanced parameters.
      * @return      array       The array holing the queued items.
      */
-    public function enqueueStyles( $aSRCs, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueStyles( $aSRCs, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueStyles' ) ) {
             return $this->oResource->_enqueueStyles( $aSRCs, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -144,7 +144,7 @@ 
      * </ul>
      * @return      string      The style handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */    
-    public function enqueueStyle( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueStyle( $sSRC, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueStyle' ) ) {
             return $this->oResource->_enqueueStyle( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );     
         }
@@ -171,7 +171,7 @@ 
      * @param       array       (optional) The argument array for more advanced parameters.
      * @return      array        The array holding the queued items.
      */
-    public function enqueueScripts( $aSRCs, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueScripts( $aSRCs, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueScripts' ) ) {
             return $this->oResource->_enqueueScripts( $aSRCs, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -212,7 +212,7 @@ 
      * </ul>
      * @return      string      The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */
-    public function enqueueScript( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {    
+    public function enqueueScript( $sSRC, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {    
         if ( method_exists( $this->oResource, '_enqueueScript' ) ) {
             return $this->oResource->_enqueueScript( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -237,7 +237,7 @@ 
     * @access       public
     * @return       void
     */     
-    public function addLinkToPluginDescription( $sTaggedLinkHTML1, $sTaggedLinkHTML2=null, $_and_more=null ) {
+    public function addLinkToPluginDescription( $sTaggedLinkHTML1, $sTaggedLinkHTML2 = null, $_and_more = null ) {
         if ( method_exists( $this->oLink, '_addLinkToPluginDescription' ) ) {
             $this->oLink->_addLinkToPluginDescription( func_get_args() );     
         }
@@ -261,7 +261,7 @@ 
     * @access       public
     * @return       void
     */    
-    public function addLinkToPluginTitle( $sTaggedLinkHTML1, $sTaggedLinkHTML2=null, $_and_more=null ) {    
+    public function addLinkToPluginTitle( $sTaggedLinkHTML1, $sTaggedLinkHTML2 = null, $_and_more = null ) {    
         if ( method_exists( $this->oLink, '_addLinkToPluginTitle' ) ) {
             $this->oLink->_addLinkToPluginTitle( func_get_args() );     
         }
@@ -317,7 +317,7 @@ 
      * @param       string      (optional) The class selector used in the message HTML element. 'error' and 'updated' are prepared by WordPress but it's not limited to them and can pass a custom name. Default: 'error'.
      * @param       string      (optional) The ID of the message. If not set, the hash of the message will be used.
      */
-    public function setAdminNotice( $sMessage, $sClassSelector='error', $sID='' ) {
+    public function setAdminNotice( $sMessage, $sClassSelector = 'error', $sID = '' ) {
             
         $sID = $sID ? $sID : md5( $sMessage );
         
@@ -354,9 +354,9 @@ 
      * @param       boolean         If true, the passed key(s) will be appended to the property; otherwise, it will override the property.
      * @return      void
      */
-    public function setDisallowedQueryKeys( $asQueryKeys, $bAppend=true ) {
+    public function setDisallowedQueryKeys( $asQueryKeys, $bAppend = true ) {
         
-        if ( ! $bAppend ) {
+        if ( !$bAppend ) {
             $this->oProp->aDisallowedQueryKeys = ( array ) $asQueryKeys;
             return;
         }
@@ -399,7 +399,7 @@ 
      * @param       mixed       $vDefault       the default value that will be returned if nothing is stored.
      * @return      mixed       If the field ID is not specified
      */
-    static public function getOption( $sOptionKey, $asKey=null, $vDefault=null ) {
+    static public function getOption( $sOptionKey, $asKey = null, $vDefault = null ) {
         return AdminPageFramework_WPUtility::getOption( $sOptionKey, $asKey, $vDefault );
     }    
     

Braces +2 added lines, -2 removed lines

@@ -212,7 +212,7 @@ 
      * </ul>
      * @return      string      The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */
-    public function enqueueScript( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {    
+    public function enqueueScript( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
         if ( method_exists( $this->oResource, '_enqueueScript' ) ) {
             return $this->oResource->_enqueueScript( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -261,7 +261,7 @@ 
     * @access       public
     * @return       void
     */    
-    public function addLinkToPluginTitle( $sTaggedLinkHTML1, $sTaggedLinkHTML2=null, $_and_more=null ) {    
+    public function addLinkToPluginTitle( $sTaggedLinkHTML1, $sTaggedLinkHTML2=null, $_and_more=null ) {
         if ( method_exists( $this->oLink, '_addLinkToPluginTitle' ) ) {
             $this->oLink->_addLinkToPluginTitle( func_get_args() );     
         }

Doc Comments +5 added lines

@@ -224,6 +224,7 @@ 
     * @param        string      the tagged HTML link text.
     * @param        string      (optional) another tagged HTML link text.
     * @param        string      (optional) add more as many as want by adding items to the next parameters.
+    * @param string $sTaggedLinkHTML1
     * @access       public
     * @return       void
     */     
@@ -249,6 +250,9 @@ 
     * @param        string      the tagged HTML link text.
     * @param        string      (optional) another tagged HTML link text.
     * @param        string      (optional) add more as many as want by adding items to the next parameters.
+    * @param null|string $sTaggedLinkHTML1
+    * @param string $sTaggedLinkHTML2
+    * @param string $_and_more
     * @access       public
     * @return       void
     */    
@@ -268,6 +272,7 @@ 
      * 
      * @since       3.1.0
      * @since       3.3.1       Moved from `AdminPageFramework`.
+     * @param string $sLabel
      */  
     public function setPluginSettingsLinkLabel( $sLabel ) {
         $this->oProp->sLabelPluginSettingsLink = $sLabel;

development/factory/admin_page/AdminPageFramework_Controller_Page.php

Spacing +14 added lines, -14 removed lines

@@ -93,7 +93,7 @@ 
      * @return      void
      */             
     public function addInPageTabs( /* $aTab1, $aTab2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $asTab ) { 
+        foreach ( func_get_args() as $asTab ) { 
             $this->addInPageTab( $asTab ); 
         }
     }
@@ -115,7 +115,7 @@ 
         
         // Target page slug - will be applied when no page slug is specified.
         static $__sTargetPageSlug; 
-        if ( ! is_array( $asInPageTab ) ) {
+        if ( !is_array( $asInPageTab ) ) {
             $__sTargetPageSlug = is_string( $asInPageTab ) 
                 ? $asInPageTab 
                 : $__sTargetPageSlug; // set the target page slug
@@ -130,23 +130,23 @@ 
         );
         
         // Set the target page slug for next calls
-        $__sTargetPageSlug  = $aInPageTab[ 'page_slug' ]; 
+        $__sTargetPageSlug = $aInPageTab[ 'page_slug' ]; 
 
         // Required keys
-        if ( ! isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) { 
+        if ( !isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) { 
             return; 
         }
         
         // Count - the number of added in-page tabs.
-        $_aElements         = $this->oUtil->getElement(
+        $_aElements = $this->oUtil->getElement(
             $this->oProp->aInPageTabs,
             $aInPageTab[ 'page_slug' ],
             array()
         );
-        $_iCountElement      = count( $_aElements );
+        $_iCountElement = count( $_aElements );
 
         // Pre-format
-        $aInPageTab         = array( 
+        $aInPageTab = array( 
             'page_slug' => $this->oUtil->sanitizeSlug( $aInPageTab[ 'page_slug' ] ),
             'tab_slug'  => $this->oUtil->sanitizeSlug( $aInPageTab[ 'tab_slug' ] ),
             'order'     => $this->oUtil->getAOrB(
@@ -174,7 +174,7 @@ 
      * @param       boolean     $bShow If false, the page title will not be displayed.
      * @return      void
      */ 
-    public function setPageTitleVisibility( $bShow=true, $sPageSlug='' ) {
+    public function setPageTitleVisibility( $bShow = true, $sPageSlug = '' ) {
         $this->_setPageProperty( 
             'bShowPageTitle', 
             'show_page_title', 
@@ -198,7 +198,7 @@ 
      * @remark      Page-heading tabs and in-page tabs are different. The former displays page titles and the latter displays tab titles.
      * @remark      If the second parameter is omitted, it sets the default value.
      */ 
-    public function setPageHeadingTabsVisibility( $bShow=true, $sPageSlug='' ) {
+    public function setPageHeadingTabsVisibility( $bShow = true, $sPageSlug = '' ) {
         $this->_setPageProperty( 
             'bShowPageHeadingTabs', 
             'show_page_heading_tabs', 
@@ -219,7 +219,7 @@ 
      * @param       string      $sPageSlug  The page to apply the visibility setting. If not set, it applies to all the pages.
      * @remark      If the second parameter is omitted, it sets the default value.
      */
-    public function setInPageTabsVisibility( $bShow=true, $sPageSlug='' ) {
+    public function setInPageTabsVisibility( $bShow = true, $sPageSlug = '' ) {
         $this->_setPageProperty( 
             'bShowInPageTabs', 
             'show_in_page_tabs', 
@@ -242,7 +242,7 @@ 
      * @param       string      $sPageSlug      The page slug that applies the setting.    
      * @remark      If the second parameter is omitted, it sets the default value.
      */     
-    public function setInPageTabTag( $sTag='h3', $sPageSlug='' ) {      
+    public function setInPageTabTag( $sTag = 'h3', $sPageSlug = '' ) {      
         $this->_setPageProperty( 
             'sInPageTabTag', 
             'in_page_tab_tag', 
@@ -265,9 +265,9 @@ 
      * @param       string      $sPageSlug  The page slug that applies the setting.    
      * @remark      If the second parameter is omitted, it sets the default value.
      */
-    public function setPageHeadingTabTag( $sTag='h2', $sPageSlug='' ) {
+    public function setPageHeadingTabTag( $sTag = 'h2', $sPageSlug = '' ) {
         $this->_setPageProperty( 
-            'sPageHeadingTabTag',   // property name
+            'sPageHeadingTabTag', // property name
             'page_heading_tab_tag', // property key
             $sTag, // value
             $sPageSlug  // page slug
@@ -289,7 +289,7 @@ 
             
             $this->oProp->{$sPropertyName} = $mValue;
        
-            foreach( $this->oProp->aPages as &$_aPage ) {
+            foreach ( $this->oProp->aPages as &$_aPage ) {
                 $_aPage[ $sPropertyKey ] = $mValue;
 
             }                        

Braces +4 added lines, -4 removed lines

@@ -93,7 +93,7 @@ 
      * @return      void
      */             
     public function addInPageTabs( /* $aTab1, $aTab2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $asTab ) { 
+        foreach( func_get_args() as $asTab ) {
             $this->addInPageTab( $asTab ); 
         }
     }
@@ -111,7 +111,7 @@ 
      * @remark      In-page tabs are different from page-heading tabs which are automatically added with page titles.
      * @return      void
      */         
-    public function addInPageTab( $asInPageTab ) {    
+    public function addInPageTab( $asInPageTab ) {
         
         // Target page slug - will be applied when no page slug is specified.
         static $__sTargetPageSlug; 
@@ -133,7 +133,7 @@ 
         $__sTargetPageSlug  = $aInPageTab[ 'page_slug' ]; 
 
         // Required keys
-        if ( ! isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) { 
+        if ( ! isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) {
             return; 
         }
         
@@ -242,7 +242,7 @@ 
      * @param       string      $sPageSlug      The page slug that applies the setting.    
      * @remark      If the second parameter is omitted, it sets the default value.
      */     
-    public function setInPageTabTag( $sTag='h3', $sPageSlug='' ) {      
+    public function setInPageTabTag( $sTag='h3', $sPageSlug='' ) {
         $this->_setPageProperty( 
             'sInPageTabTag', 
             'in_page_tab_tag', 

Doc Comments +4 added lines, -37 removed lines

@@ -52,43 +52,6 @@ 
      * @since       2.0.0
      * @since       3.0.0     Changed the scope to public. Added page slug target support. 
      * @since       3.3.1       Moved from `AdminPageFramework_Page`.
-     * @param       array     $aTab1 The in-page tab array.
-     * <h4>In-Page Tab Array</h4>
-     * <ul>
-     *     <li>**page_slug** - (string) the page slug that the tab belongs to.</li>
-     *     <li>**tab_slug** -  (string) the tab slug. Non-alphabetical characters should not be used including dots(.) and hyphens(-).</li>
-     *     <li>**title** - (string) the title of the tab.</li>
-     *     <li>**order** - (optional, integer) the order number of the tab. The lager the number is, the lower the position it is placed in the menu.</li>
-     *     <li>**show_in_page_tab** - (optional, boolean) default: `true`. If this is set to `false`, the tab title will not be displayed in the tab navigation menu; however, it is still accessible from the direct URL.</li>
-     *     <li>**parent_tab_slug** - (optional, string) this needs to be set if the above `show_in_page_tab` argument is `false` so that the parent tab will be emphasized as active when the hidden page is accessed.</li>
-     *     <li>**url** - [3.5.0+] (optional, string) If this is set, the link url of the navigation tab will be this url. Use this to create link only tab.</li>
-     *     <li>**capability** - [3.6.0+] (optional, string) The capability level.</li>
-     *     <li>**if** - [3.6.0+] (optional, boolean) Whether the tab should be displayed or not.</li>
-     *     <li>style - [3.6.3+] (optional) (string|array) The path or url of a stylesheet which gets loaded in the head tag. Or inline CSS rules.
-     * When custom arguments need to be set such as whether it should be inserted in the footer, set an array holding the following arguments.
-     *          <ul>
-     *              <li>src - (required, string) the source url or path.</li>    
-     *              <li>handle_id - (optional, string) The handle ID of the stylesheet.</li>    
-     *              <li>dependencies - (optional, array) The dependency array.</li>    
-     *              <li>version - (optional, string) The stylesheet version number.</li>    
-     *              <li>media - (optional, string) the description of the field which is inserted into the after the input field tag.</li>    
-     *          </ul>
-     *     </li>
-     *     <li>script - [3.6.3+] (optional) (string|array) The path or url of a JavaScript script which gets loaded in the head tag. Or an inline JavaScript script.
-     * When custom arguments need to be set such as whether it should be inserted in the footer, set an array holding the following arguments.
-     *          <ul>
-     *              <li>src - (required, string) the source url or path.</li>    
-     *              <li>handle_id - (optional, string) The handle ID of the stylesheet.</li>    
-     *              <li>dependencies - (optional, array) The dependency array.</li>    
-     *              <li>version - (optional, string) The stylesheet version number.</li>    
-     *              <li>translation - (optional, array) an array holding translation key-value pairs.</li>    
-     *          </ul>
-     *     </li>
-     *     <li>show_debug_info - [3.8.8+] (optional, boolean) Whether to show debug information. If `WP_DEBUG` is false, the debug output will not be displayed. Default: `true`.</li>
-     * </ul>
-     * @param       array       $aTab2      Another in-page tab array.
-     * @param       array       $_and_more  (optional) Add in-page tab arrays as many as necessary to the next parameters.
-     * @param       string      $sPageSlug  (optional) If the passed parameter item is a string, it will be stored as the target page slug so that it will be applied to the next passed tab arrays as the page_slug element.
      * @remark      Accepts variadic parameters; the number of accepted parameters are not limited to three.
      * @remark      In-page tabs are different from page-heading tabs which is automatically added with page titles.  
      * @return      void
@@ -278,6 +241,10 @@ 
         /**
          * Sets a page property.
          * @since       3.6.0
+         * @param string $sPropertyName
+         * @param string $sPropertyKey
+         * @param boolean|string $mValue
+         * @param string $sPageSlug
          * @return      void
          */
         private function _setPageProperty( $sPropertyName, $sPropertyKey, $mValue, $sPageSlug ) {

development/factory/admin_page/AdminPageFramework_Model_Form.php

Spacing +17 added lines, -17 removed lines

@@ -70,14 +70,14 @@ 
      * @since       3.3.0
      * @since       3.3.1       Moved from `AdminPageFramework_Setting_Base`.
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         parent::__construct( $sOptionKey, $sCallerPath, $sCapability, $sTextDomain );
         
         if ( $this->oProp->bIsAdminAjax ) {
             return;
         }
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
 
@@ -127,7 +127,7 @@ 
             'section_title' => null,
             'section_id'    => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
 
@@ -145,7 +145,7 @@ 
                 'help_tab_id'               => $_sRootSectionID,
                 'help_tab_content'          => "<span class='contextual-help-tab-title'>" 
                         . $aFieldset[ 'title' ] 
-                    . "</span> - " . PHP_EOL
+                    . "</span> - ".PHP_EOL
                     . $aFieldset[ 'help' ],
                 'help_tab_sidebar_content'  => $aFieldset[ 'help_aside' ] 
                     ? $aFieldset[ 'help_aside' ] 
@@ -179,7 +179,7 @@ 
         public function _registerHelpPaneItemsOfFormSections( $aSectionsets ) {            
 // @todo Test if help pane item gets displayed        
 
-            foreach( $aSectionsets as $_aSectionset ) {
+            foreach ( $aSectionsets as $_aSectionset ) {
 // @todo check capability and conditions                
                 $_aSectionset = $_aSectionset + array(
                     'help'          => null,
@@ -213,13 +213,13 @@ 
      */
     public function _replyToDetermineSectionsetVisibility( $bVisible, $aSectionset ) {
 
-        if ( ! current_user_can( $aSectionset[ 'capability' ] ) ) {
+        if ( !current_user_can( $aSectionset[ 'capability' ] ) ) {
             return false;
         }
-        if ( ! $aSectionset[ 'if' ] ) { 
+        if ( !$aSectionset[ 'if' ] ) { 
             return false;
         }
-        if ( ! $this->_isSectionOfCurrentPage( $aSectionset ) ) { 
+        if ( !$this->_isSectionOfCurrentPage( $aSectionset ) ) { 
             return false;
         }
         return $bVisible;
@@ -237,15 +237,15 @@ 
         private function _isSectionOfCurrentPage( array $aSectionset ) {
         
             // Make sure the value type is string so that when the page_slug is not set, it won't match.
-            $_sCurrentPageSlug  = ( string ) $this->oProp->getCurrentPageSlug();
+            $_sCurrentPageSlug = ( string ) $this->oProp->getCurrentPageSlug();
             
             // Make sure if it's in the loading page.
-            if ( $aSectionset[ 'page_slug' ] !== $_sCurrentPageSlug  ) { 
+            if ( $aSectionset[ 'page_slug' ] !== $_sCurrentPageSlug ) { 
                 return false; 
             }
 
             // If no tag is specified, the user wants to display the section regardless of the tab.
-            if ( ! $aSectionset[ 'tab_slug' ] ) {
+            if ( !$aSectionset[ 'tab_slug' ] ) {
                 return true;
             }
                 
@@ -261,7 +261,7 @@ 
      */
     public function _replyToDetermineFieldsetVisibility( $bVisible, $aFieldset ) {
         
-        $_sCurrentPageSlug  = $this->oProp->getCurrentPageSlug();
+        $_sCurrentPageSlug = $this->oProp->getCurrentPageSlug();
         
         // If the specified field does not exist, do nothing.
         if ( $aFieldset[ 'page_slug' ] !== $_sCurrentPageSlug ) { 
@@ -291,7 +291,7 @@ 
             array( $_sSectionPath, 'page_slug' ), 
             null 
         );        
-        $aFieldset[ 'tab_slug' ]        = $this->oUtil->getElement( 
+        $aFieldset[ 'tab_slug' ] = $this->oUtil->getElement( 
             $aSectionsets, 
             array( $_sSectionPath, 'tab_slug' ), 
             null 
@@ -302,11 +302,11 @@ 
             $aSectionsets,
             $_sSectionPath
         );
-        $aFieldset[ 'section_title' ]   = $this->oUtil->getElement( 
+        $aFieldset[ 'section_title' ] = $this->oUtil->getElement( 
             $_aSectionset, 
             'title'
         );
-        $aFieldset[ 'capability' ]   = $aFieldset[ 'capability' ]
+        $aFieldset[ 'capability' ] = $aFieldset[ 'capability' ]
             ? $aFieldset[ 'capability' ]
             : $this->_replyToGetCapabilityForForm( 
                 $this->oUtil->getElement( $_aSectionset, 'capability' ),
@@ -355,13 +355,13 @@ 
             
             // Find the capability of the parent section if nested.
             if ( 0 < $aSectionset[ '_nested_depth' ] ) {
-                $_aSectionPath         = $aSectionset[ '_section_path_array' ];
+                $_aSectionPath = $aSectionset[ '_section_path_array' ];
                 array_pop( $_aSectionPath ); // remove the last element
                 $_sParentCapability = $this->oUtil->getElement(
                     $this->oForm->aSectionsets,
                     array_merge( $_aSectionPath, array( 'capability' ) )
                 );
-                if( $_sParentCapability ) {
+                if ( $_sParentCapability ) {
                     return $_sParentCapability;
                 }
             }

Braces +6 added lines, -6 removed lines

@@ -176,7 +176,7 @@ 
          * @return      void
          * @since       DEVVER
          */
-        public function _registerHelpPaneItemsOfFormSections( $aSectionsets ) {            
+        public function _registerHelpPaneItemsOfFormSections( $aSectionsets ) {
 // @todo Test if help pane item gets displayed        
 
             foreach( $aSectionsets as $_aSectionset ) {
@@ -216,10 +216,10 @@ 
         if ( ! current_user_can( $aSectionset[ 'capability' ] ) ) {
             return false;
         }
-        if ( ! $aSectionset[ 'if' ] ) { 
+        if ( ! $aSectionset[ 'if' ] ) {
             return false;
         }
-        if ( ! $this->_isSectionOfCurrentPage( $aSectionset ) ) { 
+        if ( ! $this->_isSectionOfCurrentPage( $aSectionset ) ) {
             return false;
         }
         return $bVisible;
@@ -240,7 +240,7 @@ 
             $_sCurrentPageSlug  = ( string ) $this->oProp->getCurrentPageSlug();
             
             // Make sure if it's in the loading page.
-            if ( $aSectionset[ 'page_slug' ] !== $_sCurrentPageSlug  ) { 
+            if ( $aSectionset[ 'page_slug' ] !== $_sCurrentPageSlug  ) {
                 return false; 
             }
 
@@ -264,7 +264,7 @@ 
         $_sCurrentPageSlug  = $this->oProp->getCurrentPageSlug();
         
         // If the specified field does not exist, do nothing.
-        if ( $aFieldset[ 'page_slug' ] !== $_sCurrentPageSlug ) { 
+        if ( $aFieldset[ 'page_slug' ] !== $_sCurrentPageSlug ) {
             return false; 
         }        
         return parent::_replyToDetermineFieldsetVisibility( $bVisible, $aFieldset );
@@ -277,7 +277,7 @@ 
      */
     public function _replyToFormatFieldsetDefinition( $aFieldset, $aSectionsets ) {
         
-        if ( empty( $aFieldset ) ) { 
+        if ( empty( $aFieldset ) ) {
             return $aFieldset; 
         }
         

development/factory/admin_page/AdminPageFramework_Model_Page.php

Spacing +5 added lines, -5 removed lines

@@ -37,13 +37,13 @@ 
      */         
     public function _replyToFinalizeInPageTabs() {
 
-        if ( ! $this->oProp->isPageAdded() ) { 
+        if ( !$this->oProp->isPageAdded() ) { 
             return; 
         }
 
-        foreach( $this->oProp->aPages as $_sPageSlug => $_aPage ) {
+        foreach ( $this->oProp->aPages as $_sPageSlug => $_aPage ) {
             
-            if ( ! isset( $this->oProp->aInPageTabs[ $_sPageSlug ] ) ) { 
+            if ( !isset( $this->oProp->aInPageTabs[ $_sPageSlug ] ) ) { 
                 continue; 
             }
             
@@ -80,8 +80,8 @@ 
          * @since       3.6.0
          */
         private function _getDefaultInPageTab( $sPageSlug, $aInPageTabs ) {
-            foreach( $aInPageTabs as $_aInPageTab ) {                 
-                if ( ! isset( $_aInPageTab[ 'tab_slug' ] ) ) { 
+            foreach ( $aInPageTabs as $_aInPageTab ) {                 
+                if ( !isset( $_aInPageTab[ 'tab_slug' ] ) ) { 
                     continue; 
                 }                
                 // Regardless of whether it's a hidden tab, it is stored as the default in-page tab.

Braces +4 added lines, -4 removed lines

@@ -37,13 +37,13 @@ 
      */         
     public function _replyToFinalizeInPageTabs() {
 
-        if ( ! $this->oProp->isPageAdded() ) { 
+        if ( ! $this->oProp->isPageAdded() ) {
             return; 
         }
 
         foreach( $this->oProp->aPages as $_sPageSlug => $_aPage ) {
             
-            if ( ! isset( $this->oProp->aInPageTabs[ $_sPageSlug ] ) ) { 
+            if ( ! isset( $this->oProp->aInPageTabs[ $_sPageSlug ] ) ) {
                 continue; 
             }
             
@@ -80,8 +80,8 @@ 
          * @since       3.6.0
          */
         private function _getDefaultInPageTab( $sPageSlug, $aInPageTabs ) {
-            foreach( $aInPageTabs as $_aInPageTab ) {                 
-                if ( ! isset( $_aInPageTab[ 'tab_slug' ] ) ) { 
+            foreach( $aInPageTabs as $_aInPageTab ) {
+                if ( ! isset( $_aInPageTab[ 'tab_slug' ] ) ) {
                     continue; 
                 }                
                 // Regardless of whether it's a hidden tab, it is stored as the default in-page tab.

development/factory/admin_page/_model/AdminPageFramework_CustomSubmitFields.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
     public function __construct( array $aPostElement ) {
             
         $this->aPost    = $aPostElement;
-        $this->sInputID = $this->getInputID( $aPostElement['submit'] ); // the submit element must be set by the field type.
+        $this->sInputID = $this->getInputID( $aPostElement[ 'submit' ] ); // the submit element must be set by the field type.
     
     }
     
@@ -47,10 +47,10 @@ 
      * @since   2.0.0
      * @since   3.4.0   Changed the name from `getElement()`.
      */ 
-    protected function getSubmitValueByType( $aElement, $sInputID, $sElementKey='format' ) {
+    protected function getSubmitValueByType( $aElement, $sInputID, $sElementKey = 'format' ) {
             
         return $this->getElement( 
-            $aElement,  // subject array
+            $aElement, // subject array
             array( $sInputID, $sElementKey ), // dimensional keys
             null    // default
         );
@@ -80,7 +80,7 @@ 
         // Only the pressed element will be stored in the array.
         // The input tag: name="__import[submit][my_section_my_import_field_the_index]" value="Import Button"
         // The array structure:  array( 'my_section_my_import_field_the_index' => 'Import Button' )
-        foreach( $aSubmitElement as $sInputID => $v ) { // $aSubmitElement should have been set in the constructor.
+        foreach ( $aSubmitElement as $sInputID => $v ) { // $aSubmitElement should have been set in the constructor.
             $this->sInputID = $sInputID;
             return $this->sInputID;
         }     

Braces +3 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
      * 
      * @since 2.1.5
      */
-    public function getSiblingValue( $sKey ) {    
+    public function getSiblingValue( $sKey ) {
         return $this->getSubmitValueByType( $this->aPost, $this->sInputID, $sKey );
     }
     
@@ -80,7 +80,8 @@ 
         // Only the pressed element will be stored in the array.
         // The input tag: name="__import[submit][my_section_my_import_field_the_index]" value="Import Button"
         // The array structure:  array( 'my_section_my_import_field_the_index' => 'Import Button' )
-        foreach( $aSubmitElement as $sInputID => $v ) { // $aSubmitElement should have been set in the constructor.
+        foreach( $aSubmitElement as $sInputID => $v ) {
+// $aSubmitElement should have been set in the constructor.
             $this->sInputID = $sInputID;
             return $this->sInputID;
         }     

development/factory/admin_page/_model/AdminPageFramework_ExportOptions.php

Spacing +5 added lines, -5 removed lines

@@ -94,9 +94,9 @@ 
      * @since       2.0.0
      * @since       3.5.4       Added the `$aHeader` parameter. Deprecated the `$sFileName` parameter as it is included in the $aHeader definition.
      */ 
-    public function doExport( $vData, $sFormatType=null, array $aHeader=array() ) {
+    public function doExport( $vData, $sFormatType = null, array $aHeader = array() ) {
 
-        $sFormatType    = isset( $sFormatType ) 
+        $sFormatType = isset( $sFormatType ) 
             ? $sFormatType 
             : $this->sFormatType;
 
@@ -122,9 +122,9 @@ 
          * @since       3.5.4
          * @return      void
          */
-        private function _outputHTTPHeader( array $aHeader, $sKey='' ) {
+        private function _outputHTTPHeader( array $aHeader, $sKey = '' ) {
             
-            foreach( $aHeader as $_sKey => $_asValue ) {
+            foreach ( $aHeader as $_sKey => $_asValue ) {
                 
                 // Nested items. Set the key to overrider array keys.
                 if ( is_array( $_asValue ) ) {
@@ -164,7 +164,7 @@ 
                     return;
                 case 'json': // for json.
                     echo json_encode( ( array ) $vData );
-                    return ;
+                    return;
                 case 'array': // for serialized PHP array.
                 default: // for anything else, 
                     echo serialize( ( array ) $vData );

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

Spacing +6 added lines, -6 removed lines

@@ -36,7 +36,7 @@ 
             : 'add_filter';
             
         $_sFunctionName(
-            $this->sActionHookPrefix . $this->oFactory->oProp->sClassName,
+            $this->sActionHookPrefix.$this->oFactory->oProp->sClassName,
             array( $this, $this->sCallbackName ),
             $this->iHookPriority,
             $this->iCallbackParameters
@@ -56,17 +56,17 @@ 
      * @since   3.6.3       Moved from `AdminPageFramework_Validation`.
      * @return  array       The intact stored options.
      */
-    protected function _confirmSubmitButtonAction( $sPressedInputName, $sSectionID, $sType='reset' ) {
+    protected function _confirmSubmitButtonAction( $sPressedInputName, $sSectionID, $sType = 'reset' ) {
         
-        switch( $sType ) {
+        switch ( $sType ) {
             default:
             case 'reset':
                 $_sFieldErrorMessage = $this->oFactory->oMsg->get( 'reset_options' );
-                $_sTransientKey      =  'apf_rc_' . md5( $sPressedInputName . get_current_user_id() );
+                $_sTransientKey      = 'apf_rc_'.md5( $sPressedInputName.get_current_user_id() );
                 break;
             case 'email':
                 $_sFieldErrorMessage = $this->oFactory->oMsg->get( 'send_email' );
-                $_sTransientKey      =  'apf_ec_' . md5( $sPressedInputName . get_current_user_id() );
+                $_sTransientKey      = 'apf_ec_'.md5( $sPressedInputName.get_current_user_id() );
                 break;                
         }
         
@@ -88,7 +88,7 @@ 
         $this->oFactory->setFieldErrors( $_aErrors );
             
         // Set a flag that the confirmation is displayed
-        $this->setTransient( $_sTransientKey, $sPressedInputName, 60*2 );
+        $this->setTransient( $_sTransientKey, $sPressedInputName, 60 * 2 );
         
         // Set the admin notice
         $this->oFactory->setSettingNotice( $this->oFactory->oMsg->get( 'confirm_perform_task' ), 'error confirmation' );

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

Braces +6 added lines, -4 removed lines

@@ -181,7 +181,8 @@ 
              */
             private function _validateEachField( array $aData, array $aInputsToParse ) {
                 
-                foreach( $aInputsToParse as $_sID => $_aSectionOrFields ) { // $_sID is either a section id or a field id
+                foreach( $aInputsToParse as $_sID => $_aSectionOrFields ) {
+// $_sID is either a section id or a field id
                     
                     // For each section
                     if ( $this->oFactory->oForm->isSection( $_sID ) ) {
@@ -192,7 +193,8 @@ 
                         }                             
                         
                         // Call the validation callback method.
-                        foreach( $_aSectionOrFields as $_sFieldID => $_aFields ) { // For fields
+                        foreach( $_aSectionOrFields as $_sFieldID => $_aFields ) {
+// For fields
                             $aData[ 'aInput' ][ $_sID ][ $_sFieldID ] = $this->_getValidatedData(
                                 "validation_{$this->oFactory->oProp->sClassName}_{$_sID}_{$_sFieldID}", 
                                 $aData[ 'aInput' ][ $_sID ][ $_sFieldID ], 
@@ -276,7 +278,7 @@ 
              */
             private function _validateTabFields( array $aData ) {
                 
-                if ( ! $aData[ 'sTabSlug' ] || ! $aData[ 'sPageSlug' ] ) { 
+                if ( ! $aData[ 'sTabSlug' ] || ! $aData[ 'sPageSlug' ] ) {
                     return $aData; 
                 }
                     
@@ -337,7 +339,7 @@ 
              */
             private function _validatePageFields( array $aData ) {
            
-                if ( ! $aData[ 'sPageSlug' ] ) { 
+                if ( ! $aData[ 'sPageSlug' ] ) {
                     return $aData[ 'aInput' ]; 
                 }
 

Spacing +30 added lines, -30 removed lines

@@ -89,7 +89,7 @@ 
                     $this->oFactory
                 ),               
                 'aStoredTabDataWODynamicElements' => array(),
-                'aEmbeddedDataWODynamicElements'  => array(),   // stores page meta box field options. This will be updated inside the validation methods.
+                'aEmbeddedDataWODynamicElements'  => array(), // stores page meta box field options. This will be updated inside the validation methods.
                 'aSubmitInformation'    => $aSubmitInformation, // 3.5.0+
             );
             
@@ -104,7 +104,7 @@ 
                 call_user_func_array( 
                     array( $this->oFactory, 'validate' ), // triggers __call()
                     array( $_aData[ 'aInput' ], $_aData[ 'aStoredData' ], $this->oFactory, $_aData[ 'aSubmitInformation' ] )
-                ),    // 3.5.3+
+                ), // 3.5.3+
                 $_aData[ 'aStoredData' ],
                 $_aData[ 'aSubmitInformation' ] // 3.5.0+
             );
@@ -115,7 +115,7 @@ 
             
             // If everything fine, return the filtered input data. 
             $this->_bHasFieldErrors = $this->oFactory->hasFieldError();
-            if ( ! $this->_bHasFieldErrors ) {
+            if ( !$this->_bHasFieldErrors ) {
                 return $_aInput;
             }
             
@@ -129,7 +129,7 @@ 
             );            
                         
             // 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 = $_aInput;
             throw $_oException;
 
@@ -152,13 +152,13 @@ 
              */
             private function _getInputByUnset( array $aInputs ) {
                 
-                $_sUnsetKey = '__unset_' . $this->oFactory->oProp->sStructureType;
-                if ( ! isset( $_POST[ $_sUnsetKey ] ) ) {
+                $_sUnsetKey = '__unset_'.$this->oFactory->oProp->sStructureType;
+                if ( !isset( $_POST[ $_sUnsetKey ] ) ) {
                     return $aInputs;
                 }
                 
                 $_aUnsetElements = array_unique( $_POST[ $_sUnsetKey ] );
-                foreach( $_aUnsetElements as $_sFlatInputName ) {
+                foreach ( $_aUnsetElements as $_sFlatInputName ) {
                     $_aDimensionalKeys = explode( '|', $_sFlatInputName );
                     
                     // The first element is the option key; the section or field dimensional keys follow.
@@ -181,22 +181,22 @@ 
              */
             private function _validateEachField( array $aData, array $aInputsToParse ) {
                 
-                foreach( $aInputsToParse as $_sID => $_aSectionOrFields ) { // $_sID is either a section id or a field id
+                foreach ( $aInputsToParse as $_sID => $_aSectionOrFields ) { // $_sID is either a section id or a field id
                     
                     // For each section
                     if ( $this->oFactory->oForm->isSection( $_sID ) ) {
                         
                         // If the parsing item does not belong to the current page, do not call the validation callback method.
-                        if ( ! $this->_isValidSection( $_sID, $aData[ 'sPageSlug' ], $aData[ 'sTabSlug' ] ) ) {
+                        if ( !$this->_isValidSection( $_sID, $aData[ 'sPageSlug' ], $aData[ 'sTabSlug' ] ) ) {
                             continue;
                         }                             
                         
                         // Call the validation callback method.
-                        foreach( $_aSectionOrFields as $_sFieldID => $_aFields ) { // For fields
+                        foreach ( $_aSectionOrFields as $_sFieldID => $_aFields ) { // For fields
                             $aData[ 'aInput' ][ $_sID ][ $_sFieldID ] = $this->_getValidatedData(
                                 "validation_{$this->oFactory->oProp->sClassName}_{$_sID}_{$_sFieldID}", 
                                 $aData[ 'aInput' ][ $_sID ][ $_sFieldID ], 
-                                $this->getElement( $aData, array( 'aStoredData', $_sID, $_sFieldID ), null ),    // isset( $aData[ 'aStoredData' ][ $_sID ][ $_sFieldID ] ) ? $aData[ 'aStoredData' ][ $_sID ][ $_sFieldID ] : null,
+                                $this->getElement( $aData, array( 'aStoredData', $_sID, $_sFieldID ), null ), // isset( $aData[ 'aStoredData' ][ $_sID ][ $_sFieldID ] ) ? $aData[ 'aStoredData' ][ $_sID ][ $_sFieldID ] : null,
                                 $aData[ 'aSubmitInformation' ]    // 3.5.0+
                             );
                         }
@@ -217,7 +217,7 @@ 
                         $aData[ 'aInput' ][ $_sID ] = $this->_getValidatedData(
                             "validation_{$this->oFactory->oProp->sClassName}_{$_sID}", 
                             $_aSectionInput,
-                            $this->getElement( $aData, array( 'aStoredData', $_sID ), null ),    // isset( $aData[ 'aStoredData' ][ $_sID ] ) ? $aData[ 'aStoredData' ][ $_sID ] : null,
+                            $this->getElement( $aData, array( 'aStoredData', $_sID ), null ), // isset( $aData[ 'aStoredData' ][ $_sID ] ) ? $aData[ 'aStoredData' ][ $_sID ] : null,
                             $aData[ 'aSubmitInformation' ]
                         );     
                         
@@ -226,7 +226,7 @@ 
                     }
                                         
                     // Check if the parsing item (the default section) belongs to the current page; if not, do not call the validation callback method.
-                    if ( ! $this->_isValidSection( '_default', $aData[ 'sPageSlug' ], $aData[ 'sTabSlug' ] ) ) {
+                    if ( !$this->_isValidSection( '_default', $aData[ 'sPageSlug' ], $aData[ 'sTabSlug' ] ) ) {
                         continue;
                     }  
                     
@@ -234,7 +234,7 @@ 
                     $aData[ 'aInput' ][ $_sID ] = $this->_getValidatedData(
                         "validation_{$this->oFactory->oProp->sClassName}_{$_sID}",
                         $aData[ 'aInput' ][ $_sID ],
-                        $this->getElement( $aData, array( 'aStoredData', $_sID ), null ),    // isset( $aData[ 'aStoredData' ][ $_sID ] ) ? $aData[ 'aStoredData' ][ $_sID ] : null,
+                        $this->getElement( $aData, array( 'aStoredData', $_sID ), null ), // isset( $aData[ 'aStoredData' ][ $_sID ] ) ? $aData[ 'aStoredData' ][ $_sID ] : null,
                         $aData[ 'aSubmitInformation' ]
                     );
                     
@@ -276,22 +276,22 @@ 
              */
             private function _validateTabFields( array $aData ) {
 
-                if ( ! $aData[ 'sTabSlug' ] || ! $aData[ 'sPageSlug' ] ) { 
+                if ( !$aData[ 'sTabSlug' ] || !$aData[ 'sPageSlug' ] ) { 
                     return $aData; 
                 }
 
-                $aData[ 'aStoredTabData' ]        = $this->oFactory->oForm->getTabOptions( 
+                $aData[ 'aStoredTabData' ] = $this->oFactory->oForm->getTabOptions( 
                     $aData[ 'aStoredData' ], 
                     $aData[ 'sPageSlug' ], 
                     $aData[ 'sTabSlug' ] 
                 ); // respects page meta box fields
-                $aData[ 'aStoredTabData' ]        = $this->addAndApplyFilter(
+                $aData[ 'aStoredTabData' ] = $this->addAndApplyFilter(
                     $this->oFactory, 
                     "validation_saved_options_{$aData[ 'sPageSlug' ]}_{$aData[ 'sTabSlug' ]}", 
                     $aData[ 'aStoredTabData' ], 
                     $this->oFactory 
                 );
-                $_aOtherTabOptions  = $this->oFactory->oForm->getOtherTabOptions( 
+                $_aOtherTabOptions = $this->oFactory->oForm->getOtherTabOptions( 
                     $aData[ 'aStoredData' ], 
                     $aData[ 'sPageSlug' ], 
                     $aData[ 'sTabSlug' ] 
@@ -345,7 +345,7 @@ 
              */
             private function _validatePageFields( array $aData ) {
            
-                if ( ! $aData[ 'sPageSlug' ] ) { 
+                if ( !$aData[ 'sPageSlug' ] ) { 
                     return $aData[ 'aInput' ]; 
                 }
 
@@ -357,7 +357,7 @@ 
                 $_aPageOptionsWODynamicElements = $this->addAndApplyFilter( 
                     $this->oFactory, 
                     "validation_saved_options_without_dynamic_elements_{$aData[ 'sPageSlug' ]}", 
-                    $this->oFactory->oForm->getPageOptions( $aData[ 'aStoredDataWODynamicElements' ], $aData[ 'sPageSlug' ] ),     // united with the in-page tab specific data in order to override the page-specific dynamic elements.
+                    $this->oFactory->oForm->getPageOptions( $aData[ 'aStoredDataWODynamicElements' ], $aData[ 'sPageSlug' ] ), // united with the in-page tab specific data in order to override the page-specific dynamic elements.
                     $this->oFactory 
                 );                
 
@@ -370,15 +370,15 @@ 
                 // Validate the input data.
                 $aData[ 'aInput' ] = $this->_getValidatedData(
                     "validation_{$aData[ 'sPageSlug' ]}", 
-                    $aData[ 'aInput' ],                   // new values
-                    $_aPageOptions,                     // stored page options
+                    $aData[ 'aInput' ], // new values
+                    $_aPageOptions, // stored page options
                     $aData[ 'aSubmitInformation' ]        // submit information 3.5.0+
                 );
 
                 // If it's in a tab-page, drop the elements which belong to the tab so that arrayed-options will not be merged such as multiple select options.
-                $_aPageOptions = $aData[ 'sTabSlug' ] && ! empty( $aData[ 'aStoredTabData' ] ) 
+                $_aPageOptions = $aData[ 'sTabSlug' ] && !empty( $aData[ 'aStoredTabData' ] ) 
                     ? $this->invertCastArrayContents( $_aPageOptions, $aData[ 'aStoredTabData' ] ) 
-                    : ( ! $aData[ 'sTabSlug' ] // if the tab is not specified, do not merge the input array with the page options as the input array already includes the page options. This is for dynamic elements(repeatable sections).
+                    : ( !$aData[ 'sTabSlug' ] // if the tab is not specified, do not merge the input array with the page options as the input array already includes the page options. This is for dynamic elements(repeatable sections).
                         ? array()
                         : $_aPageOptions
                     );    
@@ -433,13 +433,13 @@ 
                  * @param       array       $aStoredData    The stored option.
                  * @param       array       $aSubmitInfo    [3.5.0+] The form submit information such as the field ID of the pressed submit field.
                  */
-                private function _getValidatedData( $sFilterName, $aInputs, $aStoredData, $aSubmitInfo=array() ) {
+                private function _getValidatedData( $sFilterName, $aInputs, $aStoredData, $aSubmitInfo = array() ) {
                     return $this->addAndApplyFilter( 
-                        $this->oFactory,          // caller
-                        $sFilterName,   // hook name
-                        $aInputs,        // 1st argument
-                        $aStoredData,   // 2nd argument
-                        $this->oFactory,          // 3rd argument
+                        $this->oFactory, // caller
+                        $sFilterName, // hook name
+                        $aInputs, // 1st argument
+                        $aStoredData, // 2nd argument
+                        $this->oFactory, // 3rd argument
                         $aSubmitInfo    // 4th argument 3.5.0+
                     );                    
                 }  

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

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
         $_sLinkURL = $this->_getPressedSubmitButtonData( $aSubmits, 'href' );
-        if ( ! $_sLinkURL ) {
+        if ( !$_sLinkURL ) {
             return;
         }        
         $this->goToURL( $_sLinkURL );