michaeluno / admin-page-framework

development/factory/page_meta_box/_model/AdminPageFramework_Property_page_meta_box.php

Indentation +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
      * @since       3.5.3
      * @return      string      The found page slug. An empty string if not found.
      * @remark      Do not return `null` when not found as some framework methods check the retuened value with `isset()` and if null is given, `isset()` yields `false` while it does `true` for an emtpy string ('').
-    */
+     */
     public function getCurrentPageSlug() {
         return $this->getHTTPQueryGET( 'page', '' );
     }

Spacing +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
     /**
      * Sets up hooks and properties.
      */
-    public function __construct( $oCaller, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='page_meta_box' ) {
+    public function __construct( $oCaller, $sClassName, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework', $sStructureType = 'page_meta_box' ) {
 
         // Let them overload.
         unset(
@@ -105,7 +105,7 @@ 
     public function _getScreenIDOfPage( $sPageSlug ) {
         $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug );
         return $_oAdminPage
-            ? $_oAdminPage->oProp->aPages[ $sPageSlug ][ '_page_hook' ] . ( is_network_admin() ? '-network' : '' )
+            ? $_oAdminPage->oProp->aPages[ $sPageSlug ][ '_page_hook' ].( is_network_admin() ? '-network' : '' )
             : '';
     }
 
@@ -115,7 +115,7 @@ 
      * @since       3.0.0
      * @return      boolean     Returns true if it is of framework's added page; otherwise, false.
      */
-    public function isPageAdded( $sPageSlug='' ) {
+    public function isPageAdded( $sPageSlug = '' ) {
         $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug );
         return $_oAdminPage
             ? $_oAdminPage->oProp->isPageAdded( $sPageSlug )
@@ -132,7 +132,7 @@ 
     public function isCurrentTab( $sTabSlug ) {
 
         $_sCurrentPageSlug = $this->getHTTPQueryGET( 'page' );
-        if ( ! $_sCurrentPageSlug ) {
+        if ( !$_sCurrentPageSlug ) {
             return false;
         }
         $_sCurrentTabSlug = $this->getHTTPQueryGET( 'tab', $this->getDefaultInPageTab( $_sCurrentPageSlug ) );
@@ -179,7 +179,7 @@ 
      */
     public function getDefaultInPageTab( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return '';
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -194,7 +194,7 @@ 
      */
     public function getOptionKey( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return '';
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -221,7 +221,7 @@ 
             $GLOBALS,
             array( 'aAdminPageFramework', 'aPageClasses' )
         );
-        foreach( $_aPageClasses as $_oAdminPage ) {
+        foreach ( $_aPageClasses as $_oAdminPage ) {
             if ( $_oAdminPage->oProp->isPageAdded( $sPageSlug ) ) {
                 return $_oAdminPage;
             }

development/factory/post_type/AdminPageFramework_PostType_Model.php

Spacing +10 added lines, -10 removed lines

@@ -37,7 +37,7 @@ 
 
         if ( $this->oProp->bIsAdmin ) {
 
-            add_action( 'load_' . $this->oProp->sPostType, array( $this, '_replyToSetUpHooksForModel' ) );
+            add_action( 'load_'.$this->oProp->sPostType, array( $this, '_replyToSetUpHooksForModel' ) );
 
             if ( $this->oProp->sCallerPath ) {
                 new AdminPageFramework_PostType_Model__FlushRewriteRules( $this );
@@ -64,11 +64,11 @@ 
 
         // Properties - sets translatable labels.
         $this->oProp->aColumnHeaders = array(
-            'cb'        => '<input type="checkbox" />',     // Checkbox for bulk actions.
-            'title'     => $this->oMsg->get( 'title' ),     // Post title. Includes "edit", "quick edit", "trash" and "view" links. If $mode (set from `$_REQUEST`['mode']) is 'excerpt', a post excerpt is included between the title and links.
-            'author'    => $this->oMsg->get( 'author' ),    // Post author.
+            'cb'        => '<input type="checkbox" />', // Checkbox for bulk actions.
+            'title'     => $this->oMsg->get( 'title' ), // Post title. Includes "edit", "quick edit", "trash" and "view" links. If $mode (set from `$_REQUEST`['mode']) is 'excerpt', a post excerpt is included between the title and links.
+            'author'    => $this->oMsg->get( 'author' ), // Post author.
             'comments'  => '<div class="comment-grey-bubble"></div>', // Number of pending comments.
-            'date'      => $this->oMsg->get( 'date' ),      // The date and publish status of the post.
+            'date'      => $this->oMsg->get( 'date' ), // The date and publish status of the post.
         );
 
     }
@@ -130,7 +130,7 @@ 
         echo $this->oUtil->addAndApplyFilter(
             $this,
             "cell_{$this->oProp->sPostType}_{$sColumnKey}",
-            '',  // value to be filtered - cell output
+            '', // value to be filtered - cell output
             $iPostID
         );
     }
@@ -179,7 +179,7 @@ 
      * @internal
      */
     public function _replyToRegisterTaxonomies() {
-        foreach( $this->oProp->aTaxonomies as $_sTaxonomySlug => $_aArguments ) {
+        foreach ( $this->oProp->aTaxonomies as $_sTaxonomySlug => $_aArguments ) {
             $this->_registerTaxonomy(
                 $_sTaxonomySlug,
                 $this->oUtil->getAsArray( $this->oProp->aTaxonomyObjectTypes[ $_sTaxonomySlug ] ), // object types
@@ -195,8 +195,8 @@ 
      */
     public function _registerTaxonomy( $sTaxonomySlug, array $aObjectTypes, array $aArguments ) {
 
-        if ( ! in_array( $this->oProp->sPostType, $aObjectTypes ) ) {
-            $aObjectTypes[] = $this->oProp->sPostType;
+        if ( !in_array( $this->oProp->sPostType, $aObjectTypes ) ) {
+            $aObjectTypes[ ] = $this->oProp->sPostType;
         }
         register_taxonomy(
             $sTaxonomySlug,
@@ -234,7 +234,7 @@ 
      */
     public function _replyToRemoveTexonomySubmenuPages() {
 
-        foreach( $this->oProp->aTaxonomyRemoveSubmenuPages as $sSubmenuPageSlug => $sTopLevelPageSlug ) {
+        foreach ( $this->oProp->aTaxonomyRemoveSubmenuPages as $sSubmenuPageSlug => $sTopLevelPageSlug ) {
 
             remove_submenu_page( $sTopLevelPageSlug, $sSubmenuPageSlug );
 

development/factory/post_type/_model/AdminPageFramework_Property_post_type.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         'title'     => 'Title', // Post title. Includes "edit", "quick edit", "trash" and "view" links. If $mode (set from `$_REQUEST[ 'mode' ]`) is 'excerpt', a post excerpt is included between the title and links.
         'author'    => 'Author', // Post author.
         'comments'  => '<div class="comment-grey-bubble"></div>', // Number of pending comments.
-        'date'      => 'Date',     // The date and publish status of the post.
+        'date'      => 'Date', // The date and publish status of the post.
     );
 
     /**

development/factory/post_type/_view/AdminPageFramework_PageLoadInfo_post_type.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         if ( in_array( $oProp->sClassName, self::$aClassNames ) )
             return self::$_oInstance;
         
-        self::$aClassNames[] = $oProp->sClassName;
+        self::$aClassNames[ ] = $oProp->sClassName;
         self::$_oInstance = new AdminPageFramework_PageLoadInfo_post_type( $oProp, $oMsg );
         
         return self::$_oInstance;

Braces +5 added lines, -3 removed lines

@@ -26,8 +26,9 @@ 
      */
     public static function instantiate( $oProp, $oMsg ) {
         
-        if ( in_array( $oProp->sClassName, self::$aClassNames ) )
-            return self::$_oInstance;
+        if ( in_array( $oProp->sClassName, self::$aClassNames ) ) {
+                    return self::$_oInstance;
+        }
         
         self::$aClassNames[] = $oProp->sClassName;
         self::$_oInstance = new AdminPageFramework_PageLoadInfo_post_type( $oProp, $oMsg );
@@ -43,7 +44,8 @@ 
     public function _replyToSetPageLoadInfoInFooter() {
 
         // Some users set `$_GET[ 'post_type' ]` element even in regular admin pages. In that case, do not load the style to avoid duplicates.
-        if ( isset( $_GET[ 'page' ] ) && $_GET[ 'page' ] ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'page' ] ) && $_GET[ 'page' ] ) {
+// sanitization unnecessary
             return;
         }
     

development/factory/_common/_abstract/AdminPageFramework_Factory_Router.php

Spacing +20 added lines, -20 removed lines

@@ -169,8 +169,8 @@ 
         }
 
         // Call the user constructor.
-        $this->start();     // defined in the controller class.
-        $this->oUtil->addAndDoAction( $this, 'start_' . $this->oProp->sClassName, $this );
+        $this->start(); // defined in the controller class.
+        $this->oUtil->addAndDoAction( $this, 'start_'.$this->oProp->sClassName, $this );
 
     }
         /**
@@ -179,8 +179,8 @@ 
          * @return      array
          */
         private function ___getSubClassNames() {
-            foreach( $this->_aSubClassPrefixes as $_sObjectVariableName => $_sPrefix ) {
-                $this->aSubClassNames[ $_sObjectVariableName ] = $_sPrefix . $this->_sStructureType;
+            foreach ( $this->_aSubClassPrefixes as $_sObjectVariableName => $_sPrefix ) {
+                $this->aSubClassNames[ $_sObjectVariableName ] = $_sPrefix.$this->_sStructureType;
             }
             return $this->aSubClassNames + $this->_aSubClassNames;
         }
@@ -194,15 +194,15 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) {
+        if ( !$this->_isInThePage() ) {
             return;
         }
 
-        if ( ! isset( $this->oResource ) ) {
+        if ( !isset( $this->oResource ) ) {
             $this->oResource = $this->_replyTpSetAndGetInstance_oResource();
         }
 
-        if ( ! isset(  $this->oLink ) ) {
+        if ( !isset( $this->oLink ) ) {
             $this->oLink = $this->_replyTpSetAndGetInstance_oLink();
         }
 
@@ -216,10 +216,10 @@ 
      * Calls the load method and callbacks.
      * @since       3.8.14
      */
-    protected function _load( $aActions=array() ) {
+    protected function _load( $aActions = array() ) {
         $aActions = empty( $aActions )
             ? array(
-                'load_' . $this->oProp->sClassName,
+                'load_'.$this->oProp->sClassName,
             )
             : $aActions;
         $this->load();
@@ -231,7 +231,7 @@ 
      */
     protected function _setUp() {
         $aActions = array(
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
         );
         $this->setUp();
         $this->oUtil->addAndDoActions( $this, $aActions, $this );
@@ -258,14 +258,14 @@ 
          * @since   3.8.19
          */
         protected function _isWordPressCoreAjaxRequest() {
-            if ( ! isset( $GLOBALS[ 'pagenow' ] ) ) {
+            if ( !isset( $GLOBALS[ 'pagenow' ] ) ) {
                 return false;
             }
             if ( 'admin-ajax.php' !== $GLOBALS[ 'pagenow' ] ) {
                 return false;
             }
             return in_array(
-                isset( $_POST[ 'action' ] ) ? $_POST[ 'action' ] : '',  // sanitization unnecessary as just checking
+                isset( $_POST[ 'action' ] ) ? $_POST[ 'action' ] : '', // sanitization unnecessary as just checking
                 array( 'heartbeat', 'closed-postboxes', 'meta-box-order' ),
                 true
             );
@@ -293,7 +293,7 @@ 
      * @return  boolean
      */
     protected function _isValidAjaxReferrer() {
-        if ( ! $this->oProp->bIsAdminAjax ) {
+        if ( !$this->oProp->bIsAdminAjax ) {
             return false;
         }
         return true;
@@ -309,7 +309,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) {
+        if ( !$this->_isInThePage() ) {
             return;
         }
 
@@ -375,7 +375,7 @@ 
         // Set and return the sub class object instance.
         if ( isset( $this->aSubClassNames[ $sPropertyName ] ) ) {
             return call_user_func(
-                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}"  )
+                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}" )
             );
         }
 
@@ -409,7 +409,7 @@ 
          */
         public function _replyTpSetAndGetInstance_oMsg() {
             $this->oMsg = call_user_func_array(
-                array( $this->aSubClassNames[ 'oMsg' ], 'getInstance'),
+                array( $this->aSubClassNames[ 'oMsg' ], 'getInstance' ),
                 array( $this->oProp->sTextDomain )  // parameters
             );
             return $this->oMsg;
@@ -474,7 +474,7 @@ 
      *
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {
+    public function __call( $sMethodName, $aArguments = null ) {
 
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
 
@@ -504,7 +504,7 @@ 
 
             // Check if the method name does not contain a backslash.
             if ( false === strpos( $sMethodName, "\\" ) ) {
-                return $this->oUtil->getElement( $aArguments, 0 );  // the first element - the filter value
+                return $this->oUtil->getElement( $aArguments, 0 ); // the first element - the filter value
             }
 
             // If the method name contains a backslash, the user may be using a name space.
@@ -515,7 +515,7 @@ 
                     array( $this, $_sAutoCallbackMethodName ),
                     $aArguments
                 )
-                : $this->oUtil->getElement( $aArguments, 0 );   // the first argument
+                : $this->oUtil->getElement( $aArguments, 0 ); // the first argument
 
         }
 
@@ -525,7 +525,7 @@ 
          */
         private function _triggerUndefinedMethodWarning( $sMethodName ) {
             trigger_error(
-                AdminPageFramework_Registry::NAME . ': '
+                AdminPageFramework_Registry::NAME.': '
                     . sprintf(
                         __( 'The method is not defined: %1$s', $this->oProp->sTextDomain ),
                         $sMethodName

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

Spacing +16 added lines, -16 removed lines

@@ -320,7 +320,7 @@ 
         'caller_id'                         => '',
         'structure_type'                    => '',
         'action_hook_form_registration'     => '',
-        'autoload_min_resource'             => true,    // will be overridden by the $bAutoloadMinifiedResource property.
+        'autoload_min_resource'             => true, // will be overridden by the $bAutoloadMinifiedResource property.
     );
 
     /**
@@ -328,14 +328,14 @@ 
      * @since       3.7.0
      */
     public $aFormCallbacks = array(
-        'hfID'              => null,    // the input id attribute
-        'hfTagID'           => null,    // the field container id attribute
-        'hfName'            => null,    // the field name attribute
-        'hfNameFlat'        => null,    // the flat field name attribute
+        'hfID'              => null, // the input id attribute
+        'hfTagID'           => null, // the field container id attribute
+        'hfName'            => null, // the field name attribute
+        'hfNameFlat'        => null, // the flat field name attribute
         // @todo Document the differences between `hfName` and `hfInputName`
-        'hfInputName'       => null,    // 3.6.0+   the field input name attribute
-        'hfInputNameFlat'   => null,    // 3.6.0+   the flat field input name
-        'hfClass'           => null,    // the class attribute
+        'hfInputName'       => null, // 3.6.0+   the field input name attribute
+        'hfInputNameFlat'   => null, // 3.6.0+   the flat field input name
+        'hfClass'           => null, // the class attribute
     );
 
     /**
@@ -468,7 +468,7 @@ 
              * @internal
              */
             private function ___getURLQuery() {
-                if ( ! $this->bIsAdminAjax ) {
+                if ( !$this->bIsAdminAjax ) {
                     return $this->getHTTPQueryGET( array(), array() );
                 }
                 parse_str(
@@ -483,7 +483,7 @@ 
          * @since       3.7.9
          */
         private function ___setGlobals() {
-            if ( ! isset( $GLOBALS[ 'aAdminPageFramework' ] ) ) {
+            if ( !isset( $GLOBALS[ 'aAdminPageFramework' ] ) ) {
                 $GLOBALS[ 'aAdminPageFramework' ] = array(
                     'aFieldFlags' => array()
                 );
@@ -511,7 +511,7 @@ 
             'action_hook_form_registration'     => $this->bIsAdminAjax
                 ? 'admin_init'  // 3.8.14
                 : $this->_sFormRegistrationHook,
-            'autoload_min_resource'             => $this->bAutoloadMinifiedResource,    // [3.8.31]
+            'autoload_min_resource'             => $this->bAutoloadMinifiedResource, // [3.8.31]
         ) + $this->aFormArguments;
     }
     /**
@@ -555,8 +555,8 @@ 
             'hfTagID'                           => array( $this->oCaller, '_replyToGetInputTagIDAttribute' ), // the fields & fieldset & field row container id attribute
             'hfName'                            => array( $this->oCaller, '_replyToGetFieldNameAttribute' ), // the input name attribute
             'hfNameFlat'                        => array( $this->oCaller, '_replyToGetFlatFieldName' ), // the flat input name attribute
-            'hfInputName'                       => array( $this->oCaller, '_replyToGetInputNameAttribute' ),    // 3.6.0+   the field input name attribute
-            'hfInputNameFlat'                   => array( $this->oCaller, '_replyToGetFlatInputName' ),    // 3.6.0+   the flat field input name
+            'hfInputName'                       => array( $this->oCaller, '_replyToGetInputNameAttribute' ), // 3.6.0+   the field input name attribute
+            'hfInputNameFlat'                   => array( $this->oCaller, '_replyToGetFlatInputName' ), // 3.6.0+   the flat field input name
             'hfClass'                           => array( $this->oCaller, '_replyToGetInputClassAttribute' ), // the class attribute
             'hfSectionName'                     => array( $this->oCaller, '_replyToGetSectionName' ), // 3.6.0+
         ) + $this->aFormCallbacks;
@@ -630,7 +630,7 @@ 
      * @remark      The information can be used to embed into the footer etc.
      * @return      array       The information of the script.
      */
-    protected function getCallerInfo( $sCallerPath='' ) {
+    protected function getCallerInfo( $sCallerPath = '' ) {
 
         if ( isset( self::$_aScriptDataCaches[ $sCallerPath ] ) ) {
             return self::$_aScriptDataCaches[ $sCallerPath ];
@@ -748,11 +748,11 @@ 
 
         // 3.7.9 Moved from the constructor to make it lighter.
         if ( 'sClassHash' === $sName ) {
-            $this->sClassHash       = md5( $this->sClassName );
+            $this->sClassHash = md5( $this->sClassName );
             return $this->sClassHash;
         }
         if ( 'sScriptType' === $sName ) {
-            $this->sScriptType      = $this->_getCallerType( $this->sCallerPath );    // 3.7.6+
+            $this->sScriptType = $this->_getCallerType( $this->sCallerPath ); // 3.7.6+
             return $this->sScriptType;
         }
         if ( 'oUtil' === $sName ) {

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

Spacing +17 added lines, -17 removed lines

@@ -83,11 +83,11 @@ 
         'nonce_verification_failed'             => 'A problem occurred while processing the form data. Please try again.',
         'check_max_input_vars'                  => 'Not all form fields could not be sent. '
             . 'Please check your server settings of PHP <code>max_input_vars</code> and consult the server administrator to increase the value. '
-            . '<code>max input vars</code>: %1$s. <code>$_POST</code> count: %2$s',  // 3.5.11+ // sanitization unnecessary as it is just a literal string
-        'send_email'                            => 'Is it okay to send the email?',     // 3.3.0+
-        'email_sent'                            => 'The email has been sent.',  // 3.3.0+, 3.3.5+ deprecated, 3.8.32 Re-added
+            . '<code>max input vars</code>: %1$s. <code>$_POST</code> count: %2$s', // 3.5.11+ // sanitization unnecessary as it is just a literal string
+        'send_email'                            => 'Is it okay to send the email?', // 3.3.0+
+        'email_sent'                            => 'The email has been sent.', // 3.3.0+, 3.3.5+ deprecated, 3.8.32 Re-added
         'email_scheduled'                       => 'The email has been scheduled.', // 3.3.5+, 3.8.32 deprecated
-        'email_could_not_send'                  => 'There was a problem sending the email',     // 3.3.0+
+        'email_could_not_send'                  => 'There was a problem sending the email', // 3.3.0+
 
         // AdminPageFramework_PostType
         'title'                                 => 'Title',
@@ -114,14 +114,14 @@ 
         'upload_file'                           => 'Upload File',
         'use_this_file'                         => 'Use This File',
         'select_file'                           => 'Select File',
-        'remove_value'                          => 'Remove Value',  // 3.2.0+
-        'select_all'                            => 'Select All',    // 3.3.0+
-        'select_none'                           => 'Select None',   // 3.3.0+
+        'remove_value'                          => 'Remove Value', // 3.2.0+
+        'select_all'                            => 'Select All', // 3.3.0+
+        'select_none'                           => 'Select None', // 3.3.0+
         'no_term_found'                         => 'No term found.', // 3.3.2+
 
         // AdminPageFramework_Form_View___Script_{...}
         'select'                                => 'Select', // 3.4.2+
-        'insert'                                => 'Insert',  // 3.4.2+
+        'insert'                                => 'Insert', // 3.4.2+
         'use_this'                              => 'Use This', // 3.4.2+
         'return_to_library'                     => 'Return to Library', // 3.4.2+
 
@@ -133,8 +133,8 @@ 
 
         // Repeatable sections & fields
         'repeatable_section_is_disabled'        => 'The ability to repeat sections is disabled.', // 3.8.13+
-        'repeatable_field_is_disabled'          => 'The ability to repeat fields is disabled.',   // 3.8.13+
-        'warning_caption'                       => 'Warning',   // 3.8.13+
+        'repeatable_field_is_disabled'          => 'The ability to repeat fields is disabled.', // 3.8.13+
+        'warning_caption'                       => 'Warning', // 3.8.13+
 
         // AdminPageFramework_FormField
         'allowed_maximum_number_of_fields'      => 'The allowed maximum number of fields is {0}.',
@@ -216,7 +216,7 @@ 
      * @remark      This class should be instantiated via this method.
      * @return      AdminPageFramework_Message
      */
-    public static function getInstance( $sTextDomain='admin-page-framework' ) {
+    public static function getInstance( $sTextDomain = 'admin-page-framework' ) {
 
         $_oInstance = isset( self::$_aInstancesByTextDomain[ $sTextDomain ] ) && ( self::$_aInstancesByTextDomain[ $sTextDomain ] instanceof AdminPageFramework_Message )
             ? self::$_aInstancesByTextDomain[ $sTextDomain ]
@@ -229,7 +229,7 @@ 
          * Ensures that only one instance of this class object exists. ( no multiple instances of this object )
          * @deprecated  3.2.0
          */
-        public static function instantiate( $sTextDomain='admin-page-framework' ) {
+        public static function instantiate( $sTextDomain = 'admin-page-framework' ) {
             return self::getInstance( $sTextDomain );
         }
 
@@ -237,14 +237,14 @@ 
      * Sets up properties.
      * @param string $sTextDomain
      */
-    public function __construct( $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sTextDomain = 'admin-page-framework' ) {
 
         $this->_sTextDomain = $sTextDomain;
 
         // Fill the $aMessages property with the keys extracted from the $aDefaults property
         // with the value of null.  The null is set to let it trigger the __get() method
         // so that each translation item gets processed individually.
-        $this->aMessages    = array_fill_keys(
+        $this->aMessages = array_fill_keys(
             array_keys( $this->aDefaults ),
             null
         );
@@ -279,13 +279,13 @@ 
      * @param       string       $sKey
      * @return      string|array
      */
-    public function get( $sKey='' ) {
-        if ( ! $sKey ) {
+    public function get( $sKey = '' ) {
+        if ( !$sKey ) {
             return $this->_getAllMessages();
         }
         return isset( $this->aMessages[ $sKey ] )
             ? __( $this->aMessages[ $sKey ], $this->_sTextDomain )
-            : __( $this->{$sKey}, $this->_sTextDomain );     // triggers __get()
+            : __( $this->{$sKey}, $this->_sTextDomain ); // triggers __get()
     }
         /**
          * Returns the all registered messag items.

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' ] ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'settings-notice' ] ) && !$_GET[ 'settings-notice' ] ) {    // sanitization unnecessary
             return; 
         }
             

Braces +4 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,13 @@ 
         
         // 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' ] ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) {
+// sanitization unnecessary
             return; 
         }
             

development/factory/_common/form/_model/AdminPageFramework_Form_Model___LastInput.php

Spacing +5 added lines, -5 removed lines

@@ -56,7 +56,7 @@ 
             $_sPageSlug = $this->getHTTPQueryGET( 'page', '' );
             $_sTabSlug  = $this->getHTTPQueryGET( 'tab', '' );
             $_sUserID   = get_current_user_id();
-            return "apf_li_" . md5(
+            return "apf_li_".md5(
                 $_sPageNow
                 . $_sPageSlug
                 . $_sTabSlug
@@ -94,13 +94,13 @@ 
          * @return      void
          */
         public function _replyToSave() {
-            if ( ! isset( self::$_aLastInputs ) ) {
+            if ( !isset( self::$_aLastInputs ) ) {
                 return;
             }
             $this->setTransient(
                 $this->sTransientKey,
                 self::$_aLastInputs,
-                60*60     // store it for 1 hour
+                60 * 60     // store it for 1 hour
             );
         }
 
@@ -120,9 +120,9 @@ 
             $_aLastInputs = self::$_aCaches[ $this->sTransientKey ];
         } else {
             $_aLastInputs = $this->getTransient( $this->sTransientKey );
-            self::$_aCaches[ $this->sTransientKey ]  = $_aLastInputs;
+            self::$_aCaches[ $this->sTransientKey ] = $_aLastInputs;
             if ( false !== $_aLastInputs ) {
-                $this->delete();    // deletes at the end of the script.
+                $this->delete(); // deletes at the end of the script.
             }
         }