michaeluno / admin-page-framework

development/factory/widget/AdminPageFramework_Widget_Factory.php

Doc Comments +1 added lines

@@ -23,6 +23,7 @@
      * Sets up internal properties.
      * 
      * @since       3.2.0
+     * @param AdminPageFramework_Widget_Model $oCaller
      * @return      void
      */
 	public function __construct( $oCaller, $sWidgetTitle, array $aArguments=array() ) {

Indentation +11 added lines, -11 removed lines

@@ -25,21 +25,21 @@ 
      * @since       3.2.0
      * @return      void
      */
-	public function __construct( $oCaller, $sWidgetTitle, array $aArguments=array() ) {
+    public function __construct( $oCaller, $sWidgetTitle, array $aArguments=array() ) {
 		
         $aArguments = $aArguments 
             + array( 
                 'classname'     => 'admin_page_framework_widget',
                 'description'   => '',  
             );
-		parent::__construct( 
+        parent::__construct( 
             $oCaller->oProp->sClassName,  // base id 
             $sWidgetTitle,      // widget title
             $aArguments         // widget arguments
         );
         $this->oCaller = $oCaller;
         
-	}
+    }
     
     /**
      * Displays the widget contents in the front end.
@@ -49,7 +49,7 @@ 
      * whether the title should be visible or not depending on the content.
      * @return      void
      */
-	public function widget( $aArguments, $aFormData ) {
+    public function widget( $aArguments, $aFormData ) {
            
         echo $aArguments[ 'before_widget' ];
         
@@ -72,9 +72,9 @@ 
 
         echo $_sContent;
         
-		echo $aArguments[ 'after_widget' ];
+        echo $aArguments[ 'after_widget' ];
 		
-	}
+    }
         /**
          * Returns the widget title.
          * 
@@ -103,7 +103,7 @@ 
             if ( ! $_sTitle ) {
                 return '';
             }
-           return $aArguments['before_title'] 
+            return $aArguments['before_title'] 
                 . $_sTitle 
             . $aArguments['after_title'];           
             
@@ -115,7 +115,7 @@ 
      * @since       3.2.0
      * @return      mixed       The validated form data. The type should be an array but it is dealt by the framework user it will be unknown.
      */
-	public function update( $aSubmittedFormData, $aSavedFormData ) {
+    public function update( $aSubmittedFormData, $aSavedFormData ) {
                 
         return $this->oCaller->oUtil->addAndApplyFilters(
             $this->oCaller, 
@@ -128,14 +128,14 @@ 
             $this->oCaller
         );
  
-	}
+    }
     
     /**
      * Constructs the widget form.
      * 
      * @return      void
      */
-	public function form( $aFormData ) {
+    public function form( $aFormData ) {
 
         // Trigger the load() method and load_{...} actions. The user sets up the form.
         $this->oCaller->load( $this->oCaller );
@@ -193,7 +193,7 @@ 
             // $this->oCaller
         // );   
        
-	}
+    }
     
         /**
          * 

Spacing +24 added lines, -24 removed lines

@@ -25,7 +25,7 @@ 
      * @since       3.2.0
      * @return      void
      */
-	public function __construct( $oCaller, $sWidgetTitle, array $aArguments=array() ) {
+	public function __construct( $oCaller, $sWidgetTitle, array $aArguments = array() ) {
 		
         $aArguments = $aArguments 
             + array( 
@@ -34,17 +34,17 @@ 
             );
             
 		parent::__construct( 
-            $oCaller->oProp->sClassName,  // base id 
-            $sWidgetTitle,      // widget title
+            $oCaller->oProp->sClassName, // base id 
+            $sWidgetTitle, // widget title
             $aArguments         // widget arguments
         );
         $this->oCaller = $oCaller;
     
         // Set up callbacks for field element outputs such as for name and it attributes.
         $this->oCaller->oProp->aFormCallbacks = array( 
-            'hfID'          => array( $this, 'get_field_id' ),    // defined in the WP_Widget class.  
-            'hfTagID'       => array( $this, 'get_field_id' ),    // defined in the WP_Widget class.  
-            'hfName'        => array( $this, '_replyToGetFieldName' ),  // defined in the WP_Widget class.  
+            'hfID'          => array( $this, 'get_field_id' ), // defined in the WP_Widget class.  
+            'hfTagID'       => array( $this, 'get_field_id' ), // defined in the WP_Widget class.  
+            'hfName'        => array( $this, '_replyToGetFieldName' ), // defined in the WP_Widget class.  
             'hfInputName'   => array( $this, '_replyToGetFieldInputName' ),
             // 'hfClass'       => array( $this, '_replyToAddClassSelector' ),
             // 'hfNameFlat'    => array( $this, '_replyToGetFlatFieldName' ), // @deprecated 3.6.0+ the same as the framework factory method.
@@ -68,7 +68,7 @@ 
         
         $this->oCaller->oUtil->addAndDoActions( 
             $this->oCaller, 
-            'do_' . $this->oCaller->oProp->sClassName, 
+            'do_'.$this->oCaller->oProp->sClassName, 
             $this->oCaller
         );
        
@@ -99,7 +99,7 @@ 
          */
         private function _getTitle( array $aArguments, array $aFormData ) {
                 
-            if ( ! $this->oCaller->oProp->bShowWidgetTitle ) {
+            if ( !$this->oCaller->oProp->bShowWidgetTitle ) {
                 return '';
             }
             
@@ -113,12 +113,12 @@ 
                 $aFormData,
                 $this->id_base 
             );
-            if ( ! $_sTitle ) {
+            if ( !$_sTitle ) {
                 return '';
             }
-           return $aArguments['before_title'] 
+           return $aArguments[ 'before_title' ] 
                 . $_sTitle 
-            . $aArguments['after_title'];           
+            . $aArguments[ 'after_title' ];           
             
         }
             
@@ -134,7 +134,7 @@ 
             $this->oCaller, 
             "validation_{$this->oCaller->oProp->sClassName}", 
             call_user_func_array( 
-                array( $this->oCaller, 'validate' ),    // triggers __call()
+                array( $this->oCaller, 'validate' ), // triggers __call()
                 array( $aSubmittedFormData, $aSavedFormData, $this->oCaller ) // parameters
             ), // 3.5.3+                        
             $aSavedFormData,
@@ -179,8 +179,8 @@ 
         $this->oCaller->oForm = new AdminPageFramework_Form_widget(
             array(
                 'register_if_action_already_done' => false, // do not register fields right away
-            ) + $this->oCaller->oProp->aFormArguments,  // form arguments  
-            $this->oCaller->oProp->aFormCallbacks,  // callbacks
+            ) + $this->oCaller->oProp->aFormArguments, // form arguments  
+            $this->oCaller->oProp->aFormCallbacks, // callbacks
             $this->oCaller->oMsg
         );
         
@@ -198,7 +198,7 @@ 
             $this->oCaller->oUtil->addAndDoActions( 
                 $this->oCaller, 
                 array(
-                    'load_' . $this->oCaller->oProp->sClassName, 
+                    'load_'.$this->oCaller->oProp->sClassName, 
                 ),
                 $this->oCaller 
             );            
@@ -217,12 +217,12 @@ 
             $_aParams      = func_get_args() + array( null, null, null );
             $aField        = $_aParams[ 1 ];
             
-            $_sSectionIndex = isset( $aField['section_id'], $aField['_section_index'] ) 
-                ? "[{$aField['_section_index']}]" 
+            $_sSectionIndex = isset( $aField[ 'section_id' ], $aField[ '_section_index' ] ) 
+                ? "[{$aField[ '_section_index' ]}]" 
                 : "";             
             $_sID           = $this->oCaller->isSectionSet( $aField )
-                ? $aField['section_id'] . "]" . $_sSectionIndex . "[" . $aField[ 'field_id' ]
-                : $aField['field_id'];
+                ? $aField[ 'section_id' ]."]".$_sSectionIndex."[".$aField[ 'field_id' ]
+                : $aField[ 'field_id' ];
             return $this->get_field_name( $_sID );
         
         }    
@@ -241,15 +241,15 @@ 
             $_sIndex = $this->oCaller->oUtil->getAOrB(
                 '0' !== $sIndex && empty( $sIndex ),
                 '',
-                "[" . $sIndex . "]"
+                "[".$sIndex."]"
             );                        
-            $_sSectionIndex = isset( $aField['section_id'], $aField['_section_index'] ) 
-                ? "[{$aField['_section_index']}]" 
+            $_sSectionIndex = isset( $aField[ 'section_id' ], $aField[ '_section_index' ] ) 
+                ? "[{$aField[ '_section_index' ]}]" 
                 : "";             
             $_sID           = $this->oCaller->isSectionSet( $aField )
-                ? $aField['section_id'] . "]" . $_sSectionIndex . "[" . $aField[ 'field_id' ]
+                ? $aField[ 'section_id' ]."]".$_sSectionIndex."[".$aField[ 'field_id' ]
                 : $aField[ 'field_id' ];
-            return $this->get_field_name( $_sID ) . $_sIndex;
+            return $this->get_field_name( $_sID ).$_sIndex;
             
         }
         

development/utility/AdminPageFramework_PluginBootstrap.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @param       string      $sPluginFilePath        The plugin file path.
      * @param       string      $sPluginHookPrefix      The plugin hook slug without underscore. This will be used to construct hook names.
      * @param       string      $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
-     * @param       string      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
+     * @param       integer      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
      */
     public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
         

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
      * @param       string      $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
      * @param       string      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
      */
-    public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
+    public function __construct( $sPluginFilePath, $sPluginHookPrefix = '', $sSetUpHook = 'plugins_loaded', $iPriority = 10 ) {
         
         // Check if it has been loaded.
         if ( $this->_hasLoaded() ) {
@@ -76,7 +76,7 @@ 
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
                  
         // 8. Schedule to load plugin specific components.
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
+        if ( !$this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
             $this->_replyToLoadPluginComponents();
         } else {
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
@@ -113,13 +113,13 @@ 
          */
         protected function _registerClasses() {
             
-            if ( ! class_exists( 'AdminPageFramework_RegisterClasses' ) ) {
+            if ( !class_exists( 'AdminPageFramework_RegisterClasses' ) ) {
                 return;
             }                        
             // Register classes
             new AdminPageFramework_RegisterClasses( 
-                $this->getScanningDirs(),   // scanning directory paths
-                array(),                    // autoloader options
+                $this->getScanningDirs(), // scanning directory paths
+                array(), // autoloader options
                 $this->getClasses()         // pre-generated class list
             );
                     

Braces +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
                  
         // 8. Schedule to load plugin specific components.
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
+        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
             $this->_replyToLoadPluginComponents();
         } else {
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );

development/utility/AdminPageFramework_Requirement.php

Doc Comments +3 added lines

@@ -134,6 +134,7 @@ 
          * Returns a php warning if present.
          * @since       3.5.3
          * @internal
+         * @param string $sType
          * @return      string      The warning.
          */
         private function _getWarningByType( $sType ) {
@@ -226,6 +227,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.
              */
             private function _getWarningsByFunctionName( $sFuncName, $aSubjects ) {
@@ -285,6 +287,7 @@ 
     /**
      * Deactivates the plugin.
      * @since       3.4.6
+     * @param string $sPluginFilePath
      */
     public function deactivatePlugin( $sPluginFilePath, $sMessage='', $bIsOnActivation=false ) {
         

Indentation +1 added lines, -1 removed lines

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

Spacing +24 added lines, -24 removed lines

@@ -76,12 +76,12 @@ 
      * 
      * @since       3.4.6
      */ 
-    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 ];
             }
@@ -105,9 +105,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(
@@ -120,10 +120,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.
@@ -137,7 +137,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' ] ) ) {
@@ -164,7 +164,7 @@ 
          * @since       3.4.6
          */
         private function _checkWordPressVersion( $sWordPressVersion ) {
-            return version_compare( $GLOBALS['wp_version'], $sWordPressVersion, ">=" );
+            return version_compare( $GLOBALS[ 'wp_version' ], $sWordPressVersion, ">=" );
         }
         
         /**
@@ -230,9 +230,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;                
@@ -252,7 +252,7 @@ 
          */    
         public function _replyToPrintAdminNotices() {
             
-            $_aWarnings     = array_unique( $this->aWarnings );
+            $_aWarnings = array_unique( $this->aWarnings );
             if ( empty( $_aWarnings ) ) {
                 return;
             }
@@ -270,12 +270,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 );
@@ -286,12 +286,12 @@ 
      * Deactivates the plugin.
      * @since       3.4.6
      */
-    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;
             }
         }
@@ -301,8 +301,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>" );
             
         }
         

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();
         

development/utility/readme_parser/AdminPageFramework_WPReadmeParser.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,6 @@ 
      *      'code_block'        =>  ...,
      *      
      * )`
-     * @param       array       $aOptions           The options array which determines the behaviour of the class.
      * @since       3.5.0
      * @since       3.6.0       Made it accept string content to be passed to the first parameter.
      */
@@ -156,6 +155,7 @@ 
          * Returns the parsed text.
          * @since       3.5.0
          * @internal
+         * @param string $sContent
          */
         private function _getParsedText( $sContent ) {
 

Spacing +6 added lines, -6 removed lines

@@ -85,7 +85,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 )
@@ -121,7 +121,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;
         }
@@ -132,7 +132,7 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public function get( $sSectionName='' ) {
+    public function get( $sSectionName = '' ) {
         return $sSectionName
             ? $this->getSection( $sSectionName )
             : $this->_getParsedText( $this->sText );
@@ -194,7 +194,7 @@ 
          */
         public function _replyToReplaceCodeBlocks( $aMatches ) {
             
-            if ( ! isset( $aMatches[ 1 ] ) ) {
+            if ( !isset( $aMatches[ 1 ] ) ) {
                 return $aMatches[ 0 ];
             }
             
@@ -217,7 +217,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 ] );
@@ -237,7 +237,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 );

include/class/admin/_abstract/AdminPageFrameworkLoader_AdminPage_Tab_ReadMeBase.php

Doc Comments +2 added lines

@@ -20,6 +20,8 @@
     /**
      * 
      * @since       3.5.3
+     * @param string $sFilePath
+     * @param string $sTOCTitle
      */
     protected function _getReadmeContents( $sFilePath, $sTOCTitle, $asSections=array() ) {
         

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 +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         foreach( ( array ) $asSections as $_sSection  ) {
             $_sContent .= $_oWPReadmeParser->getSection( $_sSection );  
         }        
-        if ( $sTOCTitle ) {            
+        if ( $sTOCTitle ) {
             $_oTOC = new AdminPageFramework_TableOfContents(
                 $_sContent,
                 4,
@@ -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/tool/generator/AdminPageFrameworkLoader_AdminPage_Tool_Generator_Generator.php

Doc Comments +4 added lines, -5 removed lines

@@ -257,6 +257,7 @@ 
          * Generates the framework zip data.
          * 
          * @since       3.5.4
+         * @param string $sFrameworkDirPath
          * @return      string      The binary zip data.
          */
         private function _getDownloadFrameworkZipFile( $sFrameworkDirPath, $sDestinationPath ) {
@@ -308,11 +309,7 @@ 
                  * 
                  * @since       3.5.4
                  * @remark      string      $sPath              The path to check. It can be a directory or a file.
-                 * @param       string      $sPathInArchive     The parsing directory path set to the archive. 
-                 * The passed path for the archive has a trailing slash. It starts with a directory name.
-                 * e.g.
-                 * `utility/AdminPageFramework_WPReadmeParser/`
-                 * `factory/widget/model/`
+                 * @param string $sPath
                  * @return      boolean
                  */
                 private function _isAllowedArchivePath( $sPath ) {
@@ -567,6 +564,7 @@ 
                  * Modifies the text domain in the given file contents.
                  * 
                  * @since       3.5.4
+                 * @param string $sFileContents
                  * @return      string
                  */                
                 private function _modifyTextDomain( $sFileContents ) {
@@ -585,6 +583,7 @@ 
                      * Retrieves the value from the $_POST array by the given field ID.
                      * 
                      * @since       3.5.4
+                     * @param string $sFieldID
                      * @return      string
                      */
                     private function _getFormSubmitValueByFieldIDAsString( $sFieldID ) {

Spacing +32 added lines, -32 removed lines

@@ -30,7 +30,7 @@ 
         $this->oFactory = $oFactory;
         
         add_action( 
-            'export_name_' . $this->sPageSlug . '_' . $this->sTabSlug, 
+            'export_name_'.$this->sPageSlug.'_'.$this->sTabSlug, 
             array( $this, 'replyToFilterFileName' ), 
             10, 
             5 
@@ -76,7 +76,7 @@ 
                 'title'             => __( 'Class Prefix', 'admin-page-framework-loader' ),
                 'type'              => 'text',
                 'description'       => __( 'Set alphanumeric characters for the class names.', 'admin-page-framework-loader' )
-                    . ' ' .  __( 'For example, if you set here <code>MyPluginName_</code>, you will need to extend the class named <code>MyClassName_AdminPageFramework</code> instead of <code>AdminPageFramework</code>.', 'admin-page-framework-loader' )
+                    . ' '.__( 'For example, if you set here <code>MyPluginName_</code>, you will need to extend the class named <code>MyClassName_AdminPageFramework</code> instead of <code>AdminPageFramework</code>.', 'admin-page-framework-loader' )
                     . ' e.g.<code>MyPluginName_</code>',
                 'attributes'        => array(
                     'size'          => 30,
@@ -126,8 +126,8 @@ 
                 'label_min_width'   => 0,
                 'order'             => 100,
                 'value'             => __( 'Download', 'adimn-page-framework-demo' ),
-                'file_name'         => 'admin-page-framework.zip',  // the default file name. This will be modified by the filter.
-                'format'            => 'text',  // 'json', 'text', 'array'      
+                'file_name'         => 'admin-page-framework.zip', // the default file name. This will be modified by the filter.
+                'format'            => 'text', // 'json', 'text', 'array'      
                 'description'       => $oFactory->oUtil->getAOrB(
                     class_exists( 'ZipArchive' ),
                     __( 'Download the compiled framework files as a zip file.', 'admin-page-framework-loader' ),
@@ -181,8 +181,8 @@ 
         
         // the class prefix must not contain white spaces and some other characters not supported in PHP class names.
         preg_match( 
-            '/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*$/',     // pattern - allowed characters for variables in PHP.
-            $aInput[ 'class_prefix' ],     // subject
+            '/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*$/', // pattern - allowed characters for variables in PHP.
+            $aInput[ 'class_prefix' ], // subject
             $_aMatches 
         );
         if ( $aInput[ 'class_prefix' ] && empty( $_aMatches ) ) {
@@ -190,13 +190,13 @@ 
             $_bVerified = false;
         }
                         
-        if ( ! $aInput[ 'text_domain' ] ) {
+        if ( !$aInput[ 'text_domain' ] ) {
             $_aErrors[ 'text_domain' ] = __( 'The text domain cannot be empty.', 'admin-page-framework-loader' );
             $_bVerified = false;
         }
                 
         // An invalid value is found. Set a field error array and an admin notice and return the old values.
-        if ( ! $_bVerified ) {
+        if ( !$_bVerified ) {
             $oAdminPage->setFieldErrors( $_aErrors );     
             $oAdminPage->setSettingNotice( __( 'There was something wrong with your input.', 'admin-page-framework-loader' ) );
             return $aOldInput;
@@ -238,8 +238,8 @@ 
      */
     public function replyToDownloadFramework( $aSavedData, $sSubmittedFieldID, $sSubmittedInputID, $oAdminPage ) {
         
-        $_sFrameworkDirPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/library/admin-page-framework';
-        if ( ! file_exists( $_sFrameworkDirPath ) ) {
+        $_sFrameworkDirPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/library/admin-page-framework';
+        if ( !file_exists( $_sFrameworkDirPath ) ) {
             return $aSavedData;
         }
         
@@ -248,7 +248,7 @@ 
             $_sFrameworkDirPath, 
             $_sTempFile
         );
-        header( "Content-Length: " . strlen( $_sData ) ); 
+        header( "Content-Length: ".strlen( $_sData ) ); 
         unlink( $_sTempFile );
         return $_sData;
         
@@ -265,9 +265,9 @@ 
                 $sFrameworkDirPath, 
                 $sDestinationPath, 
                 array(
-                    'include_directory'             => false,   // wrap contents in a sub-directory
+                    'include_directory'             => false, // wrap contents in a sub-directory
                     'additional_source_directories' => apply_filters(
-                        AdminPageFrameworkLoader_Registry::HOOK_SLUG . '_filter_generator_additional_source_directories',
+                        AdminPageFrameworkLoader_Registry::HOOK_SLUG.'_filter_generator_additional_source_directories',
                         array() // directory paths 
                     ),
                 ),                
@@ -278,7 +278,7 @@ 
                 ) 
             );
             $_bSucceed = $_oZip->compress();
-            if ( ! $_bSucceed ) {
+            if ( !$_bSucceed ) {
                 return '';
             }
             return file_get_contents( $sDestinationPath );
@@ -298,7 +298,7 @@ 
                 
                 // Check if it belongs to selected components.
                 if ( false === $this->_isAllowedArchivePath( $sPathInArchive ) ) {
-                    return '';  // empty value will drop the entry
+                    return ''; // empty value will drop the entry
                 }                
                 return $sPathInArchive;
                            
@@ -317,7 +317,7 @@ 
                  */
                 private function _isAllowedArchivePath( $sPath ) {
                     
-                    foreach( $this->_getDisallowedArchiveDirectoryPaths() as $_sDisallowedPath ) {
+                    foreach ( $this->_getDisallowedArchiveDirectoryPaths() as $_sDisallowedPath ) {
                         $_bHasPrefix = $this->oFactory->oUtil->hasPrefix( 
                             ltrim( $_sDisallowedPath, '/' ), // needle
                             ltrim( $sPath, '/' ) // haystack
@@ -386,7 +386,7 @@ 
                     // List paths.
                     $_aAllComponentsPaths       = array();
                     $_aSelectedComponentsPaths  = array();
-                    foreach( $this->_aComponentPaths as $_sKey => $_aPaths ) {
+                    foreach ( $this->_aComponentPaths as $_sKey => $_aPaths ) {
                     
                         // Extract all component paths.
                         $_aAllComponentsPaths = array_merge(
@@ -440,7 +440,7 @@ 
             public function _replyToModifyFileContents( $sFileContents, $sPathInArchive ) {
 
                 $sFileContents = apply_filters(
-                    AdminPageFrameworkLoader_Registry::HOOK_SLUG . '_filter_generator_file_contents',
+                    AdminPageFrameworkLoader_Registry::HOOK_SLUG.'_filter_generator_file_contents',
                     $sFileContents, 
                     $sPathInArchive,
                     $this->oFactory->oUtil->getElement(
@@ -454,7 +454,7 @@ 
                 );
             
                 // Check the file extension.
-                if ( ! in_array( pathinfo( $sPathInArchive, PATHINFO_EXTENSION ), array( 'php', 'css', 'js' ) ) ) {
+                if ( !in_array( pathinfo( $sPathInArchive, PATHINFO_EXTENSION ), array( 'php', 'css', 'js' ) ) ) {
                     return $sFileContents;
                 }
                 
@@ -506,12 +506,12 @@ 
                             $this->_getCheckedComponents()
                         );
                         $_aInsert = array(
-                            'Included Components: ' . implode( ', ', $_aCheckedComponents ),
-                            'Generated on ' . date( 'Y-m-d' ),  // today's date
+                            'Included Components: '.implode( ', ', $_aCheckedComponents ),
+                            'Generated on '.date( 'Y-m-d' ), // today's date
                         );
                         return preg_replace(
                             '#\*/#', // needle - matches '*/'
-                            implode( PHP_EOL . ' ', $_aInsert ) . ' \0', // replacement \0 is a back-reference to '*/'
+                            implode( PHP_EOL.' ', $_aInsert ).' \0', // replacement \0 is a back-reference to '*/'
                             $sFileContents, // subject
                             1 // replace only the first occurrence
                         );      
@@ -525,14 +525,14 @@ 
                     private function _modifyClassNameOfInclusionList( $sFileContents ) {
                         // Replace the array key names.
                         $sFileContents = preg_replace_callback(
-                            '/(["\'])(.+)\1(?=\s?+=>)/',  // pattern '
-                            array( $this, '_replyToModifyPathName' ),   // callable
+                            '/(["\'])(.+)\1(?=\s?+=>)/', // pattern '
+                            array( $this, '_replyToModifyPathName' ), // callable
                             $sFileContents // subject
                         );     
                         // Replace the registry class names.
                         return preg_replace_callback(
-                            '/(=>\s?+)(.+)(?=::)/',  // pattern '
-                            array( $this, '_replyToModifyPathName' ),   // callable
+                            '/(=>\s?+)(.+)(?=::)/', // pattern '
+                            array( $this, '_replyToModifyPathName' ), // callable
                             $sFileContents // subject
                         );     
                     }
@@ -557,7 +557,7 @@ 
                     return strlen( $_sPrefix )
                         ? str_replace( 
                             'AdminPageFramework', // search 
-                            $_sPrefix . 'AdminPageFramework', // replace
+                            $_sPrefix.'AdminPageFramework', // replace
                             $sSubject // subject
                         )
                         : $sSubject;
@@ -589,7 +589,7 @@ 
                      */
                     private function _getFormSubmitValueByFieldIDAsString( $sFieldID ) {
                          
-                        static $_aCaches=array();
+                        static $_aCaches = array();
                         $_aCaches[ $sFieldID ] = isset( $_aCaches[ $sFieldID ] )
                             ? $_aCaches[ $sFieldID ]
                             : $this->oFactory->oUtil->getElement(
@@ -623,9 +623,9 @@ 
                 'public',
             ),
             'Content-Description'       => 'File Transfer',
-            'Content-type'              => 'application/octet-stream',   // 'application/zip' may work as well
+            'Content-type'              => 'application/octet-stream', // 'application/zip' may work as well
             'Content-Transfer-Encoding' => 'binary',
-            'Content-Disposition'       => 'attachment; filename="' . $sFileName .'";',
+            'Content-Disposition'       => 'attachment; filename="'.$sFileName.'";',
             // 'Content-Length'            => strlen( $mData ),
         ) + $aHeader;
         
@@ -662,10 +662,10 @@ 
         $_sFileNameWOExtension = trim( $_sFileNameWOExtension );
         return $this->oFactory->oUtil->getAOrB(
                 $_sFileNameWOExtension,
-                $_sFileNameWOExtension . '-admin-page-framework',
+                $_sFileNameWOExtension.'-admin-page-framework',
                 'admin-page-framework'
             )
-            . '.' . AdminPageFramework_Registry::VERSION
+            . '.'.AdminPageFramework_Registry::VERSION
             . '.zip';
         
     }

Braces +1 added lines, -1 removed lines

@@ -168,7 +168,7 @@
      * )      
      * </code>
      */
-    public function replyToFilterFileName( $sFileName, $sFieldID, $sInputID, $vExportingData, $oAdminPage ) { 
+    public function replyToFilterFileName( $sFileName, $sFieldID, $sInputID, $vExportingData, $oAdminPage ) {
 
         return isset( 
                 $_POST[ $this->oFactory->oProp->sOptionKey ][ $this->sSectionID ][ 'minified_script_name' ] 

include/class/admin/admin-page-framework/tool/minifier/AdminPageFrameworkLoader_AdminPage_Tool_Minifier_Minifier.php

Doc Comments +1 added lines

@@ -139,6 +139,7 @@
          * Modifies the class names of the minified script.
          * 
          * @since       3.4.6
+         * @param string $sCode
          */
         private function _modifyClassNames( $sCode ) {
 

Spacing +9 added lines, -9 removed lines

@@ -26,7 +26,7 @@ 
             4 
         );
         add_action( 
-            'export_name_' . $this->sPageSlug . '_' . $this->sTabSlug, 
+            'export_name_'.$this->sPageSlug.'_'.$this->sTabSlug, 
             array( $this, 'replyToFilterFileName' ), 10, 5 
         );
         
@@ -41,7 +41,7 @@ 
                 'title'             => __( 'Class Prefix', 'admin-page-framework-loader' ),
                 'type'              => 'text',
                 'description'       => __( 'Set alphanumeric characters for the class names', 'admin-page-framework-loader' )
-                    . ' ' .  __( 'For example, if you set here <code>MyPluginName_</code>, you will need to extend the class named <code>MyClassName_AdminPageFramework</code> instead of <code>AdminPageFramework</code>.', 'admin-page-framework-loader' )
+                    . ' '.__( 'For example, if you set here <code>MyPluginName_</code>, you will need to extend the class named <code>MyClassName_AdminPageFramework</code> instead of <code>AdminPageFramework</code>.', 'admin-page-framework-loader' )
                     . ' e.g.<code>MyPluginName_</code>',
                 'attributes'        => array(
                     'size'          => 30,
@@ -68,8 +68,8 @@ 
                 'type'              => 'export',
                 'label_min_width'   => 0,
                 'value'             => __( 'Download', 'adimn-page-framework-demo' ),
-                'file_name'         => 'admin-page-framework.min.php',  // the default file name. This will be modified by the filter.
-                'format'            => 'text',  // 'json', 'text', 'array'      
+                'file_name'         => 'admin-page-framework.min.php', // the default file name. This will be modified by the filter.
+                'format'            => 'text', // 'json', 'text', 'array'      
                 'description'       => __( 'Download the minified version.', 'admin-page-framework-loader' ),
             ) 
         );          
@@ -101,13 +101,13 @@ 
         if ( $aInput[ 'class_prefix' ] && empty( $_aMatches ) ) {
       
             // $variable[ 'sectioni_id' ]['field_id']
-            $_aErrors['class_prefix'] = __( 'The prefix must consist of alphanumeric with underscores.', 'admin-page-framework-loader' );
+            $_aErrors[ 'class_prefix' ] = __( 'The prefix must consist of alphanumeric with underscores.', 'admin-page-framework-loader' );
             $_bVerified = false;
                     
         }
 
         /* 4. An invalid value is found. */
-        if ( ! $_bVerified ) {
+        if ( !$_bVerified ) {
 
             /* 4-1. Set the error array for the input fields. */
             $oAdminPage->setFieldErrors( $_aErrors );     
@@ -128,7 +128,7 @@ 
      */
     public function replyToDownloadMinifiedVersion( $aSavedData, $sSubmittedFieldID, $sSubmittedInputID, $oAdminPage ) {
         
-        $_sMinifiedVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/library/admin-page-framework.min.php';
+        $_sMinifiedVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/library/admin-page-framework.min.php';
         if ( file_exists( $_sMinifiedVersionPath ) ) {
             return $this->_modifyClassNames( file_get_contents( $_sMinifiedVersionPath ) );
         }
@@ -147,8 +147,8 @@ 
                 : '';             
             
             return str_replace( 
-                'AdminPageFramework',         // search 
-                $_sPrefix . 'AdminPageFramework',         // replace
+                'AdminPageFramework', // search 
+                $_sPrefix.'AdminPageFramework', // replace
                 $sCode // subject
             );
         

Braces +1 added lines, -1 removed lines

@@ -168,7 +168,7 @@
      * )      
      * </code>
      */
-    public function replyToFilterFileName( $sFileName, $sFieldID, $sInputID, $vExportingData, $oAdminPage ) { 
+    public function replyToFilterFileName( $sFileName, $sFieldID, $sInputID, $vExportingData, $oAdminPage ) {
 
         return isset( 
                 $_POST[ $this->oFactory->oProp->sOptionKey ][ $this->sSectionID ][ 'minified_script_name' ] 

admin-page-framework-loader.php

Braces +5 added lines, -4 removed lines

@@ -185,7 +185,7 @@ 
      * @return      void
      */ 
     static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {  
+        if ( ! is_admin() ) {
             return; 
         }
         self::$_aAdminNotices[] = array(
@@ -200,7 +200,7 @@ 
          * @return      void
          */
         static public function _replyToSetAdminNotice() {
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {                
+            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
                 echo "<div class='" . esc_attr( $_aAdminNotice['class_attribute'] ) . " notice is-dismissible'>"
                         ."<p>" 
                             . sprintf( 
@@ -221,7 +221,7 @@ 
 if ( defined( 'DOING_UNINSTALL' ) ) { return; }
 
 // Include the library file 
-if ( ! class_exists( 'AdminPageFramework' ) ) {    
+if ( ! class_exists( 'AdminPageFramework' ) ) {
     // The development version is available in the repository cloned via GitHub.
     $_sDevelopmentVersionPath = dirname( __FILE__ ) . '/development/admin-page-framework.php';
     $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
@@ -250,7 +250,8 @@ 
 
 // Include the framework loader plugin components.
 include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/boot/AdminPageFrameworkLoader_Bootstrap.php' );
-if ( class_exists( 'AdminPageFrameworkLoader_Bootstrap' ) ) {   // for backward compatibility
+if ( class_exists( 'AdminPageFrameworkLoader_Bootstrap' ) ) {
+// for backward compatibility
     new AdminPageFrameworkLoader_Bootstrap( 
         AdminPageFrameworkLoader_Registry::$sFilePath,
         AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix

Indentation +29 added lines, -29 removed lines

@@ -16,15 +16,15 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-	const VERSION        = '3.7.0';    // <--- 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 VERSION        = '3.7.0';    // <--- 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 DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
-	const URI            = 'http://en.michaeluno.jp/';
-	const AUTHOR         = 'miunosoft (Michael Uno)';
-	const AUTHOR_URI     = 'http://en.michaeluno.jp/';
-	const COPYRIGHT      = 'Copyright (c) 2015, Michael Uno';
-	const LICENSE        = 'GPL v2 or later';
+    const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
+    const URI            = 'http://en.michaeluno.jp/';
+    const AUTHOR         = 'miunosoft (Michael Uno)';
+    const AUTHOR_URI     = 'http://en.michaeluno.jp/';
+    const COPYRIGHT      = 'Copyright (c) 2015, Michael Uno';
+    const LICENSE        = 'GPL v2 or later';
     const CONTRIBUTORS   = '';
 	
 }
@@ -55,7 +55,7 @@ 
      * @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.
@@ -70,12 +70,12 @@ 
      * 
      * These will be accessed from the bootstrap script.
      */
-	const TEXT_DOMAIN              = 'admin-page-framework-loader';
-	const TEXT_DOMAIN_PATH         = '/language';    
+    const TEXT_DOMAIN              = 'admin-page-framework-loader';
+    const TEXT_DOMAIN_PATH         = '/language';    
     	    
-	// These properties will be defined in the setUp() method.
-	static public $sFilePath = '';
-	static public $sDirPath  = '';
+    // These properties will be defined in the setUp() method.
+    static public $sFilePath = '';
+    static public $sDirPath  = '';
 	
     /**
      * Requirements.
@@ -136,30 +136,30 @@ 
     static public $aTaxonomies = array(
     );
     
-	/**
-	 * Sets up static properties.
+    /**
+     * Sets up static properties.
      * @return      void
-	 */
-	static public function setUp( $sPluginFilePath=null ) {
+     */
+    static public function setUp( $sPluginFilePath=null ) {
 	                    
-		self::$sFilePath = $sPluginFilePath ? $sPluginFilePath : __FILE__;
-		self::$sDirPath  = dirname( self::$sFilePath );
+        self::$sFilePath = $sPluginFilePath ? $sPluginFilePath : __FILE__;
+        self::$sDirPath  = dirname( self::$sFilePath );
 	    
-	}    
+    }    
 	
-	/**
-	 * Returns the URL with the given relative path to the plugin path.
-	 * 
-	 * <h3>Example</h3>
+    /**
+     * Returns the URL with the given relative path to the plugin path.
+     * 
+     * <h3>Example</h3>
      * <code>
      * AdminPageFrameworkLoader_Registry::getPluginURL( 'asset/css/meta_box.css' );
      * </code>
      * @since       3.5.0
      * @return      string
-	 */
-	public static function getPluginURL( $sRelativePath='' ) {
-		return plugins_url( $sRelativePath, self::$sFilePath );
-	}
+     */
+    public static function getPluginURL( $sRelativePath='' ) {
+        return plugins_url( $sRelativePath, self::$sFilePath );
+    }
     
     /**
      * Returns the information of this class.

Spacing +23 added lines, -23 removed lines

@@ -16,16 +16,16 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-	const VERSION        = '3.7.0';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+	const VERSION        = '3.7.0'; // <--- 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://en.michaeluno.jp/';
 	const AUTHOR         = 'miunosoft (Michael Uno)';
 	const AUTHOR_URI     = 'http://en.michaeluno.jp/';
 	const COPYRIGHT      = 'Copyright (c) 2015, Michael Uno';
 	const LICENSE        = 'GPL v2 or later';
-    const CONTRIBUTORS   = '';
+    const CONTRIBUTORS = '';
 	
 }
 /**
@@ -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';
         
     
     /**
@@ -117,7 +117,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',
@@ -140,7 +140,7 @@ 
 	 * Sets up static properties.
      * @return      void
 	 */
-	static public function setUp( $sPluginFilePath=null ) {
+	static public function setUp( $sPluginFilePath = null ) {
 	                    
 		self::$sFilePath = $sPluginFilePath ? $sPluginFilePath : __FILE__;
 		self::$sDirPath  = dirname( self::$sFilePath );
@@ -157,7 +157,7 @@ 
      * @since       3.5.0
      * @return      string
 	 */
-	public static function getPluginURL( $sRelativePath='' ) {
+	public static function getPluginURL( $sRelativePath = '' ) {
 		return plugins_url( $sRelativePath, self::$sFilePath );
 	}
     
@@ -184,13 +184,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' ) );
     }
@@ -200,12 +200,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>";
@@ -217,24 +217,24 @@ 
 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' ) ) { return; }
+if ( !defined( 'ABSPATH' ) ) { return; }
 if ( defined( 'DOING_UNINSTALL' ) ) { return; }
 
 // Include the library file 
-if ( ! class_exists( 'AdminPageFramework' ) ) {    
+if ( !class_exists( 'AdminPageFramework' ) ) {    
     // The development version is available in the repository cloned via GitHub.
-    $_sDevelopmentVersionPath = dirname( __FILE__ ) . '/development/admin-page-framework.php';
+    $_sDevelopmentVersionPath = dirname( __FILE__ ).'/development/admin-page-framework.php';
     $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
     include( 
         $_bDebugMode && file_exists( $_sDevelopmentVersionPath )
             ? $_sDevelopmentVersionPath
-            : dirname( __FILE__ ) . '/library/admin-page-framework/admin-page-framework.php'
+            : dirname( __FILE__ ).'/library/admin-page-framework/admin-page-framework.php'
     );
 } 
  
 if ( 
-    ! class_exists( 'AdminPageFramework_Registry' ) 
-    || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
+    !class_exists( 'AdminPageFramework_Registry' ) 
+    || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
     || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
 ) {
     AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -249,7 +249,7 @@ 
 }
 
 // Include the framework loader plugin components.
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/boot/AdminPageFrameworkLoader_Bootstrap.php' );
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/boot/AdminPageFrameworkLoader_Bootstrap.php' );
 if ( class_exists( 'AdminPageFrameworkLoader_Bootstrap' ) ) {   // for backward compatibility
     new AdminPageFrameworkLoader_Bootstrap( 
         AdminPageFrameworkLoader_Registry::$sFilePath,

development/admin-page-framework.php

Spacing +8 added lines, -8 removed lines

@@ -55,7 +55,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,12 +98,12 @@ 
      * 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::$sFileURI                 = plugins_url( '', 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,8 +130,8 @@ 
      */
     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(
@@ -179,9 +179,9 @@ 
  */
 final class AdminPageFramework_Bootstrap {
 
-    public function __construct( $sLibraryPath=__FILE__ ) {
+    public function __construct( $sLibraryPath = __FILE__ ) {
 
-        if ( ! $this->_isLoadable() ) {
+        if ( !$this->_isLoadable() ) {
             return;
         }