michaeluno / admin-page-framework

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

Braces +1 added lines, -1 removed lines

@@ -400,7 +400,7 @@
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
                  * @return      array       removed menu item.
                  */
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
  
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex(
                         $nSubMenuPageIndex,

Doc Comments +5 added lines, -1 removed lines

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

Spacing +51 added lines, -51 removed lines

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

development/factory/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/format/AdminPageFramework_Format_SubMenuLink.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
         protected function _getFormattedSubMenuLinkArray( array $aSubMenuLink ) {
             
             // If the set URL is not valid, return.
-            if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) {
+            if ( !filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) {
                 return array();
             }
             

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;
                     }
@@ -132,10 +132,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.
                     }
                     

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Model.php

Braces +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
      */
     public function _replyToRegisterMetaBoxes( $sPageHook='' ) {
         foreach( $this->oProp->aPageSlugs as $_sKey => $_asPage ) {
-            if ( is_string( $_asPage ) )  {
+            if ( is_string( $_asPage ) ) {
                 $this->_registerMetaBox( $_asPage );
                 continue;
             }

Indentation +1 added lines, -1 removed lines

@@ -256,7 +256,7 @@
 
     }
 
-       /**
+        /**
          * Extracts meta box form fields options array from the given options array of an admin page.
          *
          * @since       3.5.6

Spacing +21 added lines, -21 removed lines

@@ -45,11 +45,11 @@ 
     public function _replyToSetUpValidationHooks( $oScreen ) {
 
         // Validation hooks
-        foreach( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
+        foreach ( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
 
             if ( is_scalar( $_asTabArrayOrPageSlug ) ) {
                 $_sPageSlug = $_asTabArrayOrPageSlug;
-                add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 );  // 3.4.1+
+                add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 ); // 3.4.1+
                 add_filter( "validation_{$_sPageSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
                 add_filter( "options_update_status_{$_sPageSlug}", array( $this, '_replyToModifyOptionsUpdateStatus' ) );
                 continue;
@@ -58,7 +58,7 @@ 
             // At this point, the array key is the page slug. It means the user specified the tab(s).
             $_sPageSlug = $_sIndexOrPageSlug;
             $_aTabs     = $_asTabArrayOrPageSlug;
-            foreach( $_aTabs as $_sTabSlug ) {
+            foreach ( $_aTabs as $_sTabSlug ) {
                 add_filter( "validation_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
                 add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 ); // 3.4.1+
                 add_filter( "options_update_status_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToModifyOptionsUpdateStatus' ) );
@@ -78,9 +78,9 @@ 
      * @return      void
      * @callback    action      add_meta_boxes
      */
-    public function _replyToRegisterMetaBoxes( $sPageHook='' ) {
-        foreach( $this->oProp->aPageSlugs as $_sKey => $_asPage ) {
-            if ( is_string( $_asPage ) )  {
+    public function _replyToRegisterMetaBoxes( $sPageHook = '' ) {
+        foreach ( $this->oProp->aPageSlugs as $_sKey => $_asPage ) {
+            if ( is_string( $_asPage ) ) {
                 $this->_registerMetaBox( $_asPage );
                 continue;
             }
@@ -96,8 +96,8 @@ 
          * @return      void
          */
         private function _registerMetaBoxes( $sPageSlug, $asPage ) {
-            foreach( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
-                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) {
+            foreach ( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
+                if ( !$this->oProp->isCurrentTab( $_sTabSlug ) ) {
                     continue;
                 }
                 $this->_registerMetaBox( $sPageSlug );
@@ -114,12 +114,12 @@ 
          */
         private function _registerMetaBox( $sPageSlug ) {
             add_meta_box(
-                $this->oProp->sMetaBoxID,                       // id
-                $this->oProp->sTitle,                           // title
+                $this->oProp->sMetaBoxID, // id
+                $this->oProp->sTitle, // title
                 array( $this, '_replyToPrintMetaBoxContents' ), // callback
                 $this->oProp->_getScreenIDOfPage( $sPageSlug ), // screen ID
-                $this->oProp->sContext,                         // context
-                $this->oProp->sPriority,                        // priority
+                $this->oProp->sContext, // context
+                $this->oProp->sPriority, // priority
                 null                                            // argument (deprecated)
             );
         }
@@ -167,17 +167,17 @@ 
 
         $_aNewMetaBoxInputs      = $this->oForm->getSubmittedData( $_POST );
         $_aOldMetaBoxInputs      = $this->oUtil->castArrayContents(
-            $this->oForm->getDataStructureFromAddedFieldsets(),   // model
+            $this->oForm->getDataStructureFromAddedFieldsets(), // model
             $aOldPageOptions        // data source
         );
 
         // Apply filters - third party scripts will have access to the input.
         $_aNewMetaBoxInputsRaw   = $_aNewMetaBoxInputs; // copy one for validation errors.
         $_aNewMetaBoxInputs      = call_user_func_array(
-            array( $this, 'validate' ),     // triggers __call()
+            array( $this, 'validate' ), // triggers __call()
             array( $_aNewMetaBoxInputs, $_aOldMetaBoxInputs, $this, $aSubmitInfo )
         ); // 3.5.3+
-        $_aNewMetaBoxInputs      = $this->oUtil->addAndApplyFilters(
+        $_aNewMetaBoxInputs = $this->oUtil->addAndApplyFilters(
             $this,
             "validation_{$this->oProp->sClassName}",
             $_aNewMetaBoxInputs,
@@ -215,7 +215,7 @@ 
      */
     public function _replyToModifyOptionsUpdateStatus( $aStatus ) {
 
-        if ( ! $this->hasFieldError() ) {
+        if ( !$this->hasFieldError() ) {
             return $aStatus;
         }
         return array(
@@ -245,12 +245,12 @@ 
 
         $_aPageOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $_aPageOptions
         );
 
         return $this->oUtil->castArrayContents(
-            $this->oForm->getDataStructureFromAddedFieldsets(),   // model
+            $this->oForm->getDataStructureFromAddedFieldsets(), // model
             $_aPageOptions        // data source
         );
 
@@ -267,9 +267,9 @@ 
         private function _getPageMetaBoxOptionsFromPageOptions( array $aPageOptions, array $aFieldsets ) {
 
             $_aOptions = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
-                if ( '_default' === $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aField ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+                if ( '_default' === $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aField ) {
                         if ( array_key_exists( $_aField[ 'field_id' ], $aPageOptions ) ) {
                             $_aOptions[ $_aField[ 'field_id' ] ] = $aPageOptions[ $_aField[ 'field_id' ] ];
                         }

development/factory/post_type/AdminPageFramework_PostType_View.php

Braces +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
      */
     public function _replyToModifyActionLinks( $aActionLinks, $oPost ) {
         
-        if ( $oPost->post_type !== $this->oProp->sPostType ){
+        if ( $oPost->post_type !== $this->oProp->sPostType ) {
             return $aActionLinks;
         }
 

Spacing +16 added lines, -16 removed lines

@@ -31,7 +31,7 @@ 
 
         if ( $this->oProp->bIsAdmin ) {
 
-            add_action( 'load_' . $this->oProp->sPostType, array( $this, '_replyToSetUpHooksForView' ) );
+            add_action( 'load_'.$this->oProp->sPostType, array( $this, '_replyToSetUpHooksForView' ) );
 
             // 3.5.10+
             add_action( 'admin_menu', array( $this, '_replyToRemoveAddNewSidebarMenu' ) );
@@ -105,19 +105,19 @@ 
             private function _removeAddNewSidebarSubMenu( $sMenuKey, $sPostTypeSlug ) {
 
                 // Remove the default post type menu item.
-                if ( ! isset( $GLOBALS[ 'submenu' ][ $sMenuKey ] ) ) {
+                if ( !isset( $GLOBALS[ 'submenu' ][ $sMenuKey ] ) ) {
                     // logged-in users of an insufficient access level don't have the menu to be registered.
                     return;
                 }
 
                 foreach ( $GLOBALS[ 'submenu' ][ $sMenuKey ] as $_iIndex => $_aSubMenu ) {
 
-                    if ( ! isset( $_aSubMenu[ 2 ] ) ) {
+                    if ( !isset( $_aSubMenu[ 2 ] ) ) {
                         continue;
                     }
 
                     // Remove the default Add New entry.
-                    if ( 'post-new.php?post_type=' . $sPostTypeSlug === $_aSubMenu[ 2 ] ) {
+                    if ( 'post-new.php?post_type='.$sPostTypeSlug === $_aSubMenu[ 2 ] ) {
                         unset( $GLOBALS[ 'submenu' ][ $sMenuKey ][ $_iIndex ] );
                         break;
                     }
@@ -134,7 +134,7 @@ 
      */
     public function _replyToModifyActionLinks( $aActionLinks, $oPost ) {
 
-        if ( $oPost->post_type !== $this->oProp->sPostType ){
+        if ( $oPost->post_type !== $this->oProp->sPostType ) {
             return $aActionLinks;
         }
 
@@ -156,12 +156,12 @@ 
      */
     public function _replyToAddAuthorTableFilter() {
 
-        if ( ! $this->oProp->bEnableAuthorTableFileter ) {
+        if ( !$this->oProp->bEnableAuthorTableFileter ) {
             return;
         }
 
         if (
-            ! ( isset( $_GET[ 'post_type' ] ) && post_type_exists( $_GET[ 'post_type' ] )
+            !( isset( $_GET[ 'post_type' ] ) && post_type_exists( $_GET[ 'post_type' ] )
             && in_array( strtolower( $_GET[ 'post_type' ] ), array( $this->oProp->sPostType ) ) )
         ) {
             return;
@@ -201,7 +201,7 @@ 
 
         foreach ( get_object_taxonomies( $GLOBALS[ 'typenow' ] ) as $_sTaxonomySulg ) {
 
-            if ( ! in_array( $_sTaxonomySulg, $this->oProp->aTaxonomyTableFilters ) ) {
+            if ( !in_array( $_sTaxonomySulg, $this->oProp->aTaxonomyTableFilters ) ) {
                 continue;
             }
 
@@ -215,7 +215,7 @@ 
             // Echo the drop down list based on the passed array argument.
             wp_dropdown_categories(
                 array(
-                    'show_option_all'   => $this->oMsg->get( 'show_all' ) . ' ' . $_oTaxonomy->label,
+                    'show_option_all'   => $this->oMsg->get( 'show_all' ).' '.$_oTaxonomy->label,
                     'taxonomy'          => $_sTaxonomySulg,
                     'name'              => $_oTaxonomy->name,
                     'orderby'           => 'name',
@@ -235,24 +235,24 @@ 
      *
      * @internal
      */
-    public function _replyToGetTableFilterQueryForTaxonomies( $oQuery=null ) {
+    public function _replyToGetTableFilterQueryForTaxonomies( $oQuery = null ) {
 
         if ( 'edit.php' != $this->oProp->sPageNow ) {
             return $oQuery;
         }
 
-        if ( ! isset( $GLOBALS[ 'typenow' ] ) ) {
+        if ( !isset( $GLOBALS[ 'typenow' ] ) ) {
             return $oQuery;
         }
 
         foreach ( get_object_taxonomies( $GLOBALS[ 'typenow' ] ) as $sTaxonomySlug ) {
 
-            if ( ! in_array( $sTaxonomySlug, $this->oProp->aTaxonomyTableFilters ) ) {
+            if ( !in_array( $sTaxonomySlug, $this->oProp->aTaxonomyTableFilters ) ) {
                 continue;
             }
 
             $sVar = &$oQuery->query_vars[ $sTaxonomySlug ];
-            if ( ! isset( $sVar ) ) {
+            if ( !isset( $sVar ) ) {
                 continue;
             }
 
@@ -291,7 +291,7 @@ 
         $_sStyle = trim( $_sStyle );
 
         // Print out the filtered styles.
-        if ( ! empty( $_sStyle ) ) {
+        if ( !empty( $_sStyle ) ) {
             echo "<style type='text/css' id='admin-page-framework-style-post-type'>"
                     . $this->oProp->sStyle
                 . "</style>";
@@ -347,10 +347,10 @@ 
      */
     public function _replyToFilterPostTypeContent( $sContent ) {
 
-        if ( ! is_singular() ) {
+        if ( !is_singular() ) {
             return $sContent;
         }
-        if ( ! is_main_query() ) {
+        if ( !is_main_query() ) {
             return $sContent;
         }
         global $post;

include/class/admin/admin-page-framework/help/AdminPageFrameworkLoader_AdminPage_Help_Example.php

Braces +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         /**
          * @return      string
          */
-        private function getReadMeContents()  {
+        private function getReadMeContents() {
             return $this->_getReadmeContents(
                 AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/text/examples.txt',
                 '',

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
 
         $_aItems     = $this->getContentsByHeader( $this->getReadmeContents(), 3 );
         $_iLastIndex = count( $_aItems ) - 1;
-        foreach( $_aItems as $_iIndex => $_aContent ) {
+        foreach ( $_aItems as $_iIndex => $_aContent ) {
 
             $_oParser   = new AdminPageFramework_WPReadmeParser;
             $_oParser->setText( $_aContent[ 1 ] );
@@ -31,7 +31,7 @@ 
             $oAdminPage->addSettingSections(
                 $this->sPageSlug, // the target page slug
                 array(
-                    'section_id'        => 'examples_' . $_iIndex,
+                    'section_id'        => 'examples_'.$_iIndex,
                     'title'             => $_aContent[ 0 ],
                     'collapsible'       => array(
                         'toggle_all_button' => $_iLastIndex === $_iIndex
@@ -52,9 +52,9 @@ 
         /**
          * @return      string
          */
-        private function getReadMeContents()  {
+        private function getReadMeContents() {
             return $this->_getReadmeContents(
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/text/examples.txt',
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/text/examples.txt',
                 '',
                 array( 'Examples' )
             );

include/class/admin/admin-page-framework/help/AdminPageFrameworkLoader_AdminPage_Help_Tip.php

Braces +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         /**
          * @return      string
          */
-        private function getReadMeContents()  {
+        private function getReadMeContents() {
             return $this->_getReadmeContents(
                 AdminPageFrameworkLoader_Registry::$sDirPath . '/readme.txt', // source path
                 '', // TOC title

Spacing +4 added lines, -4 removed lines

@@ -25,14 +25,14 @@ 
 
         $_aItems     = $this->getContentsByHeader( $this->getReadmeContents(), 4 );
         $_iLastIndex = count( $_aItems ) - 1;
-        foreach( $_aItems as $_iIndex => $_aContent ) {
+        foreach ( $_aItems as $_iIndex => $_aContent ) {
 
             $_oParser   = new AdminPageFramework_WPReadmeParser( $_aContent[ 1 ] );
             $_sContent  = $_oParser->get();
             $oAdminPage->addSettingSections(
                 $this->sPageSlug, // the target page slug
                 array(
-                    'section_id'        => 'tips_' . $_iIndex,
+                    'section_id'        => 'tips_'.$_iIndex,
                     'title'             => $_aContent[ 0 ],
                     'collapsible'       => array(
                         'toggle_all_button' => $_iLastIndex === $_iIndex
@@ -53,9 +53,9 @@ 
         /**
          * @return      string
          */
-        private function getReadMeContents()  {
+        private function getReadMeContents() {
             return $this->_getReadmeContents(
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/readme.txt', // source path
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/readme.txt', // source path
                 '', // TOC title
                 array( 'Other Notes' )  // sections
             );

include/class/utility/AdminPageFrameworkLoader_Utility.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             ),
             2 => __( 'Header already sent.', 'admin-page-framework-loader' ),
         );
-        if ( ! class_exists( 'AdminPageFramework_AdminNotice' ) ) {
+        if ( !class_exists( 'AdminPageFramework_AdminNotice' ) ) {
             return;
         }
         new AdminPageFramework_AdminNotice(