michaeluno / admin-page-framework

include/class/event/AdminPageFrameworkLoader_Event.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     public function __construct() {
         
         new AdminPageFrameworkLoader_Event_Action_GetDevelopmentVersion(
-            AdminPageFrameworkLoader_Registry::HOOK_SLUG . '_action_get_development_version'  // action name
+            AdminPageFrameworkLoader_Registry::HOOK_SLUG.'_action_get_development_version'  // action name
         );
                 
     }

include/class/event/AdminPageFrameworkLoader_Event_Action_GetDevelopmentVersion.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function replyToDoAction() {
         AdminPageFramework_WPUtility::setTransient(
-            AdminPageFrameworkLoader_Registry::TRANSIENT_PREFIX . 'devver',
+            AdminPageFrameworkLoader_Registry::TRANSIENT_PREFIX.'devver',
             $this->_getVersion(), // data - if an error occurs, an empty string will be given
             604800 // for one week
         );        
@@ -55,7 +55,7 @@ 
             $_sPageBody = $this->_getPageBody();
             $_aHeaders  = $_oUtil->getScriptData( 
                 $_sPageBody, 
-                '',  /// context
+                '', /// context
                 array( 'version' => 'Version' ) 
             );
             return $_oUtil->getElement(

Braces +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      * 
      * @since       3.6.2
      */
-    public function __construct( $sActionName ) {  
+    public function __construct( $sActionName ) {
         add_action(
             $sActionName,
             array( $this, 'replyToDoAction' )

include/class/utility/AdminPageFrameworkLoader_FeedList.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
      * 
      * @return      array
      */
-    public function get( $iItems=0 ) {
+    public function get( $iItems = 0 ) {
         
         $_aOutput   = array();
         $_aURLs     = $this->_aURLs;
@@ -56,7 +56,7 @@ 
             return $_aOutput;
         }
                                  
-        $_oFeed     = fetch_feed( $_aURLs );
+        $_oFeed = fetch_feed( $_aURLs );
         foreach ( $_oFeed->get_items() as $_oItem ) {
             $_aOutput[ $_oItem->get_title() ] = array( 
                 'content'        => $_oItem->get_content(),
@@ -64,7 +64,7 @@ 
                 'title'          => $_oItem->get_title(),
                 'date'           => $_oItem->get_date( 'j F Y, g:i a' ),
                 'author'         => $_oItem->get_author(),
-                'link'           => $_oItem->get_permalink(),    // get_link() may be used as well        
+                'link'           => $_oItem->get_permalink(), // get_link() may be used as well        
             );
         }
 

uninstall.php

Indentation +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
  */
 $_sMaingPluginFileName  = 'admin-page-framework-loader.php';
 if ( file_exists( dirname( __FILE__ ). '/' . $_sMaingPluginFileName ) ) {
-   include( $_sMaingPluginFileName );
+    include( $_sMaingPluginFileName );
 }
 
 if ( class_exists( 'AdminPageFrameworkLoader_Registry' ) ) :

Spacing +11 added lines, -11 removed lines

@@ -9,10 +9,10 @@ 
  * @since        3.5.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( !defined( 'ABSPATH' ) ) {
     exit;
 }
-if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
+if ( !defined( 'WP_UNINSTALL_PLUGIN' ) ) {
     exit;
 }
 
@@ -21,15 +21,15 @@ 
  * We are going to load the main file to get the registry class. And in the main file, 
  * if this constant is set, it will return after declaring the registry class.
  **/
-if ( ! defined( 'DOING_UNINSTALL' ) ) {
-    define( 'DOING_UNINSTALL', true  );
+if ( !defined( 'DOING_UNINSTALL' ) ) {
+    define( 'DOING_UNINSTALL', true );
 }
 
 /**
  * Set the main plugin file name here.
  */
-$_sMaingPluginFileName  = 'admin-page-framework-loader.php';
-if ( file_exists( dirname( __FILE__ ). '/' . $_sMaingPluginFileName ) ) {
+$_sMaingPluginFileName = 'admin-page-framework-loader.php';
+if ( file_exists( dirname( __FILE__ ).'/'.$_sMaingPluginFileName ) ) {
    include( $_sMaingPluginFileName );
 }
 
@@ -41,12 +41,12 @@ 
     // Delete transients
     $_aPrefixes = array(
         AdminPageFrameworkLoader_Registry::TRANSIENT_PREFIX, // the plugin transients
-        'apf_',      // the admin page framework transients
+        'apf_', // the admin page framework transients
     );
-    foreach( $_aPrefixes as $_sPrefix ) {
-        if ( ! $_sPrefix ) { continue; }
-        $GLOBALS['wpdb']->query( "DELETE FROM `" . $GLOBALS['table_prefix'] . "options` WHERE `option_name` LIKE ( '_transient_%{$_sPrefix}%' )" );
-        $GLOBALS['wpdb']->query( "DELETE FROM `" . $GLOBALS['table_prefix'] . "options` WHERE `option_name` LIKE ( '_transient_timeout_%{$_sPrefix}%' )" );    
+    foreach ( $_aPrefixes as $_sPrefix ) {
+        if ( !$_sPrefix ) { continue; }
+        $GLOBALS[ 'wpdb' ]->query( "DELETE FROM `".$GLOBALS[ 'table_prefix' ]."options` WHERE `option_name` LIKE ( '_transient_%{$_sPrefix}%' )" );
+        $GLOBALS[ 'wpdb' ]->query( "DELETE FROM `".$GLOBALS[ 'table_prefix' ]."options` WHERE `option_name` LIKE ( '_transient_timeout_%{$_sPrefix}%' )" );    
     }
     
 endif;
\ No newline at end of file

development/factory/_abstract/controller/AdminPageFramework_Resource_Base.php

Doc Comments +4 added lines

@@ -232,6 +232,8 @@ 
      * @remark      The meta box class may be instantiated multiple times so prevent echoing the same styles multiple times.
      * @parameter   string      $sIDPrefix   The id selector embedded in the script tag.
      * @parameter   string      $sClassName  The class name that identify the call group. This is important for the meta-box class because it can be instantiated multiple times in one particular page.
+     * @param string $sIDPrefix
+     * @param string $sClassName
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
 
@@ -315,6 +317,8 @@ 
      * @remark      The meta box class may be instantiated multiple times so prevent echoing the same styles multiple times.
      * @parametr    string      $sIDPrefix      The id selector embedded in the script tag.
      * @parametr    string      $sClassName     The class name that identify the call group. This is important for the meta-box class because it can be instantiated multiple times in one particular page.
+     * @param string $sIDPrefix
+     * @param string $sClassName
      */
     protected function _printCommonScripts( $sIDPrefix, $sClassName ) {
         

Indentation +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
         return "{$_aField['field_id']}{$_sKey}";
     }
 
-   /**
+    /**
      * Adds input fields
      * 
      * @internal

Spacing +45 added lines, -45 removed lines

@@ -36,19 +36,19 @@ 
     
         /* The system internal keys. */
         'sSRC'          => null,
-        'aPostTypes'    => array(),     // for meta box class
+        'aPostTypes'    => array(), // for meta box class
         'sPageSlug'     => null,    
         'sTabSlug'      => null,
-        'sType'         => null,        // script or style
+        'sType'         => null, // script or style
         
         /* The below keys are for users. */
         'handle_id'     => null,
         'dependencies'  => array(),
-        'version'       => false,       // although the type should be string, the wp_enqueue_...() functions want false as the default value.
-        'translation'   => array(),     // only for scripts
-        'in_footer'     => false,       // only for scripts
-        'media'         => 'all',       // only for styles     
-        'attributes'    => array(),     // 3.3.0+ - the attribute array
+        'version'       => false, // although the type should be string, the wp_enqueue_...() functions want false as the default value.
+        'translation'   => array(), // only for scripts
+        'in_footer'     => false, // only for scripts
+        'media'         => 'all', // only for styles     
+        'attributes'    => array(), // 3.3.0+ - the attribute array
         
     );    
       
@@ -59,7 +59,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */
-    protected $_sClassSelector_Style    = 'admin-page-framework-style';
+    protected $_sClassSelector_Style = 'admin-page-framework-style';
     
     /**
      * Stores the class selector used to the class-specific script.
@@ -68,7 +68,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */    
-    protected $_sClassSelector_Script   = 'admin-page-framework-script';
+    protected $_sClassSelector_Script = 'admin-page-framework-script';
       
     /**
      * Stores hand IDs by resource url to look up handle id and add custom arguments.
@@ -142,8 +142,8 @@ 
      * @internal
      */
     
-    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs=array() ) {}
-    public function _forceToEnqueueScript( $sSRC, $aCustomArgs=array() ) {}
+    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs = array() ) {}
+    public function _forceToEnqueueScript( $sSRC, $aCustomArgs = array() ) {}
     
     /**
      * A helper function for the _replyToEnqueueScripts() and the `_replyToEnqueueStyle()` methods.
@@ -201,7 +201,7 @@ 
                     }
                     
                     $_sAttributes   = $this->getAttributes( $_aAttributes );
-                    $_sModifiedURL  = $sSanitizedURL . "' " . rtrim( $_sAttributes, "'\"" );    // '"
+                    $_sModifiedURL  = $sSanitizedURL."' ".rtrim( $_sAttributes, "'\"" ); // '"
 
                     return $_sModifiedURL;                    
                     
@@ -249,19 +249,19 @@ 
          */
         private function _getStyleTag( $oCaller, $sIDPrefix ) {
             
-            $_sStyle     = $this->addAndApplyFilters( 
+            $_sStyle = $this->addAndApplyFilters( 
                 $oCaller, 
                 array(
-                    "style_common_admin_page_framework",            // 3.2.1+
+                    "style_common_admin_page_framework", // 3.2.1+
                     "style_common_{$this->oProp->sClassName}",
                 ), 
                 AdminPageFramework_CSS::getDefaultCSS() 
             );
-            $_sStyle     = $this->isDebugMode()
+            $_sStyle = $this->isDebugMode()
                 ? trim( $_sStyle )
                 : $this->minifyCSS( $_sStyle );
             if ( $_sStyle ) {
-                echo "<style type='text/css' id='" . esc_attr( $sIDPrefix ) . "'>"
+                echo "<style type='text/css' id='".esc_attr( $sIDPrefix )."'>"
                         . $_sStyle
                     . "</style>";
             }
@@ -275,20 +275,20 @@ 
          */        
         private function _getIEStyleTag( $oCaller, $sIDPrefix ) {
                 
-            $_sStyleIE   = $this->addAndApplyFilters( 
+            $_sStyleIE = $this->addAndApplyFilters( 
                 $oCaller, 
                 array(
-                    "style_ie_common_admin_page_framework",         // 3.2.1+
+                    "style_ie_common_admin_page_framework", // 3.2.1+
                     "style_ie_common_{$this->oProp->sClassName}", 
                 ),
                 AdminPageFramework_CSS::getDefaultCSSIE() 
             );
             
-            $_sStyleIE   = $this->isDebugMode()
+            $_sStyleIE = $this->isDebugMode()
                 ? trim( $_sStyleIE )
                 : $this->minifyCSS( $_sStyleIE );
             return $_sStyleIE
-                ? "<!--[if IE]><style type='text/css' id='" . esc_attr( $sIDPrefix . "-ie" ) . "'>"
+                ? "<!--[if IE]><style type='text/css' id='".esc_attr( $sIDPrefix."-ie" )."'>"
                         . $_sStyleIE
                     . "</style><![endif]-->"
                 : '';
@@ -321,13 +321,13 @@ 
         $_sScript = $this->addAndApplyFilters( 
             $this->oProp->_getCallerObject(), 
             array(
-                "script_common_admin_page_framework",       // 3.2.1+
+                "script_common_admin_page_framework", // 3.2.1+
                 "script_common_{$this->oProp->sClassName}", 
             ),
             AdminPageFramework_Property_Base::$_sDefaultScript 
         );
         if ( $_sScript ) {
-            echo "<script type='text/javascript' id='" . esc_attr( $sIDPrefix ) . "'>"
+            echo "<script type='text/javascript' id='".esc_attr( $sIDPrefix )."'>"
                     . $_sScript
                 . "</script>";
         }
@@ -344,7 +344,7 @@ 
      */
     protected function _printClassSpecificStyles( $sIDPrefix ) {
            
-        $_oCaller   = $this->oProp->_getCallerObject();     
+        $_oCaller = $this->oProp->_getCallerObject();     
         echo $this->_getClassSpecificStyleTag( $_oCaller, $sIDPrefix );
         echo $this->_getClassSpecificIEStyleTag( $_oCaller, $sIDPrefix );
         
@@ -372,7 +372,7 @@ 
                 ? trim( $_sStyle )
                 : $this->minifyCSS( $_sStyle );
             if ( $_sStyle ) {
-                return "<style type='text/css' id='" . esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" ) . "'>"
+                return "<style type='text/css' id='".esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" )."'>"
                         . $_sStyle
                     . "</style>";
                 $_iCallCount++;
@@ -398,7 +398,7 @@ 
                 ? trim( $_sStyleIE )
                 : $this->minifyCSS( $_sStyleIE );
             if ( $_sStyleIE ) {
-                return "<!--[if IE]><style type='text/css' id='" . esc_attr( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" ) . "'>" 
+                return "<!--[if IE]><style type='text/css' id='".esc_attr( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" )."'>" 
                         . $_sStyleIE
                     . "</style><![endif]-->";
                 $_iCallCountIE++;
@@ -425,7 +425,7 @@ 
             $this->oProp->sScript 
         );
         if ( $_sScript ) {
-            echo "<script type='text/javascript' id='" . esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" ) . "'>" 
+            echo "<script type='text/javascript' id='".esc_attr( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" )."'>" 
                     . $_sScript
                 . "</script>"; 
             $_iCallCount++;
@@ -449,10 +449,10 @@ 
     public function _replyToAddStyle() {
     
         $_oCaller = $this->oProp->_getCallerObject();     
-        if ( ! $_oCaller->_isInThePage() ) { return; }
+        if ( !$_oCaller->_isInThePage() ) { return; }
         
         $this->_printCommonStyles( 'admin-page-framework-style-common', get_class() );
-        $this->_printClassSpecificStyles( $this->_sClassSelector_Style . '-' . $this->oProp->sFieldsType );
+        $this->_printClassSpecificStyles( $this->_sClassSelector_Style.'-'.$this->oProp->sFieldsType );
  
     }
     /**
@@ -467,10 +467,10 @@ 
     public function _replyToAddScript() {
 
         $_oCaller = $this->oProp->_getCallerObject();     
-        if ( ! $_oCaller->_isInThePage() ) { return; }
+        if ( !$_oCaller->_isInThePage() ) { return; }
         
         $this->_printCommonScripts( 'admin-page-framework-script-common', get_class() );
-        $this->_printClassSpecificScripts( $this->_sClassSelector_Script . '-' . $this->oProp->sFieldsType );
+        $this->_printClassSpecificScripts( $this->_sClassSelector_Script.'-'.$this->oProp->sFieldsType );
         
     }        
     
@@ -485,28 +485,28 @@ 
     protected function _enqueueSRC( $aEnqueueItem ) {
         
         // For styles
-        if ( 'style' === $aEnqueueItem['sType'] ) {
+        if ( 'style' === $aEnqueueItem[ 'sType' ] ) {
             wp_enqueue_style( 
-                $aEnqueueItem['handle_id'], 
-                $aEnqueueItem['sSRC'], 
-                $aEnqueueItem['dependencies'], 
-                $aEnqueueItem['version'], 
-                $aEnqueueItem['media']
+                $aEnqueueItem[ 'handle_id' ], 
+                $aEnqueueItem[ 'sSRC' ], 
+                $aEnqueueItem[ 'dependencies' ], 
+                $aEnqueueItem[ 'version' ], 
+                $aEnqueueItem[ 'media' ]
             );
             return;
         }
 
         // For scripts
         wp_enqueue_script( 
-            $aEnqueueItem['handle_id'], 
-            $aEnqueueItem['sSRC'], 
-            $aEnqueueItem['dependencies'], 
-            $aEnqueueItem['version'], 
-            did_action( 'admin_body_class' ) ? true : $aEnqueueItem['in_footer'] 
+            $aEnqueueItem[ 'handle_id' ], 
+            $aEnqueueItem[ 'sSRC' ], 
+            $aEnqueueItem[ 'dependencies' ], 
+            $aEnqueueItem[ 'version' ], 
+            did_action( 'admin_body_class' ) ? true : $aEnqueueItem[ 'in_footer' ] 
         );
     
-        if ( $aEnqueueItem['translation'] ) {
-            wp_localize_script( $aEnqueueItem['handle_id'], $aEnqueueItem['handle_id'], $aEnqueueItem['translation'] );
+        if ( $aEnqueueItem[ 'translation' ] ) {
+            wp_localize_script( $aEnqueueItem[ 'handle_id' ], $aEnqueueItem[ 'handle_id' ], $aEnqueueItem[ 'translation' ] );
         }
         
     }
@@ -522,7 +522,7 @@ 
      * @internal
      */    
     public function _replyToEnqueueStyles() {        
-        foreach( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
+        foreach ( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
             $this->_enqueueSRCByConditoin( $_aEnqueuingStyle );
             unset( $this->oProp->aEnqueuingStyles[ $_sKey ] );
         }
@@ -539,7 +539,7 @@ 
      * @internal
      */
     public function _replyToEnqueueScripts() {     
-        foreach( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
+        foreach ( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
             $this->_enqueueSRCByConditoin( $_aEnqueuingScript );     
             unset( $this->oProp->aEnqueuingScripts[ $_sKey ] );
         }

Braces +3 added lines, -3 removed lines

@@ -232,7 +232,7 @@ 
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
                 
-        if ( self::$_bCommonStyleLoaded ) { 
+        if ( self::$_bCommonStyleLoaded ) {
             return; 
         }
         self::$_bCommonStyleLoaded = true;
@@ -521,7 +521,7 @@ 
      * @since       3.2.0   Changed it unset the enqueued item so that the method can be called multiple times.
      * @internal
      */    
-    public function _replyToEnqueueStyles() {        
+    public function _replyToEnqueueStyles() {
         foreach( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
             $this->_enqueueSRCByConditoin( $_aEnqueuingStyle );
             unset( $this->oProp->aEnqueuingStyles[ $_sKey ] );
@@ -538,7 +538,7 @@ 
      * @since       3.2.0   Changed it unset the enqueued item so that the method can be called multiple times.
      * @internal
      */
-    public function _replyToEnqueueScripts() {     
+    public function _replyToEnqueueScripts() {
         foreach( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
             $this->_enqueueSRCByConditoin( $_aEnqueuingScript );     
             unset( $this->oProp->aEnqueuingScripts[ $_sKey ] );

development/factory/_abstract/model/form_definition/AdminPageFramework_FormDefinition_Meta.php

Doc Comments +1 added lines

@@ -173,6 +173,7 @@
          * 
          * @internal
          * @since       3.5.3
+         * @param integer $iObjectID
          * @return      void
          */
         private function _updateMetaDatumByFuncitonName( $iObjectID, $_vValue, array $aSavedMeta, $_sSectionOrFieldID, $_sFunctionName ) {

Indentation +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
                 
                 // The first element is the option key; the section or field dimensional keys follow.
                 if ( '__dummy_option_key' === $_aDimensionalKeys[ 0 ] ) {
-                     array_shift( $_aDimensionalKeys );
+                        array_shift( $_aDimensionalKeys );
                 }
                 
                 $this->unsetDimensionalArrayElement( 

Spacing +19 added lines, -19 removed lines

@@ -32,15 +32,15 @@ 
 
         // Construct an array consisting of the submitted registered field values.
         $_aInput = array();
-        foreach( $aFieldDefinitionArrays as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $aFieldDefinitionArrays as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // If a section is not set,
             if ( '_default' == $_sSectionID ) {
                 $_aFields = $_aSubSectionsOrFields;
-                foreach( $_aFields as $_aField ) {
-                    $_aInput[ $_aField['field_id'] ] = $this->getElement(
+                foreach ( $_aFields as $_aField ) {
+                    $_aInput[ $_aField[ 'field_id' ] ] = $this->getElement(
                         $_POST, // subject
-                        $_aField['field_id'],    // dimensional keys
+                        $_aField[ 'field_id' ], // dimensional keys
                         null   // default value
                     );                            
                 }
@@ -50,18 +50,18 @@ 
             // At this point, the section is set
             $_aInput[ $_sSectionID ] = $this->getElementAsArray(
                 $_aInput, // subjct
-                $_sSectionID,    // dimensional keys
+                $_sSectionID, // dimensional keys
                 array()   // default value
             );      
             
             // If the section does not contain sub sections,
-            if ( ! count( $this->getIntegerKeyElements( $_aSubSectionsOrFields ) ) ) {
+            if ( !count( $this->getIntegerKeyElements( $_aSubSectionsOrFields ) ) ) {
                 
                 $_aFields = $_aSubSectionsOrFields;
-                foreach( $_aFields as $_aField ) {
-                    $_aInput[ $_sSectionID ][ $_aField['field_id'] ] = $this->getElement(
+                foreach ( $_aFields as $_aField ) {
+                    $_aInput[ $_sSectionID ][ $_aField[ 'field_id' ] ] = $this->getElement(
                         $_POST, // subjct
-                        array( $_sSectionID, $_aField['field_id'] ),    // dimensional keys
+                        array( $_sSectionID, $_aField[ 'field_id' ] ), // dimensional keys
                         null   // default value
                     );
                 }
@@ -71,10 +71,10 @@ 
                 
             // Otherwise, it's sub-sections. 
             // Since the registered fields don't have information how many items the user added, parse the submitted data.
-            foreach( $_POST[ $_sSectionID ] as $_iIndex => $_aFields ) { // will include the main section as well.
+            foreach ( $_POST[ $_sSectionID ] as $_iIndex => $_aFields ) { // will include the main section as well.
                 $_aInput[ $_sSectionID ][ $_iIndex ] = $this->getElement(
                     $_POST, // subjct
-                    array( $_sSectionID, $_iIndex ),    // dimensional keys
+                    array( $_sSectionID, $_iIndex ), // dimensional keys
                     null   // default value
                 );
             }        
@@ -97,9 +97,9 @@ 
      * @param       string      $sFieldsType    The type of object. Currently 'post_meta_box' or 'user_meta' is accepted.
      * @return      void
      */
-    public function updateMetaDataByType( $iObjectID, array $aInput, array $aSavedMeta, $sFieldsType='post_meta_box' ) {
+    public function updateMetaDataByType( $iObjectID, array $aInput, array $aSavedMeta, $sFieldsType = 'post_meta_box' ) {
         
-        if ( ! $iObjectID ) {
+        if ( !$iObjectID ) {
             return;
         }
            
@@ -111,7 +111,7 @@ 
             // 'widget'            => null,
             'user_meta'         => 'update_user_meta',               
         );
-        if ( ! in_array( $sFieldsType, array_keys( $_aFunctionNameMapByFieldsType ) ) ) {
+        if ( !in_array( $sFieldsType, array_keys( $_aFunctionNameMapByFieldsType ) ) ) {
             return;
         }
         $_sFunctionName = $this->getElement( $_aFunctionNameMapByFieldsType, $sFieldsType );
@@ -140,16 +140,16 @@ 
          */
         private function _getInputByUnset( array $aInput ) {
             
-            $_sUnsetKey = '__unset_' . $this->sFieldsType;
-            if ( ! isset( $_POST[ $_sUnsetKey ] ) ) {
+            $_sUnsetKey = '__unset_'.$this->sFieldsType;
+            if ( !isset( $_POST[ $_sUnsetKey ] ) ) {
                 return $aInput;
             }
             
             $_aUnsetElements = array_unique( $_POST[ $_sUnsetKey ] );
-            foreach( $_aUnsetElements as $_sFlatInputName ) {
+            foreach ( $_aUnsetElements as $_sFlatInputName ) {
                 
                 $_aDimensionalKeys = explode( '|', $_sFlatInputName );
-                if ( ! isset( $_aDimensionalKeys[ 0 ] ) ) {
+                if ( !isset( $_aDimensionalKeys[ 0 ] ) ) {
                     continue;
                 }
                 
@@ -183,7 +183,7 @@ 
 
             $_vSavedValue = $this->getElement(
                 $aSavedMeta, // subject
-                $_sSectionOrFieldID,    // dimensional keys
+                $_sSectionOrFieldID, // dimensional keys
                 null   // default value
             );
                 

Braces +4 added lines, -3 removed lines

@@ -71,7 +71,8 @@ 
                 
             // Otherwise, it's sub-sections. 
             // Since the registered fields don't have information how many items the user added, parse the submitted data.
-            foreach( $_POST[ $_sSectionID ] as $_iIndex => $_aFields ) { // will include the main section as well.
+            foreach( $_POST[ $_sSectionID ] as $_iIndex => $_aFields ) {
+// will include the main section as well.
                 $_aInput[ $_sSectionID ][ $_iIndex ] = $this->getElement(
                     $_POST, // subjct
                     array( $_sSectionID, $_iIndex ),    // dimensional keys
@@ -177,7 +178,7 @@ 
          */
         private function _updateMetaDatumByFuncitonName( $iObjectID, $_vValue, array $aSavedMeta, $_sSectionOrFieldID, $_sFunctionName ) {
             
-            if ( is_null( $_vValue ) ) { 
+            if ( is_null( $_vValue ) ) {
                 return;
             }
 
@@ -189,7 +190,7 @@ 
                 
             // PHP can compare even array contents with the == operator. See http://www.php.net/manual/en/language.operators.array.php
             // if the input value and the saved meta value are the same, no need to update it.
-            if ( $_vValue == $_vSavedValue ) { 
+            if ( $_vValue == $_vSavedValue ) {
                 return; 
             }
             

development/factory/_abstract/model/format/AdminPageFramework_Format_CollapsibleSection.php

Doc Comments +2 added lines

@@ -77,6 +77,8 @@
     }
         /**
          * @since       3.6.0
+         * @param boolean $abCollapsible
+         * @param string $sTitle
          * @return      array
          */
         private function _getArguments( $abCollapsible, $sTitle, array $aSection ) {

Spacing +11 added lines, -11 removed lines

@@ -23,10 +23,10 @@ 
      * @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.
+        '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.
     );   
     
@@ -40,7 +40,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.
@@ -83,14 +83,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
@@ -114,7 +114,7 @@ 
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {            
+                if ( !$aSection[ 'repeatable' ] ) {            
                     return $sToggleAll;
                 }
                 
@@ -124,13 +124,13 @@ 
                 }
                 
                 // Disable the toggle all button for middle sub-sections in repeatable sections.
-                if ( ! $aSection[ '_is_first_index' ] && ! $aSection[ '_is_last_index' ] ) {
+                if ( !$aSection[ '_is_first_index' ] && !$aSection[ '_is_last_index' ] ) {
                     return 0;
                 }            
                 
                 $_aToggleAll = $this->getAOrB(
-                    true === $sToggleAll || 1 ===  $sToggleAll, // evaluate
-                    array( 'top-right', 'bottom-right' ),   // if true
+                    true === $sToggleAll || 1 === $sToggleAll, // evaluate
+                    array( 'top-right', 'bottom-right' ), // if true
                     explode( ',', $sToggleAll ) // if false
                 );            
                 $_aToggleAll = $this->getAOrB(

Braces +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {            
+                if ( ! $aSection[ 'repeatable' ] ) {
                     return $sToggleAll;
                 }
                 

development/factory/_abstract/view/AdminPageFramework_PageLoadInfo_Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
          * Converts bytes to HR.
          *
          * @access private
-         * @param mixed $bytes
+         * @param mixed $nBytes
          * @remark This is influenced by the work of Mike Jolley.
          * @see http://mikejolley.com/projects/wp-page-load-stats/
          */

Spacing +9 added lines, -9 removed lines

@@ -19,7 +19,7 @@ 
     
     function __construct( $oProp, $oMsg ) {
         
-        if ( $oProp->bIsAdminAjax || ! $oProp->bIsAdmin ) {
+        if ( $oProp->bIsAdminAjax || !$oProp->bIsAdmin ) {
             return;
         }     
         
@@ -58,7 +58,7 @@ 
         self::$_bLoadedPageLoadInfo = true;     
         
         $_nSeconds                 = timer_stop( 0 );
-        $_nQueryCount             = get_num_queries();
+        $_nQueryCount = get_num_queries();
         $_nMemoryUsage             = round( $this->_convertBytesToHR( memory_get_usage() ), 2 );
         $_nMemoryPeakUsage         = round( $this->_convertBytesToHR( memory_get_peak_usage() ), 2 );
         $_nMemoryLimit             = round( $this->_convertBytesToHR( $this->_convertToNumber( WP_MEMORY_LIMIT ) ), 2 );
@@ -67,10 +67,10 @@ 
         return $sFooterHTML
             . "<div id='admin-page-framework-page-load-stats'>"
                 . "<ul>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'queries_in_seconds' ), $_nQueryCount, $_nSeconds ) . "</li>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'out_of_x_memory_used' ), $_nMemoryUsage, $_nMemoryLimit, round( ( $_nMemoryUsage / $_nMemoryLimit ), 2 ) * 100 . '%' ) . "</li>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'peak_memory_usage' ), $_nMemoryPeakUsage ) . "</li>"
-                    . "<li>" . sprintf( $this->oMsg->get( 'initial_memory_usage' ), $_sInitialMemoryUsage ) . "</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'queries_in_seconds' ), $_nQueryCount, $_nSeconds )."</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'out_of_x_memory_used' ), $_nMemoryUsage, $_nMemoryLimit, round( ( $_nMemoryUsage / $_nMemoryLimit ), 2 ) * 100.'%' )."</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'peak_memory_usage' ), $_nMemoryPeakUsage )."</li>"
+                    . "<li>".sprintf( $this->oMsg->get( 'initial_memory_usage' ), $_sInitialMemoryUsage )."</li>"
                 . "</ul>"
             . "</div>";
         
@@ -88,8 +88,8 @@ 
          */
         private function _convertToNumber( $nSize ) {
             
-            $_nReturn     = substr( $nSize, 0, -1 );
-            switch( strtoupper( substr( $nSize, -1 ) ) ) {
+            $_nReturn = substr( $nSize, 0, -1 );
+            switch ( strtoupper( substr( $nSize, -1 ) ) ) {
                 case 'P':
                     $_nReturn *= 1024;
                 case 'T':
@@ -118,7 +118,7 @@ 
             $_nLog = log( $nBytes, 1024 );
             $_iPower = ( int ) $_nLog;
             $_iSize = pow( 1024, $_nLog - $_iPower );
-            return $_iSize . $_aUnits[ $_iPower ];
+            return $_iSize.$_aUnits[ $_iPower ];
         }
 
 }
\ No newline at end of file

development/factory/_abstract/view/attribute/AdminPageFramework_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

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * Admin Page Framework
- * 
- * http://en.michaeluno.jp/admin-page-framework/
- * Copyright (c) 2013-2015 Michael Uno; Licensed MIT
- * 
- */
+         * Admin Page Framework
+         * 
+         * http://en.michaeluno.jp/admin-page-framework/
+         * Copyright (c) 2013-2015 Michael Uno; Licensed MIT
+         * 
+         */
 
 /**
  * Provides methods to handle importing options.

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
      * 
      * @since       3.6.0
      */
-    public $sContext    = 'field'; 
+    public $sContext = 'field'; 
 
     /**
      * Returns the field container attribute array.
@@ -41,10 +41,10 @@ 
 
         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.
             // @deprecated 3.6.0 
             // 'data-id_model' => $this->aArguments[ '_fields_container_id_model' ], // 3.3.1+
-            'class'         => "admin-page-framework-field admin-page-framework-field-" . $this->aArguments[ 'type' ]
+            'class'         => "admin-page-framework-field admin-page-framework-field-".$this->aArguments[ 'type' ]
                 . $this->getAOrB(
                     $this->aArguments[ 'attributes' ][ 'disabled' ],
                     ' disabled',