michaeluno / admin-page-framework

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

Spacing +9 added lines, -9 removed lines

@@ -30,15 +30,15 @@
         'page_slug'         => null,
         'tab_slug'          => null,
         'title'             => null,
-        'order'             => 10,      // (integer)
-        'show_in_page_tab'  => true,    // 3.6.0+ (boolean)
-        'parent_tab_slug'   => null,    // this needs to be set if the above show_in_page_tab is false so that the framework can mark the parent tab to be active when the hidden page is accessed.
-        'url'               => null,    // 3.5.0+ This allows the user set custom link.
-        'disabled'          => null,    // 3.5.10+ (boolean) If true, the link will be unlinked.
-        'attributes'        => null,    // 3.5.10+ (array) Applies to the navigation tab bar element.
-        'capability'        => null,    // 3.6.0+ (string)
-        'if'                => true,    // 3.6.0+ (boolean)
-        'show_debug_info'   => null,    // 3.8.8+ (boolean, optional) Whether to show debug information. If not set, the existent `bShowDebugInfo` property value will be used. The initial value here is `null` as the default value will be assigned in the formatting method.
+        'order'             => 10, // (integer)
+        'show_in_page_tab'  => true, // 3.6.0+ (boolean)
+        'parent_tab_slug'   => null, // this needs to be set if the above show_in_page_tab is false so that the framework can mark the parent tab to be active when the hidden page is accessed.
+        'url'               => null, // 3.5.0+ This allows the user set custom link.
+        'disabled'          => null, // 3.5.10+ (boolean) If true, the link will be unlinked.
+        'attributes'        => null, // 3.5.10+ (array) Applies to the navigation tab bar element.
+        'capability'        => null, // 3.6.0+ (string)
+        'if'                => true, // 3.6.0+ (boolean)
+        'show_debug_info'   => null, // 3.8.8+ (boolean, optional) Whether to show debug information. If not set, the existent `bShowDebugInfo` property value will be used. The initial value here is `null` as the default value will be assigned in the formatting method.
     );
 
     /**

development/factory/admin_page/_model/AdminPageFramework_FormEmail.php

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
         }
 
         // Send mail.
-        $_bSent         = wp_mail(
+        $_bSent = wp_mail(
             $this->_getEmailArgument( $aInput, $aEmailOptions, 'to', $sSubmitSectionID ),
             $this->_getEmailArgument( $aInput, $aEmailOptions, 'subject', $sSubmitSectionID ),
             $_bIsHTML
@@ -100,7 +100,7 @@ 
         remove_filter( 'wp_mail_from_name', array( $this, '_replyToSetEmailSenderAddress' ) );
 
         // Clean up.
-        foreach( $this->_aPathsToDelete as $_sPath ) {
+        foreach ( $this->_aPathsToDelete as $_sPath ) {
             unlink( $_sPath );
         }
 
@@ -121,7 +121,7 @@ 
             }
 
             $_aAttachments = $this->getAsArray( $asAttachments );
-            foreach( $_aAttachments as $_iIndex => $_sPathORURL ) {
+            foreach ( $_aAttachments as $_iIndex => $_sPathORURL ) {
 
                 // If it is a file path, fine.
                 if ( is_file( $_sPathORURL ) ) {
@@ -211,7 +211,7 @@ 
             }
 
             // If the key element is empty, search the corresponding item in the same section.
-            if ( ! $aEmailOptions[ $sKey ] ) {
+            if ( !$aEmailOptions[ $sKey ] ) {
                 return $this->getArrayValueByArrayKeys( $aInput, array( $sSectionID, $sKey ) );
             }
 

Indentation +1 added lines, -1 removed lines

@@ -186,7 +186,7 @@
                         $sURL
                     ) );
                 }
-       /**
+        /**
          * Sets the mail content type to HTML.
          * @since       3.3.0
          * @since       3.4.2       Moved from the validation class.

development/factory/admin_page/_model/AdminPageFramework_ImportOptions.php

Spacing +3 added lines, -3 removed lines

@@ -56,11 +56,11 @@ 
 
     }
 
-    private function getElementInFilesArray( array $aFilesImport, $sInputID, $sElementKey='error' ) {
+    private function getElementInFilesArray( array $aFilesImport, $sInputID, $sElementKey = 'error' ) {
 
         $sElementKey = strtolower( $sElementKey );
         return $this->getElement(
-            $aFilesImport,  // subject array
+            $aFilesImport, // subject array
             array( $sElementKey, $sInputID ), // dimensional keys
             null    // default
         );
@@ -88,7 +88,7 @@ 
         return $vData;
 
     }
-    public function formatImportData( &$vData, $sFormatType=null ) {
+    public function formatImportData( &$vData, $sFormatType = null ) {
 
         $sFormatType = isset( $sFormatType ) ? $sFormatType : $this->getFormatType();
         switch ( strtolower( $sFormatType ) ) {

development/factory/admin_page/form/AdminPageFramework_Form_admin_page.php

Spacing +19 added lines, -19 removed lines

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

development/factory/admin_page/AdminPageFramework_Model_Form.php

Spacing +17 added lines, -17 removed lines

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

Braces +2 added lines, -1 removed lines

@@ -74,7 +74,8 @@
         // new AdminPageFramework_Model__FormEmailHandler( $this );
 
         // Checking the GET and POST methods.
-        if ( isset( $_REQUEST[ 'apf_remote_request_test' ] ) && '_testing' === $_REQUEST[ 'apf_remote_request_test' ] ) {   // sanitization unnecessary
+        if ( isset( $_REQUEST[ 'apf_remote_request_test' ] ) && '_testing' === $_REQUEST[ 'apf_remote_request_test' ] ) {
+// sanitization unnecessary
             exit( 'OK' );
         }
 

development/factory/admin_page/AdminPageFramework_Model_Menu.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      *
      * @internal
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         parent::__construct( $sOptionKey, $sCallerPath, $sCapability, $sTextDomain );
 

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer__PageHeadingTabs.php

Spacing +12 added lines, -12 removed lines

@@ -45,7 +45,7 @@ 
         $_aPage = $this->oFactory->oProp->aPages[ $this->sPageSlug ];
 
         // If the page title is disabled, return an empty string.
-        if ( ! $_aPage[ 'show_page_title' ] ) {
+        if ( !$_aPage[ 'show_page_title' ] ) {
             return "";
         }
 
@@ -64,10 +64,10 @@ 
          */
         private function _getOutput( $aPage, $sTag ) {
 
-            $sTag  = $this->_getPageHeadingTabTag( $sTag, $aPage );
+            $sTag = $this->_getPageHeadingTabTag( $sTag, $aPage );
 
             // If the page heading tab visibility is disabled, or only one page is registered, return the title.
-            if ( ! $aPage[ 'show_page_heading_tabs' ] || count( $this->oFactory->oProp->aPages ) == 1 ) {
+            if ( !$aPage[ 'show_page_heading_tabs' ] || count( $this->oFactory->oProp->aPages ) == 1 ) {
                 return "<{$sTag}>"
                         . $aPage[ 'title' ]
                     . "</{$sTag}>";
@@ -105,9 +105,9 @@ 
             private function _getPageHeadingtabNavigationBar( array $aPages, $sTag, $sCurrentPageSlug ) {
 
                 $_oTabBar = new AdminPageFramework_TabNavigationBar(
-                    $aPages,     // tab items
+                    $aPages, // tab items
                     $sCurrentPageSlug, // active tab slug
-                    $sTag,       // container tag
+                    $sTag, // container tag
                     array(       // container attributes
                         // 'class' => '...',
                     ),
@@ -130,8 +130,8 @@ 
                  * @since           3.5.10
                  * @since       3.6.3       Moved from `AdminPageFramework_Page_View`.
                  */
-                public function _replyToFormatNavigationTabItem_PageHeadingTab( $aSubPage, $aStructure, $aPages, $aArguments=array() ) {
-                    switch( $aSubPage['type'] ) {
+                public function _replyToFormatNavigationTabItem_PageHeadingTab( $aSubPage, $aStructure, $aPages, $aArguments = array() ) {
+                    switch ( $aSubPage[ 'type' ] ) {
                         case 'link':
                             return $this->_getFormattedPageHeadingtabNavigationBarLinkItem( $aSubPage, $aStructure );
                         default:
@@ -148,10 +148,10 @@ 
                      */
                     private function _getFormattedPageHeadingtabNavigationBarPageItem( array $aSubPage, $aStructure ) {
 
-                        if ( ! isset( $aSubPage[ 'page_slug' ] ) ) {
+                        if ( !isset( $aSubPage[ 'page_slug' ] ) ) {
                             return array();
                         }
-                        if ( ! $aSubPage[ 'show_page_heading_tab' ] ) {
+                        if ( !$aSubPage[ 'show_page_heading_tab' ] ) {
                             return array();
                         }
                         return array(
@@ -182,10 +182,10 @@ 
                      */
                     private function _getFormattedPageHeadingtabNavigationBarLinkItem( array $aSubPage, $aStructure ) {
 
-                        if ( ! isset( $aSubPage[ 'href' ] ) ) {
+                        if ( !isset( $aSubPage[ 'href' ] ) ) {
                             return array();
                         }
-                        if ( ! $aSubPage[ 'show_page_heading_tab' ] ) {
+                        if ( !$aSubPage[ 'show_page_heading_tab' ] ) {
                             return array();
                         }
                         $aSubPage = array(
@@ -197,7 +197,7 @@ 
                             + array( 'class' => null )
                             + $aStructure;
 
-                        $aSubPage[ 'class' ] = trim( $aSubPage[ 'class' ] . ' link' );
+                        $aSubPage[ 'class' ] = trim( $aSubPage[ 'class' ].' link' );
                         return $aSubPage;
                     }
 

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer__ScreenIcon.php

Spacing +12 added lines, -12 removed lines

@@ -40,7 +40,7 @@ 
      */
     public function get() {
 
-        if ( ! $this->_isScreenIconVisible() ) {
+        if ( !$this->_isScreenIconVisible() ) {
             return '';
         }
         return $this->_getScreenIcon( $this->sPageSlug );
@@ -52,7 +52,7 @@ 
          */
         private function _isScreenIconVisible() {
 
-            $_bShowPageTitle        = $this->getElement(
+            $_bShowPageTitle = $this->getElement(
                 $this->oFactory->oProp->aPages,
                 array( $this->sPageSlug, 'show_page_title' )
             );
@@ -130,7 +130,7 @@ 
                 array( $sPageSlug, 'href_icon_32x32' ),
                 ''
             );
-            if ( ! $_sScreenIconPath ) {
+            if ( !$_sScreenIconPath ) {
                 return;
             }
             $_sScreenIconPath = $this->getResolvedSRC(
@@ -140,7 +140,7 @@ 
             $_aAttributes = array(
                 'style'    => $this->getInlineCSS(
                     array(
-                        'background-image' => "url('" . esc_url( $_sScreenIconPath ) . "')"
+                        'background-image' => "url('".esc_url( $_sScreenIconPath )."')"
                     )
                 )
             );
@@ -164,12 +164,12 @@ 
                 array( $sPageSlug, 'screen_icon_id' ),
                 ''
             );
-            if ( ! $_sScreenIconID ) {
+            if ( !$_sScreenIconID ) {
                 return;
             }
 
             $_aAttributes = array(
-                'id'       => "icon-" . $_sScreenIconID,
+                'id'       => "icon-".$_sScreenIconID,
             );
 
             // Go to the catch clause.
@@ -192,16 +192,16 @@ 
                 'class'    => $this->getClassAttribute(
                     $this->getAOrB(
                         empty( $_sIconIDAttribute ) && $_oScreen->post_type,
-                        sanitize_html_class( 'icon32-posts-' . $_oScreen->post_type ),
+                        sanitize_html_class( 'icon32-posts-'.$_oScreen->post_type ),
                         ''
                     ),
                     $this->getAOrB(
                         empty( $_sIconIDAttribute ) || $_sIconIDAttribute == $this->oFactory->oProp->sClassName,
-                        'generic',  // the default value
+                        'generic', // the default value
                         ''
                     )
                 ),
-                'id'       => "icon-" . $_sIconIDAttribute,
+                'id'       => "icon-".$_sIconIDAttribute,
             );
             return $this->_getScreenIconByAttributes( $_aAttributes );
 
@@ -215,7 +215,7 @@ 
              */
             private function _getScreenIDAttribute( $oScreen ) {
 
-                if ( ! empty( $oScreen->parent_base ) ) {
+                if ( !empty( $oScreen->parent_base ) ) {
                     return $oScreen->parent_base;
                 }
                 if ( 'page' === $oScreen->post_type ) {
@@ -236,10 +236,10 @@ 
             private function _getScreenIconByAttributes( array $aAttributes ) {
 
                 $aAttributes[ 'class' ] = $this->getClassAttribute(
-                    'icon32',   // required for a screen icon container element.
+                    'icon32', // required for a screen icon container element.
                     $this->getElement( $aAttributes, 'class' )
                 );
-                return "<div " . $this->getAttributes( $aAttributes ) . ">"
+                return "<div ".$this->getAttributes( $aAttributes ).">"
                         . "<br />"
                     . "</div>";
 

development/factory/page_meta_box/form/AdminPageFramework_Form_page_meta_box.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     public function construct() {
 
         add_filter(
-            'options_' . $this->aArguments[ 'caller_id' ],
+            'options_'.$this->aArguments[ 'caller_id' ],
             array( $this, '_replyToSanitizeSavedFormData' ),
             5   //  high priority as it must be done earlier
         );
@@ -53,7 +53,7 @@ 
 
         // Extract the meta box field form data (options) from the page form data (options).
         return $this->castArrayContents(
-            $this->getDataStructureFromAddedFieldsets(),    // form data structure generate from fieldsets
+            $this->getDataStructureFromAddedFieldsets(), // form data structure generate from fieldsets
             $aSavedFormData
         );