michaeluno / admin-page-framework

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

Braces +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
             return $_aNestedSections;
             
         }
-        private function getFAQContents()  {
+        private function getFAQContents() {
             
             $_aReplacements   = array(
                 '%PLUGIN_DIR_URL%'  => AdminPageFrameworkLoader_Registry::getPluginURL(),

Spacing +11 added lines, -11 removed lines

@@ -21,8 +21,8 @@ 
      */
     public function replyToLoadTab( $oAdminPage ) {
 
-        $_aSections  = $this->getContentsByHeader( $this->getFAQContents(), 4 );
-        foreach( $_aSections as $_iIndex => $_aContent ) {
+        $_aSections = $this->getContentsByHeader( $this->getFAQContents(), 4 );
+        foreach ( $_aSections as $_iIndex => $_aContent ) {
 
             $_sTitle   = $_aContent[ 0 ];
             $_sContent = $this->_getFAQSubSections( $_aContent[ 1 ] );
@@ -32,7 +32,7 @@ 
             $oAdminPage->addSettingSections(
                 $this->sPageSlug, // the target page slug
                 array(
-                    'section_id'        => 'faq_sections_' . $_iIndex,
+                    'section_id'        => 'faq_sections_'.$_iIndex,
                     'tab_slug'          => $this->sTabSlug,
                     'section_tab_slug'  => 'apf_faq',
                     'title'             => $_sTitle,
@@ -54,17 +54,17 @@ 
 
             $_aNestedSections = array();
             $_iLastIndex = count( $aItems ) - 1;
-            foreach( $aItems as $_iIndex => $_aContent ) {
+            foreach ( $aItems as $_iIndex => $_aContent ) {
 
-                $_oParser   = new AdminPageFramework_WPReadmeParser( $_aContent[ 1 ] );
+                $_oParser = new AdminPageFramework_WPReadmeParser( $_aContent[ 1 ] );
 
                 // If no sections, return the contents of the first item.
-                if ( ! $_aContent[ 0 ] ) {
+                if ( !$_aContent[ 0 ] ) {
                     return $_oParser->get();
                 }
 
-                $_aNestedSections[] = array(
-                    'section_id'        => 'faq_item_' . $_iIndex,
+                $_aNestedSections[ ] = array(
+                    'section_id'        => 'faq_item_'.$_iIndex,
                     'title'             => $_aContent[ 0 ],
                     'collapsible'       => array(
                         'toggle_all_button' => $_iLastIndex === $_iIndex
@@ -80,14 +80,14 @@ 
             return $_aNestedSections;
 
         }
-        private function getFAQContents()  {
+        private function getFAQContents() {
 
-            $_aReplacements   = array(
+            $_aReplacements = array(
                 '%PLUGIN_DIR_URL%'  => AdminPageFrameworkLoader_Registry::getPluginURL(),
                 '%WP_ADMIN_URL%'    => admin_url(),
             );
             $_oWPReadmeParser = new AdminPageFramework_WPReadmeParser(
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/readme.txt',
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/readme.txt',
                 $_aReplacements
             );
             return $_oWPReadmeParser->getRawSection( 'Frequently asked questions' );

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

Indentation +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
      */
     public function validate( $aInput, $aOldInput, $oFactory, $aSubmit ) {
     
-       // Local variables
+        // Local variables
         $_bIsValid = true;
         $_aErrors  = array();
       

Spacing +4 added lines, -4 removed lines

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

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 ) {
         

Indentation +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
     /*
      * Shared methods
      */
-    	/**
+        /**
          * Checks the src url of the enqueued script/style to determine whether or not to set up a attribute modification callback.
          *
          * If it is one of the framework added item, the method sets up a hook to modify the url to add custom attributes.

Spacing +42 added lines, -42 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.
@@ -143,8 +143,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,8 +204,8 @@ 
                         return $sSanitizedURL;
                     }
 
-                    $_sAttributes   = $this->getAttributes( $_aAttributes );
-                    return $sSanitizedURL . "' " . rtrim( $_sAttributes, "'\"" );
+                    $_sAttributes = $this->getAttributes( $_aAttributes );
+                    return $sSanitizedURL."' ".rtrim( $_sAttributes, "'\"" );
 
                 }
 
@@ -225,7 +225,7 @@ 
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
 
-        if ( $this->hasBeenCalled( 'COMMON_STYLES: ' . get_class( $this ) . '::' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_STYLES: '.get_class( $this ).'::'.__METHOD__ ) ) {
             return;
         }
         $_oCaller = $this->oProp->oCaller;
@@ -241,10 +241,10 @@ 
          */
         private function ___getCommonStyleTag( $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()
@@ -252,7 +252,7 @@ 
             $_sStyle     = $this->isDebugMode() ? $_sStyle : $this->getCSSMinified( $_sStyle );
             $_sStyle     = trim( $_sStyle );
             if ( $_sStyle ) {
-                echo "<style type='text/css' id='" . esc_attr( strtolower( $sIDPrefix ) ) . "'>"
+                echo "<style type='text/css' id='".esc_attr( strtolower( $sIDPrefix ) )."'>"
                         . $_sStyle
                     . "</style>";
             }
@@ -267,10 +267,10 @@ 
          */
         private function ___getCommonIEStyleTag( $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()
@@ -278,7 +278,7 @@ 
             $_sStyleIE  = $this->isDebugMode() ? $_sStyleIE : $this->getCSSMinified( $_sStyleIE );
             $_sStyleIE  = trim( $_sStyleIE );
             return $_sStyleIE
-                ? "<!--[if IE]><style type='text/css' id='" . esc_attr( strtolower( $sIDPrefix . "-ie" ) ) . "'>"
+                ? "<!--[if IE]><style type='text/css' id='".esc_attr( strtolower( $sIDPrefix."-ie" ) )."'>"
                         . $_sStyleIE
                     . "</style><![endif]-->"
                 : '';
@@ -297,23 +297,23 @@ 
      */
     protected function _printCommonScripts( $sIDPrefix, $sClassName ) {
 
-        if ( $this->hasBeenCalled( 'COMMON_SCRIPT: ' . get_class( $this ) . '::' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_SCRIPT: '.get_class( $this ).'::'.__METHOD__ ) ) {
             return;
         }
 
         $_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 ) {
+        if ( !$_sScript ) {
             return;
         }
-        echo "<script type='text/javascript' id='" . esc_attr( strtolower( $sIDPrefix ) ) . "'>"
+        echo "<script type='text/javascript' id='".esc_attr( strtolower( $sIDPrefix ) )."'>"
                 . '/* <![CDATA[ */'
                 . $_sScript
                 . '/* ]]> */'
@@ -331,7 +331,7 @@ 
      */
     protected function _printClassSpecificStyles( $sIDPrefix ) {
 
-        $_oCaller   = $this->oProp->oCaller;
+        $_oCaller = $this->oProp->oCaller;
         echo $this->_getClassSpecificStyleTag( $_oCaller, $sIDPrefix );
         echo $this->_getClassSpecificIEStyleTag( $_oCaller, $sIDPrefix );
 
@@ -352,18 +352,18 @@ 
             static $_iCallCount = 0;
 
             $_sFilterName = "style_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
                 return '';
             }
             $_sStyle = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyle );
             $_sStyle = $this->isDebugMode() ? $_sStyle : $this->getCSSMinified( $_sStyle );
             $_sStyle = trim( $_sStyle );
-            if ( ! $_sStyle ) {
+            if ( !$_sStyle ) {
                 return '';
             }
             $_iCallCount++;
-            $_sID = strtolower( "{$sIDPrefix}-" . $this->oProp->sClassName . "_{$_iCallCount}" );
-            return "<style type='text/css' id='" . esc_attr( $_sID ) . "'>"
+            $_sID = strtolower( "{$sIDPrefix}-".$this->oProp->sClassName."_{$_iCallCount}" );
+            return "<style type='text/css' id='".esc_attr( $_sID )."'>"
                     . $_sStyle
                 . "</style>";
 
@@ -379,18 +379,18 @@ 
             static $_iCallCountIE = 1;
 
             $_sFilterName = "style_ie_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
                 return '';
             }
             $_sStyleIE = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyleIE );
             $_sStyleIE = $this->isDebugMode() ? $_sStyleIE : $this->getCSSMinified( $_sStyleIE );
             $_sStyleIE = trim( $_sStyleIE );
-            if ( ! $_sStyleIE ) {
+            if ( !$_sStyleIE ) {
                 return '';
             }
             $_iCallCountIE++;
-            $_sID  = strtolower( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" );
-            return "<!--[if IE]><style type='text/css' id='" . esc_attr( $_sID ) . "'>"
+            $_sID = strtolower( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" );
+            return "<!--[if IE]><style type='text/css' id='".esc_attr( $_sID )."'>"
                     . $_sStyleIE
                 . "</style><![endif]-->";
 
@@ -407,18 +407,18 @@ 
 
         static $_iCallCount = 1;
         $_sFilterName = "script_{$this->oProp->sClassName}";
-        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+        if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
             return '';
         }
         $_sScript = $this->addAndApplyFilters( $this->oProp->oCaller, $_sFilterName, $this->oProp->sScript );
         $_sScript = trim( $_sScript );
-        if ( ! $_sScript ) {
+        if ( !$_sScript ) {
             return '';
         }
 
         $_iCallCount++;
         $_sID = strtolower( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" );
-        echo "<script type='text/javascript' id='" . esc_attr( $_sID ) . "'>"
+        echo "<script type='text/javascript' id='".esc_attr( $_sID )."'>"
                 . '/* <![CDATA[ */'
                 . $_sScript
                 . '/* ]]> */'
@@ -442,12 +442,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 );
 
     }
     /**
@@ -462,12 +462,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 );
 
     }
 
@@ -519,7 +519,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 ] );
         }
@@ -536,7 +536,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 +6 added lines, -3 removed lines

@@ -352,7 +352,8 @@ 
             static $_iCallCount = 0;
 
             $_sFilterName = "style_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
                 return '';
             }
             $_sStyle = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyle );
@@ -379,7 +380,8 @@ 
             static $_iCallCountIE = 1;
 
             $_sFilterName = "style_ie_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
                 return '';
             }
             $_sStyleIE = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyleIE );
@@ -407,7 +409,8 @@ 
 
         static $_iCallCount = 1;
         $_sFilterName = "script_{$this->oProp->sClassName}";
-        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
             return '';
         }
         $_sScript = $this->addAndApplyFilters( $this->oProp->oCaller, $_sFilterName, $this->oProp->sScript );

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

@@ -27,7 +27,7 @@ 
      */
     public function __construct( $oProp, $oMsg ) {
 
-        if ( ! $this->_shouldProceed( $oProp ) ) {
+        if ( !$this->_shouldProceed( $oProp ) ) {
             return;
         }
 
@@ -44,7 +44,7 @@ 
          */
         private function _shouldProceed( $oProp ) {
 
-            if ( $oProp->bIsAdminAjax || ! $oProp->bIsAdmin ) {
+            if ( $oProp->bIsAdminAjax || !$oProp->bIsAdmin ) {
                 return false;
             }
             return ( boolean ) $oProp->bShowDebugInfo;
@@ -71,7 +71,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;
         }
 
@@ -99,10 +99,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>";
 
@@ -120,8 +120,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':
@@ -150,7 +150,7 @@ 
             $_nLog = log( $nBytes, 1024 );
             $_iPower = ( int ) $_nLog;
             $_iSize = pow( 1024, $_nLog - $_iPower );
-            return $_iSize . $_aUnits[ $_iPower ];
+            return $_iSize.$_aUnits[ $_iPower ];
         }
 
 }

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

@@ -24,12 +24,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;
@@ -42,7 +42,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.
@@ -85,14 +85,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
@@ -116,7 +116,7 @@ 
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {
+                if ( !$aSection[ 'repeatable' ] ) {
                     return $sToggleAll;
                 }
                 
@@ -126,13 +126,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(

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

@@ -54,9 +54,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 );
 
     }
 

development/factory/_common/form/input/AdminPageFramework_Input_Base.php

Doc Comments +1 added lines, -3 removed lines

@@ -105,10 +105,8 @@
      * If a parameter is not passed, it returns the entire attribute array.
      * 
      * @since       3.5.3
-     * @return      string|array|null        The specified attribute value or the entire attribute array if not specified. 
+     * @return      string        The specified attribute value or the entire attribute array if not specified. 
      * If not set, null will be returned as the `getAttributes()` method will not list an attribute with the null value.
-     * @param       string      $sName      The attribute name.
-     * @param       string      $sDefault   The defaqult value if the value is not set.
      */
     public function getAttribute( /* $sName=null, $sDefault=null */ ) {
         $_aParams = func_get_args() + array(

Braces +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
      * @return      string      The set class selector(s).
      */
     public function addClass( /* $asSelectors1, $asSelectors2 */ ) {
-        foreach( func_get_args() as $_asSelectors ) {            
+        foreach( func_get_args() as $_asSelectors ) {
             $this->aAttributes['class'] = $this->getClassAttribute( 
                 $this->aAttributes['class'],
                 $_asSelectors

Spacing +7 added lines, -7 removed lines

@@ -63,9 +63,9 @@ 
      * @param       array       $aAttributes    The attribute array. A field definition array is deprecated.
      * @param       array       $aOptions       options that allows the user to set custom container tags and class selectors.
      */
-    public function __construct( array $aAttributes, array $aOptions=array() ) {
+    public function __construct( array $aAttributes, array $aOptions = array() ) {
 
-        $this->aAttributes  = $this->getElementAsArray( 
+        $this->aAttributes = $this->getElementAsArray( 
             $aAttributes, 
             'attributes', 
             $aAttributes    // if the above key is not set, this will be set
@@ -127,13 +127,13 @@ 
      * @return      string      The set class selector(s).
      */
     public function addClass( /* $asSelectors1, $asSelectors2 */ ) {
-        foreach( func_get_args() as $_asSelectors ) {            
-            $this->aAttributes['class'] = $this->getClassAttribute( 
-                $this->aAttributes['class'],
+        foreach ( func_get_args() as $_asSelectors ) {            
+            $this->aAttributes[ 'class' ] = $this->getClassAttribute( 
+                $this->aAttributes[ 'class' ],
                 $_asSelectors
             );
         }
-        return $this->aAttributes['class'];
+        return $this->aAttributes[ 'class' ];
     }
     
     /**
@@ -158,7 +158,7 @@ 
         // $this->aAttributes[ $_aParams[ 0 ] ] = $_aParams[ 1 ];
         $this->setMultiDimensionalArray(
             $this->aAttributes, 
-            $this->getElementAsArray( $_aParams, 0 ),   // $asAttributeName
+            $this->getElementAsArray( $_aParams, 0 ), // $asAttributeName
             $_aParams[ 1 ]  // $mValue
         );
     }