michaeluno / admin-page-framework

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

Spacing +9 added lines, -9 removed lines

@@ -23,17 +23,17 @@ 
     /**
      * The callback function name or the callable object to retrieve meta data.
      */
-    protected $osCallable   = 'get_post_meta';
+    protected $osCallable = 'get_post_meta';
 
     /**
      * The object ID such as post ID, user ID, and term ID.
      */
-    public $iObjectID       = 0;
+    public $iObjectID = 0;
     
     /**
      * A form fieldsets array.
      */
-    public $aFieldsets      = array();
+    public $aFieldsets = array();
     
     /**
      * Sets up properties.
@@ -57,7 +57,7 @@ 
      */
     public function get() {
 
-        if ( ! $this->iObjectID ) {
+        if ( !$this->iObjectID ) {
             return array();
         }
         return $this->_getSavedDataFromFieldsets(
@@ -76,11 +76,11 @@ 
             
             $_aMetaKeys = $this->_getMetaKeys( $iObjectID );
             $_aMetaData = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
                 
-                if ( '_default' == $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aFieldset ) {
-                        if ( ! in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
+                if ( '_default' == $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aFieldset ) {
+                        if ( !in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
                             continue;
                         }
                         $_aMetaData[ $_aFieldset[ 'field_id' ] ] = call_user_func_array(
@@ -93,7 +93,7 @@ 
                         );
                     }
                 }
-                if ( ! in_array( $_sSectionID, $_aMetaKeys ) ) {
+                if ( !in_array( $_sSectionID, $_aMetaKeys ) ) {
                     continue;
                 }                
                 $_aMetaData[ $_sSectionID ] = call_user_func_array(

Doc Comments +1 added lines

@@ -69,6 +69,7 @@
          * 
          * @since       3.7.0
          * @uses        get_post_meta()
+         * @param integer $iObjectID
          * @return      array
          */
         private function _getSavedDataFromFieldsets( $iObjectID, $aFieldsets ) {

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

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

Doc Comments +4 added lines, -1 removed lines

@@ -173,6 +173,7 @@ 
      * Returns the repeatable fields script.
      * 
      * @since 2.1.3
+     * @param string $sFieldsContainerID
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
         
@@ -261,6 +262,7 @@ 
             /**
              * @since       3.8.13
              * @param       array       $aArguments
+             * @param string $sSmallButtonSelector
              * @return      string
              */
             private function ___getAddButtonAttribtes( $aArguments, $sFieldsContainerID, $sSmallButtonSelector ) {
@@ -279,7 +281,7 @@ 
             }
             /**
              * @since       3.8.13
-             * @param       array       $aArguments
+             * @param string $sSmallButtonSelector
              * @return      string
              */
             private function ___getRemoveButtonAttributes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
@@ -323,6 +325,7 @@ 
      * Returns the sortable fields script.
      * 
      * @since 3.0.0
+     * @param string $sFieldsContainerID
      */    
     protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
     

Indentation +3 added lines, -3 removed lines

@@ -272,8 +272,8 @@ 
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
                         : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                            . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
+                            . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -285,7 +285,7 @@ 
             private function ___getRemoveButtonAttributes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
                 $_aMinusButtonAttributes = array(
                     'class'     => 'repeatable-field-remove-button button-secondary repeatable-field-button button'
-                         . $sSmallButtonSelector,
+                            . $sSmallButtonSelector,
                     'title'     => $this->oMsg->get( 'remove' ),
                     'style'     => $iFieldCount <= 1
                         ? 'visibility: hidden'

Spacing +25 added lines, -25 removed lines

@@ -67,7 +67,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 );
@@ -76,13 +76,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.
@@ -149,9 +149,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;
             }
@@ -176,8 +176,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() {
@@ -223,36 +223,36 @@ 
          * @since       3.8.0
          * @return      string
          */
-        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall=true ) {
+        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall = true ) {
 
             // @todo Move this formatting routine to the field-set formatter class.
             $_oFormatter             = new AdminPageFramework_Form_Model___Format_RepeatableField( $aArguments, $this->oMsg );
             $_aArguments             = $_oFormatter->get();
             $_sSmallButtonSelector   = $bSmall ? ' button-small' : '';
             if ( version_compare( $GLOBALS[ 'wp_version' ], '5.3', '>=' ) ) {
-                return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                        . "<a " . $this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">"
+                return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                        . "<a ".$this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">"
                             . "<span class='dashicons dashicons-minus'></span>"
                         . "</a>"
-                        . "<a " . $this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">"
+                        . "<a ".$this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">"
                             . "<span class='dashicons dashicons-plus-alt2'></span>"
                         ."</a>"
                     . "</div>"
                     . $this->getModalForDisabledRepeatableElement(
-                        'repeatable_field_disabled_' . $sFieldsContainerID,
+                        'repeatable_field_disabled_'.$sFieldsContainerID,
                         $_aArguments[ 'disabled' ]
                     );
             }
-            return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                    . "<a " . $this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">"
+            return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                    . "<a ".$this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">"
                         . "-"
                     . "</a>"
-                    . "<a " . $this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">"
+                    . "<a ".$this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">"
                         . "+"
                     ."</a>"
                 . "</div>"
                 . $this->getModalForDisabledRepeatableElement(
-                    'repeatable_field_disabled_' . $sFieldsContainerID,
+                    'repeatable_field_disabled_'.$sFieldsContainerID,
                     $_aArguments[ 'disabled' ]
                 );
 
@@ -271,9 +271,9 @@ 
                     'data-id'   => $sFieldsContainerID,
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
-                        : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                        : '#TB_inline?width='.$aArguments[ 'disabled' ][ 'box_width' ]
+                          . '&height='.$aArguments[ 'disabled' ][ 'box_height' ]
+                          . '&inlineId='.'repeatable_field_disabled_'.$sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -300,10 +300,10 @@ 
              * @return      string
              */
             private function ___getContainerAttributes( $aArguments ) {
-                $_aContainerAttributes   = array(
+                $_aContainerAttributes = array(
                     'class' => $this->getClassAttribute(
                         'admin-page-framework-repeatable-field-buttons',
-                        ! empty( $aArguments[ 'disabled' ] )
+                        !empty( $aArguments[ 'disabled' ] )
                             ? 'disabled'
                             : ''
                     ),
@@ -316,7 +316,7 @@ 
                     unset( $aArguments[ 'disabled' ] );
                 }
                 return $this->getAttributes( $_aContainerAttributes )
-                    . ' ' . $this->getDataAttributes( $aArguments );
+                    . ' '.$this->getDataAttributes( $aArguments );
             }
 
     /**

development/factory/_common/form/_view/generator/section/AdminPageFramework_Form_View___Generate_Section_Base.php

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;
         

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

Spacing +7 added lines, -7 removed lines

@@ -24,8 +24,8 @@ 
      * Represents the structure of the sub-field definition array.
      */
     static public $aStructure = array(
-        '_is_sub_field'                 => false,   // @todo change this key name as all the parsed field is technically a sub-field.
-        '_index'                        => 0,       // indicates the field index
+        '_is_sub_field'                 => false, // @todo change this key name as all the parsed field is technically a sub-field.
+        '_index'                        => 0, // indicates the field index
         '_is_multiple_fields'           => false,
         '_saved_value'                  => null,
         '_is_value_set_by_user'         => false,
@@ -39,8 +39,8 @@ 
         '_fields_container_id'          => '',
         '_fieldset_container_id'        => '',
         
-        '_field_object'                 => null,        // 3.6.0+
-        '_parent_field_object'          => null,        // 3.6.0+ Stores the parent field object to be accessed from the nested fields to generate id and name attribute models.
+        '_field_object'                 => null, // 3.6.0+
+        '_parent_field_object'          => null, // 3.6.0+ Stores the parent field object to be accessed from the nested fields to generate id and name attribute models.
     );
     
     /**
@@ -87,7 +87,7 @@ 
 
         $_aField = $this->aField + self::$aStructure;
         
-        $_aField[ '_is_sub_field' ]            = is_numeric( $this->isIndex ) && 0 < $this->isIndex;      // 3.5.3+
+        $_aField[ '_is_sub_field' ]            = is_numeric( $this->isIndex ) && 0 < $this->isIndex; // 3.5.3+
         $_aField[ '_index' ]                   = $this->isIndex;
 
         // 'input_id' - something like ({section id}_){field_id}__{index} e.g. my_section_id_my_field_id__0
@@ -134,8 +134,8 @@ 
                     'value'             => $_aField[ 'value' ],
                     'type'              => $_aField[ 'type' ], // text, password, etc.
                     'disabled'          => null,
-                    'data-id_model'     => $_aField[ '_input_id_model' ],    // 3.3.1+
-                    'data-name_model'   => $_aField[ '_input_name_model' ],  // 3.3.1+
+                    'data-id_model'     => $_aField[ '_input_id_model' ], // 3.3.1+
+                    'data-name_model'   => $_aField[ '_input_name_model' ], // 3.3.1+
                 )
             ),
             // this allows sub-fields with different field types to set the default key-values for the sub-field.

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

Spacing +12 added lines, -12 removed lines

@@ -33,23 +33,23 @@ 
      * @internal
      */ 
     static public $aStructure = array(       
-        '_section_index'            => null,    // 3.0.0+ - internally set to indicate the section index for repeatable sections.        
+        '_section_index'            => null, // 3.0.0+ - internally set to indicate the section index for repeatable sections.        
         
         'tag_id'                    => null,
-        '_tag_id_model'             => '',      // 3.6.0+   
+        '_tag_id_model'             => '', // 3.6.0+   
         
-        '_field_name'               => '',      // 3.6.0+   
-        '_field_name_model'         => '',      // 3.6.0+           
+        '_field_name'               => '', // 3.6.0+   
+        '_field_name_model'         => '', // 3.6.0+           
         
-        '_field_name_flat'          => '',      // 3.6.0+
-        '_field_name_flat_model'    => '',      // 3.6.0+   
+        '_field_name_flat'          => '', // 3.6.0+
+        '_field_name_flat_model'    => '', // 3.6.0+   
                 
-        '_field_address'            => '',      // 3.6.0+
-        '_field_address_model'      => '',      // 3.6.0+
+        '_field_address'            => '', // 3.6.0+
+        '_field_address_model'      => '', // 3.6.0+
                
-        '_parent_field_object'      => null,    // 3.6.0+   Assigned when a field creates a nested field.
+        '_parent_field_object'      => null, // 3.6.0+   Assigned when a field creates a nested field.
         
-        '_parent_tag_id'            => null,    // 3.8.0+   Set outside the class.
+        '_parent_tag_id'            => null, // 3.8.0+   Set outside the class.
         
     );        
     
@@ -92,7 +92,7 @@ 
         $_aFieldset = $this->aFieldset + self::$aStructure;
 
         // The section index must be set before generating a field tag id as it uses a section index.
-        $_aFieldset[ '_section_index' ]   = $this->iSectionIndex;
+        $_aFieldset[ '_section_index' ] = $this->iSectionIndex;
         $_oFieldTagIDGenerator = new AdminPageFramework_Form_View___Generate_FieldTagID( 
             $_aFieldset,
             $_aFieldset[ '_caller_object' ]->aCallbacks[ 'hfTagID' ]
@@ -126,7 +126,7 @@ 
         
         // 3.8.0+ Format nested fieldsets.
         if ( $this->hasFieldDefinitionsInContent( $_aFieldset ) ) {
-            foreach( $_aFieldset[ 'content' ] as &$_aNestedFieldset ) {
+            foreach ( $_aFieldset[ 'content' ] as &$_aNestedFieldset ) {
                 // The inline-mixed type has a string element.
                 if ( is_scalar( $_aNestedFieldset ) ) {
                     continue;

development/factory/_common/_abstract/AdminPageFramework_Factory_Model.php

Doc Comments +1 added lines, -2 removed lines

@@ -203,6 +203,7 @@ 
          * Constructs a WordPress filter hook name.
          * @internal
          * @since       3.8.4
+         * @param string $sPrefix
          * @return      string
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
@@ -339,8 +340,6 @@ 
      * @since       3.7.0      Changed back the visibility scope to protected as there is the `getFieldErrors()` public method.
      * @access      protected
      * @internal
-     * @param       string      $sID        deprecated
-     * @param       boolean     $bDelete    whether or not the transient should be deleted after retrieving it. 
      * @return      array
      * @deprecated  3.7.0      Use `getFieldErrors()` instead. Kept for backward compatibility.
      */

Braces +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
      * @since       3.7.0
      * @return      array       The modified section-sets definition array.
      */    
-    public function _replyToModifySectionsets( $aSectionsets ) {    
+    public function _replyToModifySectionsets( $aSectionsets ) {
         
         return $this->oUtil->addAndApplyFilter( 
             $this,  // caller factory object
@@ -364,7 +364,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

Spacing +13 added lines, -13 removed lines

@@ -31,7 +31,7 @@ 
         
         add_filter(
             // 'field_types_admin_page_framework',
-            'field_types_' . $oProp->sClassName,
+            'field_types_'.$oProp->sClassName,
             array( $this, '_replyToFilterFieldTypeDefinitions' )
         );
         
@@ -53,7 +53,7 @@ 
             'title'      => null,
             'help_aside' => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
         $this->oHelpPane->_addHelpTextForFormFields( 
@@ -73,9 +73,9 @@ 
     public function _replyToFilterFieldTypeDefinitions( $aFieldTypeDefinitions ) {
         
         // Not triggering `__call()` as the filter is fired manually in the form class.
-        if ( method_exists( $this, 'field_types_' . $this->oProp->sClassName ) ) {
+        if ( method_exists( $this, 'field_types_'.$this->oProp->sClassName ) ) {
             return call_user_func_array(
-                array( $this, 'field_types_' . $this->oProp->sClassName ),
+                array( $this, 'field_types_'.$this->oProp->sClassName ),
                 array( $aFieldTypeDefinitions )
             );
         }
@@ -96,7 +96,7 @@ 
     public function _replyToModifySectionsets( $aSectionsets ) {    
         
         return $this->oUtil->addAndApplyFilter( 
-            $this,  // caller factory object
+            $this, // caller factory object
             "sections_{$this->oProp->sClassName}", 
             $aSectionsets
         );
@@ -116,7 +116,7 @@ 
     public function _replyToModifyFieldsets( $aFieldsets, $aSectionsets ) {
 
         // Apply filters to added field-sets
-        foreach( $aFieldsets as $_sSectionPath => $_aFields ) {
+        foreach ( $aFieldsets as $_sSectionPath => $_aFields ) {
             $_aSectionPath  = explode( '|', $_sSectionPath );
             $_sFilterSuffix = implode( '_', $_aSectionPath );
             $aFieldsets[ $_sSectionPath ] = $this->oUtil->addAndApplyFilter(
@@ -125,7 +125,7 @@ 
                 $_aFields
             ); 
         }
-        $aFieldsets =  $this->oUtil->addAndApplyFilter( 
+        $aFieldsets = $this->oUtil->addAndApplyFilter( 
             $this,
             "fields_{$this->oProp->sClassName}",
             $aFieldsets
@@ -195,14 +195,14 @@ 
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
             
-            $_sFieldPart    = '_' . implode( '_', $aFieldset[ '_field_path_array' ] );
+            $_sFieldPart    = '_'.implode( '_', $aFieldset[ '_field_path_array' ] );
             $_sSectionPart  = implode( '_', $aFieldset[ '_section_path_array' ] );
             $_sSectionPart  = $this->oUtil->getAOrB(
                 '_default' === $_sSectionPart,
                 '',
-                '_' . $_sSectionPart
+                '_'.$_sSectionPart
             );
-            return $sPrefix . $this->oProp->sClassName . $_sSectionPart . $_sFieldPart;
+            return $sPrefix.$this->oProp->sClassName.$_sSectionPart.$_sFieldPart;
             
         }
                
@@ -272,9 +272,9 @@ 
     public function _replyToGetSavedFormData() {
         
         // Must update the property with the filtered value.
-        $this->oProp->aOptions =  $this->oUtil->addAndApplyFilter(
+        $this->oProp->aOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->oProp->aOptions      // subject value to be filtered
         );
         return $this->oProp->aOptions;
@@ -364,7 +364,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

development/factory/_common/form/_view/AdminPageFramework_Form_View___ToolTip.php

Doc Comments +1 added lines, -2 removed lines

@@ -42,8 +42,6 @@ 
      * Sets up properties.
      * @since       3.6.0
      * @since       3.7.0           Changed the parameter structure.
-     * @param       array|string    $asArguments        The content output or the tooltip argument array.
-     * @param       string          $sTitleElementID    Not used at the moment.
      */
     public function __construct( /* $aArguments, $sTitleElementID */ ) {
 
@@ -170,6 +168,7 @@ 
 
     /**
      * Generates HTML attributes of the specified element.
+     * @param string $sElementKey
      * @return      string
      * @since       3.8.5
      */

Spacing +14 added lines, -14 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * Stores the default argument values.
      */
-    public $aArguments      = array(
+    public $aArguments = array(
         'attributes'    => array(
             'container'   => array(),
             'title'       => array(),
@@ -28,11 +28,11 @@ 
             'description' => array(),
             'icon'        => array(),
         ),
-        'icon'          => null,    // the icon output to override
-        'dash-icon'     => 'dashicons-editor-help',   // the dash-icon class selector for the icon
-        'icon_alt_text' => '[?]',   // [3.8.5+] alternative text when icon is not available. For WP v3.7.x or below.
+        'icon'          => null, // the icon output to override
+        'dash-icon'     => 'dashicons-editor-help', // the dash-icon class selector for the icon
+        'icon_alt_text' => '[?]', // [3.8.5+] alternative text when icon is not available. For WP v3.7.x or below.
         'title'         => null,
-        'content'       => null,    // will be assigned in the constructor
+        'content'       => null, // will be assigned in the constructor
     );
 
     public $sTitleElementID;
@@ -89,7 +89,7 @@ 
                 if ( is_string( $asContent ) ) {
                     return true;
                 }
-                if ( is_array( $asContent ) && ! $this->isAssociative( $asContent ) ) {
+                if ( is_array( $asContent ) && !$this->isAssociative( $asContent ) ) {
                     return true;
                 }
                 return false;
@@ -103,12 +103,12 @@ 
      * @return      string      The output.
      */
     public function get() {
-        if ( ! $this->aArguments[ 'content' ] ) {
+        if ( !$this->aArguments[ 'content' ] ) {
             return '';
         }
-        return "<a " . $this->_getElementAttributes( 'container', 'admin-page-framework-form-tooltip' ) . ">"
+        return "<a ".$this->_getElementAttributes( 'container', 'admin-page-framework-form-tooltip' ).">"
                 . $this->_getTipLinkIcon()
-                . "<span " . $this->_getElementAttributes( 'content', 'admin-page-framework-form-tooltip-content' ) . ">"
+                . "<span ".$this->_getElementAttributes( 'content', 'admin-page-framework-form-tooltip-content' ).">"
                     . $this->_getTipTitle()
                     . $this->_getDescriptions()
                 . "</span>"
@@ -126,7 +126,7 @@ 
                 return $this->aArguments[ 'icon' ];
             }
             if ( version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' ) ) {
-                return "<span " . $this->_getElementAttributes(
+                return "<span ".$this->_getElementAttributes(
                         'icon',
                         array(
                             'dashicons',
@@ -144,7 +144,7 @@ 
          */
         private function _getTipTitle() {
             if ( isset( $this->aArguments[ 'title' ] ) ) {
-                return "<span " . $this->_getElementAttributes( 'title', 'admin-page-framework-form-tooltip-title' ) . ">"
+                return "<span ".$this->_getElementAttributes( 'title', 'admin-page-framework-form-tooltip-title' ).">"
                     . $this->aArguments[ 'title' ]
                     . "</span>";
             }
@@ -156,9 +156,9 @@ 
          */
         private function _getDescriptions() {
             if ( isset( $this->aArguments[ 'content' ] ) ) {
-                return "<span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">"
+                return "<span ".$this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ).">"
                         . implode(
-                            "</span><span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">",
+                            "</span><span ".$this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ).">",
                             $this->getAsArray( $this->aArguments[ 'content' ] )
                         )
                     . "</span>"
@@ -177,7 +177,7 @@ 
         $_aContainerAttributes = $this->getElementAsArray(
             $this->aArguments,
             array( 'attributes', $sElementKey )
-        ) + array( 'class' => '' ) ;
+        ) + array( 'class' => '' );
         $_aContainerAttributes[ 'class' ] = $this->getClassAttribute(
             $_aContainerAttributes[ 'class' ],
             $asClassSelectors

development/factory/_common/form/_view/AdminPageFramework_Form_View___DebugInfo.php

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
  */
 class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {            
 
-    public $sStructureType   = '';
+    public $sStructureType = '';
     
     /**
      * @since       3.8.5
@@ -57,12 +57,12 @@ 
      */
     public function get() {
         
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return '';
         }
         
         return "<div class='admin-page-framework-info'>" 
-                . $this->oMsg->get( 'debug_info' ) . ': '
+                . $this->oMsg->get( 'debug_info' ).': '
                     . $this->getFrameworkNameVersion()
             . "</div>";
         
@@ -73,7 +73,7 @@ 
          */
         private function _shouldProceed() {
 
-            if ( ! $this->callBack( $this->aCallbacks[ 'show_debug_info' ], true ) ) {
+            if ( !$this->callBack( $this->aCallbacks[ 'show_debug_info' ], true ) ) {
                 return false;
             }        
             // For the generic admin pages, do no show debug information for each section.

Braces +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  * @extends     AdminPageFramework_FrameworkUtility
  * @internal
  */
-class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {            
+class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {
 
     public $sStructureType   = '';
     

development/factory/admin_page/AdminPageFramework_Controller_Menu.php

Doc Comments +3 added lines, -3 removed lines

@@ -74,6 +74,8 @@ 
      *  <li>a base64-encoded SVG using a data URI, which will be colored to match the color scheme. This should begin with 'data:image/svg+xml;base64,'.</li>
      * </ul>
      * @param       string      (optional) the position number that is passed to the <var>$position</var> parameter of the <a href="http://codex.wordpress.org/Function_Reference/add_menu_page">add_menu_page()</a> function.
+     * @param string $sIcon16x16
+     * @param integer $iMenuPosition
      * @return      void
      */
     public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16=null, $iMenuPosition=null ) {
@@ -96,6 +98,7 @@ 
          * 
          * @since       2.0.0
          * @internal
+         * @param string $sMenuLabel
          * @return      void|string     Returns the associated slug string, if true.
          */ 
         private function _isBuiltInMenuItem( $sMenuLabel ) {
@@ -160,9 +163,6 @@ 
      * @since        3.0.0       Changed the scope to public.
      * @remark       The sub menu page slug should be unique because add_submenu_page() can add one callback per page slug.
      * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
-     * @param        array       $aSubMenuItem1      a first sub-menu array. A sub-menu array can be a link or a page. For the specifications of the array structures and its arguments, refer to the parameter section of the `addSubMenuItem()` method.
-     * @param        array       $aSubMenuItem2      (optional) a second sub-menu array.
-     * @param        array       $_and_more          (optional) a third and add items as many as necessary with next parameters.
      * @access       public
      * @return       void
      */     

Indentation +14 added lines, -14 removed lines

@@ -250,20 +250,20 @@
     }
 
     /**
-    * Adds the given link into the menu on the left sidebar of the administration panel.
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public from protected.
-    * @since        3.5.0       Changed the scope to public as it was still protected.
-    * @param        string      the menu title.
-    * @param        string      the URL linked to the menu.
-    * @param        string      (optional) the <a href="http://codex.wordpress.org/Roles_and_Capabilities" target="_blank">access level</a>.
-    * @param        string      (optional) the order number. The larger it is, the lower the position it gets.
-    * @param        string      (optional) if set to false, the menu title will not be listed in the tab navigation menu at the top of the page.
-    * @access       public
-    * @return       void
-    * @internal
-    */    
+     * Adds the given link into the menu on the left sidebar of the administration panel.
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public from protected.
+     * @since        3.5.0       Changed the scope to public as it was still protected.
+     * @param        string      the menu title.
+     * @param        string      the URL linked to the menu.
+     * @param        string      (optional) the <a href="http://codex.wordpress.org/Roles_and_Capabilities" target="_blank">access level</a>.
+     * @param        string      (optional) the order number. The larger it is, the lower the position it gets.
+     * @param        string      (optional) if set to false, the menu title will not be listed in the tab navigation menu at the top of the page.
+     * @access       public
+     * @return       void
+     * @internal
+     */    
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.

Spacing +6 added lines, -6 removed lines

@@ -76,7 +76,7 @@ 
      * @param       string      (optional) the position number that is passed to the <var>$position</var> parameter of the <a href="http://codex.wordpress.org/Function_Reference/add_menu_page">add_menu_page()</a> function.
      * @return      void
      */
-    public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16=null, $iMenuPosition=null ) {
+    public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16 = null, $iMenuPosition = null ) {
 
         $sRootMenuLabel = trim( $sRootMenuLabel );
         $_sSlug         = $this->_isBuiltInMenuItem( $sRootMenuLabel ); // if true, this method returns the slug
@@ -267,16 +267,16 @@ 
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.
-        if ( ! isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
+        if ( !isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
             return; 
         }
         
         // If the set URL is not valid, return.
-        if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
+        if ( !filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
             return; 
         }
 
-        $_oFormatter   = new AdminPageFramework_Format_SubMenuLink( 
+        $_oFormatter = new AdminPageFramework_Format_SubMenuLink( 
             $aSubMenuLink, 
             $this,
             count( $this->oProp->aPages ) + 1
@@ -334,11 +334,11 @@ 
      */ 
     public function addSubMenuPage( array $aSubMenuPage ) {
 
-        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
+        if ( !isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
             return; 
         }
             
-        $_oFormatter   = new AdminPageFramework_Format_SubMenuPage( 
+        $_oFormatter = new AdminPageFramework_Format_SubMenuPage( 
             $aSubMenuPage,
             $this,
             count( $this->oProp->aPages ) + 1

Braces +3 added lines, -3 removed lines

@@ -267,12 +267,12 @@ 
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.
-        if ( ! isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
+        if ( ! isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) {
             return; 
         }
         
         // If the set URL is not valid, return.
-        if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
+        if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) {
             return; 
         }
 
@@ -334,7 +334,7 @@ 
      */ 
     public function addSubMenuPage( array $aSubMenuPage ) {
 
-        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
+        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) {
             return; 
         }