michaeluno / admin-page-framework

development/factory/_common/form/notice/AdminPageFramework_Form___SubmitNotice.php

Spacing +9 added lines, -9 removed lines

@@ -40,7 +40,7 @@ 
          * @return      string
          */
         private function _getTransientKey() {
-            return 'apf_ntc_' . get_current_user_id();
+            return 'apf_ntc_'.get_current_user_id();
         }
     
     /**
@@ -52,14 +52,14 @@ 
      * @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 hasNotice( $sType='' ) {
+    public function hasNotice( $sType = '' ) {
                 
-        if ( ! $sType ) {
+        if ( !$sType ) {
             return ( boolean ) count( self::$_aNotices );
         }
         
         // Check if there is a message of the type.
-        foreach( self::$_aNotices as $_aNotice ) {
+        foreach ( self::$_aNotices as $_aNotice ) {
             $_sClassAttribute = $this->getElement( 
                 $_aNotice, 
                 array( 
@@ -98,7 +98,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 set( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
+    public function set( $sMessage, $sType = 'error', $asAttributes = array(), $bOverride = true ) {
         
         // If the array is empty, schedule the task of saving the array at shutdown.
         if ( empty( self::$_aNotices ) ) {
@@ -108,7 +108,7 @@ 
         $_sID = md5( trim( $sMessage ) );
             
         // If the override is false and a message is already set, do not add.
-        if ( ! $bOverride && isset( self::$_aNotices[ $_sID ] ) ) {
+        if ( !$bOverride && isset( self::$_aNotices[ $_sID ] ) ) {
             return;
         }
 
@@ -118,14 +118,14 @@ 
         }
         
         $_aAttributes = $this->getAsArray( $asAttributes );
-        if ( is_string( $asAttributes ) && ! empty( $asAttributes ) ) {
+        if ( is_string( $asAttributes ) && !empty( $asAttributes ) ) {
             $_aAttributes[ 'id' ] = $asAttributes;
         }
         self::$_aNotices[ $_sID ] = array(
             'sMessage'      => $sMessage,
             'aAttributes'   => $_aAttributes + array(
                 'class'     => $sType,
-                'id'        => 'form_submit_notice_' . $_sID,
+                'id'        => 'form_submit_notice_'.$_sID,
             ),
         );
 
@@ -166,7 +166,7 @@ 
         }
             
         // By setting false to the 'settings-notice' key, it's possible to disable the notifications set with the framework.
-        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) { 
+        if ( isset( $_GET[ 'settings-notice' ] ) && !$_GET[ 'settings-notice' ] ) { 
             return; 
         }
             

Braces +3 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
          * @return      void
          */
         public function _replyToSaveNotices() {
-            if ( empty( self::$_aNotices ) ) { 
+            if ( empty( self::$_aNotices ) ) {
                 return; 
             }            
             $_bResult = $this->setTransient( 
@@ -161,12 +161,12 @@ 
         
         // Retrieve the notifications set in a transient.
         $_aNotices = $this->_getNotices();
-        if ( false === $_aNotices ) { 
+        if ( false === $_aNotices ) {
             return; 
         }
             
         // By setting false to the 'settings-notice' key, it's possible to disable the notifications set with the framework.
-        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) { 
+        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) {
             return; 
         }
             

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Meta.php

Spacing +2 added lines, -2 removed lines

@@ -38,11 +38,11 @@
      * Retrieves meta data by given keys and type (user or post).
      * @return      array
      */
-    static public function getMetaDataByKeys( $iObjectID, $aKeys, $sMetaType='post' ) {
+    static public function getMetaDataByKeys( $iObjectID, $aKeys, $sMetaType = 'post' ) {
                
         $_aSavedMeta = array();
                     
-        if ( ! $iObjectID ) {
+        if ( !$iObjectID ) {
             return $_aSavedMeta;
         }
         

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_SystemInformation.php

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
                 : @mysql_get_server_info(),
         );
         
-        foreach( ( array ) $wpdb->get_results( "SHOW VARIABLES", ARRAY_A ) as $_iIndex => $_aItem ) {
+        foreach ( ( array ) $wpdb->get_results( "SHOW VARIABLES", ARRAY_A ) as $_iIndex => $_aItem ) {
             
             $_aItem     = array_values( $_aItem );
             $_sKey      = array_shift( $_aItem );
@@ -63,8 +63,8 @@ 
     static public function getMySQLErrorLogPath() {
         
         $_aMySQLInfo = self::getMySQLInfo();
-        return isset( $_aMySQLInfo['log_error'] )
-            ? $_aMySQLInfo['log_error']
+        return isset( $_aMySQLInfo[ 'log_error' ] )
+            ? $_aMySQLInfo[ 'log_error' ]
             : '';
         
     }
@@ -73,7 +73,7 @@ 
      * Returns a PHP error log.
      * @since       3.4.6
      */
-    static public function getMySQLErrorLog( $iLines=1 ) {
+    static public function getMySQLErrorLog( $iLines = 1 ) {
         
         $_sLog = self::getFileTailContents( self::getMySQLErrorLogPath(), $iLines );
         

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_HTML.php

Spacing +6 added lines, -6 removed lines

@@ -38,9 +38,9 @@ 
      
         $_sQuoteCharactor   = "'";
         $_aOutput           = array();
-        foreach( $aAttributes as $_sAttribute => $_mProperty ) {
+        foreach ( $aAttributes as $_sAttribute => $_mProperty ) {
             if ( is_scalar( $_mProperty ) ) {
-                $_aOutput[] = "{$_sAttribute}={$_sQuoteCharactor}" . esc_attr( $_mProperty ) . "{$_sQuoteCharactor}";
+                $_aOutput[ ] = "{$_sAttribute}={$_sQuoteCharactor}".esc_attr( $_mProperty )."{$_sQuoteCharactor}";
             }            
         }     
         return implode( ' ', $_aOutput );
@@ -79,11 +79,11 @@ 
      * @since       3.6.0
      * @return      string
      */
-    static public function getHTMLTag( $sTagName, array $aAttributes, $sValue=null ) {
+    static public function getHTMLTag( $sTagName, array $aAttributes, $sValue = null ) {
         $_sTag = tag_escape( $sTagName );
         return null === $sValue
-            ? "<" . $_sTag . " " . self::getAttributes( $aAttributes ) . " />"
-            : "<" . $_sTag . " " . self::getAttributes( $aAttributes ) . ">"
+            ? "<".$_sTag." ".self::getAttributes( $aAttributes )." />"
+            : "<".$_sTag." ".self::getAttributes( $aAttributes ).">"
                     . $sValue
                 . "</{$_sTag}>";        
     }    
@@ -93,7 +93,7 @@ 
          * @return      string
          * @deprecated  3.6.0       Use `getHTMLTag()` instead.
          */
-        static public function generateHTMLTag( $sTagName, array $aAttributes, $sValue=null ) {
+        static public function generateHTMLTag( $sTagName, array $aAttributes, $sValue = null ) {
             return self::getHTMLTag( $sTagName, $aAttributes, $sValue );
         }
     

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_SiteInformation.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      * @since       3.5.3
      * @return      string      The site language.
      */
-    static public function getSiteLanguage( $sDefault='en_US' ) {
+    static public function getSiteLanguage( $sDefault = 'en_US' ) {
         return defined( 'WPLANG' ) && WPLANG ? WPLANG : $sDefault;
     }
         

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_ArraySetter.php

Braces +3 added lines, -3 removed lines

@@ -204,7 +204,7 @@ 
         
         $_aInvert = array();
         foreach( $aModel as $_isKey => $_v ) {
-            if ( array_key_exists( $_isKey, $aSubject ) ) { 
+            if ( array_key_exists( $_isKey, $aSubject ) ) {
                 continue; 
             }
             $_aInvert[ $_isKey ] = $_v;
@@ -256,11 +256,11 @@ 
      */ 
     public static function uniteArraysRecursive( $aPrecedence, $aDefault ) {
                 
-        if ( is_null( $aPrecedence ) ) { 
+        if ( is_null( $aPrecedence ) ) {
             $aPrecedence = array(); 
         }
         
-        if ( ! is_array( $aDefault ) || ! is_array( $aPrecedence ) ) { 
+        if ( ! is_array( $aDefault ) || ! is_array( $aPrecedence ) ) {
             return $aPrecedence; 
         }
             

Spacing +18 added lines, -18 removed lines

@@ -33,7 +33,7 @@ 
      * @return      integer
      * @internal
      */        
-    static public function sortArrayByKey( $a, $b, $sKey='order' ) {
+    static public function sortArrayByKey( $a, $b, $sKey = 'order' ) {
         return isset( $a[ $sKey ], $b[ $sKey ] )
             ? $a[ $sKey ] - $b[ $sKey ]
             : 1;
@@ -76,7 +76,7 @@ 
     static public function unsetDimensionalArrayElement( &$mSubject, array $aKeys ) {
         
         $_sKey = array_shift( $aKeys );
-        if ( ! empty( $aKeys ) ) {
+        if ( !empty( $aKeys ) ) {
             if ( isset( $mSubject[ $_sKey ] ) && is_array( $mSubject[ $_sKey ] ) ) {
                 self::unsetDimensionalArrayElement( $mSubject[ $_sKey ], $aKeys );
             }
@@ -97,8 +97,8 @@ 
     static public function setMultiDimensionalArray( &$mSubject, array $aKeys, $mValue ) {
 
         $_sKey = array_shift( $aKeys );
-        if ( ! empty( $aKeys ) ) {
-            if( ! isset( $mSubject[ $_sKey ] ) || ! is_array( $mSubject[ $_sKey ] ) ) {
+        if ( !empty( $aKeys ) ) {
+            if ( !isset( $mSubject[ $_sKey ] ) || !is_array( $mSubject[ $_sKey ] ) ) {
                 $mSubject[ $_sKey ] = array();
             }
             self::setMultiDimensionalArray( $mSubject[ $_sKey ], $aKeys, $mValue );
@@ -152,10 +152,10 @@ 
 
         $_aNumeric      = self::getIntegerKeyElements( $aSubject );
         $_aAssociative  = self::invertCastArrayContents( $aSubject, $_aNumeric );
-        foreach( $_aNumeric as &$_aElem ) {
+        foreach ( $_aNumeric as &$_aElem ) {
             $_aElem = self::uniteArrays( $_aElem, $_aAssociative );
         }
-        if ( ! empty( $_aAssociative ) ) {
+        if ( !empty( $_aAssociative ) ) {
             array_unshift( $_aNumeric, $_aAssociative ); // insert the main section to the beginning of the array.
         }
         return $_aNumeric;
@@ -178,10 +178,10 @@ 
     public static function castArrayContents( array $aModel, array $aSubject ) {
         
         $_aCast = array();
-        foreach( $aModel as $_isKey => $_v ) {
+        foreach ( $aModel as $_isKey => $_v ) {
             $_aCast[ $_isKey ] = self::getElement(
-                $aSubject,  // subject array
-                $_isKey,    // key
+                $aSubject, // subject array
+                $_isKey, // key
                 null        // default
             );                 
         }
@@ -202,7 +202,7 @@ 
     public static function invertCastArrayContents( array $aModel, array $aSubject ) {
         
         $_aInvert = array();
-        foreach( $aModel as $_isKey => $_v ) {
+        foreach ( $aModel as $_isKey => $_v ) {
             if ( array_key_exists( $_isKey, $aSubject ) ) { 
                 continue; 
             }
@@ -228,7 +228,7 @@ 
     public static function uniteArrays( /* $aPrecedence, $aArray1, $aArray2, ... */ ) {
                 
         $_aArray = array();
-        foreach( array_reverse( func_get_args() ) as $_aArg ) {
+        foreach ( array_reverse( func_get_args() ) as $_aArg ) {
             $_aArray = self::uniteArraysRecursive( 
                 self::getAsArray( $_aArg ), 
                 $_aArray 
@@ -259,7 +259,7 @@ 
             $aPrecedence = array(); 
         }
                 
-        if ( ! is_array( $aDefault ) || ! is_array( $aPrecedence ) ) { 
+        if ( !is_array( $aDefault ) || !is_array( $aPrecedence ) ) { 
             return $aPrecedence; 
         }
         
@@ -268,10 +268,10 @@ 
             return $aPrecedence;
         }
         
-        foreach( $aDefault as $sKey => $v ) {
+        foreach ( $aDefault as $sKey => $v ) {
             
             // If the precedence does not have the key, assign the default's value.
-            if ( ! array_key_exists( $sKey, $aPrecedence ) || is_null( $aPrecedence[ $sKey ] ) ) {
+            if ( !array_key_exists( $sKey, $aPrecedence ) || is_null( $aPrecedence[ $sKey ] ) ) {
                 $aPrecedence[ $sKey ] = $v;
             } else {
                 
@@ -302,8 +302,8 @@ 
      *  - NULL
      * @return      array       The modified array.
      */
-    static public function dropElementsByType( array $aArray, $aTypes=array( 'array' ) ) {
-        foreach( $aArray as $isKey => $vValue ) {
+    static public function dropElementsByType( array $aArray, $aTypes = array( 'array' ) ) {
+        foreach ( $aArray as $isKey => $vValue ) {
             if ( in_array( gettype( $vValue ), $aTypes, true ) ) {
                 unset( $aArray[ $isKey ] );
             }
@@ -319,7 +319,7 @@ 
      */
     static public function dropElementByValue( array $aArray, $vValue ) {
          
-        foreach( self::getAsArray( $vValue, true ) as $_vValue ) {
+        foreach ( self::getAsArray( $vValue, true ) as $_vValue ) {
             $_sKey = array_search( $_vValue, $aArray, true );
             if ( $_sKey === false ) {
                 continue;
@@ -341,7 +341,7 @@ 
      */
     static public function dropElementsByKey( array $aArray, $asKeys ) {
         
-        foreach( self::getAsArray( $asKeys, true ) as $_isKey ) {
+        foreach ( self::getAsArray( $asKeys, true ) as $_isKey ) {
             unset( $aArray[ $_isKey ] );
         }
         return $aArray;

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(

Doc Comments +2 added lines

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

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