michaeluno / admin-page-framework

development/utility/readme_parser/AdminPageFramework_WPReadmeParser.php

Spacing +6 added lines, -6 removed lines

@@ -96,7 +96,7 @@ 
      * @since       3.5.0
      * @since       3.6.0       Made it accept string content to be passed to the first parameter.
      */
-    public function __construct( $sFilePathOrContent='', array $aReplacements=array(), array $aCallbacks=array() ) {
+    public function __construct( $sFilePathOrContent = '', array $aReplacements = array(), array $aCallbacks = array() ) {
 
         $this->sText            = file_exists( $sFilePathOrContent )
             ? file_get_contents( $sFilePathOrContent )
@@ -132,7 +132,7 @@ 
                 '/^[\s]*==[\s]*(.+?)[\s]*==/m',
                 $sText,
                 -1,
-                PREG_SPLIT_DELIM_CAPTURE|PREG_SPLIT_NO_EMPTY
+                PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY
             );
             return $_aSections;
         }
@@ -143,7 +143,7 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public function get( $sSectionName='' ) {
+    public function get( $sSectionName = '' ) {
         return $sSectionName
             ? $this->getSection( $sSectionName )
             : $this->_getParsedText( $this->sText );
@@ -205,7 +205,7 @@ 
          */
         public function _replyToReplaceCodeBlocks( $aMatches ) {
 
-            if ( ! isset( $aMatches[ 1 ] ) ) {
+            if ( !isset( $aMatches[ 1 ] ) ) {
                 return $aMatches[ 0 ];
             }
 
@@ -228,7 +228,7 @@ 
      */
     public function getRawSection( $sSectionName ) {
 
-        $_iIndex   = array_search( $sSectionName, $this->_aSections );
+        $_iIndex = array_search( $sSectionName, $this->_aSections );
         return false === $_iIndex
             ? ''
             : trim( $this->_aSections[ $_iIndex + 1 ] );
@@ -248,7 +248,7 @@ 
         // If <?php notation is missing, highlight_string() will not highlight the syntax so add it.
         $_bHasPHPTag = "<?php" === substr( $sCode, 0, 5 );
 
-        $sCode = $_bHasPHPTag ? $sCode : "<?php " . $sCode;
+        $sCode = $_bHasPHPTag ? $sCode : "<?php ".$sCode;
 
         $sCode = str_replace( '"', "'", $sCode ); // highlight_string() crashes if double quotes are contained in the code.
         $sCode = highlight_string( $sCode, true );

development/utility/pointer_tool_tip/AdminPageFramework_PointerToolTip.php

Indentation +1 added lines, -1 removed lines

@@ -213,7 +213,7 @@
                     get_current_user_id(),
                     'dismissed_wp_pointers',
                     true
-               )
+                )
             );
             $_aValidPointers = array();
 

Spacing +11 added lines, -11 removed lines

@@ -111,18 +111,18 @@ 
          */
         private function _setHooks( $aScreenIDs ) {
 
-            foreach( $aScreenIDs as $_sScreenID ) {
-                if ( ! $_sScreenID ) {
+            foreach ( $aScreenIDs as $_sScreenID ) {
+                if ( !$_sScreenID ) {
                     continue;
                 }
                 add_filter(
-                    get_class( $this ) . '-' . $_sScreenID,
+                    get_class( $this ).'-'.$_sScreenID,
                     array( $this, '_replyToSetPointer' )
                 );
 
             }
 
-            if ( ! $this->_hasBeenCalled() ) {
+            if ( !$this->_hasBeenCalled() ) {
                 return;
             }
 
@@ -169,7 +169,7 @@ 
 
         $_aPointers = $this->_getValidPointers( $this->_getPointers() );
 
-        if ( empty( $_aPointers ) || ! is_array( $_aPointers ) ) {
+        if ( empty( $_aPointers ) || !is_array( $_aPointers ) ) {
             return;
         }
 
@@ -189,11 +189,11 @@ 
             $_oScreen   = get_current_screen();
             $_sScreenID = $_oScreen->id;
             if ( in_array( $_sScreenID, $this->aScreenIDs ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_sScreenID, array() );
+                return apply_filters( get_class( $this ).'-'.$_sScreenID, array() );
             }
 
             if ( isset( $_GET[ 'page' ] ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_GET[ 'page' ], array() );
+                return apply_filters( get_class( $this ).'-'.$_GET[ 'page' ], array() );
             }
             return array();
 
@@ -207,7 +207,7 @@ 
         private function _getValidPointers( $_aPointers ) {
 
             // Get dismissed pointers
-            $_aDismissed      = explode(
+            $_aDismissed = explode(
                 ',',
                 ( string ) get_user_meta(
                     get_current_user_id(),
@@ -235,7 +235,7 @@ 
                 $_aPointer[ 'pointer_id' ] = $_iPointerID;
 
                 // Add the pointer to $_aValidPointers array
-                $_aValidPointers[] =  $_aPointer;
+                $_aValidPointers[ ] = $_aPointer;
 
             }
             return $_aValidPointers;
@@ -311,9 +311,9 @@ 
          * @return      string
          * @internal
          */
-        public function _getInternalScript( $aPointers=array() ) {
+        public function _getInternalScript( $aPointers = array() ) {
 
-            $_aJSArray      = json_encode( $aPointers );
+            $_aJSArray = json_encode( $aPointers );
 
             /**
              * Checks check-boxes in siblings.

development/utility/zip/AdminPageFramework_Zip.php

Indentation +1 added lines, -1 removed lines

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

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/document/factory/AdminPageFramework_Documentation.php

Indentation -1 removed lines

@@ -196,7 +196,6 @@
  * For common hooks throughout the other factory components, see [Base Factory](./package-AdminPageFramework.Common.Factory.html).
  *
  * <h3>Factory Specific Hooks</h3>
-
  * <h4> Action Hooks</h4>
  * <ul>
  *     <li>**load_{page slug}** – [2.1.0+] triggered when the framework's page is loaded before the header gets sent. This will not be triggered in the admin pages that are not registered by the framework. The first parameter: class object [3.1.2+].</li>

include/class/event/AdminPageFrameworkLoader_Event_Action_GetDevelopmentVersion.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function replyToDoAction() {
         AdminPageFramework_WPUtility::setTransient(
-            AdminPageFrameworkLoader_Registry::TRANSIENT_PREFIX . 'devver',
+            AdminPageFrameworkLoader_Registry::TRANSIENT_PREFIX.'devver',
             $this->___getVersion(), // data - if an error occurs, an empty string will be given
             604800 // for one week
         );
@@ -54,7 +54,7 @@ 
             $_oUtil     = new AdminPageFramework_WPUtility;
             $_aHeaders  = $_oUtil->getScriptData(
                 $this->___getPageBody(),
-                '',  /// context
+                '', /// context
                 array( 'version' => 'Version' )
             );
             return $_oUtil->getElement(

include/class/admin/_abstract/AdminPageFrameworkLoader_AdminPage_Tab_Base.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
             ? $aTabDefinition[ 'tab_slug' ]
             : '';
 
-        if ( ! $this->sTabSlug ) {
+        if ( !$this->sTabSlug ) {
             return;
         }
 
@@ -72,7 +72,7 @@ 
 
         if ( $aTabDefinition[ 'tab_slug' ] ) {
             add_action(
-                "load_{$sPageSlug}_{$aTabDefinition['tab_slug']}",
+                "load_{$sPageSlug}_{$aTabDefinition[ 'tab_slug' ]}",
                 array( $this, 'replyToLoadTab' )
             );
             add_action(

include/class/admin/_abstract/AdminPageFrameworkLoader_AdminPage_Page_Base.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
     public function __construct( $oFactory, array $aPageArguments ) {
 
         $this->oFactory     = $oFactory;
-        $this->sPageSlug    = $aPageArguments['page_slug'];
+        $this->sPageSlug    = $aPageArguments[ 'page_slug' ];
         $this->_addPage( $aPageArguments );
         $this->construct( $oFactory );
 
@@ -58,8 +58,8 @@ 
      */
     public function replyToLoadResources( $oFactory ) {
 
-        $_sCSSPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/' . $this->sPageSlug . '.css';
-        if ( ! file_exists( $_sCSSPath ) ) {
+        $_sCSSPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/'.$this->sPageSlug.'.css';
+        if ( !file_exists( $_sCSSPath ) ) {
             return;
         }
         $this->oFactory->enqueueStyle(

include/class/admin/_abstract/AdminPageFrameworkLoader_AdminPage_Tab_ReadMeBase.php

Indentation +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
             return $_oTOC->get();
         }
         return ''
-         . $_sContent;
+            . $_sContent;
 
     }
 

Spacing +11 added lines, -11 removed lines

@@ -21,7 +21,7 @@ 
      *
      * @since       3.5.3
      */
-    protected function _getReadmeContents( $sFilePath, $sTOCTitle, $asSections=array() ) {
+    protected function _getReadmeContents( $sFilePath, $sTOCTitle, $asSections = array() ) {
 
         $_oWPReadmeParser = new AdminPageFramework_WPReadmeParser(
             $sFilePath,
@@ -34,7 +34,7 @@ 
             )
         );
         $_sContent = '';
-        foreach( ( array ) $asSections as $_sSection  ) {
+        foreach ( ( array ) $asSections as $_sSection ) {
             $_sContent .= $_oWPReadmeParser->getSection( $_sSection );
         }
         if ( $sTOCTitle ) {
@@ -66,7 +66,7 @@ 
              * @since       3.6.1
              * @return      string      The generate HTML output.
              */
-            public function _replyToProcessShortcode_embed( $aAttributes, $sURL, $sShortcode='' ) {
+            public function _replyToProcessShortcode_embed( $aAttributes, $sURL, $sShortcode = '' ) {
 
                 $sURL   = isset( $aAttributes[ 'src' ] ) ? $aAttributes[ 'src' ] : $sURL;
                 $_sHTML = wp_oembed_get( $sURL );
@@ -108,27 +108,27 @@ 
      *  array( 'Second Heading' => 'Another text', ),
      * )
      */
-    public function getContentsByHeader( $sContents, $iHeaderNumber=2 ) {
+    public function getContentsByHeader( $sContents, $iHeaderNumber = 2 ) {
 
         $_aContents = array();
         $_aSplitContents = preg_split(
             // '/^[\s]*==[\s]*(.+?)[\s]*==/m',
-            '/(<h[' . $iHeaderNumber . ']*[^>]*>.*?<\/h[' . $iHeaderNumber . ']>)/i',
+            '/(<h['.$iHeaderNumber.']*[^>]*>.*?<\/h['.$iHeaderNumber.']>)/i',
             $sContents,
             -1,
-            PREG_SPLIT_DELIM_CAPTURE|PREG_SPLIT_NO_EMPTY
+            PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY
         );
 
-        foreach( $_aSplitContents as $_iIndex => $_sSplitContent ) {
-            if ( ! preg_match( '/<h[' . $iHeaderNumber . ']*[^>]*>(.*?)<\/h[' . $iHeaderNumber . ']>/i', $_sSplitContent , $_aMatches ) ) {
+        foreach ( $_aSplitContents as $_iIndex => $_sSplitContent ) {
+            if ( !preg_match( '/<h['.$iHeaderNumber.']*[^>]*>(.*?)<\/h['.$iHeaderNumber.']>/i', $_sSplitContent, $_aMatches ) ) {
                 continue;
             }
 
-            if ( ! isset( $_aMatches[ 1 ] ) ) {
+            if ( !isset( $_aMatches[ 1 ] ) ) {
                 continue;
             }
-            if ( isset( $_aSplitContents[ $_iIndex + 1 ] ) )  {
-                $_aContents[] = array(
+            if ( isset( $_aSplitContents[ $_iIndex + 1 ] ) ) {
+                $_aContents[ ] = array(
                     $_aMatches[ 1 ],
                     $_aSplitContents[ $_iIndex + 1 ]
                 );

Braces +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
             if ( ! isset( $_aMatches[ 1 ] ) ) {
                 continue;
             }
-            if ( isset( $_aSplitContents[ $_iIndex + 1 ] ) )  {
+            if ( isset( $_aSplitContents[ $_iIndex + 1 ] ) ) {
                 $_aContents[] = array(
                     $_aMatches[ 1 ],
                     $_aSplitContents[ $_iIndex + 1 ]

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

Spacing +8 added lines, -8 removed lines

@@ -25,8 +25,8 @@ 
 
         // Tabs
         new AdminPageFrameworkLoader_AdminPage_Help_Information(
-            $oFactory,    // factory object
-            $this->sPageSlug,   // page slug
+            $oFactory, // factory object
+            $this->sPageSlug, // page slug
             array(  // tab definition
                 'tab_slug'  => 'information',
                 'title'     => __( 'Support', 'admin-page-framework-loader' ),
@@ -40,11 +40,11 @@ 
                 'title'         => __( 'Getting Started', 'admin-page-framework-loader' ),
                 'url'           => add_query_arg(
                     array(
-                        'page'  => AdminPageFrameworkLoader_Registry::$aAdminPages['about'],
+                        'page'  => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ],
                         // 'tab'   => 'guide',
                     ),
                     admin_url( 'index.php' )   // Dashboard
-                ) . '#section-getting_started__'
+                ).'#section-getting_started__'
             )
         );
         new AdminPageFrameworkLoader_AdminPage_Help_FAQ(
@@ -87,7 +87,7 @@ 
                 'title'     => __( 'About', 'admin-page-framework-loader' ),
                 'url'       => add_query_arg(
                     array(
-                        'page' => AdminPageFrameworkLoader_Registry::$aAdminPages['about']
+                        'page' => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ]
                     ),
                     admin_url( 'index.php' )   // Dashboard
                 )
@@ -105,17 +105,17 @@ 
 
         // Page meta boxes
         new AdminPageFrameworkLoader_AdminPageMetaBox_ExternalLinks(
-            null,                                           // meta box id - passing null will make it auto generate
+            null, // meta box id - passing null will make it auto generate
             __( 'Resources', 'admin-page-framework-loader' ), // title
             array( // page slugs
                 AdminPageFrameworkLoader_Registry::$aAdminPages[ 'help' ],
             ),
-            'side',                                       // context
+            'side', // context
             'default'                                     // priority
         );
 
         $oFactory->enqueueStyle(
-            AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/help.css',
+            AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/help.css',
             $this->sPageSlug
         );