michaeluno / admin-page-framework

development/factory/page_meta_box/_model/AdminPageFramework_Property_page_meta_box.php

Indentation +1 added lines, -1 removed lines

@@ -149,7 +149,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 isset( $_GET[ 'page' ] )
             ? $_GET[ '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->getElement( $_GET, 'page' );
-        if ( ! $_sCurrentPageSlug ) {
+        if ( !$_sCurrentPageSlug ) {
             return false;
         }
         $_sCurrentTabSlug = $this->getElement(
@@ -185,7 +185,7 @@ 
      */
     public function getDefaultInPageTab( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return '';
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -200,7 +200,7 @@ 
      */
     public function getOptionKey( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return '';
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -227,7 +227,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/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
         );
 

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Controller.php

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
      *
      * @since 3.0.0
      */
-    public function enqueueStyles( $aSRCs, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueStyles( $aSRCs, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueStyles' ) ) {
             return $this->oResource->_enqueueStyles( $aSRCs, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -37,7 +37,7 @@ 
      * @param array (optional) The argument array for more advanced parameters.
      * @return string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */
-    public function enqueueStyle( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueStyle( $sSRC, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueStyle' ) ) {
             return $this->oResource->_enqueueStyle( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -47,7 +47,7 @@ 
      *
      * @since 2.1.5
      */
-    public function enqueueScripts( $aSRCs, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueScripts( $aSRCs, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueScripts' ) ) {
             return $this->oResource->_enqueueScripts( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -63,7 +63,7 @@ 
      * @param             array (optional) The argument array for more advanced parameters.
      * @return string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */
-    public function enqueueScript( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueScript( $sSRC, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueScript' ) ) {
             return $this->oResource->_enqueueScript( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Router.php

Spacing +4 added lines, -4 removed lines

@@ -44,13 +44,13 @@ 
      */
     protected function _isInThePage() {
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
         // Foe admin-ajax.php, aQuery holds the referer's GET URL parameters so the check covers the cases of ajax calls.
         $_sPageSlug = $this->oUtil->getElement( $this->oProp->aQuery, array( 'page' ), '' );
-        if ( ! $this->___isAddedPage( $_sPageSlug ) ) {
+        if ( !$this->___isAddedPage( $_sPageSlug ) ) {
             return false;
         }
         return true;
@@ -63,8 +63,8 @@ 
         private function ___isAddedPage( $sPageSlug ) {
 
             // for cases that page slugs are stored with numeric index
-            if ( ! $this->oUtil->isAssociative( $this->oProp->aPageSlugs )
-                 && ! in_array( $sPageSlug, $this->oProp->aPageSlugs )
+            if ( !$this->oUtil->isAssociative( $this->oProp->aPageSlugs )
+                 && !in_array( $sPageSlug, $this->oProp->aPageSlugs )
             ) {
                 return false;
             }

development/factory/page_meta_box/AdminPageFramework_MetaBox_Page.php

Spacing +2 added lines, -2 removed lines

@@ -23,12 +23,12 @@
      * @since       3.0.0
      * @deprecated  3.7.0      Use     `AdminPageFramework_PageMetaBox` instead.
      */
-    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs = array(), $sContext = 'normal', $sPriority = 'default', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         parent::__construct( $sMetaBoxID, $sTitle, $asPageSlugs, $sContext, $sPriority, $sCapability, $sTextDomain );
 
         $this->oUtil->showDeprecationNotice(
-            'The class, ' . __CLASS__ . ',', // deprecated item
+            'The class, '.__CLASS__.',', // deprecated item
             'AdminPageFramework_PageMetaBox' // alternative
         );
 

development/factory/taxonomy_field/_controller/AdminPageFramework_Resource_taxonomy_field.php

Spacing +16 added lines, -16 removed lines

@@ -29,11 +29,11 @@ 
      * @remark      the $_deprecated parameter is just to avoid the PHP strict standards warning.
      * @internal
      */
-    public function _enqueueStyles( $aSRCs, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueStyles( $aSRCs, $aCustomArgs = array(), $_deprecated = null ) {
 
         $_aHandleIDs = array();
-        foreach( ( array ) $aSRCs as $_sSRC ) {
-            $_aHandleIDs[] = $this->_enqueueStyle( $_sSRC, $aCustomArgs );
+        foreach ( ( array ) $aSRCs as $_sSRC ) {
+            $_aHandleIDs[ ] = $this->_enqueueStyle( $_sSRC, $aCustomArgs );
         }
         return $_aHandleIDs;
 
@@ -57,9 +57,9 @@ 
      * @return      string      The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */
-    public function _enqueueStyle( $sSRC, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueStyle( $sSRC, $aCustomArgs = array(), $_deprecated = null ) {
 
-        $sSRC       = trim( $sSRC );
+        $sSRC = trim( $sSRC );
         if ( empty( $sSRC ) ) {
             return '';
         }
@@ -76,13 +76,13 @@ 
             array(
                 'sSRC'      => $sSRC,
                 'sType'     => 'style',
-                'handle_id' => 'style_' . $this->oProp->sClassName . '_' .  ( ++$this->oProp->iEnqueuedStyleIndex ),
+                'handle_id' => 'style_'.$this->oProp->sClassName.'_'.( ++$this->oProp->iEnqueuedStyleIndex ),
             ),
             self::$_aStructure_EnqueuingResources
         );
 
         // Store the attributes in another container by url.
-        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingStyles[ $_sSRCHash ]['handle_id'] ] = $this->oProp->aEnqueuingStyles[ $_sSRCHash ]['attributes'];
+        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingStyles[ $_sSRCHash ][ 'handle_id' ] ] = $this->oProp->aEnqueuingStyles[ $_sSRCHash ][ 'attributes' ];
 
         return $this->oProp->aEnqueuingStyles[ $_sSRCHash ][ 'handle_id' ];
 
@@ -94,11 +94,11 @@ 
      * @since       3.0.0
      * @remark      the $_deprecated parameter is just to avoid the PHP strict standards warning.
      */
-    public function _enqueueScripts( $aSRCs, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueScripts( $aSRCs, $aCustomArgs = array(), $_deprecated = null ) {
 
         $_aHandleIDs = array();
-        foreach( ( array ) $aSRCs as $_sSRC ) {
-            $_aHandleIDs[] = $this->_enqueueScript( $_sSRC, $aCustomArgs );
+        foreach ( ( array ) $aSRCs as $_sSRC ) {
+            $_aHandleIDs[ ] = $this->_enqueueScript( $_sSRC, $aCustomArgs );
         }
         return $_aHandleIDs;
 
@@ -123,9 +123,9 @@ 
      * @return      string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */
-    public function _enqueueScript( $sSRC, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueScript( $sSRC, $aCustomArgs = array(), $_deprecated = null ) {
 
-        $sSRC       = trim( $sSRC );
+        $sSRC = trim( $sSRC );
         if ( empty( $sSRC ) ) {
             return '';
         }
@@ -142,13 +142,13 @@ 
             array(
                 'sSRC'      => $sSRC,
                 'sType'     => 'script',
-                'handle_id' => 'script_' . $this->oProp->sClassName . '_' .  ( ++$this->oProp->iEnqueuedScriptIndex ),
+                'handle_id' => 'script_'.$this->oProp->sClassName.'_'.( ++$this->oProp->iEnqueuedScriptIndex ),
             ),
             self::$_aStructure_EnqueuingResources
         );
 
         // Store the attributes in another container by url.
-        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingScripts[ $_sSRCHash ]['handle_id'] ] = $this->oProp->aEnqueuingScripts[ $_sSRCHash ]['attributes'];
+        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingScripts[ $_sSRCHash ][ 'handle_id' ] ] = $this->oProp->aEnqueuingScripts[ $_sSRCHash ][ 'attributes' ];
 
         return $this->oProp->aEnqueuingScripts[ $_sSRCHash ][ 'handle_id' ];
 
@@ -160,7 +160,7 @@ 
      * @since       3.0.0
      * @internal
      */
-    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs=array() ) {
+    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs = array() ) {
         return $this->_enqueueStyle( $sSRC, $aCustomArgs );
     }
     /**
@@ -169,7 +169,7 @@ 
      * @since       3.0.0
      * @internal
      */
-    public function _forceToEnqueueScript( $sSRC, $aCustomArgs=array() ) {
+    public function _forceToEnqueueScript( $sSRC, $aCustomArgs = array() ) {
         return $this->_enqueueScript( $sSRC, $aCustomArgs );
     }
 

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_View.php

Indentation +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         return "{$_aField['field_id']}{$_sKey}";
     }
 
-   /**
+    /**
      * Adds input fields
      *
      * @internal

Spacing +8 added lines, -8 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;
 
@@ -155,8 +155,8 @@ 
     public function _replyToPrintColumnCell( $vValue, $sColumnSlug, $sTermID ) {
 
         $_sCellHTML = '';
-        if ( isset( $_GET['taxonomy'] ) && $_GET['taxonomy'] ) {
-            $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$_GET['taxonomy']}", $vValue, $sColumnSlug, $sTermID );
+        if ( isset( $_GET[ 'taxonomy' ] ) && $_GET[ 'taxonomy' ] ) {
+            $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$_GET[ 'taxonomy' ]}", $vValue, $sColumnSlug, $sTermID );
         }
         $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$this->oProp->sClassName}", $_sCellHTML, $sColumnSlug, $sTermID );
         $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$this->oProp->sClassName}_{$sColumnSlug}", $_sCellHTML, $sTermID ); // 3.0.2+

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 ) ) {
+        if ( isset( $_GET[ 'taxonomy' ] ) && !in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {
             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 );

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Model.php

Spacing +12 added lines, -12 removed lines

@@ -53,8 +53,8 @@ 
         return $this->_getFilteredColumnsByFilterPrefix(
             $this->oUtil->getAsArray( $aColumns ),
             'columns_',
-            isset( $_GET['taxonomy'] )  // in ajax, $_GET is not even set.
-                ? $_GET['taxonomy']
+            isset( $_GET[ 'taxonomy' ] )  // in ajax, $_GET is not even set.
+                ? $_GET[ 'taxonomy' ]
                 : ''
         );
     }
@@ -71,8 +71,8 @@ 
         return $this->_getFilteredColumnsByFilterPrefix(
             $this->oUtil->getAsArray( $aSortableColumns ),
             'sortable_columns_',
-            isset( $_GET['taxonomy'] )  // in ajax, $_GET is not even set.
-                ? $_GET['taxonomy']
+            isset( $_GET[ 'taxonomy' ] )  // in ajax, $_GET is not even set.
+                ? $_GET[ 'taxonomy' ]
                 : ''
         );
     }
@@ -86,7 +86,7 @@ 
             if ( $sTaxonomy ) {
                 $aColumns = $this->oUtil->addAndApplyFilter(
                     $this,
-                    "{$sFilterPrefix}{$_GET['taxonomy']}",
+                    "{$sFilterPrefix}{$_GET[ 'taxonomy' ]}",
                     $aColumns
                 );
             }
@@ -123,7 +123,7 @@ 
      * @since       3.7.0      No longer sets the value to `$this-oProp->aOptions` but to the form peoperty.
      * @internal
      */
-    protected function _setOptionArray( $iTermID=null, $sOptionKey ) {
+    protected function _setOptionArray( $iTermID = null, $sOptionKey ) {
         $this->oForm->aSavedData = $this->_getSavedFormData(
             $iTermID,
             $sOptionKey
@@ -139,7 +139,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
             );
@@ -173,7 +173,7 @@ 
      */
     public function _replyToValidateOptions( $iTermID ) {
 
-        if ( ! $this->_shouldProceedValidation() ) {
+        if ( !$this->_shouldProceedValidation() ) {
             return;
         }
 
@@ -182,7 +182,7 @@ 
         $_aSubmittedFormData    = $this->oForm->getSubmittedData( $_POST );
         $_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 )
@@ -193,7 +193,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
         );
@@ -215,11 +215,11 @@ 
          */
         protected function _shouldProceedValidation() {
 
-            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) {
+            if ( !isset( $_POST[ $this->oProp->sClassHash ] ) ) {
 
                 return false;
             }
-            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
+            if ( !wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
                 return false;
             }
             return true;