michaeluno / admin-page-framework

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

Spacing +4 added lines, -4 removed lines

@@ -25,14 +25,14 @@ 
             
         $_aItems     = $this->getContentsByHeader( $this->getReadmeContents(), 4 );
         $_iLastIndex = count( $_aItems ) - 1;
-        foreach( $_aItems as $_iIndex => $_aContent ) {
+        foreach ( $_aItems as $_iIndex => $_aContent ) {
 
             $_oParser   = new AdminPageFramework_WPReadmeParser( $_aContent[ 1 ] );
             $_sContent  = $_oParser->get();
             $oAdminPage->addSettingSections(    
                 $this->sPageSlug, // the target page slug  
                 array(
-                    'section_id'        => 'tips_' . $_iIndex,
+                    'section_id'        => 'tips_'.$_iIndex,
                     'title'             => $_aContent[ 0 ],
                     'collapsible'       => array(
                         'toggle_all_button' => $_iLastIndex === $_iIndex 
@@ -53,9 +53,9 @@ 
         /**
          * @return      string
          */
-        private function getReadMeContents()  {       
+        private function getReadMeContents() {       
             return $this->_getReadmeContents( 
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/readme.txt', // source path
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/readme.txt', // source path
                 '', // TOC title
                 array( 'Other Notes' )  // sections
             );

Braces +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         /**
          * @return      string
          */
-        private function getReadMeContents()  {       
+        private function getReadMeContents() {
             return $this->_getReadmeContents( 
                 AdminPageFrameworkLoader_Registry::$sDirPath . '/readme.txt', // source path
                 '', // TOC title

include/class/admin/demo/AdminPageFrameworkLoader_Demo.php

Indentation +6 added lines, -6 removed lines

@@ -9,12 +9,12 @@
  * @since        3.5.0
  */
 
- /**
-  * 
-  * 
-  * @action     do      admin_page_framework_loader_action_before_loading_demo
-  * @action     do      admin_page_framework_loader_action_after_loading_demo
-  */
+    /**
+     * 
+     * 
+     * @action     do      admin_page_framework_loader_action_before_loading_demo
+     * @action     do      admin_page_framework_loader_action_after_loading_demo
+     */
 class AdminPageFrameworkLoader_Demo {
     
     public function __construct() {

Spacing +5 added lines, -5 removed lines

@@ -19,14 +19,14 @@ 
     
     public function __construct() {
         
-        if ( ! $this->_shouldLoadDemo() ) {
+        if ( !$this->_shouldLoadDemo() ) {
             return;
         }
         
         do_action( 'admin_page_framework_loader_action_before_loading_demo' );
         
         // Include example components.
-        include( AdminPageFrameworkLoader_Registry::$sDirPath . '/example/admin-page-framework-demo-bootstrap.php' );
+        include( AdminPageFrameworkLoader_Registry::$sDirPath.'/example/admin-page-framework-demo-bootstrap.php' );
         
         do_action( 'admin_page_framework_loader_action_after_loading_demo' );
         
@@ -43,10 +43,10 @@ 
         
             // Check if the demo is enabled.
             $_oOption = AdminPageFrameworkLoader_Option::getInstance();
-            if ( ! $_oOption->get( 'enable_admin_pages' ) ) {
+            if ( !$_oOption->get( 'enable_admin_pages' ) ) {
                 return false;
             }            
-            if ( ! $_oOption->get( 'enable_demo' ) ) {
+            if ( !$_oOption->get( 'enable_demo' ) ) {
                 $this->_setPointerToolTips();
                 return false;
             }           
@@ -74,7 +74,7 @@ 
                 'options'   => array(
                     'content' => sprintf( '<h3> %1$s </h3> <p> %2$s </p>',
                         AdminPageFrameworkLoader_Registry::NAME,
-                        __( 'Check out the functionality of Admin Page Framework by enabling the demo.','admin-page-framework-loader' )
+                        __( 'Check out the functionality of Admin Page Framework by enabling the demo.', 'admin-page-framework-loader' )
                     ),
                     'position'  => array( 'edge' => 'left', 'align' => 'middle' )
                 )

development/factory/_common/_abstract/_controller/AdminPageFramework_Resource_Base.php

Doc Comments +4 added lines

@@ -232,6 +232,8 @@ 
      * @remark      The meta box class may be instantiated multiple times so prevent echoing the same styles multiple times.
      * @parameter   string      $sIDPrefix   The id selector embedded in the script tag.
      * @parameter   string      $sClassName  The class name that identify the call group. This is important for the meta-box class because it can be instantiated multiple times in one particular page.
+     * @param string $sIDPrefix
+     * @param string $sClassName
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
 
@@ -315,6 +317,8 @@ 
      * @remark      The meta box class may be instantiated multiple times so prevent echoing the same styles multiple times.
      * @parametr    string      $sIDPrefix      The id selector embedded in the script tag.
      * @parametr    string      $sClassName     The class name that identify the call group. This is important for the meta-box class because it can be instantiated multiple times in one particular page.
+     * @param string $sIDPrefix
+     * @param string $sClassName
      */
     protected function _printCommonScripts( $sIDPrefix, $sClassName ) {
         

Spacing +30 added lines, -30 removed lines

@@ -36,19 +36,19 @@ 
     
         /* The system internal keys. */
         'sSRC'          => null,
-        'aPostTypes'    => array(),     // for meta box class
+        'aPostTypes'    => array(), // for meta box class
         'sPageSlug'     => null,    
         'sTabSlug'      => null,
-        'sType'         => null,        // script or style
+        'sType'         => null, // script or style
         
         /* The below keys are for users. */
         'handle_id'     => null,
         'dependencies'  => array(),
-        'version'       => false,       // although the type should be string, the wp_enqueue_...() functions want false as the default value.
-        'translation'   => array(),     // only for scripts
-        'in_footer'     => false,       // only for scripts
-        'media'         => 'all',       // only for styles     
-        'attributes'    => array(),     // 3.3.0+ - the attribute array
+        'version'       => false, // although the type should be string, the wp_enqueue_...() functions want false as the default value.
+        'translation'   => array(), // only for scripts
+        'in_footer'     => false, // only for scripts
+        'media'         => 'all', // only for styles     
+        'attributes'    => array(), // 3.3.0+ - the attribute array
         
     );    
       
@@ -59,7 +59,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */
-    protected $_sClassSelector_Style    = 'admin-page-framework-style';
+    protected $_sClassSelector_Style = 'admin-page-framework-style';
     
     /**
      * Stores the class selector used to the class-specific script.
@@ -68,7 +68,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */    
-    protected $_sClassSelector_Script   = 'admin-page-framework-script';
+    protected $_sClassSelector_Script = 'admin-page-framework-script';
       
     /**
      * Stores hand IDs by resource url to look up handle id and add custom arguments.
@@ -142,8 +142,8 @@ 
      * @internal
      */
     
-    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs=array() ) {}
-    public function _forceToEnqueueScript( $sSRC, $aCustomArgs=array() ) {}
+    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs = array() ) {}
+    public function _forceToEnqueueScript( $sSRC, $aCustomArgs = array() ) {}
     
     /**
      * A helper function for the _replyToEnqueueScripts() and the `_replyToEnqueueStyle()` methods.
@@ -204,7 +204,7 @@ 
                     }
                     
                     $_sAttributes   = $this->getAttributes( $_aAttributes );
-                    $_sModifiedURL  = $sSanitizedURL . "' " . rtrim( $_sAttributes, "'\"" );    // '"
+                    $_sModifiedURL  = $sSanitizedURL."' ".rtrim( $_sAttributes, "'\"" ); // '"
 
                     return $_sModifiedURL;                    
                     
@@ -252,10 +252,10 @@ 
          */
         private function _getStyleTag( $oCaller, $sIDPrefix ) {
             
-            $_sStyle     = $this->addAndApplyFilters( 
+            $_sStyle = $this->addAndApplyFilters( 
                 $oCaller, 
                 array(
-                    "style_common_admin_page_framework",            // 3.2.1+
+                    "style_common_admin_page_framework", // 3.2.1+
                     "style_common_{$this->oProp->sClassName}",
                 ), 
                 AdminPageFramework_CSS::getDefaultCSS() 
@@ -268,7 +268,7 @@ 
                 // : $this->getCSSMinified( $_sStyle );
                 
             if ( $_sStyle ) {
-                echo "<style type='text/css' id='" . esc_attr( $sIDPrefix ) . "'>"
+                echo "<style type='text/css' id='".esc_attr( $sIDPrefix )."'>"
                         . $_sStyle
                     . "</style>";
             }
@@ -282,10 +282,10 @@ 
          */        
         private function _getIEStyleTag( $oCaller, $sIDPrefix ) {
                 
-            $_sStyleIE   = $this->addAndApplyFilters( 
+            $_sStyleIE = $this->addAndApplyFilters( 
                 $oCaller, 
                 array(
-                    "style_ie_common_admin_page_framework",         // 3.2.1+
+                    "style_ie_common_admin_page_framework", // 3.2.1+
                     "style_ie_common_{$this->oProp->sClassName}", 
                 ),
                 AdminPageFramework_CSS::getDefaultCSSIE() 
@@ -296,7 +296,7 @@ 
                 // ? trim( $_sStyleIE )
                 // : $this->getCSSMinified( $_sStyleIE );
             return $_sStyleIE
-                ? "<!--[if IE]><style type='text/css' id='" . esc_attr( $sIDPrefix . "-ie" ) . "'>"
+                ? "<!--[if IE]><style type='text/css' id='".esc_attr( $sIDPrefix."-ie" )."'>"
                         . $_sStyleIE
                     . "</style><![endif]-->"
                 : '';
@@ -329,14 +329,14 @@ 
         $_sScript = $this->addAndApplyFilters( 
             $this->oProp->oCaller, 
             array(
-                "script_common_admin_page_framework",       // 3.2.1+
+                "script_common_admin_page_framework", // 3.2.1+
                 "script_common_{$this->oProp->sClassName}", 
             ),
             AdminPageFramework_Property_Base::$_sDefaultScript 
         );
         $_sScript = trim( $_sScript );
         if ( $_sScript ) {
-            echo "<script type='text/javascript' id='" . esc_attr( $sIDPrefix ) . "'>"
+            echo "<script type='text/javascript' id='".esc_attr( $sIDPrefix )."'>"
                     . '/* <![CDATA[ */'
                     . $_sScript
                     . '/* ]]> */'
@@ -355,7 +355,7 @@ 
      */
     protected function _printClassSpecificStyles( $sIDPrefix ) {
            
-        $_oCaller   = $this->oProp->oCaller;     
+        $_oCaller = $this->oProp->oCaller;     
         echo $this->_getClassSpecificStyleTag( $_oCaller, $sIDPrefix );
         echo $this->_getClassSpecificIEStyleTag( $_oCaller, $sIDPrefix );
         
@@ -386,7 +386,7 @@ 
                 // : $this->getCSSMinified( $_sStyle );
             if ( $_sStyle ) {
                 $_iCallCount++;
-                return "<style type='text/css' id='" . esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" ) . "'>"
+                return "<style type='text/css' id='".esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" )."'>"
                         . $_sStyle
                     . "</style>";
             }         
@@ -414,7 +414,7 @@ 
                 // : $this->getCSSMinified( $_sStyleIE );
             if ( $_sStyleIE ) {
                 $_iCallCountIE++;
-                return "<!--[if IE]><style type='text/css' id='" . esc_attr( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" ) . "'>" 
+                return "<!--[if IE]><style type='text/css' id='".esc_attr( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" )."'>" 
                         . $_sStyleIE
                     . "</style><![endif]-->";
             }            
@@ -442,7 +442,7 @@ 
         $_sScript = trim( $_sScript );
         if ( $_sScript ) {
             $_iCallCount++;
-            echo "<script type='text/javascript' id='" . esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" ) . "'>" 
+            echo "<script type='text/javascript' id='".esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" )."'>" 
                     . '/* <![CDATA[ */'
                     . $_sScript
                     . '/* ]]> */'
@@ -467,12 +467,12 @@ 
     public function _replyToAddStyle() {
     
         $_oCaller = $this->oProp->oCaller;     
-        if ( ! $_oCaller->isInThePage() ) {
+        if ( !$_oCaller->isInThePage() ) {
             return; 
         }
         
         $this->_printCommonStyles( 'admin-page-framework-style-common', get_class() );
-        $this->_printClassSpecificStyles( $this->_sClassSelector_Style . '-' . $this->oProp->sStructureType );
+        $this->_printClassSpecificStyles( $this->_sClassSelector_Style.'-'.$this->oProp->sStructureType );
  
     }
     /**
@@ -487,12 +487,12 @@ 
     public function _replyToAddScript() {
 
         $_oCaller = $this->oProp->oCaller;     
-        if ( ! $_oCaller->isInThePage() ) {
+        if ( !$_oCaller->isInThePage() ) {
             return; 
         }
         
         $this->_printCommonScripts( 'admin-page-framework-script-common', get_class() );
-        $this->_printClassSpecificScripts( $this->_sClassSelector_Script . '-' . $this->oProp->sStructureType );
+        $this->_printClassSpecificScripts( $this->_sClassSelector_Script.'-'.$this->oProp->sStructureType );
         
     }        
     
@@ -544,7 +544,7 @@ 
      * @internal
      */    
     public function _replyToEnqueueStyles() {        
-        foreach( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
+        foreach ( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingStyle );
             unset( $this->oProp->aEnqueuingStyles[ $_sKey ] );
         }
@@ -561,7 +561,7 @@ 
      * @internal
      */
     public function _replyToEnqueueScripts() {     
-        foreach( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
+        foreach ( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingScript );     
             unset( $this->oProp->aEnqueuingScripts[ $_sKey ] );
         }

Braces +3 added lines, -3 removed lines

@@ -235,7 +235,7 @@ 
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
 
-        if ( self::$_bCommonStyleLoaded ) { 
+        if ( self::$_bCommonStyleLoaded ) {
             return; 
         }
         self::$_bCommonStyleLoaded = true;
@@ -543,7 +543,7 @@ 
      * @since       3.2.0   Changed it unset the enqueued item so that the method can be called multiple times.
      * @internal
      */    
-    public function _replyToEnqueueStyles() {        
+    public function _replyToEnqueueStyles() {
         foreach( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingStyle );
             unset( $this->oProp->aEnqueuingStyles[ $_sKey ] );
@@ -560,7 +560,7 @@ 
      * @since       3.2.0   Changed it unset the enqueued item so that the method can be called multiple times.
      * @internal
      */
-    public function _replyToEnqueueScripts() {     
+    public function _replyToEnqueueScripts() {
         foreach( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingScript );     
             unset( $this->oProp->aEnqueuingScripts[ $_sKey ] );

development/factory/_common/_abstract/_model/AdminPageFramework_Property_Base.php

Doc Comments +2 added lines

@@ -370,6 +370,8 @@
      * Sets up necessary property values.
      * 
      * @remark      This class gets instantiated in every factory class so the constructor should be lightest as possible.
+     * @param string $sCapability
+     * @param string $sTextDomain
      */
     public function __construct( $oCaller, $sCallerPath, $sClassName, $sCapability, $sTextDomain, $sStructureType ) {
                 

Spacing +14 added lines, -14 removed lines

@@ -321,14 +321,14 @@ 
      * @since       3.7.0
      */
     public $aFormCallbacks = array(
-        'hfID'              => null,    // the input id attribute
-        'hfTagID'           => null,    // the field container id attribute
-        'hfName'            => null,    // the field name attribute
-        'hfNameFlat'        => null,    // the flat field name attribute
+        'hfID'              => null, // the input id attribute
+        'hfTagID'           => null, // the field container id attribute
+        'hfName'            => null, // the field name attribute
+        'hfNameFlat'        => null, // the flat field name attribute
         // @todo Document the differences between `hfName` and `hfInputName`
-        'hfInputName'       => null,    // 3.6.0+   the field input name attribute
-        'hfInputNameFlat'   => null,    // 3.6.0+   the flat field input name 
-        'hfClass'           => null,    // the class attribute       
+        'hfInputName'       => null, // 3.6.0+   the field input name attribute
+        'hfInputNameFlat'   => null, // 3.6.0+   the flat field input name 
+        'hfClass'           => null, // the class attribute       
     );
             
     /**
@@ -437,7 +437,7 @@ 
              */
             private function ___getURLQuery() {
 
-                if ( ! $this->bIsAdminAjax ) {
+                if ( !$this->bIsAdminAjax ) {
                     return $_GET;
                 }
                 parse_str(
@@ -453,7 +453,7 @@ 
          * @since       3.7.9
          */
         private function ___setGlobals() {
-            if ( ! isset( $GLOBALS[ 'aAdminPageFramework' ] ) ) {
+            if ( !isset( $GLOBALS[ 'aAdminPageFramework' ] ) ) {
                 $GLOBALS[ 'aAdminPageFramework' ] = array( 
                     'aFieldFlags' => array() 
                 );
@@ -524,8 +524,8 @@ 
             'hfTagID'                           => array( $this->oCaller, '_replyToGetInputTagIDAttribute' ), // the fields & fieldset & field row container id attribute
             'hfName'                            => array( $this->oCaller, '_replyToGetFieldNameAttribute' ), // the input name attribute
             'hfNameFlat'                        => array( $this->oCaller, '_replyToGetFlatFieldName' ), // the flat input name attribute
-            'hfInputName'                       => array( $this->oCaller, '_replyToGetInputNameAttribute' ),    // 3.6.0+   the field input name attribute
-            'hfInputNameFlat'                   => array( $this->oCaller, '_replyToGetFlatInputName' ),    // 3.6.0+   the flat field input name                 
+            'hfInputName'                       => array( $this->oCaller, '_replyToGetInputNameAttribute' ), // 3.6.0+   the field input name attribute
+            'hfInputNameFlat'                   => array( $this->oCaller, '_replyToGetFlatInputName' ), // 3.6.0+   the flat field input name                 
             'hfClass'                           => array( $this->oCaller, '_replyToGetInputClassAttribute' ), // the class attribute
             'hfSectionName'                     => array( $this->oCaller, '_replyToGetSectionName' ), // 3.6.0+            
         ) + $this->aFormCallbacks;        
@@ -599,7 +599,7 @@ 
      * @remark      The information can be used to embed into the footer etc.
      * @return      array       The information of the script.
      */  
-    protected function getCallerInfo( $sCallerPath='' ) {
+    protected function getCallerInfo( $sCallerPath = '' ) {
 
         if ( isset( self::$_aScriptDataCaches[ $sCallerPath ] ) ) {
             return self::$_aScriptDataCaches[ $sCallerPath ];
@@ -717,11 +717,11 @@ 
 
         // 3.7.9 Moved from the constructor to make it lighter.
         if ( 'sClassHash' === $sName ) {
-            $this->sClassHash       = md5( $this->sClassName );    
+            $this->sClassHash = md5( $this->sClassName );    
             return $this->sClassHash;
         }
         if ( 'sScriptType' === $sName ) {
-            $this->sScriptType      = $this->_getCallerType( $this->sCallerPath );    // 3.7.6+        
+            $this->sScriptType = $this->_getCallerType( $this->sCallerPath ); // 3.7.6+        
             return $this->sScriptType;
         }
         if ( 'oUtil' === $sName ) {

development/factory/_common/_abstract/_view/AdminPageFramework_PageLoadInfo_Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
          * Converts bytes to HR.
          *
          * @access private
-         * @param mixed $bytes
+         * @param mixed $nBytes
          * @remark This is influenced by the work of Mike Jolley.
          * @see http://mikejolley.com/projects/wp-page-load-stats/
          */

Spacing +10 added lines, -10 removed lines

@@ -28,7 +28,7 @@ 
      */
     public function __construct( $oProp, $oMsg ) {
                 
-        if ( ! $this->_shouldProceed( $oProp ) ) {
+        if ( !$this->_shouldProceed( $oProp ) ) {
             return;
         }
             
@@ -45,7 +45,7 @@ 
          */
         private function _shouldProceed( $oProp ) {
         
-            if ( $oProp->bIsAdminAjax || ! $oProp->bIsAdmin ) {
+            if ( $oProp->bIsAdminAjax || !$oProp->bIsAdmin ) {
                 return false;
             }     
             return ( boolean ) $oProp->bShowDebugInfo;
@@ -72,7 +72,7 @@ 
     public function _replyToGetPageLoadInfo( $sFooterHTML ) {
         
         // 3.8.8+ The `bShowDebugInfo` property may be updated by the user during the page load.
-        if ( ! $this->oProp->bShowDebugInfo ) {
+        if ( !$this->oProp->bShowDebugInfo ) {
             return $sFooterHTML;
         }
         
@@ -100,10 +100,10 @@ 
             $_sInitialMemoryUsage = round( $this->_convertBytesToHR( $this->_nInitialMemoryUsage ), 2 );
             return "<div id='admin-page-framework-page-load-stats'>"
                 . "<ul>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'queries_in_seconds' ), $_nQueryCount, $_nSeconds ) . "</li>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'out_of_x_memory_used' ), $_nMemoryUsage, $_nMemoryLimit, round( ( $_nMemoryUsage / $_nMemoryLimit ), 2 ) * 100 . '%' ) . "</li>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'peak_memory_usage' ), $_nMemoryPeakUsage ) . "</li>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'initial_memory_usage' ), $_sInitialMemoryUsage ) . "</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'queries_in_seconds' ), $_nQueryCount, $_nSeconds )."</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'out_of_x_memory_used' ), $_nMemoryUsage, $_nMemoryLimit, round( ( $_nMemoryUsage / $_nMemoryLimit ), 2 ) * 100.'%' )."</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'peak_memory_usage' ), $_nMemoryPeakUsage )."</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'initial_memory_usage' ), $_sInitialMemoryUsage )."</li>"
                 . "</ul>"
             . "</div>";          
             
@@ -121,8 +121,8 @@ 
          */
         private function _convertToNumber( $nSize ) {
             
-            $_nReturn     = substr( $nSize, 0, -1 );
-            switch( strtoupper( substr( $nSize, -1 ) ) ) {
+            $_nReturn = substr( $nSize, 0, -1 );
+            switch ( strtoupper( substr( $nSize, -1 ) ) ) {
                 case 'P':
                     $_nReturn *= 1024;
                 case 'T':
@@ -151,7 +151,7 @@ 
             $_nLog = log( $nBytes, 1024 );
             $_iPower = ( int ) $_nLog;
             $_iSize = pow( 1024, $_nLog - $_iPower );
-            return $_iSize . $_aUnits[ $_iPower ];
+            return $_iSize.$_aUnits[ $_iPower ];
         }
 
 }

Braces +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
             return $sFooterHTML;
         }
         
-        if ( self::$_bLoadedPageLoadInfo ) { 
+        if ( self::$_bLoadedPageLoadInfo ) {
             return $sFooterHTML; 
         }
         self::$_bLoadedPageLoadInfo = true;     

development/factory/_common/form/_model/AdminPageFramework_Form_Model___LastInput.php

Doc Comments -1 removed lines

@@ -110,7 +110,6 @@
      * Retrieves the settings error array set by the user in the validation callback.
      * 
      * @since       3.7.8
-     * @param       boolean     $bDelete    whether or not the transient should be deleted after retrieving it. 
      * @return      array
      */
     public function get() {

Spacing +5 added lines, -5 removed lines

@@ -56,7 +56,7 @@ 
             $_sPageSlug = $this->getElement( $_GET, 'page', '' );
             $_sTabSlug  = $this->getElement( $_GET, 'tab', '' );            
             $_sUserID   = get_current_user_id();
-            return "apf_li_" . md5( 
+            return "apf_li_".md5( 
                 $_sPageNow 
                 . $_sPageSlug
                 . $_sTabSlug
@@ -94,13 +94,13 @@ 
          * @return      void
          */ 
         public function _replyToSave() {
-            if ( ! isset( self::$_aLastInputs ) ) { 
+            if ( !isset( self::$_aLastInputs ) ) { 
                 return; 
             }
             $this->setTransient( 
                 $this->sTransientKey,
                 self::$_aLastInputs, 
-                60*60     // store it for 1 hour 
+                60 * 60     // store it for 1 hour 
             );    
         }    
     
@@ -120,9 +120,9 @@ 
             $_aLastInputs = self::$_aCaches[ $this->sTransientKey ];
         } else {
             $_aLastInputs = $this->getTransient( $this->sTransientKey );
-            self::$_aCaches[ $this->sTransientKey ]  = $_aLastInputs;
+            self::$_aCaches[ $this->sTransientKey ] = $_aLastInputs;
             if ( false !== $_aLastInputs ) {
-                $this->delete();    // deletes at the end of the script.
+                $this->delete(); // deletes at the end of the script.
             }
         }        
         

Braces +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
          * @return      void
          */ 
         public function _replyToSave() {
-            if ( ! isset( self::$_aLastInputs ) ) { 
+            if ( ! isset( self::$_aLastInputs ) ) {
                 return; 
             }
             $this->setTransient( 

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___Format_CollapsibleSection.php

Doc Comments +2 added lines

@@ -77,6 +77,8 @@
     }
         /**
          * @since       3.6.0
+         * @param boolean $abCollapsible
+         * @param string $sTitle
          * @return      array
          */
         private function _getArguments( $abCollapsible, $sTitle, array $aSection ) {

Spacing +13 added lines, -13 removed lines

@@ -23,12 +23,12 @@ 
      * @since       3.6.0       Moved from `AdminPageFramework_FormDefinition`.
      */
     static public $aStructure = array(
-        'title'                     => null,        // (string)  the section title will be assigned by default in the section formatting method.
-        'is_collapsed'              => true,        // (boolean) whether it is already collapsed or expanded
-        'toggle_all_button'         => null,        // (boolean|string|array) the position of where to display the toggle-all button that toggles the folding state of all collapsible sections. Accepts the following values. 'top-right', 'top-left', 'bottom-right', 'bottom-left'. If true is passed, the default 'top-right' will be used. To not to display, do not set any or pass `false` or `null`.
-        'collapse_others_on_expand' => true,        // (boolean) whether the other collapsible sections should be folded when the section is unfolded.
-        'container'                 => 'sections',  // (string) the container element that collapsible styling gets applied to. Either 'sections' or 'section' is accepted.
-        'type'                      => 'box',       // 3.7.0+  (string)  supported types 'box', 'button' Default: `box`. The `button` type is only supported when the `container` argument is `section`.
+        'title'                     => null, // (string)  the section title will be assigned by default in the section formatting method.
+        'is_collapsed'              => true, // (boolean) whether it is already collapsed or expanded
+        'toggle_all_button'         => null, // (boolean|string|array) the position of where to display the toggle-all button that toggles the folding state of all collapsible sections. Accepts the following values. 'top-right', 'top-left', 'bottom-right', 'bottom-left'. If true is passed, the default 'top-right' will be used. To not to display, do not set any or pass `false` or `null`.
+        'collapse_others_on_expand' => true, // (boolean) whether the other collapsible sections should be folded when the section is unfolded.
+        'container'                 => 'sections', // (string) the container element that collapsible styling gets applied to. Either 'sections' or 'section' is accepted.
+        'type'                      => 'box', // 3.7.0+  (string)  supported types 'box', 'button' Default: `box`. The `button` type is only supported when the `container` argument is `section`.
     );   
     
     public $abCollapsible = false;
@@ -41,7 +41,7 @@ 
      * @remark      The framework will not pass this parameter when formatting a section definition array.
      * It will be passed when the framework is rendering a form table to generate collapsible elements.
      */
-    public $aSection      = array();
+    public $aSection = array();
     
     /**
      * Sets up properties.
@@ -84,14 +84,14 @@ 
             
             $_aCollapsible = $this->getAsArray( $this->abCollapsible ) + array(
                 'title'     => $sTitle,
-            ) +  self::$aStructure;
+            ) + self::$aStructure;
             
             $_aCollapsible[ 'toggle_all_button' ] = implode( 
                 ',', 
                 $this->getAsArray( $_aCollapsible[ 'toggle_all_button' ] ) 
             );
             
-            if ( ! empty( $aSection ) ) {
+            if ( !empty( $aSection ) ) {
                 $_aCollapsible[ 'toggle_all_button' ] = $this->_getToggleAllButtonArgument( 
                     $_aCollapsible[ 'toggle_all_button' ], 
                     $aSection
@@ -115,7 +115,7 @@ 
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {            
+                if ( !$aSection[ 'repeatable' ] ) {            
                     return $sToggleAll;
                 }
                 
@@ -125,13 +125,13 @@ 
                 }
                 
                 // Disable the toggle all button for middle sub-sections in repeatable sections.
-                if ( ! $aSection[ '_is_first_index' ] && ! $aSection[ '_is_last_index' ] ) {
+                if ( !$aSection[ '_is_first_index' ] && !$aSection[ '_is_last_index' ] ) {
                     return 0;
                 }            
                 
                 $_aToggleAll = $this->getAOrB(
-                    true === $sToggleAll || 1 ===  $sToggleAll, // evaluate
-                    array( 'top-right', 'bottom-right' ),   // if true
+                    true === $sToggleAll || 1 === $sToggleAll, // evaluate
+                    array( 'top-right', 'bottom-right' ), // if true
                     explode( ',', $sToggleAll ) // if false
                 );            
                 $_aToggleAll = $this->getAOrB(

Braces +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {            
+                if ( ! $aSection[ 'repeatable' ] ) {
                     return $sToggleAll;
                 }
                 

development/factory/_common/form/_view/generator/field/AdminPageFramework_Form_View___Generate_Field_Base.php

Doc Comments -1 removed lines

@@ -23,7 +23,6 @@
      * @internal
      * @since       3.5.3
      * @since       3.6.0       Moved from `AdminPageFramework_FormDefinition`.
-     * @param       array       $aFieldset     a field definition array.
      * @return      boolean
      */
     protected function _isSectionSet() {

development/factory/_common/form/_view/generator/field/AdminPageFramework_Form_View___Generate_FieldInputName.php

Doc Comments +1 added lines

@@ -60,6 +60,7 @@
         /**
          * Applies the subject string to the set callback filter function.
          * @since       3.6.0
+         * @param string $sSubject
          */
         protected function _getFiltered( $sSubject ) {
             return is_callable( $this->hfCallback )

Spacing +2 added lines, -2 removed lines

@@ -51,9 +51,9 @@
         $_sIndex = $this->getAOrB(
             '0' !== $this->sIndex && empty( $this->sIndex ),
             '',
-            "[" . $this->sIndex . "]"
+            "[".$this->sIndex."]"
         );        
-        return $this->_getFiltered( $this->_getFieldName() . $_sIndex );
+        return $this->_getFiltered( $this->_getFieldName().$_sIndex );
         
     }