michaeluno / admin-page-framework

development/factory/post_type/_view/AdminPageFramework_PageLoadInfo_post_type.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
         if ( in_array( $oProp->sClassName, self::$aClassNames ) )
             return self::$_oInstance;
         
-        self::$aClassNames[] = $oProp->sClassName;
+        self::$aClassNames[ ] = $oProp->sClassName;
         self::$_oInstance = new AdminPageFramework_PageLoadInfo_PostType( $oProp, $oMsg );
         
         return self::$_oInstance;
@@ -44,7 +44,7 @@ 
     public function _replyToSetPageLoadInfoInFooter() {
 
         // Some users sets $_GET['post_type'] element even in regular admin pages. In that case, do not load the style to avoid duplicates.
-        if ( isset( $_GET['page'] ) && $_GET['page'] ) { return; }
+        if ( isset( $_GET[ 'page' ] ) && $_GET[ 'page' ] ) { return; }
     
         // For post type pages
         if ( 

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
      */
     public static function instantiate( $oProp, $oMsg ) {
         
-        if ( in_array( $oProp->sClassName, self::$aClassNames ) )
-            return self::$_oInstance;
+        if ( in_array( $oProp->sClassName, self::$aClassNames ) ) {
+                    return self::$_oInstance;
+        }
         
         self::$aClassNames[] = $oProp->sClassName;
         self::$_oInstance = new AdminPageFramework_PageLoadInfo_PostType( $oProp, $oMsg );

development/factory/post_type/AdminPageFramework_PostType_Router.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
      */
     public function _replyToDetermineToLoadAdmin( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) {
+        if ( !$this->_isInThePage() ) {
             return;
         }
         
@@ -115,7 +115,7 @@ 
     public function _isInThePage() {
         
         // If it's not in one of the post type's pages
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
@@ -124,7 +124,7 @@ 
             return true;
         }        
         
-        if ( ! in_array( $this->oProp->sPageNow, array( 'edit.php', 'edit-tags.php', 'post.php', 'post-new.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'edit.php', 'edit-tags.php', 'post.php', 'post-new.php' ) ) ) {
             return false;
         }
                 

development/cli/AdminPageFramework_MinifiedVersionHeader.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * If accessed from a console, include the registry class to laod 'AdminPageFramework_Registry_Base'.
  */
 if ( php_sapi_name() === 'cli' ) {
-    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ) . '/admin-page-framework.php';
+    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ).'/admin-page-framework.php';
     if ( file_exists( $_sFrameworkFilePath ) ) {
         include_once( $_sFrameworkFilePath );
     }

development/document/AdminPageFramework_NetworkAdmin_Documentation.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * Copyright (c) 2013-2016 Michael Uno; Licensed MIT
  * 
  */
-if ( ! class_exists( 'AdminPageFramework_NetworkAdmin_Documentation', false ) ) :
+if ( !class_exists( 'AdminPageFramework_NetworkAdmin_Documentation', false ) ) :
 /**
  * Provides an abstract base to create admin pages in the network admin area.
  *

development/utility/toc/AdminPageFramework_TableOfContents.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * Generates an HTML Table of Contents block.
- *    
- * @package      Admin Page Framework
- * @copyright    Copyright (c) 2015, <Michael Uno>
- * @author       Michael Uno
- * @authorurl    http://michaeluno.jp
- */
+     * Generates an HTML Table of Contents block.
+     *    
+     * @package      Admin Page Framework
+     * @copyright    Copyright (c) 2015, <Michael Uno>
+     * @author       Michael Uno
+     * @authorurl    http://michaeluno.jp
+     */
 
 /**
  * Generates an HTML Table of Contents block.

Spacing +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
      * @param       integer     $iDepth     The header number to parse.
      * @param       string      $sTitle     The heading title which appears at the beginning of the output.
      */
-    public function __construct( $sHTML, $iDepth=4, $sTitle='' ) {
+    public function __construct( $sHTML, $iDepth = 4, $sTitle = '' ) {
         
         $this->sTitle   = $sTitle;
         $this->sHTML    = $sHTML;
@@ -64,26 +64,26 @@ 
      */
     public function getTOC() {
         
-        $iDepth     = $this->iDepth;
+        $iDepth = $this->iDepth;
         
         // get the headings down to the specified depth
         $this->sHTML = preg_replace_callback(
-            '/<h[2-' . $iDepth . ']*[^>]*>.*?<\/h[2-' . $iDepth . ']>/i',
+            '/<h[2-'.$iDepth.']*[^>]*>.*?<\/h[2-'.$iDepth.']>/i',
             array( $this, '_replyToInsertNamedElement' ),
             $this->sHTML
         );
         
         $_aOutput = array();
-        foreach( $this->_aMatches as $_iIndex => $_sMatch ) {
+        foreach ( $this->_aMatches as $_iIndex => $_sMatch ) {
             $_sMatch = strip_tags( $_sMatch, '<h1><h2><h3><h4><h5><h6><h7><h8>' );
-            $_sMatch = preg_replace( '/<h([1-' . $iDepth . '])>/', '<li class="toc$1"><a href="#toc_' . $_iIndex . '">', $_sMatch );
-            $_sMatch = preg_replace( '/<\/h[1-' . $iDepth . ']>/', '</a></li>', $_sMatch );
-            $_aOutput[] = $_sMatch;
+            $_sMatch = preg_replace( '/<h([1-'.$iDepth.'])>/', '<li class="toc$1"><a href="#toc_'.$_iIndex.'">', $_sMatch );
+            $_sMatch = preg_replace( '/<\/h[1-'.$iDepth.']>/', '</a></li>', $_sMatch );
+            $_aOutput[ ] = $_sMatch;
         }
 
         // plug the results into appropriate HTML tags
         $this->sTitle = $this->sTitle
-            ? '<p class="toc-title">' . $this->sTitle . '</p>'
+            ? '<p class="toc-title">'.$this->sTitle.'</p>'
             : '';
 
         return '<div class="toc">'
@@ -98,9 +98,9 @@ 
         public function _replyToInsertNamedElement( $aMatches ) {
             static $_icount = -1;
             $_icount++;
-            $this->_aMatches[] = $aMatches[ 0 ];
+            $this->_aMatches[ ] = $aMatches[ 0 ];
 
-            return "<span class='toc_header_link' id='toc_{$_icount}'></span>" . PHP_EOL
+            return "<span class='toc_header_link' id='toc_{$_icount}'></span>".PHP_EOL
                 . $aMatches[ 0 ];
         }
     

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;
                     }

Spacing +27 added lines, -27 removed lines

@@ -50,8 +50,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.
     );
@@ -64,7 +64,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;
@@ -97,7 +97,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;
         }
         
@@ -107,19 +107,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 ),
@@ -146,7 +146,7 @@ 
          * @since       3.6.0       Changed the name from `_compressDirectory`. Changed the scope to public to allow overriding the method in an extended class.
          * @return      boolean     True on success, false otherwise.
          */
-        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 = '';
            
@@ -176,11 +176,11 @@ 
              * @since       3.6.0
              * @return      void
              */
-            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 )
@@ -188,7 +188,7 @@ 
                         : $_isIndexOrRelativeDirPath;
                         
                     // Add a directory inside the compressing directory.
-                    if( $_sInsideDirPrefix ) {
+                    if ( $_sInsideDirPrefix ) {
                         $this->_addRelativeDir(
                             $oZip,
                             $_sInsideDirPrefix,
@@ -206,7 +206,7 @@ 
                             $_sSourceDirPath,
                             $_sIterationItem,
                             $aCallbacks,
-                            $sRootDirName . $_sInsideDirPrefix
+                            $sRootDirName.$_sInsideDirPrefix
                         );
                     }
                 }
@@ -220,8 +220,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 ),
@@ -240,10 +240,10 @@ 
                  * @param       string          $sInsidePathPrefix      The prefix to add to the inside archive directory structure. 
                  * @return      void
                  */
-                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 (
@@ -261,18 +261,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
                             ),
@@ -301,7 +301,7 @@ 
          * @since       3.6.0       Changed the name from `_compressFile`. Changed the scope from `private` to allow overriding in an extended class.
          * @return      boolean     True on success, false otherwise.
          */
-        public function _replyToCompressFile( ZipArchive $oZip, $sSourceFilePath, $aCallbacks=null ) {
+        public function _replyToCompressFile( ZipArchive $oZip, $sSourceFilePath, $aCallbacks = null ) {
             $this->_addFromString(
                 $oZip,
                 basename( $sSourceFilePath ),
@@ -334,7 +334,7 @@ 
      * @return      boolean
      */
     private function isFeasible( $sSource ) {
-        if ( ! extension_loaded( 'zip' ) ) {
+        if ( !extension_loaded( 'zip' ) ) {
             return false;
         }
 
@@ -359,7 +359,7 @@ 
      */
     private function _addEmptyDir( ZipArchive $oZip, $sInsidePath, $oCallable ) {
         $sInsidePath = $this->_getFilteredArchivePath( $sInsidePath, $oCallable );
-        if ( ! strlen( $sInsidePath ) ) {
+        if ( !strlen( $sInsidePath ) ) {
             return;
         }
         $oZip->addEmptyDir( $sInsidePath );
@@ -371,10 +371,10 @@ 
      * @remark      If the path is empty, it will not process.
      * @return      void
      */
-    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/admin-page-framework.php

Spacing +10 added lines, -10 removed lines

@@ -10,7 +10,7 @@ 
  * @package     AdminPageFramework
  */
 
-if ( ! class_exists( 'AdminPageFramework_Registry', false ) ) :
+if ( !class_exists( 'AdminPageFramework_Registry', false ) ) :
 /**
  * Facilitates WordPress plugin and theme development.
  *
@@ -56,7 +56,7 @@ 
 final class AdminPageFramework_Registry extends AdminPageFramework_Registry_Base {
 
     const TEXT_DOMAIN        = 'admin-page-framework';
-    const TEXT_DOMAIN_PATH   = '/language';  // not used at the moment
+    const TEXT_DOMAIN_PATH   = '/language'; // not used at the moment
 
     /**
      * Indicates whether the framework is loaded from the minified version or not.
@@ -98,11 +98,11 @@ 
      * Sets up static properties.
      * @return      void
      */
-    static public function setUp( $sFilePath=__FILE__ ) {
+    static public function setUp( $sFilePath = __FILE__ ) {
 
         self::$sFilePath                = $sFilePath;
         self::$sDirPath                 = dirname( self::$sFilePath );
-        self::$sIncludeClassListPath    = self::$sDirPath . '/admin-page-framework-include-class-list.php';
+        self::$sIncludeClassListPath    = self::$sDirPath.'/admin-page-framework-include-class-list.php';
         self::$aClassFiles              = self::_getClassFilePathList( self::$sIncludeClassListPath );
         self::$sAutoLoaderPath          = isset( self::$aClassFiles[ 'AdminPageFramework_RegisterClasses' ] )
             ? self::$aClassFiles[ 'AdminPageFramework_RegisterClasses' ]
@@ -117,7 +117,7 @@ 
          * @return      array
          */
         static private function _getClassFilePathList( $sInclusionClassListPath ) {
-            $aClassFiles = array();    // this will be updated if the inclusion below is successful.
+            $aClassFiles = array(); // this will be updated if the inclusion below is successful.
             include( $sInclusionClassListPath );
             return $aClassFiles;
         }
@@ -130,11 +130,11 @@ 
      */
     static public function getVersion() {
 
-        if ( ! isset( self::$sAutoLoaderPath ) ) {
-            trigger_error( 'Admin Page Framework: ' . ' : ' . sprintf( __( 'The method is called too early. Perform <code>%2$s</code> earlier.', 'admin-page-framework' ), __METHOD__, 'setUp()' ), E_USER_WARNING );
+        if ( !isset( self::$sAutoLoaderPath ) ) {
+            trigger_error( 'Admin Page Framework: '.' : '.sprintf( __( 'The method is called too early. Perform <code>%2$s</code> earlier.', 'admin-page-framework' ), __METHOD__, 'setUp()' ), E_USER_WARNING );
             return self::VERSION;
         }
-        $_aMinifiedVesionSuffix     = array(
+        $_aMinifiedVesionSuffix = array(
             0 => '',
             1 => '.min',
         );
@@ -163,7 +163,7 @@ 
 }
 endif;
 
-if ( ! class_exists( 'AdminPageFramework_Bootstrap', false ) ) :
+if ( !class_exists( 'AdminPageFramework_Bootstrap', false ) ) :
 /**
  * Loads the Admin Page Framework library.
  *
@@ -184,7 +184,7 @@ 
      */
     public function __construct( $sLibraryPath ) {
 
-        if ( ! $this->_isLoadable() ) {
+        if ( !$this->_isLoadable() ) {
             return;
         }
 

Indentation +9 added lines, -9 removed lines

@@ -1,14 +1,14 @@
 <?php
 /**
- * Admin Page Framework
- *
- * Facilitates WordPress plugin and theme development.
- *
- * @author      Michael Uno <michael@michaeluno.jp>
- * @copyright   2013-2016 (c) Michael Uno
- * @license     MIT <http://opensource.org/licenses/MIT>
- * @package     AdminPageFramework
- */
+     * Admin Page Framework
+     *
+     * Facilitates WordPress plugin and theme development.
+     *
+     * @author      Michael Uno <michael@michaeluno.jp>
+     * @copyright   2013-2016 (c) Michael Uno
+     * @license     MIT <http://opensource.org/licenses/MIT>
+     * @package     AdminPageFramework
+     */
 
 if ( ! class_exists( 'AdminPageFramework_Registry', false ) ) :
 /**

include/class/admin/admin-page-framework/help/AdminPageFrameworkLoader_AdminPage_Help_Report_Report.php

Indentation +6 added lines, -6 removed lines

@@ -1,10 +1,10 @@ 
 <?php
 /**
- * Admin Page Framework Loader
- * 
- * http://en.michaeluno.jp/admin-page-framework/
- * Copyright (c) 2013-2016 Michael Uno; Licensed GPLv2
- */
+     * Admin Page Framework Loader
+     * 
+     * http://en.michaeluno.jp/admin-page-framework/
+     * Copyright (c) 2013-2016 Michael Uno; Licensed GPLv2
+     */
 
 /**
  * Adds the 'Report' form section to the 'Report' tab.
@@ -163,7 +163,7 @@ 
      */
     public function validate( $aInput, $aOldInput, $oFactory, $aSubmit ) {
     
-       // Local variables
+        // Local variables
         $_bIsValid = true;
         $_aErrors  = array();
       

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
                 'title'             => __( 'Your Name', 'admin-page-framework-loader' ),
                 'type'              => 'text',
                 'default'           => $_oCurrentUser->user_lastname || $_oCurrentUser->user_firstname
-                    ? $_oCurrentUser->user_lastname . ' ' .  $_oCurrentUser->user_lastname
+                    ? $_oCurrentUser->user_lastname.' '.$_oCurrentUser->user_lastname
                     : '',
                 'attributes'        => array(
                     'required'      => 'required',
@@ -112,7 +112,7 @@ 
             array(
                 'field_id'          => 'allow_sending_system_information',
                 'title'             => __( 'Confirmation', 'admin-page-framework-loader' )
-                    . ' (' . __( 'required', 'admin-page-framework-loader' ) . ')',
+                    . ' ('.__( 'required', 'admin-page-framework-loader' ).')',
                 'type'              => 'checkbox',
                 'label'             => __( 'I understand that the system information including a PHP version and WordPress version etc. will be sent along with the messages to help developer trouble-shoot the problem.', 'admin-page-framework-loader' ),
                 'attributes'        => array(
@@ -167,12 +167,12 @@ 
         $_bIsValid = true;
         $_aErrors  = array();
       
-        if ( ! $aInput[ 'allow_sending_system_information' ] ) {
+        if ( !$aInput[ 'allow_sending_system_information' ] ) {
             $_bIsValid = false;
             $_aErrors[ 'allow_sending_system_information' ] = __( 'We need necessary information to help you.', 'admin-page-framework-loader' );
         }
         
-        if ( ! $_bIsValid ) {
+        if ( !$_bIsValid ) {
         
             $oFactory->setFieldErrors( $_aErrors );
             $oFactory->setSettingNotice( __( 'Please help us to help you.', 'admin-page-framework-loader' ) );

development/cli/AdminPageFramework_InclusionClassFilesHeader.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * If accessed from a console, include the registry class to laod 'AdminPageFramework_Registry_Base'.
  */
 if ( php_sapi_name() === 'cli' ) {
-    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ) . '/admin-page-framework.php';
+    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ).'/admin-page-framework.php';
     if ( file_exists( $_sFrameworkFilePath ) ) {
         include_once( $_sFrameworkFilePath );
     }