michaeluno / admin-page-framework

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/generator/field/AdminPageFramework_Form_View___Generate_Field_Base.php

Doc Comments -1 removed lines

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

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

Doc Comments +1 added lines

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

Spacing +2 added lines, -2 removed lines

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

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

Doc Comments +1 added lines

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

Spacing +2 added lines, -2 removed lines

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

development/factory/_common/form/AdminPageFramework_Form_Controller.php

Doc Comments +2 added lines

@@ -121,6 +121,8 @@
     }
     /**
      * @since       DEVVER
+     * @param string $sKey
+     * @param string $sValue
      * @return      void
      */
     public function addResource( $sKey, $sValue ) {

Spacing +9 added lines, -9 removed lines

@@ -45,7 +45,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 hasSubmitNotice( $sType='' ) {
+    public function hasSubmitNotice( $sType = '' ) {
         return $this->oSubmitNotice->hasNotice( $sType );
     }
     
@@ -71,7 +71,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 setSubmitNotice( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
+    public function setSubmitNotice( $sMessage, $sType = 'error', $asAttributes = array(), $bOverride = true ) {
         $this->oSubmitNotice->set(
             $sMessage, 
             $sType, 
@@ -98,7 +98,7 @@ 
         
         $this->aSectionsets[ $aSectionset[ 'section_id' ] ] = $aSectionset;    
         $this->aFieldsets[ $aSectionset[ 'section_id' ] ]   = $this->getElement(
-            $this->aFieldsets,  // subject array
+            $this->aFieldsets, // subject array
             $aSectionset[ 'section_id' ], // key
             array()      // default
         );                                
@@ -113,7 +113,7 @@ 
      */
     public function removeSection( $sSectionID ) {
         
-        if ( '_default' === $sSectionID ){ 
+        if ( '_default' === $sSectionID ) { 
             return; 
         }
         unset( 
@@ -143,7 +143,7 @@ 
      * @return      void
      */
     public function addResource( $sKey, $sValue ) {
-        self::$_aResources[ $sKey ][] = $sValue;
+        self::$_aResources[ $sKey ][ ] = $sValue;
     }
     
     /**
@@ -166,7 +166,7 @@ 
     public function addField( $asFieldset ) {
 
         // If it is a target section, update the property and return.
-        if ( ! $this->_isFieldsetDefinition( $asFieldset ) ) {
+        if ( !$this->_isFieldsetDefinition( $asFieldset ) ) {
             $this->_asTargetSectionID = $this->_getTargetSectionID( $asFieldset );
             return $this->_asTargetSectionID;
         }
@@ -175,13 +175,13 @@ 
         
         // Set the target section ID
         $this->_asTargetSectionID = $this->getElement(
-            $_aFieldset,  // subject array
+            $_aFieldset, // subject array
             'section_id', // key
             $this->_asTargetSectionID // default
         );                               
 
         // Required Keys - 3.8.0+ Now 'type' can be omitted.
-        if ( ! isset( $_aFieldset[ 'field_id' ] ) ) { 
+        if ( !isset( $_aFieldset[ 'field_id' ] ) ) { 
             return null; 
         }         
                 
@@ -284,7 +284,7 @@ 
      */     
     public function removeField( $sFieldID ) {
                
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
 
             if ( array_key_exists( $sFieldID, $_aSubSectionsOrFields ) ) {
                 unset( $this->aFieldsets[ $_sSectionID ][ $sFieldID ] );

Braces +2 added lines, -2 removed lines

@@ -113,7 +113,7 @@ 
      */
     public function removeSection( $sSectionID ) {
         
-        if ( '_default' === $sSectionID ){ 
+        if ( '_default' === $sSectionID ) {
             return; 
         }
         unset( 
@@ -181,7 +181,7 @@ 
         );                               
 
         // Required Keys - 3.8.0+ Now 'type' can be omitted.
-        if ( ! isset( $_aFieldset[ 'field_id' ] ) ) { 
+        if ( ! isset( $_aFieldset[ 'field_id' ] ) ) {
             return null; 
         }         
                 

development/factory/_common/form/AdminPageFramework_Form_Model.php

Doc Comments -2 removed lines

@@ -391,7 +391,6 @@ 
      * 
      * @since       DEVVER
      * @param       array       $aSavedData
-     * @param       boolean     $bOnlyFieldsets     Whether to format only the fieldsets. The taxonomy field factory uses this parameter.
      */
     protected function _formatElementDefinitions( array $aSavedData ) {
                 
@@ -427,7 +426,6 @@ 
      * Changed the name from `_getFieldErrors()`. 
      * @access      public      The field type class accesses this method to render nested fields.
      * @internal
-     * @param       boolean     $bDelete    whether or not the transient should be deleted after retrieving it. 
      * @return      array
      */
     public function getFieldErrors() {

Spacing +20 added lines, -20 removed lines

@@ -43,10 +43,10 @@ 
      * @since       3.7.0
      * @return      array
      */
-    public function getSubmittedData( array $aDataToParse, $bExtractFromFieldStructure=true, $bStripSlashes=true ) {
+    public function getSubmittedData( array $aDataToParse, $bExtractFromFieldStructure = true, $bStripSlashes = true ) {
                 
         // Extracts the form data from the subject data for parsing
-        $_aSubmittedFormData    = $bExtractFromFieldStructure
+        $_aSubmittedFormData = $bExtractFromFieldStructure
             ? $this->castArrayContents( 
                 $this->getDataStructureFromAddedFieldsets(), // form data (options) structure
                 $aDataToParse   // the subject data array, usually $_POST.
@@ -54,7 +54,7 @@ 
             : $aDataToParse;
 
         // 3.6.0 - sorts dynamic eleemnts.        
-        $_aSubmittedFormData    = $this->getSortedInputs( $_aSubmittedFormData ); 
+        $_aSubmittedFormData = $this->getSortedInputs( $_aSubmittedFormData ); 
         
         return $bStripSlashes
             ? stripslashes_deep( $_aSubmittedFormData ) // fixes magic quotes
@@ -79,12 +79,12 @@ 
             array_merge(
                 $this->getElementAsArray( 
                     $_POST,
-                    '__repeatable_elements_' . $this->aArguments[ 'structure_type' ],
+                    '__repeatable_elements_'.$this->aArguments[ 'structure_type' ],
                     array()
                 ),
                 $this->getElementAsArray( 
                     $_POST,
-                    '__sortable_elements_' . $this->aArguments[ 'structure_type' ],
+                    '__sortable_elements_'.$this->aArguments[ 'structure_type' ],
                     array()
                 )
             )
@@ -147,9 +147,9 @@ 
      * Changed the name from `getFieldsModel()`.
      * @return      array
      */
-    public function getDataStructureFromAddedFieldsets()  {
+    public function getDataStructureFromAddedFieldsets() {
                     
-        $_aFormDataStructure  = array();
+        $_aFormDataStructure = array();
         foreach ( $this->getAsArray( $this->aFieldsets ) as $_sSectionID => $_aFieldsets ) {
 
             if ( $_sSectionID != '_default' ) {                
@@ -158,7 +158,7 @@ 
             }
             
             // For default field items.
-            foreach( $_aFieldsets as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aFieldsets as $_sFieldID => $_aFieldset ) {
                 $_aFormDataStructure[ $_aFieldset[ 'field_id' ] ] = $_aFieldset;
             }
 
@@ -190,7 +190,7 @@ 
             $aSubject,
             $this->getElementAsArray(
                 $_POST,
-                '__repeatable_elements_' . $this->aArguments[ 'structure_type' ]
+                '__repeatable_elements_'.$this->aArguments[ 'structure_type' ]
             )
         );
         return $_oFilterRepeatableElements->get();
@@ -203,13 +203,13 @@ 
     public function _replyToRegisterFormItems() {
 
         // Check if the form should be created or not.
-        if ( ! $this->isInThePage() ) {
+        if ( !$this->isInThePage() ) {
             return;
         }
                 
         // Load field type definitions.
-        $this->_setFieldTypeDefinitions( 'admin_page_framework' );      // site-wide
-        $this->_setFieldTypeDefinitions( $this->aArguments[ 'caller_id' ] );  // per class 
+        $this->_setFieldTypeDefinitions( 'admin_page_framework' ); // site-wide
+        $this->_setFieldTypeDefinitions( $this->aArguments[ 'caller_id' ] ); // per class 
         
         // Set the options array
         $this->aSavedData = $this->_getSavedData(
@@ -237,7 +237,7 @@ 
             $this->aArguments,
             $this->aFieldsets,
             self::$_aResources,
-            $this->aFieldTypeDefinitions,   // must be called after performing `_setFieldTypeDefinitions()`.
+            $this->aFieldTypeDefinitions, // must be called after performing `_setFieldTypeDefinitions()`.
             $this->aCallbacks
         );
         self::$_aResources = $_oFieldResources->get(); // updates the property
@@ -254,10 +254,10 @@ 
         $this->callBack(
             $this->aCallbacks[ 'handle_form_data' ],
             array(
-                $this->aSavedData,      // 1st parameter
-                $this->aArguments,      // 2nd parameter
-                $this->aSectionsets,    // 3rd parameter
-                $this->aFieldsets,      // 4th parameter
+                $this->aSavedData, // 1st parameter
+                $this->aArguments, // 2nd parameter
+                $this->aSectionsets, // 3rd parameter
+                $this->aFieldsets, // 4th parameter
             )
         );        
 
@@ -271,15 +271,15 @@ 
             $this->aSectionsets = $this->callBack(
                 $this->aCallbacks[ 'secitonsets_before_registration' ],
                 array(
-                    $this->aSectionsets,    // 1st parameter
+                    $this->aSectionsets, // 1st parameter
                 )
             );
             // Let the main routine modify the fieldsets definition array.
             $this->aFieldsets = $this->callBack(
                 $this->aCallbacks[ 'fieldsets_before_registration' ],
                 array(
-                    $this->aFieldsets,    // 1st parameter
-                    $this->aSectionsets,  // 2nd parameter
+                    $this->aFieldsets, // 1st parameter
+                    $this->aSectionsets, // 2nd parameter
                 )
             );
 

Braces +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
                 array( $this, '_replyToRegisterFormItems' ),
                 100 // priority - low value is set as meta boxes use the `current_screen` action hook for `setUp()`.
             );
-        } else {                
+        } else {
             add_action(
                 $this->aArguments[ 'action_hook_form_registration' ],
                 array( $this, '_replyToRegisterFormItems' )
@@ -147,12 +147,12 @@ 
      * Changed the name from `getFieldsModel()`.
      * @return      array
      */
-    public function getDataStructureFromAddedFieldsets()  {
+    public function getDataStructureFromAddedFieldsets() {
                     
         $_aFormDataStructure  = array();
         foreach ( $this->getAsArray( $this->aFieldsets ) as $_sSectionID => $_aFieldsets ) {
 
-            if ( $_sSectionID != '_default' ) {                
+            if ( $_sSectionID != '_default' ) {
                 $_aFormDataStructure[ $_sSectionID ] = $_aFieldsets;
                 continue;
             }
@@ -185,7 +185,7 @@ 
      * @param       array       $aSubject       The subject array to modify. Usually the saved option data.
      * @return      array       The modified options array.
      */
-    public function dropRepeatableElements( array $aSubject ) {        
+    public function dropRepeatableElements( array $aSubject ) {
         $_oFilterRepeatableElements = new AdminPageFramework_Form_Model___Modifier_FilterRepeatableElements( 
             $aSubject,
             $this->getElementAsArray(

development/factory/_common/form/field_type/AdminPageFramework_FieldType_system.php

Doc Comments +3 added lines

@@ -231,6 +231,7 @@ 
             /**
              * Returns the system information by section.
              * @since       3.5.3
+             * @param integer $iPrintType
              * @return      string      The system information by section.
              */
             private function _getSystemInfoBySection( $sSectionName, $aData, $iPrintType ) {
@@ -350,6 +351,8 @@ 
                     /**
                      * 
                      * @since       3.5.12
+                     * @param string $asCategories
+                     * @param string $asRemovingCategories
                      * @return      stirng|array
                      */
                     private function _getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {

Braces +5 added lines, -5 removed lines

@@ -144,7 +144,7 @@ 
      * @internal
      * @return      array
      */
-    protected function getEnqueuingScripts() { 
+    protected function getEnqueuingScripts() {
         return array();
     }
     
@@ -162,7 +162,7 @@ 
      * @return      array
      * @internal
      */
-    protected function getEnqueuingStyles() { 
+    protected function getEnqueuingStyles() {
         return array();
     }            
 
@@ -172,7 +172,7 @@ 
      * @internal
      * @return      string
      */ 
-    protected function getScripts() { 
+    protected function getScripts() {
         return '';
     }
 
@@ -182,7 +182,7 @@ 
      * @internal
      * @return      string
      */
-    protected function getIEStyles() { 
+    protected function getIEStyles() {
         return ''; 
     }
 
@@ -223,7 +223,7 @@ 
      * @internal
      * @return      string
      */
-    protected function getField( $aField ) { 
+    protected function getField( $aField ) {
 
         $_aInputAttributes           = $aField['attributes'];
         $_aInputAttributes['class'] .= ' system';

Spacing +59 added lines, -59 removed lines

@@ -73,7 +73,7 @@ 
      * </code>
      * @var         array
      */
-    public $aFieldTypeSlugs = array( 'system', );
+    public $aFieldTypeSlugs = array( 'system',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -93,8 +93,8 @@ 
      * @remark          `$_aDefaultKeys` holds shared default key-values defined in the base class.
      */
     protected $aDefaultKeys = array(
-        'data'          =>  array(),        // [3.2.0+] Stores the data to be displayed
-        'print_type'    =>  1,              // [3.4.6+] 1: readable representation of array. 2: print_r()
+        'data'          =>  array(), // [3.2.0+] Stores the data to be displayed
+        'print_type'    =>  1, // [3.4.6+] 1: readable representation of array. 2: print_r()
         'attributes'    =>    array(
             'rows'          => 60,
             'autofocus'     => null,
@@ -233,13 +233,13 @@ 
             . "<div class='admin-page-framework-input-label-container'>"
                 . "<label for='{$aField[ 'input_id' ]}'>"
                     . $aField[ 'before_input' ]
-                    . ( $aField[ 'label' ] && ! $aField[ 'repeatable' ]
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                 . $aField[ 'label' ] 
                             . "</span>"
                         : "" 
                     )
-                    . "<textarea " . $this->getAttributes( $_aInputAttributes ) . " >"    
+                    . "<textarea ".$this->getAttributes( $_aInputAttributes )." >"    
                         . esc_textarea( $this->_getSystemInfomation( $aField[ 'value' ], $aField[ 'data' ], $aField[ 'print_type' ] ) )
                     . "</textarea>"
                     . $aField[ 'after_input' ]
@@ -254,15 +254,15 @@ 
          * @return      string      The human readable system information.
          * @internal
          */
-        private function _getSystemInfomation( $asValue=null, $asCustomData=null, $iPrintType=1 ) {
+        private function _getSystemInfomation( $asValue = null, $asCustomData = null, $iPrintType = 1 ) {
 
             if ( isset( $asValue ) ) {
                 return $asValue;
             }
 
-            $_aOutput   = array();
-            foreach( $this->_getFormattedSystemInformation( $asCustomData ) as $_sSection => $_aInfo ) {
-                $_aOutput[] = $this->_getSystemInfoBySection( $_sSection, $_aInfo, $iPrintType );
+            $_aOutput = array();
+            foreach ( $this->_getFormattedSystemInformation( $asCustomData ) as $_sSection => $_aInfo ) {
+                $_aOutput[ ] = $this->_getSystemInfoBySection( $_sSection, $_aInfo, $iPrintType );
             }
             return implode( PHP_EOL, $_aOutput );
             
@@ -277,18 +277,18 @@ 
                 
                 $_aData = $this->getAsArray( $asCustomData );
                 $_aData = $_aData + array(
-                    'Admin Page Framework'  => isset( $_aData['Admin Page Framework'] )
+                    'Admin Page Framework'  => isset( $_aData[ 'Admin Page Framework' ] )
                         ? null
                         : AdminPageFramework_Registry::getInfo(),
-                    'WordPress'             => $this->_getSiteInfoWithCache( ! isset( $_aData['WordPress'] ) ),
-                    'PHP'                   => $this->_getPHPInfo( ! isset( $_aData['PHP'] ) ),
-                    'PHP Error Log'         => $this->_getErrorLogByType( 'php', ! isset( $_aData['PHP Error Log'] ) ),
-                    'MySQL'                 => isset( $_aData['MySQL'] )
+                    'WordPress'             => $this->_getSiteInfoWithCache( !isset( $_aData[ 'WordPress' ] ) ),
+                    'PHP'                   => $this->_getPHPInfo( !isset( $_aData[ 'PHP' ] ) ),
+                    'PHP Error Log'         => $this->_getErrorLogByType( 'php', !isset( $_aData[ 'PHP Error Log' ] ) ),
+                    'MySQL'                 => isset( $_aData[ 'MySQL' ] )
                         ? null
-                        : $this->getMySQLInfo(),    // defined in the utility class.
-                    'MySQL Error Log'       => $this->_getErrorLogByType( 'mysql', ! isset( $_aData['MySQL Error Log'] ) ),
-                    'Server'                => $this->_getWebServerInfo( ! isset( $_aData['Server'] ) ),
-                    'Browser'               => $this->_getClientInfo( ! isset( $_aData['Browser'] ) ),
+                        : $this->getMySQLInfo(), // defined in the utility class.
+                    'MySQL Error Log'       => $this->_getErrorLogByType( 'mysql', !isset( $_aData[ 'MySQL Error Log' ] ) ),
+                    'Server'                => $this->_getWebServerInfo( !isset( $_aData[ 'Server' ] ) ),
+                    'Browser'               => $this->_getClientInfo( !isset( $_aData[ 'Browser' ] ) ),
                 );
                 
                 // Dropping empty elements allows the user to remove a section by setting an empty section.
@@ -305,10 +305,10 @@ 
                 switch ( $iPrintType ) {
                     default:
                     case 1: // use the framework readable representation of arrays.
-                        return $this->getReadableArrayContents( $sSectionName, $aData, 32 ) . PHP_EOL;
+                        return $this->getReadableArrayContents( $sSectionName, $aData, 32 ).PHP_EOL;
                     case 2: // use print_r()
-                        return "[{$sSectionName}]" . PHP_EOL
-                            . print_r( $aData, true ) . PHP_EOL;
+                        return "[{$sSectionName}]".PHP_EOL
+                            . print_r( $aData, true ).PHP_EOL;
                 }                      
             }
             /**
@@ -318,17 +318,17 @@ 
              * @since       3.4.6
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              */
-            private function _getClientInfo( $bGenerateInfo=true ) {
+            private function _getClientInfo( $bGenerateInfo = true ) {
                  
-                if ( ! $bGenerateInfo ) {
+                if ( !$bGenerateInfo ) {
                     return '';
                 }
                  
                 // Check the browscap value in the ini file first to prevent warnings from being populated
                 $_aBrowser = @ini_get( 'browscap' ) 
-                    ? get_browser( $_SERVER['HTTP_USER_AGENT'], true )
+                    ? get_browser( $_SERVER[ 'HTTP_USER_AGENT' ], true )
                     : array();
-                unset( $_aBrowser['browser_name_regex'] );  // this element causes output to be blank
+                unset( $_aBrowser[ 'browser_name_regex' ] ); // this element causes output to be blank
                 return empty( $_aBrowser ) 
                     ? __( 'No browser information found.', 'admin-page-framework' )
                     : $_aBrowser;
@@ -344,9 +344,9 @@ 
              * @param       string      $sType          The error log type. Either 'php' or 'mysql' is accepted.
              * @param       boolean     $bGenerateInfo  Whether to generate a log. This is for the caller method to reduce a conditinal statement.
              */
-            private function _getErrorLogByType( $sType, $bGenerateInfo=true ) {
+            private function _getErrorLogByType( $sType, $bGenerateInfo = true ) {
 
-                if ( ! $bGenerateInfo ) {
+                if ( !$bGenerateInfo ) {
                     return '';
                 }
                 switch ( $sType ) {
@@ -378,9 +378,9 @@ 
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              * @return      array      The generated site information array.
              */
-            private function _getSiteInfoWithCache( $bGenerateInfo=true ) {
+            private function _getSiteInfoWithCache( $bGenerateInfo = true ) {
                 
-                if ( ! $bGenerateInfo || isset( self::$_aSiteInfo ) ) {
+                if ( !$bGenerateInfo || isset( self::$_aSiteInfo ) ) {
                     return self::$_aSiteInfo;
                 }
                 self::$_aSiteInfo = self::_getSiteInfo();
@@ -414,8 +414,8 @@ 
                         'wp_remote_get()'                                           => $this->_getWPRemoteGetStatus(),
                         __( 'Multibite String Extension', 'admin-page-framework' )  => $this->getAOrB( function_exists( 'mb_detect_encoding' ), $this->oMsg->get( 'enabled' ), $this->oMsg->get( 'disabled' ) ),
                         __( 'WP_CONTENT_DIR Writeable', 'admin-page-framework' )    => $this->getAOrB( is_writable( WP_CONTENT_DIR ), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ), 
-                        __( 'Active Plugins', 'admin-page-framework' )              => PHP_EOL . $this->_getActivePlugins(),
-                        __( 'Network Active Plugins', 'admin-page-framework' )      => PHP_EOL . $this->_getNetworkActivePlugins(),
+                        __( 'Active Plugins', 'admin-page-framework' )              => PHP_EOL.$this->_getActivePlugins(),
+                        __( 'Network Active Plugins', 'admin-page-framework' )      => PHP_EOL.$this->_getNetworkActivePlugins(),
                         __( 'Constants', 'admin-page-framework' )                   => $this->_getDefinedConstants( 'user' ),
                     );                        
                 }
@@ -425,9 +425,9 @@ 
                      * @return      string|array
                      * @internal
                      */
-                    private function _getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {
+                    private function _getDefinedConstants( $asCategories = null, $asRemovingCategories = null ) {
                         $_asConstants = $this->getDefinedConstants( $asCategories, $asRemovingCategories );
-                        if ( ! is_array( $_asConstants ) ) {
+                        if ( !is_array( $_asConstants ) ) {
                             return $_asConstants;
                         }
                         if ( isset( $_asConstants[ 'user' ] ) ) {
@@ -463,13 +463,13 @@ 
                 private function _getActiveThemeName() {
                     
                     // If the WordPress version is less than 3.4,
-                    if ( version_compare( $GLOBALS['wp_version'], '3.4', '<' ) ) {
-                        $_aThemeData = get_theme_data( get_stylesheet_directory() . '/style.css' );
-                        return $_aThemeData['Name'] . ' ' . $_aThemeData['Version'];
+                    if ( version_compare( $GLOBALS[ 'wp_version' ], '3.4', '<' ) ) {
+                        $_aThemeData = get_theme_data( get_stylesheet_directory().'/style.css' );
+                        return $_aThemeData[ 'Name' ].' '.$_aThemeData[ 'Version' ];
                     } 
                     
                     $_oThemeData = wp_get_theme();
-                    return $_oThemeData->Name . ' ' . $_oThemeData->Version;
+                    return $_oThemeData->Name.' '.$_oThemeData->Version;
                     
                 }   
                 /**
@@ -483,10 +483,10 @@ 
                     $_aPluginList       = array();
                     $_aActivePlugins    = get_option( 'active_plugins', array() );
                     foreach ( get_plugins() as $_sPluginPath => $_aPlugin ) {
-                        if ( ! in_array( $_sPluginPath, $_aActivePlugins ) ) {
+                        if ( !in_array( $_sPluginPath, $_aActivePlugins ) ) {
                             continue;
                         }
-                        $_aPluginList[] = '    ' . $_aPlugin['Name'] . ': ' . $_aPlugin['Version'];
+                        $_aPluginList[ ] = '    '.$_aPlugin[ 'Name' ].': '.$_aPlugin[ 'Version' ];
                     }
                     return implode( PHP_EOL, $_aPluginList );
                     
@@ -498,17 +498,17 @@ 
                  */
                 private function _getNetworkActivePlugins() {
                     
-                    if ( ! is_multisite() ) {
+                    if ( !is_multisite() ) {
                         return '';
                     }
                     $_aPluginList       = array();
                     $_aActivePlugins    = get_site_option( 'active_sitewide_plugins', array() );
                     foreach ( wp_get_active_network_plugins() as $_sPluginPath ) {
-                        if ( ! array_key_exists( plugin_basename( $_sPluginPath ), $_aActivePlugins ) ) {
+                        if ( !array_key_exists( plugin_basename( $_sPluginPath ), $_aActivePlugins ) ) {
                             continue;
                         }
                         $_aPlugin       = get_plugin_data( $_sPluginPath );
-                        $_aPluginList[] = '    ' . $_aPlugin['Name'] . ' :' . $_aPlugin['Version'];
+                        $_aPluginList[ ] = '    '.$_aPlugin[ 'Name' ].' :'.$_aPlugin[ 'Version' ];
                     }
                     return implode( PHP_EOL, $_aPluginList );
                     
@@ -526,7 +526,7 @@ 
                     $_vResponse = false === $_vResponse
                         ? wp_remote_post( 
                             // 'https://www.paypal.com/cgi-bin/webscr', 
-                            add_query_arg( $_GET, admin_url( $GLOBALS['pagenow'] ) ),
+                            add_query_arg( $_GET, admin_url( $GLOBALS[ 'pagenow' ] ) ),
                             array(
                                 'sslverify'     => false,
                                 'timeout'       => 60,
@@ -549,7 +549,7 @@ 
                     $_vResponse = $this->getTransient( 'apf_rg_check' );
                     $_vResponse = false === $_vResponse
                         ? wp_remote_get( 
-                            add_query_arg( $_GET + array( 'apf_remote_request_test' => '_testing' ), admin_url( $GLOBALS['pagenow'] ) ),
+                            add_query_arg( $_GET + array( 'apf_remote_request_test' => '_testing' ), admin_url( $GLOBALS[ 'pagenow' ] ) ),
                             array(
                                 'sslverify'     => false,
                                 'timeout'       => 60,
@@ -574,7 +574,7 @@ 
                         if ( is_wp_error( $mResponse ) ) {
                             return true;
                         }
-                        if ( $mResponse[ 'response'][ 'code' ] < 200 ) {
+                        if ( $mResponse[ 'response' ][ 'code' ] < 200 ) {
                             return true;
                         }
                         if ( $mResponse[ 'response' ][ 'code' ] >= 300 ) {
@@ -597,13 +597,13 @@ 
              * @since       3.4.6
              * @since       3.5.3       Added the $bGenerateInfo parameter. This is to reduce conditional statement in the caller method.
              */
-            private function _getPHPInfo( $bGenerateInfo=true ) {
+            private function _getPHPInfo( $bGenerateInfo = true ) {
                 
-                if ( ! $bGenerateInfo || isset( self::$_aPHPInfo ) ) {
+                if ( !$bGenerateInfo || isset( self::$_aPHPInfo ) ) {
                     return self::$_aPHPInfo;
                 }
                 
-                $_oErrorReporting   = new AdminPageFramework_ErrorReporting;
+                $_oErrorReporting = new AdminPageFramework_ErrorReporting;
                 self::$_aPHPInfo = array(
                     __( 'Version', 'admin-page-framework' )                 => phpversion(),
                     __( 'Safe Mode', 'admin-page-framework' )               => $this->getAOrB( @ini_get( 'safe_mode' ), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ), 
@@ -614,16 +614,16 @@ 
                     __( 'Max Execution Time', 'admin-page-framework' )      => @ini_get( 'max_execution_time' ),
                     __( 'Max Input Vars', 'admin-page-framework' )          => @ini_get( 'max_input_vars' ),
                     __( 'Argument Separator', 'admin-page-framework' )      => @ini_get( 'arg_separator.output' ),
-                    __( 'Allow URL File Open', 'admin-page-framework' )     => $this->getAOrB( @ini_get( 'allow_url_fopen' ),    $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ),
-                    __( 'Display Errors', 'admin-page-framework' )          => $this->getAOrB( @ini_get( 'display_errors' ),     $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
-                    __( 'Log Errors', 'admin-page-framework' )              => $this->getAOrB( @ini_get( 'log_errors' ),         $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
+                    __( 'Allow URL File Open', 'admin-page-framework' )     => $this->getAOrB( @ini_get( 'allow_url_fopen' ), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ),
+                    __( 'Display Errors', 'admin-page-framework' )          => $this->getAOrB( @ini_get( 'display_errors' ), $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
+                    __( 'Log Errors', 'admin-page-framework' )              => $this->getAOrB( @ini_get( 'log_errors' ), $this->oMsg->get( 'on' ), $this->oMsg->get( 'off' ) ),
                     __( 'Error log location', 'admin-page-framework' )      => @ini_get( 'error_log' ),
                     __( 'Error Reporting Level', 'admin-page-framework' )   => $_oErrorReporting->getErrorLevel(),
-                    __( 'FSOCKOPEN', 'admin-page-framework' )               => $this->getAOrB( function_exists( 'fsockopen' ),   $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    __( 'cURL', 'admin-page-framework' )                    => $this->getAOrB( function_exists( 'curl_init' ),   $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    __( 'SOAP', 'admin-page-framework' )                    => $this->getAOrB( class_exists( 'SoapClient' ),     $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    __( 'SUHOSIN', 'admin-page-framework' )                 => $this->getAOrB( extension_loaded( 'suhosin' ),    $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
-                    'ini_set()'                                             => $this->getAOrB( function_exists( 'ini_set' ),     $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'FSOCKOPEN', 'admin-page-framework' )               => $this->getAOrB( function_exists( 'fsockopen' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'cURL', 'admin-page-framework' )                    => $this->getAOrB( function_exists( 'curl_init' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'SOAP', 'admin-page-framework' )                    => $this->getAOrB( class_exists( 'SoapClient' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    __( 'SUHOSIN', 'admin-page-framework' )                 => $this->getAOrB( extension_loaded( 'suhosin' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
+                    'ini_set()'                                             => $this->getAOrB( function_exists( 'ini_set' ), $this->oMsg->get( 'supported' ), $this->oMsg->get( 'not_supported' ) ),
                 ) 
                 + $this->getPHPInfo()
                 + array( 
@@ -642,11 +642,11 @@ 
              * @since       3.5.3       Added the $bGenerateInfo paramter. This is to reduce conditional statment in the caller method.
              * @return      array|string
              */                      
-            private function _getWebServerInfo( $bGenerateInfo=true ) {
+            private function _getWebServerInfo( $bGenerateInfo = true ) {
                         
                 return $bGenerateInfo 
                     ? array(
-                        __( 'Web Server', 'admin-page-framework' )                  => $_SERVER['SERVER_SOFTWARE'],
+                        __( 'Web Server', 'admin-page-framework' )                  => $_SERVER[ 'SERVER_SOFTWARE' ],
                         'SSL'                                                       => $this->getAOrB( is_ssl(), $this->oMsg->get( 'yes' ), $this->oMsg->get( 'no' ) ),
                         __( 'Session', 'admin-page-framework' )                     => $this->getAOrB( isset( $_SESSION ), $this->oMsg->get( 'enabled' ), $this->oMsg->get( 'disabled' ) ),
                         __( 'Session Name', 'admin-page-framework' )                => esc_html( @ini_get( 'session.name' ) ),