michaeluno / admin-page-framework

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

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
             )
         );
 
-        if ( ! $this->_isEnabled( $_aTab ) ) {
+        if ( !$this->_isEnabled( $_aTab ) ) {
             return array();
         }
 
@@ -77,7 +77,7 @@ 
                 : esc_url(
                     $this->getElement(
                         $_aTab,
-                        'url',  // if the 'url' argument is set, use it. Otherwise, use the below gnerated url.
+                        'url', // if the 'url' argument is set, use it. Otherwise, use the below gnerated url.
                         $this->getQueryAdminURL(
                             array(
                                 'page'  => $this->aArguments[ 'page_slug' ],
@@ -106,7 +106,7 @@ 
          * @return      boolean
          */
         private function _isEnabled( $aTab ) {
-            return ! in_array(
+            return !in_array(
                 false,
                 array(
                     ( bool ) current_user_can( $aTab[ 'capability' ] ), // whether the user has the sufficient capability level

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

Spacing +5 added lines, -5 removed lines

@@ -23,12 +23,12 @@ 
      * @static
      */
     static public $aStructure = array(
-        'src'           => null,    // (required, string) the source url or path
+        'src'           => null, // (required, string) the source url or path
         'handle_id'     => null,
         'dependencies'  => array(),
-        'version'       => false,       // although the type should be string, the wp_enqueue_...() functions want false as the default value.
-        'translation'   => array(),     // only for scripts
-        'in_footer'     => false,       // only for scripts
+        'version'       => false, // although the type should be string, the wp_enqueue_...() functions want false as the default value.
+        'translation'   => array(), // only for scripts
+        'in_footer'     => false, // only for scripts
         // '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_PageResource_Style.php

Spacing +6 added lines, -6 removed lines

@@ -23,11 +23,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' => '...' )`
     );
 
@@ -40,7 +40,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

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

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_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 );