michaeluno / admin-page-framework

development/cli/AdminPageFramework_InclusionClassFilesHeader.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * If accessed from a console, include the registry class to laod 'AdminPageFramework_Registry_Base'.
  */
 if ( php_sapi_name() === 'cli' ) {
-    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ) . '/admin-page-framework.php';
+    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ).'/admin-page-framework.php';
     if ( file_exists( $_sFrameworkFilePath ) ) {
         include_once( $_sFrameworkFilePath );
     }

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

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $_aParameters = func_get_args() + array(
             $this->aSubject,
         );
-        $this->aSubject  = $_aParameters[ 0 ];
+        $this->aSubject = $_aParameters[ 0 ];
         
     }
     

development/factory/_common/form/AdminPageFramework_Form_Controller.php

Braces +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
      */
     public function removeSection( $sSectionID ) {
         
-        if ( '_default' === $sSectionID ){
+        if ( '_default' === $sSectionID ) {
             return;
         }
         unset(

Spacing +9 added lines, -9 removed lines

@@ -44,7 +44,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 );
     }
 
@@ -70,7 +70,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,
@@ -97,7 +97,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
         );
@@ -112,7 +112,7 @@ 
      */
     public function removeSection( $sSectionID ) {
 
-        if ( '_default' === $sSectionID ){
+        if ( '_default' === $sSectionID ) {
             return;
         }
         unset(
@@ -151,7 +151,7 @@ 
      * @return      void
      */
     public function addResource( $sKey, $sValue ) {
-        self::$_aResources[ $sKey ][] = $sValue;
+        self::$_aResources[ $sKey ][ ] = $sValue;
     }
 
     /**
@@ -174,7 +174,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;
         }
@@ -183,13 +183,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;
         }
 
@@ -292,7 +292,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 ] );

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

Spacing +6 added lines, -6 removed lines

@@ -70,7 +70,7 @@ 
                 $aOptions // data source
             );
 
-            foreach( $aOptions as $_sSectionID => $_aSubSectionOrFields ) {
+            foreach ( $aOptions as $_sSectionID => $_aSubSectionOrFields ) {
                 
                 $_aSubSection = $this->_getSubSectionFromOptions(
                     $_sSectionID,
@@ -107,10 +107,10 @@ 
                 
                 $_aSubSection = array();
                 $_iPrevIndex  = null;
-                foreach( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
+                foreach ( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
                 
                     // If it is not a sub-section array, skip.
-                    if ( ! $this->isNumericInteger( $_isIndexOrFieldID ) ) {
+                    if ( !$this->isNumericInteger( $_isIndexOrFieldID ) ) {
                         continue;
                     }
                     
@@ -124,7 +124,7 @@ 
                     );
        
                     // Update the internal section index key
-                    foreach( $_aSubSection[ $_iIndex ] as &$_aField ) {
+                    foreach ( $_aSubSection[ $_iIndex ] as &$_aField ) {
                         $_aField[ '_section_index' ] = $_iIndex;
                     }
                     unset( $_aField ); // to be safe in PHP
@@ -150,7 +150,7 @@ 
                  */
                 private function _getSubSectionItemsFromOptions( array $_aSubSection, $_sSectionID, $_iIndex, $_iPrevIndex ) {
                     
-                    if ( ! isset( $this->aFieldsets[ $_sSectionID ] ) ) {
+                    if ( !isset( $this->aFieldsets[ $_sSectionID ] ) ) {
                         return array();
                     }
                     
@@ -159,7 +159,7 @@ 
                         : $this->getNonIntegerKeyElements( $this->aFieldsets[ $_sSectionID ] );
                         
                     // if empty, merge with the previous element.
-                    return ! empty( $_aFields )
+                    return !empty( $_aFields )
                         ? $_aFields
                         : $this->getElementAsArray(
                             $_aSubSection,

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

Spacing +13 added lines, -13 removed lines

@@ -24,12 +24,12 @@ 
      * @since       3.6.0       Moved from `AdminPageFramework_FormDefinition`.
      */
     static public $aStructure = array(
-        'title'                     => null,        // (string)  the section title will be assigned by default in the section formatting method.
-        'is_collapsed'              => true,        // (boolean) whether it is already collapsed or expanded
-        'toggle_all_button'         => null,        // (boolean|string|array) the position of where to display the toggle-all button that toggles the folding state of all collapsible sections. Accepts the following values. 'top-right', 'top-left', 'bottom-right', 'bottom-left'. If true is passed, the default 'top-right' will be used. To not to display, do not set any or pass `false` or `null`.
-        'collapse_others_on_expand' => true,        // (boolean) whether the other collapsible sections should be folded when the section is unfolded.
-        'container'                 => 'sections',  // (string) the container element that collapsible styling gets applied to. Either 'sections' or 'section' is accepted.
-        'type'                      => 'box',       // 3.7.0+  (string)  supported types 'box', 'button' Default: `box`. The `button` type is only supported when the `container` argument is `section`.
+        'title'                     => null, // (string)  the section title will be assigned by default in the section formatting method.
+        'is_collapsed'              => true, // (boolean) whether it is already collapsed or expanded
+        'toggle_all_button'         => null, // (boolean|string|array) the position of where to display the toggle-all button that toggles the folding state of all collapsible sections. Accepts the following values. 'top-right', 'top-left', 'bottom-right', 'bottom-left'. If true is passed, the default 'top-right' will be used. To not to display, do not set any or pass `false` or `null`.
+        'collapse_others_on_expand' => true, // (boolean) whether the other collapsible sections should be folded when the section is unfolded.
+        'container'                 => 'sections', // (string) the container element that collapsible styling gets applied to. Either 'sections' or 'section' is accepted.
+        'type'                      => 'box', // 3.7.0+  (string)  supported types 'box', 'button' Default: `box`. The `button` type is only supported when the `container` argument is `section`.
     );
     
     public $abCollapsible = false;
@@ -42,7 +42,7 @@ 
      * @remark      The framework will not pass this parameter when formatting a section definition array.
      * It will be passed when the framework is rendering a form table to generate collapsible elements.
      */
-    public $aSection      = array();
+    public $aSection = array();
     
     /**
      * Sets up properties.
@@ -85,14 +85,14 @@ 
             
             $_aCollapsible = $this->getAsArray( $this->abCollapsible ) + array(
                 'title'     => $sTitle,
-            ) +  self::$aStructure;
+            ) + self::$aStructure;
             
             $_aCollapsible[ 'toggle_all_button' ] = implode(
                 ',',
                 $this->getAsArray( $_aCollapsible[ 'toggle_all_button' ] )
             );
             
-            if ( ! empty( $aSection ) ) {
+            if ( !empty( $aSection ) ) {
                 $_aCollapsible[ 'toggle_all_button' ] = $this->_getToggleAllButtonArgument(
                     $_aCollapsible[ 'toggle_all_button' ],
                     $aSection
@@ -116,7 +116,7 @@ 
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {
+                if ( !$aSection[ 'repeatable' ] ) {
                     return $sToggleAll;
                 }
                 
@@ -126,13 +126,13 @@ 
                 }
                 
                 // Disable the toggle all button for middle sub-sections in repeatable sections.
-                if ( ! $aSection[ '_is_first_index' ] && ! $aSection[ '_is_last_index' ] ) {
+                if ( !$aSection[ '_is_first_index' ] && !$aSection[ '_is_last_index' ] ) {
                     return 0;
                 }
                 
                 $_aToggleAll = $this->getAOrB(
-                    true === $sToggleAll || 1 ===  $sToggleAll, // evaluate
-                    array( 'top-right', 'bottom-right' ),   // if true
+                    true === $sToggleAll || 1 === $sToggleAll, // evaluate
+                    array( 'top-right', 'bottom-right' ), // if true
                     explode( ',', $sToggleAll ) // if false
                 );
                 $_aToggleAll = $this->getAOrB(

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

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
             $this->aAttributes + $this->_getAttributes()
         );
                     
-        $_aAttributes[ 'class' ]   = $this->getClassAttribute(
+        $_aAttributes[ 'class' ] = $this->getClassAttribute(
             $this->getElement( $_aAttributes, 'class', array() ),
             $this->getElement( $this->aArguments, array( 'class', $this->sContext ), array() )
         );

development/factory/_common/form/_view/script/AdminPageFramework_Form_View___Script_CollapsibleSection.php

Spacing +5 added lines, -5 removed lines

@@ -43,7 +43,7 @@ 
         
         $_aParams               = func_get_args() + array( null );
         $_oMsg                  = $_aParams[ 0 ];
-        $_sToggleAllButtonHTML  = '"' . self::_getToggleAllButtonHTML( $_oMsg ) . '"';
+        $_sToggleAllButtonHTML  = '"'.self::_getToggleAllButtonHTML( $_oMsg ).'"';
            
         return <<<JAVASCRIPTS
 ( function( $ ) {
@@ -212,19 +212,19 @@ 
             $_sLabelToggleAll           = $oMsg->get( 'toggle_all' );
             $_sLabelToggleAllSections   = $oMsg->get( 'toggle_all_collapsible_sections' );
             $_sDashIconSort             = self::getAOrB(
-                version_compare( $GLOBALS['wp_version'], '3.8', '<' ),  // evaluate
+                version_compare( $GLOBALS[ 'wp_version' ], '3.8', '<' ), // evaluate
                 '', // true
                 'dashicons dashicons-sort' // false
             );
-            $_sText                     = self::getAOrB(
+            $_sText = self::getAOrB(
                 $_sDashIconSort, // evaluate
                 '', // true
                 $_sLabelToggleAll // false
             );
 
             return "<div class='admin-page-framework-collapsible-toggle-all-button-container'>"
-                    . "<span class='admin-page-framework-collapsible-toggle-all-button button " . $_sDashIconSort . "'"
-                        . " title='" . esc_attr( $_sLabelToggleAllSections )
+                    . "<span class='admin-page-framework-collapsible-toggle-all-button button ".$_sDashIconSort."'"
+                        . " title='".esc_attr( $_sLabelToggleAllSections )
                     . "'>"
                         . $_sText
                     . "</span>"

development/factory/_common/form/_view/sectionset/AdminPageFramework_Form_View___CollapsibleSectionTitle.php

Spacing +8 added lines, -8 removed lines

@@ -18,17 +18,17 @@ 
  */
 class AdminPageFramework_Form_View___CollapsibleSectionTitle extends AdminPageFramework_Form_View___SectionTitle {
             
-    public $aArguments      = array(
+    public $aArguments = array(
         'title'             => null,
         'tag'               => null,
         'section_index'     => null,
         'collapsible'       => array(),
         'container_type'    => 'section', // section or sections
 
-        'sectionset'        => array(),  // 3.7.0+ sectionset definition array
+        'sectionset'        => array(), // 3.7.0+ sectionset definition array
 
     );
-    public $aFieldsets               = array();
+    public $aFieldsets = array();
     public $aSavedData              = array();
     public $aFieldErrors            = array();
     public $aFieldTypeDefinitions   = array();
@@ -63,7 +63,7 @@ 
          * @param       array|boolean   $aCollapsible       The collapsible argument.
          * @param       string          $sContainer          The position context. Accepts either 'sections' or 'section'. If the set position in the argument array does not match this value, the method will return an empty string.
          */
-        private function _getCollapsibleSectionTitleBlock( array $aCollapsible, $sContainer='sections', $iSectionIndex=null ) {
+        private function _getCollapsibleSectionTitleBlock( array $aCollapsible, $sContainer = 'sections', $iSectionIndex = null ) {
 
             if ( $sContainer !== $aCollapsible[ 'container' ] ) {
                 return '';
@@ -79,10 +79,10 @@ 
             );
             
             $_aSectionset        = $this->aArguments[ 'sectionset' ];
-            $_sSectionTitleTagID = str_replace( '|', '_', $_aSectionset[ '_section_path' ]  ) . '_' . $iSectionIndex;
+            $_sSectionTitleTagID = str_replace( '|', '_', $_aSectionset[ '_section_path' ] ).'_'.$iSectionIndex;
             
             return $this->_getCollapsibleSectionsEnablerScript()
-                . "<div " . $this->getAttributes(
+                . "<div ".$this->getAttributes(
                     array(
                         'id'    => $_sSectionTitleTagID,
                         'class' => $this->getClassAttribute(
@@ -103,11 +103,11 @@ 
                                 'collapsed',
                                 ''
                             ),
-                            'admin-page-framework-collapsible-type-' . $aCollapsible[ 'type' ]
+                            'admin-page-framework-collapsible-type-'.$aCollapsible[ 'type' ]
                         ),
                     )
                     + $this->getDataAttributeArray( $aCollapsible )
-                ) . ">"
+                ).">"
                         . $_sSectionTitle
                     . "</div>";
             

development/factory/_common/form/_view/sectionset/AdminPageFramework_Form_View___Section_Base.php

Spacing +2 added lines, -2 removed lines

@@ -56,13 +56,13 @@
     public function getFieldsetOutput( $aFieldset ) {
 
         // Check if the field is visible
-        if ( ! $this->isFieldsetVisible( $aFieldset ) ) {
+        if ( !$this->isFieldsetVisible( $aFieldset ) ) {
             return '';
         }
 
         $_oFieldset = new AdminPageFramework_Form_View___Fieldset(
             $aFieldset,
-            $this->aSavedData,    // passed by reference. @todo: examine why it needs to be passed by reference.
+            $this->aSavedData, // passed by reference. @todo: examine why it needs to be passed by reference.
             $this->aFieldErrors,
             $this->aFieldTypeDefinitions,
             $this->oMsg,