michaeluno / admin-page-framework

admin-page-framework-loader.php

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.32';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.32'; // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
     const NAME           = 'Admin Page Framework - Loader'; // the name is not 'Admin Page Framework' because warning messages gets confusing.
-    const SHORTNAME      = 'Admin Page Framework';  // used for a menu title etc.
+    const SHORTNAME      = 'Admin Page Framework'; // used for a menu title etc.
     const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
     const URI            = 'http://admin-page-framework.michaeluno.jp/';
     const AUTHOR         = 'miunosoft (Michael Uno)';
@@ -55,14 +55,14 @@ 
      * @remark      This is also accessed from `uninstall.php` so do not remove.
      * @remark      Do not exceed 8 characters as a transient name allows 45 characters or less ( 40 for site transients ) so that md5 (32 characters) can be added.
      */
-    const TRANSIENT_PREFIX         = 'APFL_';
+    const TRANSIENT_PREFIX = 'APFL_';
 
     /**
      * The hook slug used for the prefix of action and filter hook names.
      *
      * @remark      The ending underscore is not necessary.
      */
-    const HOOK_SLUG                = 'admin_page_framework_loader';
+    const HOOK_SLUG = 'admin_page_framework_loader';
 
     /**
      * The text domain slug and its path.
@@ -116,7 +116,7 @@ 
      */
     static public $aAdminPages = array(
         // key => 'page slug'
-        'about'     => 'apfl_about',        // the welcome page
+        'about'     => 'apfl_about', // the welcome page
         'addon'     => 'apfl_addons',
         'tool'      => 'apfl_tools',
         'help'      => 'apfl_contact',
@@ -154,12 +154,12 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public static function getPluginURL( $sRelativePath='' ) {
+    public static function getPluginURL( $sRelativePath = '' ) {
         if ( isset( self::$_sPluginURLCache ) ) {
-            return self::$_sPluginURLCache . $sRelativePath;
+            return self::$_sPluginURLCache.$sRelativePath;
         }
         self::$_sPluginURLCache = trailingslashit( plugins_url( '', self::$sFilePath ) );
-        return self::$_sPluginURLCache . $sRelativePath;
+        return self::$_sPluginURLCache.$sRelativePath;
     }
         /**
          * @since       3.7.9
@@ -189,13 +189,13 @@ 
      * @since       3.5.0
      * @return      void
      */
-    static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {
+    static public function setAdminNotice( $sMessage, $sClassAttribute = 'error' ) {
+        if ( !is_admin() ) {
             return;
         }
-        self::$_aAdminNotices[] = array(
+        self::$_aAdminNotices[ ] = array(
             'message'           => $sMessage,
-            'class_attribute'   => trim( $sClassAttribute ) . ' notice is-dismissible',
+            'class_attribute'   => trim( $sClassAttribute ).' notice is-dismissible',
         );
         add_action( 'admin_notices', array( __CLASS__, '_replyToSetAdminNotice' ) );
     }
@@ -205,12 +205,12 @@ 
          * @return      void
          */
         static public function _replyToSetAdminNotice() {
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
-                echo "<div class='" . esc_attr( $_aAdminNotice['class_attribute'] ) . " notice is-dismissible'>"
+            foreach ( self::$_aAdminNotices as $_aAdminNotice ) {
+                echo "<div class='".esc_attr( $_aAdminNotice[ 'class_attribute' ] )." notice is-dismissible'>"
                         ."<p>"
                             . sprintf(
-                                '<strong>%1$s</strong>: ' . $_aAdminNotice['message'],
-                                self::NAME . ' ' . self::VERSION
+                                '<strong>%1$s</strong>: '.$_aAdminNotice[ 'message' ],
+                                self::NAME.' '.self::VERSION
                             )
                         . "</p>"
                     . "</div>";
@@ -222,7 +222,7 @@ 
 AdminPageFrameworkLoader_Registry::setUp( __FILE__ );
 
 // Initial checks. - Do no load if accessed directly, not exiting because the 'uninstall.php' and inclusion list generator will load this file.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( !defined( 'ABSPATH' ) ) {
     return;
 }
 if ( defined( 'DOING_UNINSTALL' ) && DOING_UNINSTALL ) {
@@ -234,8 +234,8 @@ 
 
     $_bFrameworkLoaded = class_exists( 'AdminPageFramework_Registry', false );
     if (
-        ! $_bFrameworkLoaded
-        || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
+        !$_bFrameworkLoaded
+        || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
         || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
     ) {
         AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -253,18 +253,18 @@ 
 add_action( 'admin_init', 'AdminPageFrameworkLoader_Warning' );
 
 // Include the library file - the development version will be available if you cloned the GitHub repository.
-$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/development/admin-page-framework.php';
+$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/development/admin-page-framework.php';
 $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
 $_bLoadDevelopmentVersion = $_bDebugMode && file_exists( $_sDevelopmentVersionPath );
 include(
     $_bLoadDevelopmentVersion
         ? $_sDevelopmentVersionPath
-        : AdminPageFrameworkLoader_Registry::$sDirPath . '/library/apf/admin-page-framework.php'
+        : AdminPageFrameworkLoader_Registry::$sDirPath.'/library/apf/admin-page-framework.php'
 );
 
 // Include the framework loader plugin components.
 include( AdminPageFramework_Registry::$aClassFiles[ 'AdminPageFramework_PluginBootstrap' ] );
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
 new AdminPageFrameworkLoader_Bootstrap(
     AdminPageFrameworkLoader_Registry::$sFilePath,
     AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix

development/utility/pointer_tool_tip/AdminPageFramework_PointerToolTip.php

Spacing +11 added lines, -11 removed lines

@@ -111,18 +111,18 @@ 
          */
         private function _setHooks( $aScreenIDs ) {
 
-            foreach( $aScreenIDs as $_sScreenID ) {
-                if ( ! $_sScreenID ) {
+            foreach ( $aScreenIDs as $_sScreenID ) {
+                if ( !$_sScreenID ) {
                     continue;
                 }
                 add_filter(
-                    get_class( $this ) . '-' . $_sScreenID,
+                    get_class( $this ).'-'.$_sScreenID,
                     array( $this, '_replyToSetPointer' )
                 );
 
             }
 
-            if ( ! $this->_hasBeenCalled() ) {
+            if ( !$this->_hasBeenCalled() ) {
                 return;
             }
 
@@ -169,7 +169,7 @@ 
 
         $_aPointers = $this->_getValidPointers( $this->_getPointers() );
 
-        if ( empty( $_aPointers ) || ! is_array( $_aPointers ) ) {
+        if ( empty( $_aPointers ) || !is_array( $_aPointers ) ) {
             return;
         }
 
@@ -189,10 +189,10 @@ 
             $_oScreen   = get_current_screen();
             $_sScreenID = $_oScreen->id;
             if ( in_array( $_sScreenID, $this->aScreenIDs ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_sScreenID, array() );
+                return apply_filters( get_class( $this ).'-'.$_sScreenID, array() );
             }
             if ( isset( $_GET[ 'page' ] ) ) {   // sanitization unnecessary
-                return apply_filters( get_class( $this ) . '-' . $this->getHTTPQueryGET( 'page' ), array() );
+                return apply_filters( get_class( $this ).'-'.$this->getHTTPQueryGET( 'page' ), array() );
             }
             return array();
 
@@ -206,7 +206,7 @@ 
         private function _getValidPointers( $_aPointers ) {
 
             // Get dismissed pointers
-            $_aDismissed      = explode(
+            $_aDismissed = explode(
                 ',',
                 ( string ) get_user_meta(
                     get_current_user_id(),
@@ -234,7 +234,7 @@ 
                 $_aPointer[ 'pointer_id' ] = $_iPointerID;
 
                 // Add the pointer to $_aValidPointers array
-                $_aValidPointers[] =  $_aPointer;
+                $_aValidPointers[ ] = $_aPointer;
 
             }
             return $_aValidPointers;
@@ -310,9 +310,9 @@ 
          * @return      string
          * @internal
          */
-        public function _getInternalScript( $aPointers=array() ) {
+        public function _getInternalScript( $aPointers = array() ) {
 
-            $_aJSArray      = json_encode( $aPointers );
+            $_aJSArray = json_encode( $aPointers );
 
             /**
              * Checks check-boxes in siblings.

Braces +2 added lines, -1 removed lines

@@ -191,7 +191,8 @@
             if ( in_array( $_sScreenID, $this->aScreenIDs ) ) {
                 return apply_filters( get_class( $this ) . '-' . $_sScreenID, array() );
             }
-            if ( isset( $_GET[ 'page' ] ) ) {   // sanitization unnecessary
+            if ( isset( $_GET[ 'page' ] ) ) {
+// sanitization unnecessary
                 return apply_filters( get_class( $this ) . '-' . $this->getHTTPQueryGET( 'page' ), array() );
             }
             return array();

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Model.php

Spacing +9 added lines, -9 removed lines

@@ -82,7 +82,7 @@ 
             if ( $sTaxonomy ) {
                 $aColumns = $this->oUtil->addAndApplyFilter(
                     $this,
-                    $sFilterPrefix . $this->oUtil->getHTTPQueryGET( 'taxonomy', '' ),
+                    $sFilterPrefix.$this->oUtil->getHTTPQueryGET( 'taxonomy', '' ),
                     $aColumns
                 );
             }
@@ -121,7 +121,7 @@ 
      * @param       string|null  $sOptionKey
      * @internal
      */
-    protected function _setOptionArray( $iTermID=null, $sOptionKey=null ) {
+    protected function _setOptionArray( $iTermID = null, $sOptionKey = null ) {
         $this->oForm->aSavedData = $this->_getSavedFormData( $iTermID, $sOptionKey );
     }
         /**
@@ -134,7 +134,7 @@ 
 
             return $this->oUtil->addAndApplyFilter(
                 $this, // the caller factory object
-                'options_' . $this->oProp->sClassName,
+                'options_'.$this->oProp->sClassName,
                 $this->_getSavedTermFormData( $iTermID, $sOptionKey )
                 // @todo maybe pass the term id because the user will not know whihch form data it is
             );
@@ -168,16 +168,16 @@ 
      */
     public function _replyToValidateOptions( $iTermID ) {
 
-        if ( ! $this->_shouldProceedValidation() ) {
+        if ( !$this->_shouldProceedValidation() ) {
             return;
         }
 
         $_aTaxonomyFormData     = $this->_getSavedTaxonomyFormData( $this->oProp->sOptionKey );
         $_aSavedFormData        = $this->_getSavedTermFormData( $iTermID, $this->oProp->sOptionKey );
-        $_aSubmittedFormData    = $this->oForm->getSubmittedData( $this->oForm->getHTTPRequestSanitized( $_POST ) );    // sanitization done
+        $_aSubmittedFormData    = $this->oForm->getSubmittedData( $this->oForm->getHTTPRequestSanitized( $_POST ) ); // sanitization done
         $_aSubmittedFormData    = $this->oUtil->addAndApplyFilters(
             $this,
-            'validation_' . $this->oProp->sClassName,
+            'validation_'.$this->oProp->sClassName,
             call_user_func_array(
                 array( $this, 'validate' ), // triggers __call()
                 array( $_aSubmittedFormData, $_aSavedFormData, $this )
@@ -188,7 +188,7 @@ 
 
         // @todo Examine whether it is appropriate to merge recursivly
         // as some fields will have a problem such as select with multiple options.
-        $_aTaxonomyFormData[ $iTermID ]  = $this->oUtil->uniteArrays(
+        $_aTaxonomyFormData[ $iTermID ] = $this->oUtil->uniteArrays(
             $_aSubmittedFormData,
             $_aSavedFormData
         );
@@ -210,10 +210,10 @@ 
          */
         protected function _shouldProceedValidation() {
 
-            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) {  // sanitization unnecessary
+            if ( !isset( $_POST[ $this->oProp->sClassHash ] ) ) {  // sanitization unnecessary
                 return false;
             }
-            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {  // sanitization unnecessary
+            if ( !wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {  // sanitization unnecessary
                 return false;
             }
             return true;

Braces +4 added lines, -2 removed lines

@@ -210,10 +210,12 @@
          */
         protected function _shouldProceedValidation() {
 
-            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) {  // sanitization unnecessary
+            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) {
+// sanitization unnecessary
                 return false;
             }
-            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {  // sanitization unnecessary
+            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
+// sanitization unnecessary
                 return false;
             }
             return true;

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_View.php

Spacing +6 added lines, -6 removed lines

@@ -47,7 +47,7 @@ 
             '',
             "[{$_sKey}]"
         );
-        return $_aField['field_id'] . $_sKey;
+        return $_aField[ 'field_id' ].$_sKey;
 
     }
     /**
@@ -65,7 +65,7 @@ 
             '',
             "|{$_sKey}"
         );
-        return "{$_aField['field_id']}{$_sKey}";
+        return "{$_aField[ 'field_id' ]}{$_sKey}";
     }
 
    /**
@@ -115,7 +115,7 @@ 
             $_aOutput = array();
 
             // Set nonce.
-            $_aOutput[] = wp_nonce_field(
+            $_aOutput[ ] = wp_nonce_field(
                 $this->oProp->sClassHash,
                 $this->oProp->sClassHash,
                 true,
@@ -126,17 +126,17 @@ 
             $this->_setOptionArray( $iTermID, $this->oProp->sOptionKey );
 
             // Get the field outputs
-            $_aOutput[] = $this->oForm->get( $bRenderTableRow );
+            $_aOutput[ ] = $this->oForm->get( $bRenderTableRow );
 
             // Filter the output
             $_sOutput = $this->oUtil->addAndApplyFilters(
                 $this,
-                'content_' . $this->oProp->sClassName,
+                'content_'.$this->oProp->sClassName,
                 $this->content( implode( PHP_EOL, $_aOutput ) )
             );
 
             // Do action
-            $this->oUtil->addAndDoActions( $this, 'do_' . $this->oProp->sClassName, $this );
+            $this->oUtil->addAndDoActions( $this, 'do_'.$this->oProp->sClassName, $this );
 
             return $_sOutput;
 

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Router.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
 
         parent::__construct( $oProp );
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
 
@@ -37,7 +37,7 @@ 
         );
 
         add_action(
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
             array( $this, '_replyToSetUpHooks' )
         );
 
@@ -54,7 +54,7 @@ 
      */
     protected function _isInThePage() {
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
@@ -62,11 +62,11 @@ 
             return $this->_isValidAjaxReferrer();
         }
 
-        if ( ! in_array( $this->oProp->sPageNow, array( 'edit-tags.php', 'term.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'edit-tags.php', 'term.php' ) ) ) {
             return false;
         }
 
-        if ( isset( $_GET[ 'taxonomy' ] ) && ! in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'taxonomy' ] ) && !in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {    // sanitization unnecessary
             return false;
         }
 
@@ -86,7 +86,7 @@ 
         parse_str( $_aReferrer[ 'query' ], $_aQuery );
 
         $_sBaseName = basename( $_aReferrer[ 'path' ] );
-        if ( ! in_array( $_sBaseName, array( 'edit-tags.php', 'term.php' ) ) ) {
+        if ( !in_array( $_sBaseName, array( 'edit-tags.php', 'term.php' ) ) ) {
             return false;
         }
         $_sTaxonomy = $this->oUtil->getElement( $this->oProp->aQuery, array( 'taxonomy' ), '' );
@@ -103,7 +103,7 @@ 
      */
     public function _replyToSetUpHooks( $oFactory ) {
 
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {
+        foreach ( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {
 
             // Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not.
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );

Braces +2 added lines, -1 removed lines

@@ -66,7 +66,8 @@
             return false;
         }
 
-        if ( isset( $_GET[ 'taxonomy' ] ) && ! in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'taxonomy' ] ) && ! in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {
+// sanitization unnecessary
             return false;
         }
 

development/factory/page_meta_box/_model/AdminPageFramework_Property_page_meta_box.php

Indentation +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
      * @since       3.5.3
      * @return      string      The found page slug. An empty string if not found.
      * @remark      Do not return `null` when not found as some framework methods check the retuened value with `isset()` and if null is given, `isset()` yields `false` while it does `true` for an emtpy string ('').
-    */
+     */
     public function getCurrentPageSlug() {
         return $this->getHTTPQueryGET( 'page', '' );
     }

Spacing +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
     /**
      * Sets up hooks and properties.
      */
-    public function __construct( $oCaller, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='page_meta_box' ) {
+    public function __construct( $oCaller, $sClassName, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework', $sStructureType = 'page_meta_box' ) {
 
         // Let them overload.
         unset(
@@ -105,7 +105,7 @@ 
     public function _getScreenIDOfPage( $sPageSlug ) {
         $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug );
         return $_oAdminPage
-            ? $_oAdminPage->oProp->aPages[ $sPageSlug ][ '_page_hook' ] . ( is_network_admin() ? '-network' : '' )
+            ? $_oAdminPage->oProp->aPages[ $sPageSlug ][ '_page_hook' ].( is_network_admin() ? '-network' : '' )
             : '';
     }
 
@@ -115,7 +115,7 @@ 
      * @since       3.0.0
      * @return      boolean     Returns true if it is of framework's added page; otherwise, false.
      */
-    public function isPageAdded( $sPageSlug='' ) {
+    public function isPageAdded( $sPageSlug = '' ) {
         $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug );
         return $_oAdminPage
             ? $_oAdminPage->oProp->isPageAdded( $sPageSlug )
@@ -132,7 +132,7 @@ 
     public function isCurrentTab( $sTabSlug ) {
 
         $_sCurrentPageSlug = $this->getHTTPQueryGET( 'page' );
-        if ( ! $_sCurrentPageSlug ) {
+        if ( !$_sCurrentPageSlug ) {
             return false;
         }
         $_sCurrentTabSlug = $this->getHTTPQueryGET( 'tab', $this->getDefaultInPageTab( $_sCurrentPageSlug ) );
@@ -179,7 +179,7 @@ 
      */
     public function getDefaultInPageTab( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return '';
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -194,7 +194,7 @@ 
      */
     public function getOptionKey( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return '';
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -221,7 +221,7 @@ 
             $GLOBALS,
             array( 'aAdminPageFramework', 'aPageClasses' )
         );
-        foreach( $_aPageClasses as $_oAdminPage ) {
+        foreach ( $_aPageClasses as $_oAdminPage ) {
             if ( $_oAdminPage->oProp->isPageAdded( $sPageSlug ) ) {
                 return $_oAdminPage;
             }

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Model.php

Spacing +22 added lines, -22 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)
             );
         }
@@ -165,19 +165,19 @@ 
      */
     public function _replyToValidateOptions( $aNewPageOptions, $aOldPageOptions, $oAdminPage, $aSubmitInfo ) {
 
-        $_aNewMetaBoxInputs      = $this->oForm->getSubmittedData( $this->oForm->getHTTPRequestSanitized( $_POST ) );   // sanitization done
+        $_aNewMetaBoxInputs      = $this->oForm->getSubmittedData( $this->oForm->getHTTPRequestSanitized( $_POST ) ); // sanitization done
         $_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_Model.php

Spacing +10 added lines, -10 removed lines

@@ -37,7 +37,7 @@ 
 
         if ( $this->oProp->bIsAdmin ) {
 
-            add_action( 'load_' . $this->oProp->sPostType, array( $this, '_replyToSetUpHooksForModel' ) );
+            add_action( 'load_'.$this->oProp->sPostType, array( $this, '_replyToSetUpHooksForModel' ) );
 
             if ( $this->oProp->sCallerPath ) {
                 new AdminPageFramework_PostType_Model__FlushRewriteRules( $this );
@@ -64,11 +64,11 @@ 
 
         // Properties - sets translatable labels.
         $this->oProp->aColumnHeaders = array(
-            'cb'        => '<input type="checkbox" />',     // Checkbox for bulk actions.
-            'title'     => $this->oMsg->get( 'title' ),     // Post title. Includes "edit", "quick edit", "trash" and "view" links. If $mode (set from `$_REQUEST`['mode']) is 'excerpt', a post excerpt is included between the title and links.
-            'author'    => $this->oMsg->get( 'author' ),    // Post author.
+            'cb'        => '<input type="checkbox" />', // Checkbox for bulk actions.
+            'title'     => $this->oMsg->get( 'title' ), // Post title. Includes "edit", "quick edit", "trash" and "view" links. If $mode (set from `$_REQUEST`['mode']) is 'excerpt', a post excerpt is included between the title and links.
+            'author'    => $this->oMsg->get( 'author' ), // Post author.
             'comments'  => '<div class="comment-grey-bubble"></div>', // Number of pending comments.
-            'date'      => $this->oMsg->get( 'date' ),      // The date and publish status of the post.
+            'date'      => $this->oMsg->get( 'date' ), // The date and publish status of the post.
         );
 
     }
@@ -130,7 +130,7 @@ 
         echo $this->oUtil->addAndApplyFilter(
             $this,
             "cell_{$this->oProp->sPostType}_{$sColumnKey}",
-            '',  // value to be filtered - cell output
+            '', // value to be filtered - cell output
             $iPostID
         );
     }
@@ -179,7 +179,7 @@ 
      * @internal
      */
     public function _replyToRegisterTaxonomies() {
-        foreach( $this->oProp->aTaxonomies as $_sTaxonomySlug => $_aArguments ) {
+        foreach ( $this->oProp->aTaxonomies as $_sTaxonomySlug => $_aArguments ) {
             $this->_registerTaxonomy(
                 $_sTaxonomySlug,
                 $this->oUtil->getAsArray( $this->oProp->aTaxonomyObjectTypes[ $_sTaxonomySlug ] ), // object types
@@ -195,8 +195,8 @@ 
      */
     public function _registerTaxonomy( $sTaxonomySlug, array $aObjectTypes, array $aArguments ) {
 
-        if ( ! in_array( $this->oProp->sPostType, $aObjectTypes ) ) {
-            $aObjectTypes[] = $this->oProp->sPostType;
+        if ( !in_array( $this->oProp->sPostType, $aObjectTypes ) ) {
+            $aObjectTypes[ ] = $this->oProp->sPostType;
         }
         register_taxonomy(
             $sTaxonomySlug,
@@ -234,7 +234,7 @@ 
      */
     public function _replyToRemoveTexonomySubmenuPages() {
 
-        foreach( $this->oProp->aTaxonomyRemoveSubmenuPages as $sSubmenuPageSlug => $sTopLevelPageSlug ) {
+        foreach ( $this->oProp->aTaxonomyRemoveSubmenuPages as $sSubmenuPageSlug => $sTopLevelPageSlug ) {
 
             remove_submenu_page( $sTopLevelPageSlug, $sSubmenuPageSlug );
 

development/factory/post_type/AdminPageFramework_PostType_View.php

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' ] )   // sanitization unnecessary
+            !( isset( $_GET[ 'post_type' ] ) && post_type_exists( $_GET[ 'post_type' ] )   // sanitization unnecessary
             && strtolower( $_GET[ 'post_type' ] ) == $this->oProp->sPostType )   // sanitization unnecessary
         ) {
             return;
@@ -199,7 +199,7 @@ 
 
         foreach ( get_object_taxonomies( $GLOBALS[ 'typenow' ] ) as $_sTaxonomySulg ) {
 
-            if ( ! in_array( $_sTaxonomySulg, $this->oProp->aTaxonomyTableFilters ) ) {
+            if ( !in_array( $_sTaxonomySulg, $this->oProp->aTaxonomyTableFilters ) ) {
                 continue;
             }
 
@@ -213,7 +213,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',
@@ -233,24 +233,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;
             }
 
@@ -289,7 +289,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>";
@@ -345,10 +345,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;