michaeluno / admin-page-framework

development/utility/requirement/AdminPageFramework_Requirement.php

Indentation +1 added lines, -1 removed lines

@@ -278,7 +278,7 @@
                     ?  "<strong>" . $this->_sScriptName . "</strong>:&nbsp;" 
                     : '';            
                 return $_sScripTitle
-                   . implode( '<br />', $_aWarnings );
+                    . implode( '<br />', $_aWarnings );
                 
             }
         

Braces +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
      * @since       3.4.6
      * @return      integer         The number of warnings.
      */
-    public function check() {      
+    public function check() {
         
         $_aWarnings = array();
         

Doc Comments +2 added lines

@@ -195,6 +195,7 @@ 
          * Returns a php warning if present.
          * @since       3.5.3
          * @internal
+         * @param string $sType
          * @return      string      The warning.
          */
         private function _getWarningByType( $sType ) {
@@ -294,6 +295,7 @@ 
              * if it returns non true (false), it stores the subject warning and returns the array holding the warnings.
              * 
              * @since       3.4.6
+             * @param string $sFuncName
              * @return      array           The warning array.
              * @internal
              */

Spacing +24 added lines, -24 removed lines

@@ -135,12 +135,12 @@ 
      * </ul>
      * @param       string      $sScriptName        The script name.      
      */ 
-    public function __construct( array $aRequirements=array(), $sScriptName='' ) {
+    public function __construct( array $aRequirements = array(), $sScriptName = '' ) {
         
         // Avoid undefined index warnings.
         $aRequirements          = $aRequirements + $this->_aDefaultRequirements;    
         $aRequirements          = array_filter( $aRequirements, 'is_array' );
-        foreach( array( 'php', 'mysql', 'wordpress' ) as $_iIndex => $_sName ) {
+        foreach ( array( 'php', 'mysql', 'wordpress' ) as $_iIndex => $_sName ) {
             if ( isset( $aRequirements[ $_sName ] ) ) {
                 $aRequirements[ $_sName ] = $aRequirements[ $_sName ] + $this->_aDefaultRequirements[ $_sName ];
             }
@@ -165,9 +165,9 @@ 
         $_aWarnings = array();
         
         // PHP, WordPress, MySQL
-        $_aWarnings[] = $this->_getWarningByType( 'php' );
-        $_aWarnings[] = $this->_getWarningByType( 'wordpress' );
-        $_aWarnings[] = $this->_getWarningByType( 'mysql' );
+        $_aWarnings[ ] = $this->_getWarningByType( 'php' );
+        $_aWarnings[ ] = $this->_getWarningByType( 'wordpress' );
+        $_aWarnings[ ] = $this->_getWarningByType( 'mysql' );
         
         // Ensure necessary array elements.
         $this->_aRequirements = $this->_aRequirements + array(
@@ -180,10 +180,10 @@ 
         // Check the rest.
         $_aWarnings = array_merge(
             $_aWarnings,
-            $this->_checkFunctions( $this->_aRequirements['functions'] ),
-            $this->_checkClasses( $this->_aRequirements['classes'] ),
-            $this->_checkConstants( $this->_aRequirements['constants'] ),
-            $this->_checkFiles( $this->_aRequirements['files'] )
+            $this->_checkFunctions( $this->_aRequirements[ 'functions' ] ),
+            $this->_checkClasses( $this->_aRequirements[ 'classes' ] ),
+            $this->_checkConstants( $this->_aRequirements[ 'constants' ] ),
+            $this->_checkFiles( $this->_aRequirements[ 'files' ] )
         );
         
         $this->aWarnings = array_filter( $_aWarnings ); // drop empty elements.
@@ -197,7 +197,7 @@ 
          * @return      string      The warning.
          */
         private function _getWarningByType( $sType ) {
-            if ( ! isset( $this->_aRequirements[ $sType ][ 'version' ] ) ) {
+            if ( !isset( $this->_aRequirements[ $sType ][ 'version' ] ) ) {
                 return '';
             }
             if ( $this->_checkPHPVersion( $this->_aRequirements[ $sType ][ 'version' ] ) ) {
@@ -226,7 +226,7 @@ 
          * @internal
          */
         private function _checkWordPressVersion( $sWordPressVersion ) {
-            return version_compare( $GLOBALS['wp_version'], $sWordPressVersion, ">=" );
+            return version_compare( $GLOBALS[ 'wp_version' ], $sWordPressVersion, ">=" );
         }
         
         /**
@@ -298,9 +298,9 @@ 
              */
             private function _getWarningsByFunctionName( $sFuncName, $aSubjects ) {
                 $_aWarnings = array();
-                foreach( $aSubjects as $_sSubject => $_sWarning ) {
-                    if ( ! call_user_func_array( $sFuncName, array( $_sSubject ) ) ) {
-                        $_aWarnings[] = sprintf( $_sWarning, $_sSubject );
+                foreach ( $aSubjects as $_sSubject => $_sWarning ) {
+                    if ( !call_user_func_array( $sFuncName, array( $_sSubject ) ) ) {
+                        $_aWarnings[ ] = sprintf( $_sWarning, $_sSubject );
                     }
                 }
                 return $_aWarnings;                
@@ -322,7 +322,7 @@ 
          */    
         public function _replyToPrintAdminNotices() {
             
-            $_aWarnings     = array_unique( $this->aWarnings );
+            $_aWarnings = array_unique( $this->aWarnings );
             if ( empty( $_aWarnings ) ) {
                 return;
             }
@@ -341,12 +341,12 @@ 
              */
             private function _getWarnings() {
 
-                $_aWarnings     = array_unique( $this->aWarnings );            
+                $_aWarnings = array_unique( $this->aWarnings );            
                 if ( empty( $_aWarnings ) ) {
                     return '';
                 }        
-                $_sScripTitle   = $this->_sScriptName 
-                    ?  "<strong>" . $this->_sScriptName . "</strong>:&nbsp;" 
+                $_sScripTitle = $this->_sScriptName 
+                    ? "<strong>".$this->_sScriptName."</strong>:&nbsp;" 
                     : '';            
                 return $_sScripTitle
                    . implode( '<br />', $_aWarnings );
@@ -361,12 +361,12 @@ 
      * @param       boolean     $bIsOnActivation    Whether it is called upon plugin activation hook. 
      * @return      void
      */
-    public function deactivatePlugin( $sPluginFilePath, $sMessage='', $bIsOnActivation=false ) {
+    public function deactivatePlugin( $sPluginFilePath, $sMessage = '', $bIsOnActivation = false ) {
         
         add_action( 'admin_notices', array( $this, '_replyToPrintAdminNotices' ) );
-        $this->aWarnings[] = '<strong>' . $sMessage . '</strong>';
-        if ( ! function_exists( 'deactivate_plugins' ) ) {
-            if ( ! @include( ABSPATH . '/wp-admin/includes/plugin.php' ) ) {
+        $this->aWarnings[ ] = '<strong>'.$sMessage.'</strong>';
+        if ( !function_exists( 'deactivate_plugins' ) ) {
+            if ( !@include( ABSPATH.'/wp-admin/includes/plugin.php' ) ) {
                 return;
             }
         }
@@ -376,8 +376,8 @@ 
         // Before that, we can display messages to the user.
         if ( $bIsOnActivation ) {
             
-            $_sPluginListingPage = add_query_arg( array(), $GLOBALS['pagenow'] );
-            wp_die( $this->_getWarnings() . "<p><a href='$_sPluginListingPage'>Go back</a>.</p>" );
+            $_sPluginListingPage = add_query_arg( array(), $GLOBALS[ 'pagenow' ] );
+            wp_die( $this->_getWarnings()."<p><a href='$_sPluginListingPage'>Go back</a>.</p>" );
             
         }
         

development/utility/zip/AdminPageFramework_Zip.php

Indentation +1 added lines, -1 removed lines

@@ -248,7 +248,7 @@
                         in_array( 
                             substr( $_sIterationItem, strrpos( $_sIterationItem, '/' ) + 1 ), 
                             array( '.', '..' ) 
-                       )
+                        )
                     ) {
                         return;
                     }

Braces +1 added lines, -1 removed lines

@@ -148,7 +148,7 @@
             
             $_sArchiveRootDirName = '';
            
-            if ( $bIncludeDir ) {                
+            if ( $bIncludeDir ) {
                 $_sArchiveRootDirName = $this->_getMainDirectoryName( $sSourceDirPath );
                 $this->_addEmptyDir( 
                     $oZip, 

Spacing +27 added lines, -27 removed lines

@@ -55,8 +55,8 @@ 
      * 
      * @since       3.6.0
      */
-    public $aOptions   = array(
-        'include_directory'             => false,   // (boolean) whether the contents should be put inside a root directory.
+    public $aOptions = array(
+        'include_directory'             => false, // (boolean) whether the contents should be put inside a root directory.
         'additional_source_directories' => array(),
         // 'ignoring_file_extensions'      => array(), // not implemented yet.
     );
@@ -70,7 +70,7 @@ 
      * @param       array|boolean       $abOptions
      * @param       callable            $aCallbacks
      */
-    public function __construct( $sSource, $sDestination, $abOptions=false, array $aCallbacks=array() ) {
+    public function __construct( $sSource, $sDestination, $abOptions = false, array $aCallbacks = array() ) {
         
         $this->sSource      = $sSource;
         $this->sDestination = $sDestination;
@@ -103,7 +103,7 @@ 
     public function compress() {
 
         // Check whether it is possible to perform the task.
-        if ( ! $this->isFeasible( $this->sSource ) ) {
+        if ( !$this->isFeasible( $this->sSource ) ) {
             return false;
         }
         
@@ -113,19 +113,19 @@ 
         }
 
         $_oZip = new ZipArchive();
-        if ( ! $_oZip->open( $this->sDestination, ZIPARCHIVE::CREATE ) ) {
+        if ( !$_oZip->open( $this->sDestination, ZIPARCHIVE::CREATE ) ) {
             return false;
         }
         
         $this->sSource = $this->_getSanitizedSourcePath( $this->sSource );
         // $this->sSource = str_replace( '\\', '/', realpath( $this->sSource ) );
 
-        $_aMethods      = array(
+        $_aMethods = array(
             'unknown'   => '_replyToReturnFalse',
             'directory' => '_replyToCompressDirectory',
             'file'      => '_replyToCompressFile',
         );
-        $_sMethodName   = $_aMethods[ $this->_getSourceType( $this->sSource ) ];
+        $_sMethodName = $_aMethods[ $this->_getSourceType( $this->sSource ) ];
         return call_user_func_array(
             array( $this, $_sMethodName ),
             array(
@@ -153,7 +153,7 @@ 
          * @return      boolean     True on success, false otherwise.
          * @internal
          */
-        public function _replyToCompressDirectory( ZipArchive $oZip, $sSourceDirPath, array $aCallbacks=array(), $bIncludeDir=false, array $aAdditionalSourceDirs=array() ) {
+        public function _replyToCompressDirectory( ZipArchive $oZip, $sSourceDirPath, array $aCallbacks = array(), $bIncludeDir = false, array $aAdditionalSourceDirs = array() ) {
             
             $_sArchiveRootDirName = '';
            
@@ -184,11 +184,11 @@ 
              * @return      void
              * @internal
              */
-            private function _addArchiveItems( $oZip, $aSourceDirPaths, $aCallbacks, $sRootDirName='' ) {
+            private function _addArchiveItems( $oZip, $aSourceDirPaths, $aCallbacks, $sRootDirName = '' ) {
                 
-                $sRootDirName = $sRootDirName ? rtrim( $sRootDirName, '/' ) . '/' : '';
+                $sRootDirName = $sRootDirName ? rtrim( $sRootDirName, '/' ).'/' : '';
                 
-                foreach( $aSourceDirPaths as $_isIndexOrRelativeDirPath => $_sSourceDirPath ) {
+                foreach ( $aSourceDirPaths as $_isIndexOrRelativeDirPath => $_sSourceDirPath ) {
                     
                     $_sSourceDirPath   = $this->_getSanitizedSourcePath( $_sSourceDirPath );
                     $_sInsideDirPrefix = is_integer( $_isIndexOrRelativeDirPath )
@@ -196,7 +196,7 @@ 
                         : $_isIndexOrRelativeDirPath;
                         
                     // Add a directory inside the compressing directory.
-                    if( $_sInsideDirPrefix ) {
+                    if ( $_sInsideDirPrefix ) {
                         $this->_addRelativeDir(
                             $oZip,
                             $_sInsideDirPrefix,
@@ -214,7 +214,7 @@ 
                             $_sSourceDirPath,
                             $_sIterationItem, 
                             $aCallbacks,
-                            $sRootDirName . $_sInsideDirPrefix
+                            $sRootDirName.$_sInsideDirPrefix
                         );
                     }                                
                 }
@@ -229,8 +229,8 @@ 
                     $sRelativeDirPath = str_replace( '\\', '/', $sRelativeDirPath );
                     $_aPathPartsParse = array_filter( explode( '/', $sRelativeDirPath ) );
                     $_aDirPath        = array();
-                    foreach( $_aPathPartsParse as $_sDirName ) {
-                        $_aDirPath[] = $_sDirName;
+                    foreach ( $_aPathPartsParse as $_sDirName ) {
+                        $_aDirPath[ ] = $_sDirName;
                         $this->_addEmptyDir( 
                             $oZip, 
                             implode( '/', $_aDirPath ),
@@ -250,10 +250,10 @@ 
                  * @return      void
                  * @internal
                  */
-                private function _addArchiveItem( ZipArchive $oZip, $sSource, $_sIterationItem, array $aCallbacks, $sInsidePathPrefix='' ) {
+                private function _addArchiveItem( ZipArchive $oZip, $sSource, $_sIterationItem, array $aCallbacks, $sInsidePathPrefix = '' ) {
                     
                     $_sIterationItem   = str_replace( '\\', '/', $_sIterationItem );
-                    $sInsidePathPrefix = rtrim( $sInsidePathPrefix, '/' ) . '/'; // add a trailing slash
+                    $sInsidePathPrefix = rtrim( $sInsidePathPrefix, '/' ).'/'; // add a trailing slash
                     
                     // Ignore "." and ".." folders
                     if ( 
@@ -271,18 +271,18 @@ 
                     if ( true === is_dir( $_sIterationItem ) ) {
                         $this->_addEmptyDir( 
                             $oZip, 
-                            $sInsidePathPrefix . str_replace( 
-                                $sSource . '/', 
+                            $sInsidePathPrefix.str_replace( 
+                                $sSource.'/', 
                                 '', 
-                                $_sIterationItem . '/'
+                                $_sIterationItem.'/'
                             ), 
                             $aCallbacks[ 'directory_name' ]
                         );                    
                     } else if ( true === is_file( $_sIterationItem ) ) {
                         $this->_addFromString( 
                             $oZip, 
-                            $sInsidePathPrefix . str_replace(
-                                $sSource . '/', 
+                            $sInsidePathPrefix.str_replace(
+                                $sSource.'/', 
                                 '', 
                                 $_sIterationItem
                             ),
@@ -312,7 +312,7 @@ 
          * @return      boolean     True on success, false otherwise.
          * @internal
          */
-        public function _replyToCompressFile( ZipArchive $oZip, $sSourceFilePath, $aCallbacks=null ) {
+        public function _replyToCompressFile( ZipArchive $oZip, $sSourceFilePath, $aCallbacks = null ) {
             $this->_addFromString( 
                 $oZip, 
                 basename( $sSourceFilePath ), 
@@ -345,7 +345,7 @@ 
      * @internal
      */
     private function isFeasible( $sSource ) {
-        if ( ! extension_loaded( 'zip' ) ) {
+        if ( !extension_loaded( 'zip' ) ) {
             return false;
         }
         return file_exists( $sSource );
@@ -371,7 +371,7 @@ 
      */
     private function _addEmptyDir( ZipArchive $oZip, $sInsidePath, $oCallable ) {
         $sInsidePath = $this->_getFilteredArchivePath( $sInsidePath, $oCallable );
-        if ( ! strlen( $sInsidePath ) ) {
+        if ( !strlen( $sInsidePath ) ) {
             return;
         }
         $oZip->addEmptyDir( ltrim( $sInsidePath, '/' ) );        
@@ -384,10 +384,10 @@ 
      * @return      void
      * @internal
      */
-    private function _addFromString( ZipArchive $oZip, $sInsidePath, $sSourceContents='', array $aCallbacks=array() ) {
+    private function _addFromString( ZipArchive $oZip, $sInsidePath, $sSourceContents = '', array $aCallbacks = array() ) {
         
         $sInsidePath = $this->_getFilteredArchivePath( $sInsidePath, $aCallbacks[ 'file_name' ] );
-        if ( ! strlen( $sInsidePath ) ) {
+        if ( !strlen( $sInsidePath ) ) {
             return;
         }
         $oZip->addFromString(

development/factory/page_meta_box/AdminPageFramework_PageMetaBox.php

Spacing +4 added lines, -4 removed lines

@@ -77,21 +77,21 @@
      * @param       string          $sPriority      The priority, either `high`, `core`, `default` or `low`.
      * @param       string          $sCapability    The capability. See <a href="https://codex.wordpress.org/Roles_and_Capabilities" target="_blank">Roles and Capabilities</a>.
      */
-    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' ) {
         
         if ( empty( $asPageSlugs ) ) { 
             return; 
         }
         
-        if ( ! $this->_isInstantiatable() ) { 
+        if ( !$this->_isInstantiatable() ) { 
             return; 
         }
                 
         // The property object needs to be done first before the parent constructor.
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;        
-        $this->oProp             = new $_sProprtyClassName( 
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;        
+        $this->oProp = new $_sProprtyClassName( 
             $this, 
             get_class( $this ), 
             $sCapability, 

Braces +2 added lines, -2 removed lines

@@ -78,11 +78,11 @@
      */
     public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
         
-        if ( empty( $asPageSlugs ) ) { 
+        if ( empty( $asPageSlugs ) ) {
             return; 
         }
         
-        if ( ! $this->_isInstantiatable() ) { 
+        if ( ! $this->_isInstantiatable() ) {
             return; 
         }
                 

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param       string          The text domain. Default: `admin-page-framework`.
      * @return      void
      */ 
-    function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    function __construct( $asTaxonomySlug, $sOptionKey = '', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
         
         if ( empty( $asTaxonomySlug ) ) { 
             return; 
@@ -55,7 +55,7 @@ 
         // Properties 
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;        
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;        
         $this->oProp        = new $_sProprtyClassName( 
             $this, 
             get_class( $this ), 

Braces +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      */ 
     function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
         
-        if ( empty( $asTaxonomySlug ) ) { 
+        if ( empty( $asTaxonomySlug ) ) {
             return; 
         }
         

development/factory/user_meta/AdminPageFramework_UserMeta.php

Spacing +6 added lines, -6 removed lines

@@ -39,16 +39,16 @@
      * @since       3.7.4       Changed the default capability value to `read`.
      * @todo        Examine the appropriate default capability level.
      */
-    public function __construct( $sCapability='read', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sCapability = 'read', $sTextDomain = 'admin-page-framework' ) {
         
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;        
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;        
         $this->oProp = new $_sProprtyClassName( 
-            $this,                  // the caller object
-            get_class( $this ),     // the caller class name    
-            $sCapability,           // the capability level
-            $sTextDomain,           // the text domain
+            $this, // the caller object
+            get_class( $this ), // the caller class name    
+            $sCapability, // the capability level
+            $sTextDomain, // the text domain
             $this->_sStructureType  // the structure type
         );     
         

development/factory/_common/form/field_type/AdminPageFramework_FieldType_media.php

Spacing +19 added lines, -19 removed lines

@@ -21,7 +21,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'media', );
+    public $aFieldTypeSlugs = array( 'media',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -53,7 +53,7 @@ 
         return 
             $this->_getScript_MediaUploader(
                 "admin_page_framework"
-            ) . PHP_EOL
+            ).PHP_EOL
             . $this->_getScript_RegisterCallbacks();
     }    
     
@@ -148,7 +148,7 @@ 
             $_sInsertFromURL         = esc_js( $this->oMsg->get( 'insert_from_url' ) );
             
             // If the WordPress version is 3.4.x or below
-            if ( ! function_exists( 'wp_enqueue_media' ) ) {
+            if ( !function_exists( 'wp_enqueue_media' ) ) {
                 return <<<JAVASCRIPTS
                     /**
                      * Bind/rebinds the thickbox script the given selector element.
@@ -388,10 +388,10 @@ 
             // Do not include the escaping character (backslash) in the heredoc variable declaration 
             // because the minifier script will parse it and the <<<JAVASCRIPTS and JAVASCRIPTS; parts are converted to double quotes (")
             // which causes the PHP syntax error.
-            $_sButtonHTML       = '"' . $this->_getUploaderButtonHTML_Media( $sInputID, $aButtonAttributes, $bExternalSource ) . '"';
-            $_sRpeatable        = $this->getAOrB( ! empty( $abRepeatable ), 'true', 'false' );
+            $_sButtonHTML       = '"'.$this->_getUploaderButtonHTML_Media( $sInputID, $aButtonAttributes, $bExternalSource ).'"';
+            $_sRpeatable        = $this->getAOrB( !empty( $abRepeatable ), 'true', 'false' );
             $_sExternalSource   = $this->getAOrB( $bExternalSource, 'true', 'false' );
-            $_sScript                = <<<JAVASCRIPTS
+            $_sScript = <<<JAVASCRIPTS
 if ( jQuery( 'a#select_media_{$sInputID}' ).length == 0 ) {
     jQuery( 'input#{$sInputID}' ).after( $_sButtonHTML );
 }
@@ -404,7 +404,7 @@ 
                     . '/* <![CDATA[ */'
                     . $_sScript 
                     . '/* ]]> */'
-                . "</script>". PHP_EOL;
+                . "</script>".PHP_EOL;
 
         }
             /**
@@ -414,19 +414,19 @@ 
              */
             private function _getUploaderButtonHTML_Media( $sInputID, array $aButtonAttributes, $bExternalSource ) {
                     
-                $_bIsLabelSet = isset( $aButtonAttributes['data-label'] ) && $aButtonAttributes['data-label'];
+                $_bIsLabelSet = isset( $aButtonAttributes[ 'data-label' ] ) && $aButtonAttributes[ 'data-label' ];
                 $_aAttributes = $this->_getFormattedUploadButtonAttributes_Media( 
                     $sInputID, 
                     $aButtonAttributes, 
                     $_bIsLabelSet, 
                     $bExternalSource 
                 );
-                return "<a " . $this->getAttributes( $_aAttributes ) . ">"
+                return "<a ".$this->getAttributes( $_aAttributes ).">"
                         . $this->getAOrB( 
                             $_bIsLabelSet,
-                            $_aAttributes['data-label'],
+                            $_aAttributes[ 'data-label' ],
                             $this->getAOrB(
-                                strrpos( $_aAttributes['class'], 'dashicons' ),
+                                strrpos( $_aAttributes[ 'class' ], 'dashicons' ),
                                 '',
                                 $this->oMsg->get( 'select_file' )
                             )
@@ -441,26 +441,26 @@ 
                  */
                 private function _getFormattedUploadButtonAttributes_Media( $sInputID, array $aButtonAttributes, $_bIsLabelSet, $bExternalSource ) {
                    
-                    $_aAttributes           = array(
+                    $_aAttributes = array(
                             'id'        => "select_media_{$sInputID}",
                             'href'      => '#',            
-                            'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ),    //  ? 1 : 0,
-                            'data-enable_external_source'   => ( string ) ( bool ) $bExternalSource,    //  ? 1 : 0, 
+                            'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ), //  ? 1 : 0,
+                            'data-enable_external_source'   => ( string ) ( bool ) $bExternalSource, //  ? 1 : 0, 
                         ) 
                         + $aButtonAttributes
                         + array(
                             'title'     => $_bIsLabelSet 
-                                ? $aButtonAttributes['data-label'] 
+                                ? $aButtonAttributes[ 'data-label' ] 
                                 : $this->oMsg->get( 'select_file' ),
                             'data-label' => null,
                         );
-                    $_aAttributes['class']  = $this->getClassAttribute( 
+                    $_aAttributes[ 'class' ] = $this->getClassAttribute( 
                         'select_media button button-small ',
                         $this->getAOrB(
-                            trim( $aButtonAttributes['class'] ),
-                            $aButtonAttributes['class'],
+                            trim( $aButtonAttributes[ 'class' ] ),
+                            $aButtonAttributes[ 'class' ],
                             $this->getAOrB( 
-                                ! $_bIsLabelSet && version_compare( $GLOBALS['wp_version'], '3.8', '>=' ),
+                                !$_bIsLabelSet && version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' ),
                                 'dashicons dashicons-portfolio',
                                 ''
                             )

admin-page-framework-loader.php

Braces +1 added lines, -1 removed lines

@@ -190,7 +190,7 @@
      * @return      void
      */ 
     static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {  
+        if ( ! is_admin() ) {
             return; 
         }
         self::$_aAdminNotices[] = array(

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.17b01';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.17b01'; // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
     const NAME           = 'Admin Page Framework - Loader'; // the name is not 'Admin Page Framework' because warning messages gets confusing.
-    const SHORTNAME      = 'Admin Page Framework';  // used for a menu title etc.
+    const SHORTNAME      = 'Admin Page Framework'; // used for a menu title etc.
     const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
     const URI            = 'http://admin-page-framework.michaeluno.jp/';
     const AUTHOR         = 'miunosoft (Michael Uno)';
@@ -55,14 +55,14 @@ 
      * @remark      This is also accessed from `uninstall.php` so do not remove.
      * @remark      Do not exceed 8 characters as a transient name allows 45 characters or less ( 40 for site transients ) so that md5 (32 characters) can be added.
      */
-    const TRANSIENT_PREFIX         = 'APFL_';
+    const TRANSIENT_PREFIX = 'APFL_';
 
     /**
      * The hook slug used for the prefix of action and filter hook names.
      * 
      * @remark      The ending underscore is not necessary.
      */
-    const HOOK_SLUG                = 'admin_page_framework_loader';
+    const HOOK_SLUG = 'admin_page_framework_loader';
 
     /**
      * The text domain slug and its path.
@@ -116,7 +116,7 @@ 
      */
     static public $aAdminPages = array(
         // key => 'page slug'
-        'about'     => 'apfl_about',        // the welcome page
+        'about'     => 'apfl_about', // the welcome page
         'addon'     => 'apfl_addons',
         'tool'      => 'apfl_tools',
         'help'      => 'apfl_contact',
@@ -154,12 +154,12 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public static function getPluginURL( $sRelativePath='' ) {
+    public static function getPluginURL( $sRelativePath = '' ) {
         if ( isset( self::$_sPluginURLCache ) ) {
-            return self::$_sPluginURLCache . $sRelativePath;
+            return self::$_sPluginURLCache.$sRelativePath;
         }
         self::$_sPluginURLCache = trailingslashit( plugins_url( '', self::$sFilePath ) );
-        return self::$_sPluginURLCache . $sRelativePath;
+        return self::$_sPluginURLCache.$sRelativePath;
     }
         /**
          * @since       3.7.9
@@ -189,13 +189,13 @@ 
      * @since       3.5.0
      * @return      void
      */ 
-    static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {  
+    static public function setAdminNotice( $sMessage, $sClassAttribute = 'error' ) {
+        if ( !is_admin() ) {  
             return; 
         }
-        self::$_aAdminNotices[] = array(
+        self::$_aAdminNotices[ ] = array(
             'message'           => $sMessage,
-            'class_attribute'   => trim( $sClassAttribute ) . ' notice is-dismissible',
+            'class_attribute'   => trim( $sClassAttribute ).' notice is-dismissible',
         );
         add_action( 'admin_notices', array( __CLASS__, '_replyToSetAdminNotice' ) );
     }
@@ -205,12 +205,12 @@ 
          * @return      void
          */
         static public function _replyToSetAdminNotice() {
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
-                echo "<div class='" . esc_attr( $_aAdminNotice['class_attribute'] ) . " notice is-dismissible'>"
+            foreach ( self::$_aAdminNotices as $_aAdminNotice ) {
+                echo "<div class='".esc_attr( $_aAdminNotice[ 'class_attribute' ] )." notice is-dismissible'>"
                         ."<p>"
                             . sprintf(
-                                '<strong>%1$s</strong>: ' . $_aAdminNotice['message'],
-                                self::NAME . ' ' . self::VERSION
+                                '<strong>%1$s</strong>: '.$_aAdminNotice[ 'message' ],
+                                self::NAME.' '.self::VERSION
                             )
                         . "</p>"
                     . "</div>";
@@ -222,7 +222,7 @@ 
 AdminPageFrameworkLoader_Registry::setUp( __FILE__ );
 
 // Initial checks. - Do no load if accessed directly, not exiting because the 'uninstall.php' and inclusion list generator will load this file.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( !defined( 'ABSPATH' ) ) {
     return;
 }
 if ( defined( 'DOING_UNINSTALL' ) && DOING_UNINSTALL ) {
@@ -234,8 +234,8 @@ 
 
     $_bFrameworkLoaded = class_exists( 'AdminPageFramework_Registry', false );
     if (
-        ! $_bFrameworkLoaded
-        || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
+        !$_bFrameworkLoaded
+        || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
         || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
     ) {
         AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -253,18 +253,18 @@ 
 add_action( 'admin_init', 'AdminPageFrameworkLoader_Warning' );
 
 // Include the library file - the development version will be available if you cloned the GitHub repository.
-$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/development/admin-page-framework.php';
+$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/development/admin-page-framework.php';
 $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
 $_bLoadDevelopmentVersion = $_bDebugMode && file_exists( $_sDevelopmentVersionPath );
 include(
     $_bLoadDevelopmentVersion
         ? $_sDevelopmentVersionPath
-        : AdminPageFrameworkLoader_Registry::$sDirPath . '/library/apf/admin-page-framework.php'
+        : AdminPageFrameworkLoader_Registry::$sDirPath.'/library/apf/admin-page-framework.php'
 );
 
 // Include the framework loader plugin components.
 include( AdminPageFramework_Registry::$aClassFiles[ 'AdminPageFramework_PluginBootstrap' ] );
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
 new AdminPageFrameworkLoader_Bootstrap(
     AdminPageFrameworkLoader_Registry::$sFilePath,
     AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Router.php

Braces +1 added lines, -1 removed lines

@@ -80,7 +80,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 );

Spacing +5 added lines, -5 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' )
         );
         
@@ -58,11 +58,11 @@ 
             return true;
         }
         
-        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;
         }        
         
@@ -79,7 +79,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/post_type/AdminPageFramework_PostType.php

Spacing +7 added lines, -7 removed lines

@@ -104,7 +104,7 @@ 
     * @param        string      The text domain of the caller script.
     * @return       void
     */
-    public function __construct( $sPostType, $aArguments=array(), $sCallerPath=null, $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sPostType, $aArguments = array(), $sCallerPath = null, $sTextDomain = 'admin-page-framework' ) {
         
         if ( empty( $sPostType ) ) { 
             return; 
@@ -112,16 +112,16 @@ 
 
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;           
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;           
         $this->oProp = new $_sProprtyClassName( 
             $this, 
             $this->_getCallerScriptPath( $sCallerPath ),
-            get_class( $this ),     // class name
-            'publish_posts',        // capability
-            $sTextDomain,           // text domain
+            get_class( $this ), // class name
+            'publish_posts', // capability
+            $sTextDomain, // text domain
             $this->_sStructureType  // structure type
         );
-        $this->oProp->sPostType     = AdminPageFramework_WPUtility::sanitizeSlug( $sPostType );
+        $this->oProp->sPostType = AdminPageFramework_WPUtility::sanitizeSlug( $sPostType );
         
         // Post type argument array structure
         // @see http://codex.wordpress.org/Function_Reference/register_post_type#Arguments
@@ -144,7 +144,7 @@ 
                 return $sCallerPath;
             }
             
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return null;
             }
             $_sPageNow = AdminPageFramework_Utility::getElement( $GLOBALS, 'pagenow' );

Indentation +77 added lines, -78 removed lines

@@ -25,84 +25,83 @@
     protected $_sStructureType = 'post_type';
     
     /**
-    * The constructor of the class object.
-    * 
-    * Registers necessary hooks and sets up internal properties.
-    * 
-    * <h4>Example</h4>
-    * <code>new APF_PostType( 
-    *     'apf_posts',     // post type slug
-    *       array( 
-    *           'labels' => array(
-    *               'name'               => 'Demo',
-    *               'all_items'          => __( 'Sample Posts', 'admin-page-framework-demo' ),
-    *               'singular_name'      => 'Demo',
-    *               'add_new'            => __( 'Add New', 'admin-page-framework-demo' ),
-    *               'add_new_item'       => __( 'Add New APF Post', 'admin-page-framework-demo' ),
-    *               'edit'               => __( 'Edit', 'admin-page-framework-demo' ),
-    *               'edit_item'          => __( 'Edit APF Post', 'admin-page-framework-demo' ),
-    *               'new_item'           => __( 'New APF Post', 'admin-page-framework-demo' ),
-    *               'view'               => __( 'View', 'admin-page-framework-demo' ),
-    *               'view_item'          => __( 'View APF Post', 'admin-page-framework-demo' ),
-    *               'search_items'       => __( 'Search APF Post', 'admin-page-framework-demo' ),
-    *               'not_found'          => __( 'No APF Post found', 'admin-page-framework-demo' ),
-    *               'not_found_in_trash' => __( 'No APF Post found in Trash', 'admin-page-framework-demo' ),
-    *               'parent'             => __( 'Parent APF Post', 'admin-page-framework-demo' ),
-    *               
-    *               // (framework specific)
-    *               'plugin_action_link' => __( 'APF Posts', 'admin-page-framework-demo' ), // framework specific key. [3.7.3+]
-    *           ),
-    *           'public'            => true,
-    *           'menu_position'     => 110,
-    *           'supports'          => array( 'title' ), // e.g. array( 'title', 'editor', 'comments', 'thumbnail', 'excerpt' ),    
-    *           'taxonomies'        => array( '' ),
-    *           'has_archive'       => true,
-    *           'show_admin_column' => true, // [3.5+ core] this is for custom taxonomies to automatically add the column in the listing table.
-    *           'menu_icon'         => $this->oProp->bIsAdmin 
-    *               ? ( 
-    *                   version_compare( $GLOBALS['wp_version'], '3.8', '>=' ) 
-    *                       ? 'dashicons-wordpress' 
-    *                       : plugins_url( 'asset/image/wp-logo_16x16.png', APFDEMO_FILE ) 
-    *               )
-    *               : null, // do not call the function in the front-end.
-    *               
-    *           // (framework specific) this sets the screen icon for the post type for WordPress v3.7.1 or below.
-    *           // a file path can be passed instead of a url, plugins_url( 'asset/image/wp-logo_32x32.png', APFDEMO_FILE )
-    *           'screen_icon' => dirname( APFDEMO_FILE  ) . '/asset/image/wp-logo_32x32.png', 
-    *           
-    *           // [3.5.10+] (framework specific) default: true
-    *           'show_submenu_add_new'  => true, 
-    *           
-    *           // [3.7.4+] (framework specific) default: 10
-    *           'submenu_order_manage'  => 20,
-    *           'submenu_order_addnew'  => 21,
-    * 
-    *       )     
-    * );</code>
-    * 
-    * <h4>Framework Specific Post Type Arguments</h4>
-    * In addition to the post type argument structure defined by the WordPress core, there are arguments defined by the framework.
-    * 
-    * - screen_icon - For WordPress 3.7.x or below, set an icon url or path for the 32x32 screen icon displayed in the post listing page.
-    * - show_submenu_add_new - [3.5.10+]    (boolean) Whether the sub-menu item of `Add New` should be displayed.
-    * - submenu_order_manage - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `5`
-    * - submenu_order_addnew - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `10`
-    *
-    * <h4>Framework Specific Post Type Label Arguments</h4>
-    * - plugin_listing_table_title_cell_link' - [3.0.6+] Deprecated [3.7.3] use the `plugin_action_link` argument instead.
-    * - plugin_action_link' - [3.7.3+] If the caller script is a plugin, this determines the label of the action link embedded in the plugin listing page (plugins.php).
-    * To disable the action link, set an empty string `''`. 
-
-    * 
-    * @since        2.0.0
-    * @since        2.1.6       Added the $sTextDomain parameter.
-    * @see          http://codex.wordpress.org/Function_Reference/register_post_type#Arguments
-    * @param        string      The post type slug.
-    * @param        array       The <a href="http://codex.wordpress.org/Function_Reference/register_post_type#Arguments">argument array</a> passed to register_post_type().
-    * @param        string      The path of the caller script. This is used to retrieve the script information to insert it into the footer. If not set, the framework tries to detect it.
-    * @param        string      The text domain of the caller script.
-    * @return       void
-    */
+     * The constructor of the class object.
+     * 
+     * Registers necessary hooks and sets up internal properties.
+     * 
+     * <h4>Example</h4>
+     * <code>new APF_PostType( 
+     *     'apf_posts',     // post type slug
+     *       array( 
+     *           'labels' => array(
+     *               'name'               => 'Demo',
+     *               'all_items'          => __( 'Sample Posts', 'admin-page-framework-demo' ),
+     *               'singular_name'      => 'Demo',
+     *               'add_new'            => __( 'Add New', 'admin-page-framework-demo' ),
+     *               'add_new_item'       => __( 'Add New APF Post', 'admin-page-framework-demo' ),
+     *               'edit'               => __( 'Edit', 'admin-page-framework-demo' ),
+     *               'edit_item'          => __( 'Edit APF Post', 'admin-page-framework-demo' ),
+     *               'new_item'           => __( 'New APF Post', 'admin-page-framework-demo' ),
+     *               'view'               => __( 'View', 'admin-page-framework-demo' ),
+     *               'view_item'          => __( 'View APF Post', 'admin-page-framework-demo' ),
+     *               'search_items'       => __( 'Search APF Post', 'admin-page-framework-demo' ),
+     *               'not_found'          => __( 'No APF Post found', 'admin-page-framework-demo' ),
+     *               'not_found_in_trash' => __( 'No APF Post found in Trash', 'admin-page-framework-demo' ),
+     *               'parent'             => __( 'Parent APF Post', 'admin-page-framework-demo' ),
+     *               
+     *               // (framework specific)
+     *               'plugin_action_link' => __( 'APF Posts', 'admin-page-framework-demo' ), // framework specific key. [3.7.3+]
+     *           ),
+     *           'public'            => true,
+     *           'menu_position'     => 110,
+     *           'supports'          => array( 'title' ), // e.g. array( 'title', 'editor', 'comments', 'thumbnail', 'excerpt' ),    
+     *           'taxonomies'        => array( '' ),
+     *           'has_archive'       => true,
+     *           'show_admin_column' => true, // [3.5+ core] this is for custom taxonomies to automatically add the column in the listing table.
+     *           'menu_icon'         => $this->oProp->bIsAdmin 
+     *               ? ( 
+     *                   version_compare( $GLOBALS['wp_version'], '3.8', '>=' ) 
+     *                       ? 'dashicons-wordpress' 
+     *                       : plugins_url( 'asset/image/wp-logo_16x16.png', APFDEMO_FILE ) 
+     *               )
+     *               : null, // do not call the function in the front-end.
+     *               
+     *           // (framework specific) this sets the screen icon for the post type for WordPress v3.7.1 or below.
+     *           // a file path can be passed instead of a url, plugins_url( 'asset/image/wp-logo_32x32.png', APFDEMO_FILE )
+     *           'screen_icon' => dirname( APFDEMO_FILE  ) . '/asset/image/wp-logo_32x32.png', 
+     *           
+     *           // [3.5.10+] (framework specific) default: true
+     *           'show_submenu_add_new'  => true, 
+     *           
+     *           // [3.7.4+] (framework specific) default: 10
+     *           'submenu_order_manage'  => 20,
+     *           'submenu_order_addnew'  => 21,
+     * 
+     *       )     
+     * );</code>
+     * 
+     * <h4>Framework Specific Post Type Arguments</h4>
+     * In addition to the post type argument structure defined by the WordPress core, there are arguments defined by the framework.
+     * 
+     * - screen_icon - For WordPress 3.7.x or below, set an icon url or path for the 32x32 screen icon displayed in the post listing page.
+     * - show_submenu_add_new - [3.5.10+]    (boolean) Whether the sub-menu item of `Add New` should be displayed.
+     * - submenu_order_manage - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `5`
+     * - submenu_order_addnew - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `10`
+     *
+     * <h4>Framework Specific Post Type Label Arguments</h4>
+     * - plugin_listing_table_title_cell_link' - [3.0.6+] Deprecated [3.7.3] use the `plugin_action_link` argument instead.
+     * - plugin_action_link' - [3.7.3+] If the caller script is a plugin, this determines the label of the action link embedded in the plugin listing page (plugins.php).
+     * To disable the action link, set an empty string `''`. 
+     * 
+     * @since        2.0.0
+     * @since        2.1.6       Added the $sTextDomain parameter.
+     * @see          http://codex.wordpress.org/Function_Reference/register_post_type#Arguments
+     * @param        string      The post type slug.
+     * @param        array       The <a href="http://codex.wordpress.org/Function_Reference/register_post_type#Arguments">argument array</a> passed to register_post_type().
+     * @param        string      The path of the caller script. This is used to retrieve the script information to insert it into the footer. If not set, the framework tries to detect it.
+     * @param        string      The text domain of the caller script.
+     * @return       void
+     */
     public function __construct( $sPostType, $aArguments=array(), $sCallerPath=null, $sTextDomain='admin-page-framework' ) {
         
         if ( empty( $sPostType ) ) { 

Braces +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
  * @since           2.0.0
  * @package         AdminPageFramework/Factory/PostType
  */
-abstract class AdminPageFramework_PostType extends AdminPageFramework_PostType_Controller {    
+abstract class AdminPageFramework_PostType extends AdminPageFramework_PostType_Controller {
       
     /**
      * Defines the class object structure type.
@@ -105,7 +105,7 @@ 
     */
     public function __construct( $sPostType, $aArguments=array(), $sCallerPath=null, $sTextDomain='admin-page-framework' ) {
         
-        if ( empty( $sPostType ) ) { 
+        if ( empty( $sPostType ) ) {
             return; 
         }