michaeluno / admin-page-framework

development/factory/admin_page/_view/AdminPageFramework_View__Resource.php

Spacing +13 added lines, -13 removed lines

@@ -68,9 +68,9 @@ 
                  */
                 public function _appendInternalAssets( $sInternal, &$aContainer ) {
                     $_aInternals = array_unique( $aContainer );
-                    $sInternal   = PHP_EOL . $sInternal;
-                    foreach( $_aInternals as $_iIndex => $_sInternal ) {
-                        $sInternal .= $_sInternal . PHP_EOL;
+                    $sInternal   = PHP_EOL.$sInternal;
+                    foreach ( $_aInternals as $_iIndex => $_sInternal ) {
+                        $sInternal .= $_sInternal.PHP_EOL;
                         unset( $_aInternals[ $_iIndex ] );
                     }
                     $aContainer = $_aInternals; // update the container array.
@@ -84,23 +84,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' )
             );              
@@ -117,10 +117,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 );
                 }
             }
 
@@ -145,7 +145,7 @@ 
                     }
                     
                     // Insert the CSS rule in the head tag.
-                    $this->aCSSRules[] = $_sSRC;
+                    $this->aCSSRules[ ] = $_sSRC;
                     
                 }                    
      
@@ -170,7 +170,7 @@ 
                     }                
                     
                     // Insert the scripts in the head tag.                
-                    $this->aScripts[] = $_sSRC;
+                    $this->aScripts[ ] = $_sSRC;
                     
                 }        
                 

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_Path.php

Spacing +7 added lines, -7 removed lines

@@ -29,8 +29,8 @@ 
     static public function getRelativePath( $from, $to ) {
         
         // some compatibility fixes for Windows paths
-        $from = is_dir( $from ) ? rtrim( $from, '\/') . '/' : $from;
-        $to   = is_dir( $to )   ? rtrim( $to, '\/') . '/'   : $to;
+        $from = is_dir( $from ) ? rtrim( $from, '\/' ).'/' : $from;
+        $to   = is_dir( $to ) ? rtrim( $to, '\/' ).'/' : $to;
         $from = str_replace( '\\', '/', $from );
         $to   = str_replace( '\\', '/', $to );
 
@@ -38,21 +38,21 @@ 
         $to       = explode( '/', $to );
         $relPath  = $to;
 
-        foreach( $from as $depth => $dir ) {
+        foreach ( $from as $depth => $dir ) {
             // find first non-matching dir
-            if( $dir === $to[ $depth ] ) {
+            if ( $dir === $to[ $depth ] ) {
                 // ignore this directory
                 array_shift( $relPath );
             } else {
                 // get number of remaining dirs to $from
                 $remaining = count( $from ) - $depth;
-                if( $remaining > 1 ) {
+                if ( $remaining > 1 ) {
                     // add traversals up to first matching dir
                     $padLength = ( count( $relPath ) + $remaining - 1 ) * -1;
                     $relPath = array_pad( $relPath, $padLength, '..' );
                     break;
                 } else {
-                    $relPath[ 0 ] = './' . $relPath[ 0 ];
+                    $relPath[ 0 ] = './'.$relPath[ 0 ];
                 }
             }
         }
@@ -81,7 +81,7 @@ 
                 6, // the second parameter: limit
             )
         );
-        foreach( $_aBackTrace as $_aDebugInfo )  {     
+        foreach ( $_aBackTrace as $_aDebugInfo ) {     
             $_sCallerFilePath = $_aDebugInfo[ 'file' ];
             if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) { 
                 continue; 

Braces +2 added lines, -2 removed lines

@@ -81,9 +81,9 @@
                 6, // the second parameter: limit
             )
         );
-        foreach( $_aBackTrace as $_aDebugInfo )  {     
+        foreach( $_aBackTrace as $_aDebugInfo ) {
             $_sCallerFilePath = $_aDebugInfo[ 'file' ];
-            if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) { 
+            if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) {
                 continue; 
             }
             break; // catch the first found item.

development/factory/_common/_abstract/AdminPageFramework_Factory_Model.php

Doc Comments +1 added lines, -2 removed lines

@@ -203,6 +203,7 @@ 
          * Constructs a WordPress filter hook name.
          * @internal
          * @since       3.8.4
+         * @param string $sPrefix
          * @return      string
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
@@ -339,8 +340,6 @@ 
      * @since       3.7.0      Changed back the visibility scope to protected as there is the `getFieldErrors()` public method.
      * @access      protected
      * @internal
-     * @param       string      $sID        deprecated
-     * @param       boolean     $bDelete    whether or not the transient should be deleted after retrieving it. 
      * @return      array
      * @deprecated  3.7.0      Use `getFieldErrors()` instead. Kept for backward compatibility.
      */

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
         
         add_filter(
             // 'field_types_admin_page_framework',
-            'field_types_' . $oProp->sClassName,
+            'field_types_'.$oProp->sClassName,
             array( $this, '_replyToFilterFieldTypeDefinitions' )
         );
         
@@ -65,7 +65,7 @@ 
             'title'      => null,
             'help_aside' => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
         $this->oHelpPane->_addHelpTextForFormFields( 
@@ -85,9 +85,9 @@ 
     public function _replyToFilterFieldTypeDefinitions( $aFieldTypeDefinitions ) {
         
         // Not triggering `__call()` as the filter is fired manually in the form class.
-        if ( method_exists( $this, 'field_types_' . $this->oProp->sClassName ) ) {
+        if ( method_exists( $this, 'field_types_'.$this->oProp->sClassName ) ) {
             return call_user_func_array(
-                array( $this, 'field_types_' . $this->oProp->sClassName ),
+                array( $this, 'field_types_'.$this->oProp->sClassName ),
                 array( $aFieldTypeDefinitions )
             );
         }
@@ -108,7 +108,7 @@ 
     public function _replyToModifySectionsets( $aSectionsets ) {    
         
         return $this->oUtil->addAndApplyFilter( 
-            $this,  // caller factory object
+            $this, // caller factory object
             "sections_{$this->oProp->sClassName}", 
             $aSectionsets
         );
@@ -128,7 +128,7 @@ 
     public function _replyToModifyFieldsets( $aFieldsets, $aSectionsets ) {
 
         // Apply filters to added fieldsets
-        foreach( $aFieldsets as $_sSectionPath => $_aFields ) {
+        foreach ( $aFieldsets as $_sSectionPath => $_aFields ) {
             $_aSectionPath  = explode( '|', $_sSectionPath );
             $_sFilterSuffix = implode( '_', $_aSectionPath );
             $aFieldsets[ $_sSectionPath ] = $this->oUtil->addAndApplyFilter(
@@ -137,7 +137,7 @@ 
                 $_aFields
             ); 
         }
-        $aFieldsets =  $this->oUtil->addAndApplyFilter( 
+        $aFieldsets = $this->oUtil->addAndApplyFilter( 
             $this,
             "fields_{$this->oProp->sClassName}",
             $aFieldsets
@@ -207,14 +207,14 @@ 
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
             
-            $_sFieldPart    = '_' . implode( '_', $aFieldset[ '_field_path_array' ] );
+            $_sFieldPart    = '_'.implode( '_', $aFieldset[ '_field_path_array' ] );
             $_sSectionPart  = implode( '_', $aFieldset[ '_section_path_array' ] );
             $_sSectionPart  = $this->oUtil->getAOrB(
                 '_default' === $_sSectionPart,
                 '',
-                '_' . $_sSectionPart
+                '_'.$_sSectionPart
             );
-            return $sPrefix . $this->oProp->sClassName . $_sSectionPart . $_sFieldPart;
+            return $sPrefix.$this->oProp->sClassName.$_sSectionPart.$_sFieldPart;
             
         }
                
@@ -284,9 +284,9 @@ 
     public function _replyToGetSavedFormData() {
         
         // Must update the property with the filtered value.
-        $this->oProp->aOptions =  $this->oUtil->addAndApplyFilter(
+        $this->oProp->aOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->oProp->aOptions      // subject value to be filtered
         );
         return $this->oProp->aOptions;
@@ -366,7 +366,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

Braces +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
      * but it requires to call the setUp() method in the overridden method so it's not that useful.
      * @internal
      */
-    protected function _setUp() { 
+    protected function _setUp() {
         $this->setUp();
     }    
     
@@ -105,7 +105,7 @@ 
      * @since       3.7.0
      * @return      array       The modified sectionsets definition array.
      */    
-    public function _replyToModifySectionsets( $aSectionsets ) {    
+    public function _replyToModifySectionsets( $aSectionsets ) {
         
         return $this->oUtil->addAndApplyFilter( 
             $this,  // caller factory object
@@ -366,7 +366,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }