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

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

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/_view/generator/section/AdminPageFramework_Form_View___Generate_Section_Base.php

Doc Comments +1 added lines

@@ -51,6 +51,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

@@ -93,8 +93,8 @@
         
         // Extract the first part as it does not have braces
         $_sName = array_shift( $aParts );
-        foreach( $aParts as $_sPart ) {
-            $_sName .= '[' . $_sPart . ']';
+        foreach ( $aParts as $_sPart ) {
+            $_sName .= '['.$_sPart.']';
         }
         return $_sName;