michaeluno / admin-page-framework

development/factory/admin_page/_model/format/AdminPageFramework_Format_PageResource_Style.php

Spacing +6 added lines, -6 removed lines

@@ -24,11 +24,11 @@ 
      * @static
      */     
     static public $aStructure = array(    
-        'src'           => null,    // (required, string) the source url or path
-        'handle_id'     => null,    // (optional, string) The handle ID of the stylesheet
-        'dependencies'  => null,    // (optional, array) The dependency array.
-        'version'       => null,    // (optional, string) The stylesheet version number.
-        'media'         => null,    // (optional, string) the description of the field which is inserted into the after the input field tag.
+        'src'           => null, // (required, string) the source url or path
+        'handle_id'     => null, // (optional, string) The handle ID of the stylesheet
+        'dependencies'  => null, // (optional, array) The dependency array.
+        'version'       => null, // (optional, string) The stylesheet version number.
+        'media'         => null, // (optional, string) the description of the field which is inserted into the after the input field tag.
         // 'attributes'    => null,    // (optional, array) [3.3.0+] attributes array. `array( 'data-id' => '...' )`
     );        
     
@@ -41,7 +41,7 @@ 
         $_aParameters = func_get_args() + array( 
             $this->asSubject, 
         );
-        $this->asSubject             = $_aParameters[ 0 ];
+        $this->asSubject = $_aParameters[ 0 ];
     }
     
     /**

development/factory/admin_page/_model/format/AdminPageFramework_Format_SubMenuItem.php

Indentation -1 removed lines

@@ -19,7 +19,6 @@
     
     /**
      * Represents the structure of the sub-field definition array.
-  
      */
     static public $aStructure = array(
     );

development/factory/admin_page/_model/format/AdminPageFramework_Format_SubMenuPage.php

Spacing +7 added lines, -7 removed lines

@@ -32,8 +32,8 @@ 
         'page_slug'                 => null, // (required)
         'type'                      => 'page', // this is used to compare with the link type.
         'title'                     => null, 
-        'page_title'                => null,    // (optional) 3.3.0+ When the page title is different from the above 'title' argument, set this.
-        'menu_title'                => null,    // (optional) 3.3.0+ When the menu title is different from the above 'title' argument, set this.
+        'page_title'                => null, // (optional) 3.3.0+ When the page title is different from the above 'title' argument, set this.
+        'menu_title'                => null, // (optional) 3.3.0+ When the menu title is different from the above 'title' argument, set this.
         'screen_icon'               => null, // this will become either href_icon_32x32 or screen_icon_id
         'capability'                => null, 
         'order'                     => null,
@@ -113,12 +113,12 @@ 
             
             $aSubMenuPage = $aSubMenuPage 
                 + array(
-                    'show_page_title'           => $this->oFactory->oProp->bShowPageTitle,       // boolean
+                    'show_page_title'           => $this->oFactory->oProp->bShowPageTitle, // boolean
                     'show_page_heading_tabs'    => $this->oFactory->oProp->bShowPageHeadingTabs, // boolean
-                    'show_in_page_tabs'         => $this->oFactory->oProp->bShowInPageTabs,      // boolean
-                    'in_page_tab_tag'           => $this->oFactory->oProp->sInPageTabTag,        // string
-                    'page_heading_tab_tag'      => $this->oFactory->oProp->sPageHeadingTabTag,   // string
-                    'capability'                => $this->oFactory->oProp->sCapability,  // 3.6.0+
+                    'show_in_page_tabs'         => $this->oFactory->oProp->bShowInPageTabs, // boolean
+                    'in_page_tab_tag'           => $this->oFactory->oProp->sInPageTabTag, // string
+                    'page_heading_tab_tag'      => $this->oFactory->oProp->sPageHeadingTabTag, // string
+                    'capability'                => $this->oFactory->oProp->sCapability, // 3.6.0+
                 )       
                 + self::$aStructure;
 

development/factory/admin_page/_view/AdminPageFramework_View__PageMataBoxRenderer.php

Indentation -1 removed lines

@@ -20,7 +20,6 @@
            
     /**
      * Renders a registered meta box.
- 
      * @return      void
      * @param       string $sContext `side`, `normal`, or `advanced`.
      * @since       3.0.0

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     public function render( $sContext ) {
                
         // If nothing is registered do not render even the container.
-        if ( ! $this->doesMetaBoxExist() ) {
+        if ( !$this->doesMetaBoxExist() ) {
             return;
         }
         

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer__InPageTabs.php

Spacing +4 added lines, -4 removed lines

@@ -73,7 +73,7 @@ 
             $_sTag              = $this->_getInPageTabTag( $sTag, $_aPage );
              
             // If the in-page tabs' visibility is set to false, returns the title.
-            if ( ! $_aPage[ 'show_in_page_tabs' ] ) {
+            if ( !$_aPage[ 'show_in_page_tabs' ] ) {
                 return isset( $aInPageTabs[ $_sCurrentTabSlug ][ 'title' ] ) 
                     ? "<{$_sTag}>" 
                             . $aInPageTabs[ $_sCurrentTabSlug ][ 'title' ]
@@ -100,9 +100,9 @@ 
             private function _getInPageTabNavigationBar( array $aTabs, $sActiveTab, $sCurrentPageSlug, $sTag ) {
 
                 $_oTabBar = new AdminPageFramework_TabNavigationBar(
-                    $aTabs,      // tabs
+                    $aTabs, // tabs
                     $sActiveTab, // active tab slug
-                    $sTag,       // container tag
+                    $sTag, // container tag
                     array(       // container attributes
                         'class' => 'in-page-tab',
                     ),
@@ -130,7 +130,7 @@ 
                  * @since           3.5.10
                  * @since           3.6.3       Moved from `AdminPageFramework_Page_View`.
                  */
-                public function _replyToFormatNavigationTabItem_InPageTab( array $aTab, array $aStructure, array $aTabs, array $aArguments=array() ) {
+                public function _replyToFormatNavigationTabItem_InPageTab( array $aTab, array $aStructure, array $aTabs, array $aArguments = array() ) {
                     $_oFormatter = new AdminPageFramework_Format_NavigationTab_InPageTab(
                         $aTab,
                         $aStructure,

development/factory/admin_page/_view/AdminPageFramework_View__Resource.php

Spacing +13 added lines, -13 removed lines

@@ -67,9 +67,9 @@ 
                  */
                 public function _appendInlineAssets( $sInline, &$aContainer ) {
                     $_aInlines = array_unique( $aContainer );
-                    $sInline   = PHP_EOL . $sInline;
-                    foreach( $_aInlines as $_iIndex => $_sInline ) {
-                        $sInline .= $_sInline . PHP_EOL;
+                    $sInline   = PHP_EOL.$sInline;
+                    foreach ( $_aInlines as $_iIndex => $_sInline ) {
+                        $sInline .= $_sInline.PHP_EOL;
                         unset( $_aInlines[ $_iIndex ] );
                     }
                     $aContainer = $_aInlines; // update the container array.
@@ -83,23 +83,23 @@ 
         private function _parseAssets( $oFactory ) {
             
             // page
-            $_aPageStyles      = $this->getElementAsArray(
+            $_aPageStyles = $this->getElementAsArray(
                 $oFactory->oProp->aPages,
                 array( $this->sCurrentPageSlug, 'style' )
             );               
             $this->_enqueuePageAssets( $_aPageStyles, 'style' );
             
-            $_aPageScripts     = $this->getElementAsArray(
+            $_aPageScripts = $this->getElementAsArray(
                 $oFactory->oProp->aPages,
                 array( $this->sCurrentPageSlug, 'script' )
             );
             $this->_enqueuePageAssets( $_aPageScripts, 'script' );
             
             // In-page tabs 
-            if ( ! $this->sCurrentTabSlug ) {
+            if ( !$this->sCurrentTabSlug ) {
                 return;
             }        
-            $_aInPageTabStyles  = $this->getElementAsArray(
+            $_aInPageTabStyles = $this->getElementAsArray(
                 $oFactory->oProp->aInPageTabs,
                 array( $this->sCurrentPageSlug, $this->sCurrentTabSlug, 'style' )
             );              
@@ -116,10 +116,10 @@ 
              * @since       3.6.3
              * @return      void
              */    
-            private function _enqueuePageAssets( array $aPageAssets, $sType='style' ) {
-                $_sMathodName = "_enqueueAsset_" . $sType;
-                foreach( $aPageAssets as $_asPageAsset ) {
-                    $this->{$_sMathodName}( $_asPageAsset);
+            private function _enqueuePageAssets( array $aPageAssets, $sType = 'style' ) {
+                $_sMathodName = "_enqueueAsset_".$sType;
+                foreach ( $aPageAssets as $_asPageAsset ) {
+                    $this->{$_sMathodName}( $_asPageAsset );
                 }
             }
 
@@ -144,7 +144,7 @@ 
                     }
                     
                     // Insert the CSS rule in the head tag.
-                    $this->aCSSRules[] = $_sSRC;
+                    $this->aCSSRules[ ] = $_sSRC;
                     
                 }                    
      
@@ -169,7 +169,7 @@ 
                     }                
                     
                     // Insert the scripts in the head tag.                
-                    $this->aScripts[] = $_sSRC;
+                    $this->aScripts[ ] = $_sSRC;
                     
                 }        
                 

development/factory/meta_box/AdminPageFramework_MetaBox_Model.php

Indentation +2 added lines, -2 removed lines

@@ -136,11 +136,11 @@
      * 
      * Triggered when a post has not been created so no post id is assigned.
      * 
-	 * @internal
+     * @internal
      * @since       3.3.0
      * @callback    filter      wp_insert_attachment_data
      * @callback    filter      wp_insert_post_data
-	 * @param       array       $aPostData      An array of slashed post data.
+     * @param       array       $aPostData      An array of slashed post data.
      * @param       array       $aUnmodified    An array of sanitized, but otherwise unmodified post data.
      */
     public function _replyToFilterSavingData( $aPostData, $aUnmodified ) {

Doc Comments +2 added lines

@@ -32,6 +32,8 @@
      * Sets up properties and hooks.
      * 
      * @since       3.0.4
+     * @param string $sMetaBoxID
+     * @param string $sTitle
      */
     public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {     
                 

Spacing +19 added lines, -19 removed lines

@@ -22,12 +22,12 @@ 
      * Sets up hooks.
      * @since       3.7.9
      */
-    public function __construct( $sMetaBoxID, $sTitle, $asPostTypeOrScreenID=array( 'post' ), $sContext='normal', $sPriority='default', $sCapability='edit_posts', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sMetaBoxID, $sTitle, $asPostTypeOrScreenID = array( 'post' ), $sContext = 'normal', $sPriority = 'default', $sCapability = 'edit_posts', $sTextDomain = 'admin-page-framework' ) {
          
         // This is important to set the hooks before the parent constructor 
         // as the setUp wil be called in there if the default action hook (current_screen) is already triggered.
-        add_action( 'set_up_' . $this->oProp->sClassName, array( $this, '_replyToSetUpHooks' ) );
-        add_action( 'set_up_' . $this->oProp->sClassName, array( $this, '_replyToSetUpValidationHooks' ) );
+        add_action( 'set_up_'.$this->oProp->sClassName, array( $this, '_replyToSetUpHooks' ) );
+        add_action( 'set_up_'.$this->oProp->sClassName, array( $this, '_replyToSetUpValidationHooks' ) );
                 
         parent::__construct(
             $sMetaBoxID,
@@ -107,14 +107,14 @@ 
      * @callback    action      add_meta_boxes
      */
     public function _replyToRegisterMetaBoxes() {
-        foreach( $this->oProp->aPostTypes as $sPostType ) {
+        foreach ( $this->oProp->aPostTypes as $sPostType ) {
             add_meta_box(
-                $this->oProp->sMetaBoxID,                       // id
-                $this->oProp->sTitle,                           // title
+                $this->oProp->sMetaBoxID, // id
+                $this->oProp->sTitle, // title
                 array( $this, '_replyToPrintMetaBoxContents' ), // callback
-                $sPostType,                                     // post type
-                $this->oProp->sContext,                         // context
-                $this->oProp->sPriority,                        // priority
+                $sPostType, // post type
+                $this->oProp->sContext, // context
+                $this->oProp->sPriority, // priority
                 null                                            // argument - deprecated $this->oForm->aFields
             );
         }
@@ -146,7 +146,7 @@ 
          * @internal
          * @return      integer     The found post ID. `0` if not found.
          */
-        private function _getPostID()  {
+        private function _getPostID() {
             
             // for an editing post page.
             if ( isset( $GLOBALS[ 'post' ]->ID ) ) {
@@ -179,14 +179,14 @@ 
     public function _replyToFilterSavingData( $aPostData, $aUnmodified ) {
 
         // Perform initial checks.
-        if ( ! $this->_shouldProceedValidation( $aUnmodified ) ) {
+        if ( !$this->_shouldProceedValidation( $aUnmodified ) ) {
             return $aPostData;
         }
                 
         // Retrieve the submitted data. 
-        $_aInputs       = $this->oForm->getSubmittedData(
-            $_POST,     // subject data to be parsed
-            true,       // extract data with the fieldset structure
+        $_aInputs = $this->oForm->getSubmittedData(
+            $_POST, // subject data to be parsed
+            true, // extract data with the fieldset structure
             false       // strip slashes
         );
         $_aInputsRaw    = $_aInputs; // store one for the last input array.
@@ -221,8 +221,8 @@ 
         }
                     
         $this->oForm->updateMetaDataByType(
-            $_iPostID,   // object id
-            $_aInputs,   // user submit form data
+            $_iPostID, // object id
+            $_aInputs, // user submit form data
             $this->oForm->dropRepeatableElements( $_aSavedMeta ), // Drop repeatable section elements from the saved meta array.
             $this->oForm->sStructureType   // fields type
         );
@@ -280,14 +280,14 @@ 
             }
      
             // If our nonce isn't there, or we can't verify it, bail
-            if ( ! isset( $_POST[ $this->oProp->sMetaBoxID ] ) ) {
+            if ( !isset( $_POST[ $this->oProp->sMetaBoxID ] ) ) {
                 return false;
             }
-            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sMetaBoxID ], $this->oProp->sMetaBoxID ) ) {
+            if ( !wp_verify_nonce( $_POST[ $this->oProp->sMetaBoxID ], $this->oProp->sMetaBoxID ) ) {
                 return false;
             }
             
-            if ( ! in_array( $aUnmodified[ 'post_type' ], $this->oProp->aPostTypes ) ) {
+            if ( !in_array( $aUnmodified[ 'post_type' ], $this->oProp->aPostTypes ) ) {
                 return false;
             }
             

Braces +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
          * @internal
          * @return      integer     The found post ID. `0` if not found.
          */
-        private function _getPostID()  {
+        private function _getPostID() {
             
             // for an editing post page.
             if ( isset( $GLOBALS[ 'post' ]->ID ) ) {

development/factory/meta_box/_model/AdminPageFramework_MetaBox_Model___PostMeta.php

Spacing +6 added lines, -6 removed lines

@@ -44,7 +44,7 @@ 
      */
     public function get() {
 
-        if ( ! $this->iPostID ) {
+        if ( !$this->iPostID ) {
             return array();
         }
         return $this->_getSavedDataFromFieldsets(
@@ -66,11 +66,11 @@ 
             );     
 
             $_aMetaData = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
                 
-                if ( '_default' == $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aFieldset ) {
-                        if ( ! in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
+                if ( '_default' == $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aFieldset ) {
+                        if ( !in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
                             continue;
                         }
                         $_aMetaData[ $_aFieldset[ 'field_id' ] ] = get_post_meta( 
@@ -80,7 +80,7 @@ 
                         );    
                     }
                 }
-                if ( ! in_array( $_sSectionID, $_aMetaKeys ) ) {
+                if ( !in_array( $_sSectionID, $_aMetaKeys ) ) {
                     continue;
                 }                
                 $_aMetaData[ $_sSectionID ] = get_post_meta( 

development/factory/network_admin_page/AdminPageFramework_NetworkAdmin.php

Spacing +8 added lines, -8 removed lines

@@ -30,12 +30,12 @@ 
     protected $_aBuiltInRootMenuSlugs = array(
         // All keys must be lower case to support case insensitive look-ups.
         'dashboard'     => 'index.php',
-        'sites'         => 'sites.php',         // not work
-        'themes'        => 'themes.php',        // not work
+        'sites'         => 'sites.php', // not work
+        'themes'        => 'themes.php', // not work
         'plugins'       => 'plugins.php',
         'users'         => 'users.php',
         'settings'      => 'settings.php',
-        'updates'       => 'update-core.php',   // does not work
+        'updates'       => 'update-core.php', // does not work
     );     
         
     /**
@@ -55,9 +55,9 @@ 
      * @param       string      $sTextDomain        (optional) the <a href="http://codex.wordpress.org/I18n_for_WordPress_Developers#Text_Domains" target="_blank">text domain</a> used for the framework's system messages. Default: `admin-page-framework`.
      * @return      void        returns nothing.
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_network', $sTextDomain='admin-page-framework' ){
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_network', $sTextDomain = 'admin-page-framework' ) {
             
-        if ( ! $this->_isInstantiatable() ) {
+        if ( !$this->_isInstantiatable() ) {
             return;
         }
         
@@ -65,7 +65,7 @@ 
             
         $sCallerPath = $sCallerPath 
             ? $sCallerPath 
-            : AdminPageFramework_Utility::getCallerScriptPath( __FILE__ );     // this is important to attempt to find the caller script path here when separating the library into multiple files.
+            : AdminPageFramework_Utility::getCallerScriptPath( __FILE__ ); // this is important to attempt to find the caller script path here when separating the library into multiple files.
         
         $this->oProp = new AdminPageFramework_Property_NetworkAdmin( $this, $sCallerPath, get_class( $this ), $sOptionKey, $sCapability, $sTextDomain );
         
@@ -81,7 +81,7 @@ 
      */
     protected function _isInstantiatable() {
         
-        if ( isset( $GLOBALS['pagenow'] ) && 'admin-ajax.php' === $GLOBALS['pagenow'] ) {
+        if ( isset( $GLOBALS[ 'pagenow' ] ) && 'admin-ajax.php' === $GLOBALS[ 'pagenow' ] ) {
             return false;
         }     
         
@@ -125,7 +125,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::getSiteOption( $sOptionKey, $asKey, $vDefault );
     }
     

Braces +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      * @param       string      $sTextDomain        (optional) the <a href="http://codex.wordpress.org/I18n_for_WordPress_Developers#Text_Domains" target="_blank">text domain</a> used for the framework's system messages. Default: `admin-page-framework`.
      * @return      void        returns nothing.
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_network', $sTextDomain='admin-page-framework' ){
+    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_network', $sTextDomain='admin-page-framework' ) {
             
         if ( ! $this->_isInstantiatable() ) {
             return;