michaeluno / admin-page-framework

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

@@ -28,11 +28,11 @@ 
      */
     public function __construct( $oProp, $oMsg ) {
         
-        if ( $oProp->bIsAdminAjax || ! $oProp->bIsAdmin ) {
+        if ( $oProp->bIsAdminAjax || !$oProp->bIsAdmin ) {
             return;
         }     
         
-        if ( ! defined( 'WP_DEBUG' ) || ! WP_DEBUG ) {
+        if ( !defined( 'WP_DEBUG' ) || !WP_DEBUG ) {
             return;
         }
             
@@ -78,10 +78,10 @@ 
         return $sFooterHTML
             . "<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>";
         
@@ -99,8 +99,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':
@@ -129,7 +129,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

@@ -63,7 +63,7 @@
      */
     public function _replyToGetPageLoadInfo( $sFooterHTML ) {
         
-        if ( self::$_bLoadedPageLoadInfo ) { 
+        if ( self::$_bLoadedPageLoadInfo ) {
             return; 
         }
         self::$_bLoadedPageLoadInfo = true;     

development/factory/_common/_abstract/AdminPageFramework_Factory_Controller.php

Doc Comments +2 added lines, -2 removed lines

@@ -187,7 +187,7 @@ 
      * @access      public
      * @remark      The actual registration will be performed in the `_replyToRegisterSettings()` method with the `admin_menu` hook.
      * @remark      The `$oForm` property should be created in each extended class.
-     * @param       array|string        $aSection       the section array. If a string is passed, it is considered as a target page slug that will be used as a page slug element from the next call so that the element can be omitted.
+     * @param       array|string        $aSectionset       the section array. If a string is passed, it is considered as a target page slug that will be used as a page slug element from the next call so that the element can be omitted.
      * <h4>Section Definition Array</h4>
      * <ul>
      *      <li>**section_id** - (string) the section ID. Avoid using non-alphabetic characters except underscore and numbers.</li>
@@ -299,7 +299,7 @@ 
      * @since        3.0.0   The scope changed to public to indicate the users will use.
      * @return       void
      * @remark       The $oForm property should be created in each extended class.
-     * @param        array|string       $asField        A field definition array or a string of the target section id.
+     * @param        array|string       $asFieldset        A field definition array or a string of the target section id.
     * <h4>Built-in Field Types</h4>
     * <ul>
     *       <li>**text** - a normal field to enter text input.</li>

Spacing +11 added lines, -11 removed lines

@@ -43,7 +43,7 @@ 
      * @return      array|string
      * @since       DEVVER
      */
-    public function getMessage( $sKey='' ) {
+    public function getMessage( $sKey = '' ) {
         return $this->oMsg->get( $sKey );
     }
        
@@ -63,7 +63,7 @@ 
      * @return      array       The array holing the queued items.
      * @internal
      */
-    public function enqueueStyles( $aSRCs, $_vArg2=null ) {} 
+    public function enqueueStyles( $aSRCs, $_vArg2 = null ) {} 
     
     /**
      * Enqueues a style of the given source.
@@ -83,7 +83,7 @@ 
      * @return      string      The style handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */
-    public function enqueueStyle( $sSRC, $_vArg2=null ) {}
+    public function enqueueStyle( $sSRC, $_vArg2 = null ) {}
     
     /**
      * Enqueues scripts by the given sources.
@@ -104,7 +104,7 @@ 
      * @return      array        The array holding the queued items.
      * @internal
      */    
-    public function enqueueScripts( $aSRCs, $_vArg2=null ) {}
+    public function enqueueScripts( $aSRCs, $_vArg2 = null ) {}
     /**
      * Enqueues a script by the given source.
      *  
@@ -136,7 +136,7 @@ 
      * @return      string      The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */    
-    public function enqueueScript( $sSRC, $_vArg2=null ) {}    
+    public function enqueueScript( $sSRC, $_vArg2 = null ) {}    
     
     /*
      * Help Pane
@@ -154,7 +154,7 @@ 
      * @since       2.1.0
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the `replyToRegisterHelpTabTextForMetaBox()` method.
      */ 
-    public function addHelpText( $sHTMLContent, $sHTMLSidebarContent="" ) {
+    public function addHelpText( $sHTMLContent, $sHTMLSidebarContent = "" ) {
         if ( method_exists( $this->oHelpPane, '_addHelpText' ) ) {
             $this->oHelpPane->_addHelpText( $sHTMLContent, $sHTMLSidebarContent );
         }
@@ -190,7 +190,7 @@ 
      */    
     public function addSettingSections( /* $aSection1, $aSection2=null, $_and_more=null */ ) {
         
-        foreach( func_get_args() as $_asSectionset ) { 
+        foreach ( func_get_args() as $_asSectionset ) { 
             $this->addSettingSection( $_asSectionset ); 
         }
         
@@ -261,7 +261,7 @@ 
      */
     public function addSettingSection( $aSectionset ) {
         
-        if ( ! is_array( $aSectionset ) ) { 
+        if ( !is_array( $aSectionset ) ) { 
             return; 
         }
         
@@ -294,7 +294,7 @@ 
     * @return       void
     */ 
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $_aFieldset ) { 
+        foreach ( func_get_args() as $_aFieldset ) { 
             $this->addSettingField( $_aFieldset ); 
         }
     }    
@@ -671,7 +671,7 @@ 
     * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
     * @return       void
     */      
-    public function setSettingNotice( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
+    public function setSettingNotice( $sMessage, $sType = 'error', $asAttributes = array(), $bOverride = true ) {
         $this->oForm->setSubmitNotice(
             $sMessage,
             $sType,
@@ -690,7 +690,7 @@ 
      * @param       string      $sType If empty, the method will check if a message exists in all types. Otherwise, it checks the existence of a message of the specified type.
      * @return      boolean     True if a setting notice is set; otherwise, false.
      */
-    public function hasSettingNotice( $sType='' ) {
+    public function hasSettingNotice( $sType = '' ) {
         return $this->oForm->hasSubmitNotice( $sType );
     }
 

Braces +3 added lines, -3 removed lines

@@ -169,7 +169,7 @@ 
      */    
     public function addSettingSections( /* $aSection1, $aSection2=null, $_and_more=null */ ) {
         
-        foreach( func_get_args() as $_asSectionset ) { 
+        foreach( func_get_args() as $_asSectionset ) {
             $this->addSettingSection( $_asSectionset ); 
         }
         
@@ -238,7 +238,7 @@ 
      */
     public function addSettingSection( $aSectionset ) {
         
-        if ( ! is_array( $aSectionset ) ) { 
+        if ( ! is_array( $aSectionset ) ) {
             return; 
         }
         
@@ -271,7 +271,7 @@ 
     * @return       void
     */ 
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $_aFieldset ) { 
+        foreach( func_get_args() as $_aFieldset ) {
             $this->addSettingField( $_aFieldset ); 
         }
     }    

Indentation +186 added lines, -186 removed lines

@@ -291,18 +291,18 @@ 
     }     
         
     /**
-    * Adds form fields.
-    * 
-    * It inserts the given field definition arrays into the class property and later they are parsed when fields are registered. The field definition array requires specific keys. Refer to the parameter section of this method.
-    * 
-    * @since        2.0.0
-    * @since        3.5.3       Removed the parameter declarations as they are caught with the func_get_args().
-    * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
-    * @param        array       the field definition array.
-    * @param        array       (optional) another field array.
-    * @param        array       (optional) add more field arrays to the next parameters as many as necessary.
-    * @return       void
-    */ 
+     * Adds form fields.
+     * 
+     * It inserts the given field definition arrays into the class property and later they are parsed when fields are registered. The field definition array requires specific keys. Refer to the parameter section of this method.
+     * 
+     * @since        2.0.0
+     * @since        3.5.3       Removed the parameter declarations as they are caught with the func_get_args().
+     * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
+     * @param        array       the field definition array.
+     * @param        array       (optional) another field array.
+     * @param        array       (optional) add more field arrays to the next parameters as many as necessary.
+     * @return       void
+     */ 
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
         foreach( func_get_args() as $_aFieldset ) { 
             $this->addSettingField( $_aFieldset ); 
@@ -333,158 +333,158 @@ 
      * @return       void
      * @remark       The $oForm property should be created in each extended class.
      * @param        array|string       $asField        A field definition array or a string of the target section id.
-    * <h4>Built-in Field Types</h4>
-    * <ul>
-    *       <li>[text](./class-AdminPageFramework_FieldType_text.html) - a normal field to enter short text input.</li>
-    *       <li>[password](./class-AdminPageFramework_FieldType_text.html) - a masked text input field.</li>
-    *       <li>[textarea](./class-AdminPageFramework_FieldType_textarea.html) - a text input field with multiple lines. It supports rich text editors.</li>
-    *       <li>[radio](./class-AdminPageFramework_FieldType_radio.html) - a set of radio buttons that lets the user pick an option.</li>
-    *       <li>[checkbox](./class-AdminPageFramework_FieldType_checkbox.html) - a check box that lets the user enable/disable an item.</li>
-    *       <li>[select](./class-AdminPageFramework_FieldType_select.html) - a drop-down list that lest the user pick one or more item(s) from a list.</li>
-    *       <li>[hidden](./class-AdminPageFramework_FieldType_hidden.html) - a field with invisible input values.</li>
-    *       <li>[file](./class-AdminPageFramework_FieldType_file.html) - a file uploader that lets the user upload files.</li>
-    *       <li>[image](./class-AdminPageFramework_FieldType_image.html) - a text field with an image uploader that lets the user set the image URL.</li>
-    *       <li>[media](./class-AdminPageFramework_FieldType_media.html) - a text field with a media uploader that lets the user set the file URL.</li>
-    *       <li>[color](./class-AdminPageFramework_FieldType_color.html) - a text field with a color picker.</li>
-    *       <li>[submit](./class-AdminPageFramework_FieldType_submit.html) - a submit button that lets the user send the form.</li>
-    *       <li>[export](./class-AdminPageFramework_FieldType_export.html) - a custom submit field that lets the user export stored data.</li>
-    *       <li>[import](./class-AdminPageFramework_FieldType_import.html) - a custom combination field of file and submit fields that let the user import data.</li>
-    *       <li>[posttype](./class-AdminPageFramework_FieldType_posttype.html) - a check-list of post types enabled on the site.</li>
-    *       <li>[taxonomy](./class-AdminPageFramework_FieldType_taxonomy.html) - a set of check-lists of taxonomies enabled on the site in a tabbed box.</li>
-    *       <li>[size](./class-AdminPageFramework_FieldType_size.html) - a combination field of the text and the select fields that let the user set sizes with a selectable unit.</li>
-    *       <li>[section_title](./class-AdminPageFramework_FieldType_section_title.html) - [3.0.0+] a text field type that will be placed in the section title so that it lets the user set the section title. Note that only one field with this field type is allowed per a section.</li>
-    *       <li>[system](./class-AdminPageFramework_FieldType_system.html) - [3.3.0+] a custom text area field that displays the system information including the PHP settings, the framework version, MySQL version etc.</li>
-    *       <li>[inline_mixed](./class-AdminPageFramework_FieldType_inline_mixed.html) - [3.8.0+] a field that include inner fields with different field types. </li>
-    * </ul>
-    * <h4>Field Definition Array</h4>
-    * <ul>
-    *       <li>**field_id** - (required, string) the field ID. Avoid using non-alphabetic characters except underscore and numbers.</li>
-    *       <li>**type** - (optional, string) the type of the field. The supported types are listed below. When creating nested fields, this argument can be omitted.</li>
-    *       <li>**section_id** - (optional, string) the section ID that the field belongs to. If not set, the internal `_default` section ID will be assigned.</li>
-    *       <li>**title** - (optional, string) the title of the section.</li>
-    *       <li>**description** - (optional, string) the description of the field which is inserted into the after the input field tag.</li>
-    *       <li>**tip** - (optional, string) the tip for the field which is displayed when the mouse is hovered over the field title.</li>
-    *       <li>**capability** - (optional, string) the <a href="http://codex.wordpress.org/Roles_and_Capabilities">access level</a> of the section. If the page visitor does not have sufficient capability, the section will be invisible to them.</li>
-    *       <li>**error_message** - (optional, string) the error message to display above the input field.</li>
-    *       <li>**before_field** - (optional, string) the HTML string to insert before the input field output.</li>
-    *       <li>**after_field** - (optional, string) the HTML string to insert after the input field output.</li>
-    *       <li>**if** - (optional, boolean) if the passed value is false, the section will not be registered.</li>
-    *       <li>**order** - (optional, integer) the order number of the section. The higher the number is, the lower the position it gets.</li>
-    *       <li>**label** - (optional, string) the text label(s) associated with and displayed along with the input field. Some input types can ignore this key.</li>
-    *       <li>**default** - (optional, string|array) the default value(s) assigned to the input tag's value attribute.</li>
-    *       <li>**value** - (optional, string|array) the value(s) assigned to the input tag's `value` attribute to override the default and the stored value.</li>
-    *       <li>**delimiter** - (optional, string) the HTML string that delimits multiple elements. This is available if the <var>label</var> key is passed as array. It will be enclosed in inline-block elements so the passed HTML string should not contain block elements.</li>
-    *       <li>**before_input** - (optional, string) the HTML string inserted right before the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
-    *       <li>**after_input** - (optional, string) the HTML string inserted right after the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
-    *       <li>**label_min_width** - (optional, string) the inline style property of the `min-width` of the label tag for the field. If the unit is not specified, 'px' is applied. Default: `120`. e.g. `100%`</li> 
-    *       <li>**help** - (optional, string) the help description added to the contextual help tab.</li>
-    *       <li>**help_aside** - (optional, string) the additional help description for the side bar of the contextual help tab.</li>
-    *       <li>**repeatable** - [3.0.0+] (optional, array|boolean) whether the fields should be repeatable. If it yields true, the plus and the minus buttons appear next to each field that lets the user add/remove the fields. Optionally an setting array can be passed.
-    *           <h5>Repeatable Fields Setting Array</h5>
-    *           <ul>
-    *                 <li>**max** - the allowed maximum number of fields to be repeated.</li>
-    *                 <li>**min** - the allowed minimum number of fields to be repeated.</li>
-    *           </ul>
-    *       </li>
-    *       <li>**sortable** - [3.0.0+] (optional, boolean) whether the fields should be sortable. If it yields true, the fields will be enclosed in a draggable box.
-    *       <li>**attributes** - [3.0.0+] (optional, array) holds key-value pairs representing the attribute and its property. Note that some field types have specific keys in the first dimensions. e.g. `array( 'class' => 'my_custom_class_selector', 'style' => 'background-color:#777', 'size' => 20, )` Every field holds the following nested attribute definition arrays.
-    *           <ul>
-    *               <li>**fieldrow** - the `td` tag element containing the field output.</li>
-    *               <li>**fieldset** - the `fieldset` tag element containing the field output.</li>
-    *               <li>**fields** - the `div` tag element containing the sub-fields and the main field.</li>
-    *               <li>**field** - the `div` tag element containing each field.</li>
-    *           </ul>
-    *       </li>
-    *       <li>**show_title_column** - [3.0.0+] (optional, boolean) If true, the field title column will be omitted from the output.</li>
-    *       <li>**hidden** - [3.0.0+] (optional, boolean) If true, the entire field row output will be invisible with the inline style attribute of `style="display:none"`.</li>
-    *       <li>**save** - [3.6.0+] (optional, boolean) If `false`, the field value will not be saved. Default: `true`</li>
-    *       <li>**content** - (optional, string|array) [3.6.1+] a custom section output. [3.8.0+] Supports an array to be passed for nested and inline-mixed fields. If an array holding field definitions is set, those fields will be nested.
-    *           <h4>Example</h4>
-    * <pre><code>
-    *   $this->addSettingFields(
-    *       'my_section_id', // the target section ID - pass dimensional keys of the section
-    *       array(
-    *           'field_id'      => 'Y',
-    *           'title'         => __( 'Y', 'admin-page-framework-loader' ),
-    *           'description'   => __( 'By passing an array of field definition to the <code>content</code> argument, you can nest fields.', 'admin-page-framework-loader' )
-    *               . ' ' . __( 'Also the <code>type</code> argument can be omitted.', 'admin-page-framework-loader' ),
-    *           'content'       => array(
-    *               array(
-    *                   'field_id'      => 'i',
-    *                   'title'         => __( 'i', 'admin-page-framework-loader' ),                    
-    *                   'type'          => 'textarea',
-    *               ),
-    *               array(
-    *                   'field_id'      => 'ii',
-    *                   'title'         => __( 'ii', 'admin-page-framework-loader' ),                    
-    *                   'type'          => 'color',                    
-    *               ),
-    *               array(
-    *                   'field_id'      => 'iii',
-    *                   'title'         => __( 'iii', 'admin-page-framework-loader' ),
-    *                   'repeatable'    => true,
-    *                   'sortable'      => true,
-    *                   'content'       => array(
-    *                       array(
-    *                           'field_id'      => 'a',
-    *                           'title'         => __( 'a', 'admin-page-framework-loader' ),                    
-    *                           'type'          => 'image',
-    *                           'attributes'    => array(
-    *                               'preview' => array(
-    *                                   'style' => 'max-width: 200px;',
-    *                               ),
-    *                           ),                                
-    *                       ),
-    *                       array(
-    *                           'field_id'      => 'b',
-    *                           'title'         => __( 'b', 'admin-page-framework-loader' ),
-    *                           'content'       => array(
-    *                               array(
-    *                                   'field_id'      => 'first',
-    *                                   'title'         => __( '1st', 'admin-page-framework-loader' ),                    
-    *                                   'type'          => 'color',
-    *                                   'repeatable'    => true,
-    *                                   'sortable'      => true,
-    *                               ),                                
-    *                               array(
-    *                                   'field_id'      => 'second',
-    *                                   'title'         => __( '2nd', 'admin-page-framework-loader' ),                    
-    *                                   'type'          => 'size',
-    *                               ),
-    *                               array(
-    *                                   'field_id'      => 'third',
-    *                                   'title'         => __( '3rd', 'admin-page-framework-loader' ),                    
-    *                                   'type'          => 'select',
-    *                                   'label'         => array(
-    *                                       'x' => 'X',
-    *                                       'y' => 'Y',
-    *                                       'z' => 'Z',                                        
-    *                                   ),
-    *                               ),                                    
-    *                           ),
-    *                           // 'description'   => '',
-    *                       ),                            
-    *                       array(
-    *                           'field_id'      => 'c',
-    *                           'title'         => __( 'c', 'admin-page-framework-loader' ),                    
-    *                           'type'          => 'radio',                    
-    *                           'label'         => array(
-    *                               'a' => __( 'Apple', 'admin-page-framework-loader' ),
-    *                               'b' => __( 'Banana', 'admin-page-framework-loader' ),
-    *                               'c' => __( 'Cherry', 'admin-page-framework-loader' ),
-    *                           ),
-    *                           'default'       => 'b',
-    *                       ),                        
-    *                   )
-    *               ),                    
-    *           ),
-    *       )
-    *   );
-    * </code></pre>
-    *       </li>
-    *       <li>**placement** - [3.8.0+] (optional, string) Specifies where the field is rendered. Either `normal` `section_title` or `field_title` is accepted. `field_title` can only be used by nested fields defined in the `content` argument.  Default: `normal`</li>
-    * </ul>
-    */
+     * <h4>Built-in Field Types</h4>
+     * <ul>
+     *       <li>[text](./class-AdminPageFramework_FieldType_text.html) - a normal field to enter short text input.</li>
+     *       <li>[password](./class-AdminPageFramework_FieldType_text.html) - a masked text input field.</li>
+     *       <li>[textarea](./class-AdminPageFramework_FieldType_textarea.html) - a text input field with multiple lines. It supports rich text editors.</li>
+     *       <li>[radio](./class-AdminPageFramework_FieldType_radio.html) - a set of radio buttons that lets the user pick an option.</li>
+     *       <li>[checkbox](./class-AdminPageFramework_FieldType_checkbox.html) - a check box that lets the user enable/disable an item.</li>
+     *       <li>[select](./class-AdminPageFramework_FieldType_select.html) - a drop-down list that lest the user pick one or more item(s) from a list.</li>
+     *       <li>[hidden](./class-AdminPageFramework_FieldType_hidden.html) - a field with invisible input values.</li>
+     *       <li>[file](./class-AdminPageFramework_FieldType_file.html) - a file uploader that lets the user upload files.</li>
+     *       <li>[image](./class-AdminPageFramework_FieldType_image.html) - a text field with an image uploader that lets the user set the image URL.</li>
+     *       <li>[media](./class-AdminPageFramework_FieldType_media.html) - a text field with a media uploader that lets the user set the file URL.</li>
+     *       <li>[color](./class-AdminPageFramework_FieldType_color.html) - a text field with a color picker.</li>
+     *       <li>[submit](./class-AdminPageFramework_FieldType_submit.html) - a submit button that lets the user send the form.</li>
+     *       <li>[export](./class-AdminPageFramework_FieldType_export.html) - a custom submit field that lets the user export stored data.</li>
+     *       <li>[import](./class-AdminPageFramework_FieldType_import.html) - a custom combination field of file and submit fields that let the user import data.</li>
+     *       <li>[posttype](./class-AdminPageFramework_FieldType_posttype.html) - a check-list of post types enabled on the site.</li>
+     *       <li>[taxonomy](./class-AdminPageFramework_FieldType_taxonomy.html) - a set of check-lists of taxonomies enabled on the site in a tabbed box.</li>
+     *       <li>[size](./class-AdminPageFramework_FieldType_size.html) - a combination field of the text and the select fields that let the user set sizes with a selectable unit.</li>
+     *       <li>[section_title](./class-AdminPageFramework_FieldType_section_title.html) - [3.0.0+] a text field type that will be placed in the section title so that it lets the user set the section title. Note that only one field with this field type is allowed per a section.</li>
+     *       <li>[system](./class-AdminPageFramework_FieldType_system.html) - [3.3.0+] a custom text area field that displays the system information including the PHP settings, the framework version, MySQL version etc.</li>
+     *       <li>[inline_mixed](./class-AdminPageFramework_FieldType_inline_mixed.html) - [3.8.0+] a field that include inner fields with different field types. </li>
+     * </ul>
+     * <h4>Field Definition Array</h4>
+     * <ul>
+     *       <li>**field_id** - (required, string) the field ID. Avoid using non-alphabetic characters except underscore and numbers.</li>
+     *       <li>**type** - (optional, string) the type of the field. The supported types are listed below. When creating nested fields, this argument can be omitted.</li>
+     *       <li>**section_id** - (optional, string) the section ID that the field belongs to. If not set, the internal `_default` section ID will be assigned.</li>
+     *       <li>**title** - (optional, string) the title of the section.</li>
+     *       <li>**description** - (optional, string) the description of the field which is inserted into the after the input field tag.</li>
+     *       <li>**tip** - (optional, string) the tip for the field which is displayed when the mouse is hovered over the field title.</li>
+     *       <li>**capability** - (optional, string) the <a href="http://codex.wordpress.org/Roles_and_Capabilities">access level</a> of the section. If the page visitor does not have sufficient capability, the section will be invisible to them.</li>
+     *       <li>**error_message** - (optional, string) the error message to display above the input field.</li>
+     *       <li>**before_field** - (optional, string) the HTML string to insert before the input field output.</li>
+     *       <li>**after_field** - (optional, string) the HTML string to insert after the input field output.</li>
+     *       <li>**if** - (optional, boolean) if the passed value is false, the section will not be registered.</li>
+     *       <li>**order** - (optional, integer) the order number of the section. The higher the number is, the lower the position it gets.</li>
+     *       <li>**label** - (optional, string) the text label(s) associated with and displayed along with the input field. Some input types can ignore this key.</li>
+     *       <li>**default** - (optional, string|array) the default value(s) assigned to the input tag's value attribute.</li>
+     *       <li>**value** - (optional, string|array) the value(s) assigned to the input tag's `value` attribute to override the default and the stored value.</li>
+     *       <li>**delimiter** - (optional, string) the HTML string that delimits multiple elements. This is available if the <var>label</var> key is passed as array. It will be enclosed in inline-block elements so the passed HTML string should not contain block elements.</li>
+     *       <li>**before_input** - (optional, string) the HTML string inserted right before the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
+     *       <li>**after_input** - (optional, string) the HTML string inserted right after the input tag. It will be enclosed in the <code>label</code> tag so the passed HTML string should not contain block elements.</li>
+     *       <li>**label_min_width** - (optional, string) the inline style property of the `min-width` of the label tag for the field. If the unit is not specified, 'px' is applied. Default: `120`. e.g. `100%`</li> 
+     *       <li>**help** - (optional, string) the help description added to the contextual help tab.</li>
+     *       <li>**help_aside** - (optional, string) the additional help description for the side bar of the contextual help tab.</li>
+     *       <li>**repeatable** - [3.0.0+] (optional, array|boolean) whether the fields should be repeatable. If it yields true, the plus and the minus buttons appear next to each field that lets the user add/remove the fields. Optionally an setting array can be passed.
+     *           <h5>Repeatable Fields Setting Array</h5>
+     *           <ul>
+     *                 <li>**max** - the allowed maximum number of fields to be repeated.</li>
+     *                 <li>**min** - the allowed minimum number of fields to be repeated.</li>
+     *           </ul>
+     *       </li>
+     *       <li>**sortable** - [3.0.0+] (optional, boolean) whether the fields should be sortable. If it yields true, the fields will be enclosed in a draggable box.
+     *       <li>**attributes** - [3.0.0+] (optional, array) holds key-value pairs representing the attribute and its property. Note that some field types have specific keys in the first dimensions. e.g. `array( 'class' => 'my_custom_class_selector', 'style' => 'background-color:#777', 'size' => 20, )` Every field holds the following nested attribute definition arrays.
+     *           <ul>
+     *               <li>**fieldrow** - the `td` tag element containing the field output.</li>
+     *               <li>**fieldset** - the `fieldset` tag element containing the field output.</li>
+     *               <li>**fields** - the `div` tag element containing the sub-fields and the main field.</li>
+     *               <li>**field** - the `div` tag element containing each field.</li>
+     *           </ul>
+     *       </li>
+     *       <li>**show_title_column** - [3.0.0+] (optional, boolean) If true, the field title column will be omitted from the output.</li>
+     *       <li>**hidden** - [3.0.0+] (optional, boolean) If true, the entire field row output will be invisible with the inline style attribute of `style="display:none"`.</li>
+     *       <li>**save** - [3.6.0+] (optional, boolean) If `false`, the field value will not be saved. Default: `true`</li>
+     *       <li>**content** - (optional, string|array) [3.6.1+] a custom section output. [3.8.0+] Supports an array to be passed for nested and inline-mixed fields. If an array holding field definitions is set, those fields will be nested.
+     *           <h4>Example</h4>
+     * <pre><code>
+     *   $this->addSettingFields(
+     *       'my_section_id', // the target section ID - pass dimensional keys of the section
+     *       array(
+     *           'field_id'      => 'Y',
+     *           'title'         => __( 'Y', 'admin-page-framework-loader' ),
+     *           'description'   => __( 'By passing an array of field definition to the <code>content</code> argument, you can nest fields.', 'admin-page-framework-loader' )
+     *               . ' ' . __( 'Also the <code>type</code> argument can be omitted.', 'admin-page-framework-loader' ),
+     *           'content'       => array(
+     *               array(
+     *                   'field_id'      => 'i',
+     *                   'title'         => __( 'i', 'admin-page-framework-loader' ),                    
+     *                   'type'          => 'textarea',
+     *               ),
+     *               array(
+     *                   'field_id'      => 'ii',
+     *                   'title'         => __( 'ii', 'admin-page-framework-loader' ),                    
+     *                   'type'          => 'color',                    
+     *               ),
+     *               array(
+     *                   'field_id'      => 'iii',
+     *                   'title'         => __( 'iii', 'admin-page-framework-loader' ),
+     *                   'repeatable'    => true,
+     *                   'sortable'      => true,
+     *                   'content'       => array(
+     *                       array(
+     *                           'field_id'      => 'a',
+     *                           'title'         => __( 'a', 'admin-page-framework-loader' ),                    
+     *                           'type'          => 'image',
+     *                           'attributes'    => array(
+     *                               'preview' => array(
+     *                                   'style' => 'max-width: 200px;',
+     *                               ),
+     *                           ),                                
+     *                       ),
+     *                       array(
+     *                           'field_id'      => 'b',
+     *                           'title'         => __( 'b', 'admin-page-framework-loader' ),
+     *                           'content'       => array(
+     *                               array(
+     *                                   'field_id'      => 'first',
+     *                                   'title'         => __( '1st', 'admin-page-framework-loader' ),                    
+     *                                   'type'          => 'color',
+     *                                   'repeatable'    => true,
+     *                                   'sortable'      => true,
+     *                               ),                                
+     *                               array(
+     *                                   'field_id'      => 'second',
+     *                                   'title'         => __( '2nd', 'admin-page-framework-loader' ),                    
+     *                                   'type'          => 'size',
+     *                               ),
+     *                               array(
+     *                                   'field_id'      => 'third',
+     *                                   'title'         => __( '3rd', 'admin-page-framework-loader' ),                    
+     *                                   'type'          => 'select',
+     *                                   'label'         => array(
+     *                                       'x' => 'X',
+     *                                       'y' => 'Y',
+     *                                       'z' => 'Z',                                        
+     *                                   ),
+     *                               ),                                    
+     *                           ),
+     *                           // 'description'   => '',
+     *                       ),                            
+     *                       array(
+     *                           'field_id'      => 'c',
+     *                           'title'         => __( 'c', 'admin-page-framework-loader' ),                    
+     *                           'type'          => 'radio',                    
+     *                           'label'         => array(
+     *                               'a' => __( 'Apple', 'admin-page-framework-loader' ),
+     *                               'b' => __( 'Banana', 'admin-page-framework-loader' ),
+     *                               'c' => __( 'Cherry', 'admin-page-framework-loader' ),
+     *                           ),
+     *                           'default'       => 'b',
+     *                       ),                        
+     *                   )
+     *               ),                    
+     *           ),
+     *       )
+     *   );
+     * </code></pre>
+     *       </li>
+     *       <li>**placement** - [3.8.0+] (optional, string) Specifies where the field is rendered. Either `normal` `section_title` or `field_title` is accepted. `field_title` can only be used by nested fields defined in the `content` argument.  Default: `normal`</li>
+     * </ul>
+     */
     public function addSettingField( $asFieldset ) {
         if ( method_exists( $this->oForm, 'addField' ) ) {
             $this->oForm->addField( $asFieldset );     
@@ -548,28 +548,28 @@ 
     }
     
     /**
-    * Sets the given message to be displayed in the next page load. 
-    * 
-    * This is used to inform users about the submitted input data, such as "Updated successfully." or "Problem occurred." etc. 
-    * and normally used in validation callback methods.
-    * 
-    * <h4>Example</h4>
-    * `
-    * if ( ! $bVerified ) {
-    *       $this->setFieldErrors( $aErrors );     
-    *       $this->setSettingNotice( 'There was an error in your input.' );
-    *       return $aOldPageOptions;
-    * }
-    * `
-    *
-    * @since        3.0.4     
-    * @access       public
-    * @param        string      $sMessage       the text message to be displayed.
-    * @param        string      $sType          (optional) the type of the message, either "error" or "updated"  is used.
-    * @param        array       $asAttributes   (optional) the tag attribute array applied to the message container HTML element. If a string is given, it is used as the ID attribute value.
-    * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
-    * @return       void
-    */      
+     * Sets the given message to be displayed in the next page load. 
+     * 
+     * This is used to inform users about the submitted input data, such as "Updated successfully." or "Problem occurred." etc. 
+     * and normally used in validation callback methods.
+     * 
+     * <h4>Example</h4>
+     * `
+     * if ( ! $bVerified ) {
+     *       $this->setFieldErrors( $aErrors );     
+     *       $this->setSettingNotice( 'There was an error in your input.' );
+     *       return $aOldPageOptions;
+     * }
+     * `
+     *
+     * @since        3.0.4     
+     * @access       public
+     * @param        string      $sMessage       the text message to be displayed.
+     * @param        string      $sType          (optional) the type of the message, either "error" or "updated"  is used.
+     * @param        array       $asAttributes   (optional) the tag attribute array applied to the message container HTML element. If a string is given, it is used as the ID attribute value.
+     * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
+     * @return       void
+     */      
     public function setSettingNotice( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
         $this->oForm->setSubmitNotice(
             $sMessage,

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/_model/formatter/AdminPageFramework_Form_Model___FormatFieldsets.php

Doc Comments +2 added lines

@@ -98,6 +98,7 @@ 
          * @since       DEVVER      Moved from `AdminPageFramework_FormDefinition`. Changed the name from `formatFields()`.
          * Added the `$aSectionsets` parameter.
          * @retuen      array
+         * @param string $sCapability
          */
         private function _getFieldsetsFormatted( array $aFieldsets, array $aSectionsets, $sCapability ) {
 
@@ -271,6 +272,7 @@ 
              * 
              * @since       3.0.0
              * @since       DEVVER          Moved from `AdminPageFramework_FormDefinition`. Changed the name from `formatField()`.
+             * @param integer $iCountOfElements
              * @return      array|void      An array of formatted field definition array. If required keys are not set, nothing will be returned. 
              */
             private function _getFieldsetFormatted( $aFieldset, $aSectionsets, $sCapability, $iCountOfElements, $iSubSectionIndex, $bIsSectionRepeatable, $oCallerObject ) {

Spacing +13 added lines, -13 removed lines

@@ -104,10 +104,10 @@ 
         private function _getFieldsetsFormatted( array $aFieldsets, array $aSectionsets, $sCapability ) {
 
             $_aNewFieldsets = array();
-            foreach( $aFieldsets as $_sSectionPath => $_aItems ) {
+            foreach ( $aFieldsets as $_sSectionPath => $_aItems ) {
 
                 // If the section is not set, skip.
-                if ( ! isset( $aSectionsets[ $_sSectionPath ] ) ) {               
+                if ( !isset( $aSectionsets[ $_sSectionPath ] ) ) {               
                     continue;
                 }
 
@@ -116,7 +116,7 @@ 
                     $_aItems,
                     $this->getElement(  // 3.6.0+ Get the section's capability
                         $aSectionsets, 
-                        array( $_sSectionPath, 'capability', ),
+                        array( $_sSectionPath, 'capability',),
                         $sCapability
                     ),
                     $aSectionsets
@@ -176,16 +176,16 @@ 
                  */
                 private function _getNormalFieldsetsFormatted( $aItems, $sCapability, $aSectionsets, $_abSectionRepeatable ) {
                     
-                    $_aNewItems     = array();
-                    foreach( $aItems as $_sFieldID => $_aFieldset ) {
+                    $_aNewItems = array();
+                    foreach ( $aItems as $_sFieldID => $_aFieldset ) {
                         
                         // Insert the formatted field definition array. The fields count is needed to set each order value.
-                        $_aFieldset        = $this->_getFieldsetFormatted(
+                        $_aFieldset = $this->_getFieldsetFormatted(
                             $_aFieldset, 
                             $aSectionsets,
                             $sCapability, 
                             count( $_aNewItems ), // index of elements - zero based
-                            null,   // sub-section index
+                            null, // sub-section index
                             $_abSectionRepeatable,
                             $this->oCallerForm
                         );
@@ -204,7 +204,7 @@ 
                  * @return      boolean
                  */
                 private function _isSubSections( $aItems, $_abSectionRepeatable ) {
-                    if ( ! empty( $_abSectionRepeatable ) ) {
+                    if ( !empty( $_abSectionRepeatable ) ) {
                         return true;
                     }
                     return ( boolean ) count( $this->getIntegerKeyElements( $aItems ) );
@@ -215,9 +215,9 @@ 
                 private function _getSubSectionsFormatted( $aItems, $sCapability, $aSectionsets, $_abSectionRepeatable ) {
                              
                     $_aNewFieldset = array();
-                    foreach( $this->numerizeElements( $aItems ) as $_iSubSectionIndex => $_aFieldsets ) {
+                    foreach ( $this->numerizeElements( $aItems ) as $_iSubSectionIndex => $_aFieldsets ) {
                                       
-                        foreach( $_aFieldsets as $_aFieldset ) {
+                        foreach ( $_aFieldsets as $_aFieldset ) {
                             $_iCountElement = count( $this->getElementAsArray( $_aNewFieldset, $_iSubSectionIndex ) );
                             $_aFieldset     = $this->_getFieldsetFormatted( 
                                 $_aFieldset, 
@@ -231,7 +231,7 @@ 
                             if ( empty( $_aFieldset ) ) {
                                 continue;
                             }
-                            $_aNewFieldset[ $_iSubSectionIndex ][ $_aFieldset['field_id'] ] = $_aFieldset;
+                            $_aNewFieldset[ $_iSubSectionIndex ][ $_aFieldset[ 'field_id' ] ] = $_aFieldset;
                         }
                         uasort( $_aNewFieldset[ $_iSubSectionIndex ], array( $this, 'sortArrayByKey' ) );
                         
@@ -258,7 +258,7 @@ 
                 }
                 
                 $_aSortedFields = array();
-                foreach( $aSectionsets as $_sSectionPath => $_aSecitonset ) { 
+                foreach ( $aSectionsets as $_sSectionPath => $_aSecitonset ) { 
                     if ( isset( $aFieldsets[ $_sSectionPath ] ) ) {
                         $_aSortedFields[ $_sSectionPath ] = $aFieldsets[ $_sSectionPath ];
                     }
@@ -277,7 +277,7 @@ 
             private function _getFieldsetFormatted( $aFieldset, $aSectionsets, $sCapability, $iCountOfElements, $iSubSectionIndex, $bIsSectionRepeatable, $oCallerObject ) {
 
                 // 3.8.0+ Dropped the check for $aFieldset[ 'type' ] to allow it to be omitted.
-                if ( ! isset( $aFieldset[ 'field_id' ] ) ) { 
+                if ( !isset( $aFieldset[ 'field_id' ] ) ) { 
                     return; 
                 }
                 

Braces +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
             foreach( $aFieldsets as $_sSectionPath => $_aItems ) {
 
                 // If the section is not set, skip.
-                if ( ! isset( $aSectionsets[ $_sSectionPath ] ) ) {               
+                if ( ! isset( $aSectionsets[ $_sSectionPath ] ) ) {
                     continue;
                 }
 
@@ -258,7 +258,7 @@ 
                 }
                 
                 $_aSortedFields = array();
-                foreach( $aSectionsets as $_sSectionPath => $_aSecitonset ) { 
+                foreach( $aSectionsets as $_sSectionPath => $_aSecitonset ) {
                     if ( isset( $aFieldsets[ $_sSectionPath ] ) ) {
                         $_aSortedFields[ $_sSectionPath ] = $aFieldsets[ $_sSectionPath ];
                     }
@@ -277,7 +277,7 @@ 
             private function _getFieldsetFormatted( $aFieldset, $aSectionsets, $sCapability, $iCountOfElements, $iSubSectionIndex, $bIsSectionRepeatable, $oCallerObject ) {
 
                 // 3.8.0+ Dropped the check for $aFieldset[ 'type' ] to allow it to be omitted.
-                if ( ! isset( $aFieldset[ 'field_id' ] ) ) { 
+                if ( ! isset( $aFieldset[ 'field_id' ] ) ) {
                     return; 
                 }
                 

development/factory/_common/form/_view/attribute/AdminPageFramework_Form_View___Attribute_Field.php

Doc Comments -1 removed lines

@@ -30,7 +30,6 @@
      * Returns the field container attribute array.
      * 
      * @remark      Formatting each sub-field should be performed prior to callign this method.
-     * @param       array       $aField     The (sub-)field definition array. This should have been formatted already.
      * @return      array       The generated field container attribute array.
      * @internal   
      * @since       3.5.3

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
      * 
      * @since       3.6.0
      */
-    public $sContext    = 'field'; 
+    public $sContext = 'field'; 
 
     /**
      * Returns the field container attribute array.
@@ -41,7 +41,7 @@ 
     protected function _getAttributes() {
         
         // 3.8.0+ Supports omitting the `type` argument.
-        $_sFieldTypeSelector   = $this->getAOrB(
+        $_sFieldTypeSelector = $this->getAOrB(
             $this->aArguments[ 'type' ],
             " admin-page-framework-field-{$this->aArguments[ 'type' ]}",
             ''
@@ -66,7 +66,7 @@ 
         
         return array(
             'id'            => $this->aArguments[ '_field_container_id' ],
-            'data-type'     => $this->aArguments[ 'type' ],   // referred by the repeatable field JavaScript script.
+            'data-type'     => $this->aArguments[ 'type' ], // referred by the repeatable field JavaScript script.
             'class'         => "admin-page-framework-field{$_sFieldTypeSelector}{$_sNestedFieldSelector}{$_sMixedFieldSelector}{$_sChildFieldSelector}"
                 . $this->getAOrB(
                     $this->aArguments[ 'attributes' ][ 'disabled' ],

development/factory/_common/form/_view/fieldset/AdminPageFramework_Form_View___Fieldset___FieldError.php

Doc Comments +1 added lines

@@ -60,6 +60,7 @@
          * 
          * @since       3.1.0
          * @since       DEVVER      Moved from `AdminPageFramework_Form_View___Fieldset`.
+         * @param string $sHeadingMessage
          * @return      string      The error string message. An empty value if not found.
          */
         private function _getFieldError( $aErrors, $aSectionPath, $aFieldPath, $sHeadingMessage ) {

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         private function _getFieldError( $aErrors, $aSectionPath, $aFieldPath, $sHeadingMessage ) {
 
             // If this field has a section and the error element is set
-            $_aErrorPath   = array_merge( $aSectionPath, $aFieldPath );
+            $_aErrorPath = array_merge( $aSectionPath, $aFieldPath );
             if ( $this->_hasFieldError( $aErrors, $_aErrorPath ) ) {
                 return "<span class='field-error'>*&nbsp;"
                         . $sHeadingMessage

development/factory/_common/form/_view/fieldset/AdminPageFramework_Form_View___Fieldset_Base.php

Doc Comments +2 added lines

@@ -176,6 +176,7 @@ 
      * Returns the repeatable fields script.
      * 
      * @since 2.1.3
+     * @param string $sFieldsContainerID
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
 
@@ -222,6 +223,7 @@ 
      * Returns the sortable fields script.
      * 
      * @since 3.0.0
+     * @param string $sFieldsContainerID
      */    
     protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
     

Indentation +1 added lines, -1 removed lines

@@ -239,7 +239,7 @@
             
             return "<div class='admin-page-framework-repeatable-field-buttons' {$_sSettingsAttributes} >"
                 . "<a class='repeatable-field-remove-button button-secondary repeatable-field-button button {$_sSmallButtonSelector}{$_sDashiconMinus}' href='#' title='{$_sRemove}' {$_sVisibility} data-id='{$sFieldsContainerID}'>"
-                  . '-' // ( $_bDashiconSupported ? '' : '-' )
+                    . '-' // ( $_bDashiconSupported ? '' : '-' )
                 . "</a>"
                 . "<a class='repeatable-field-add-button button-secondary repeatable-field-button button {$_sSmallButtonSelector}{$_sDashiconPlus}' href='#' title='{$_sAdd}' data-id='{$sFieldsContainerID}'>" 
                     . '+' // ( $_bDashiconSupported ? '' : '+' )

Spacing +14 added lines, -14 removed lines

@@ -68,7 +68,7 @@ 
      * @param       object      $oMsg                   An object storing the system messages.
      * @param       array       $aCallbacks             An array storing the form-field specific callbacks.     
      */
-    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks=array() ) {
+    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks = array() ) {
 
         // Set up the properties that will be accessed later in the methods.
         $this->aFieldset                = $this->_getFormatted( $aFieldset, $aFieldTypeDefinitions );
@@ -77,13 +77,13 @@ 
         $this->aErrors                  = $this->getAsArray( $aErrors );
         $this->oMsg                     = $oMsg;
         $this->aCallbacks               = $aCallbacks + array(
-            'hfID'              => null,    // the input id attribute
-            'hfTagID'           => null,    // the fieldset/field row container id attribute
-            'hfName'            => null,    // the input name attribute
-            'hfNameFlat'        => null,    // the flat input name attribute                
+            'hfID'              => null, // the input id attribute
+            'hfTagID'           => null, // the fieldset/field row container id attribute
+            'hfName'            => null, // the input name attribute
+            'hfNameFlat'        => null, // the flat input name attribute                
             'hfInputName'       => null,
             'hfInputNameFlat'   => null,
-            'hfClass'           => null,    // the class attribute
+            'hfClass'           => null, // the class attribute
         );        
         
         // 2. Load necessary JavaScript scripts.
@@ -150,9 +150,9 @@ 
          * @since   3.2.0   Added the `$sFieldsType` parameter.
          * @internal
          */
-        private function _loadScripts( $sStructureType='' ) {
+        private function _loadScripts( $sStructureType = '' ) {
 
-            if ( 'widget' === $sStructureType && ! self::$_bIsLoadedJSScripts_Widget ) {
+            if ( 'widget' === $sStructureType && !self::$_bIsLoadedJSScripts_Widget ) {
                 new AdminPageFramework_Form_View___Script_Widget;
                 self::$_bIsLoadedJSScripts_Widget = true;
             }
@@ -177,8 +177,8 @@ 
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
         
-        $_sSmallButtons         = '"' . $this->_getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, true ) . '"';
-        $_sNestedFieldsButtons  = '"' . $this->_getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, false ) . '"';
+        $_sSmallButtons         = '"'.$this->_getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, true ).'"';
+        $_sNestedFieldsButtons  = '"'.$this->_getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, false ).'"';
         $_aJSArray              = json_encode( $aSettings );
         $_sScript               = <<<JAVASCRIPTS
 jQuery( document ).ready( function() {
@@ -224,7 +224,7 @@ 
          * @since       3.8.0
          * @return      string
          */
-        private function _getRepeatableButtonHTML( $sFieldsContainerID, array $aSettings, $iFieldCount, $bSmall=true ) {
+        private function _getRepeatableButtonHTML( $sFieldsContainerID, array $aSettings, $iFieldCount, $bSmall = true ) {
             
             $_sAdd                  = $this->oMsg->get( 'add' );
             $_sRemove               = $this->oMsg->get( 'remove' );            
@@ -233,9 +233,9 @@ 
             $_sSmallButtonSelector  = $bSmall ? ' button-small' : '';
                 
             // Not using dash-icons at the moment.
-            $_bDashiconSupported    = false;    // version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' );
-            $_sDashiconPlus         = '';       // $_bDashiconSupported ? 'dashicons dashicons-plus' : '';
-            $_sDashiconMinus        = '';       // $_bDashiconSupported ? 'dashicons dashicons-minus' : '';            
+            $_bDashiconSupported    = false; // version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' );
+            $_sDashiconPlus         = ''; // $_bDashiconSupported ? 'dashicons dashicons-plus' : '';
+            $_sDashiconMinus        = ''; // $_bDashiconSupported ? 'dashicons dashicons-minus' : '';            
             
             return "<div class='admin-page-framework-repeatable-field-buttons' {$_sSettingsAttributes} >"
                 . "<a class='repeatable-field-remove-button button-secondary repeatable-field-button button {$_sSmallButtonSelector}{$_sDashiconMinus}' href='#' title='{$_sRemove}' {$_sVisibility} data-id='{$sFieldsContainerID}'>"

Braces +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
                 self::$_bIsLoadedJSScripts_Widget = true;
             }
             
-            if ( self::$_bIsLoadedJSScripts ) { 
+            if ( self::$_bIsLoadedJSScripts ) {
                 return; 
             }
             self::$_bIsLoadedJSScripts = true;
@@ -255,7 +255,7 @@ 
      * 
      * @since 3.0.0
      */    
-    protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
+    protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {
     
         $_sScript = <<<JAVASCRIPTS
     jQuery( document ).ready( function() {

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