michaeluno / admin-page-framework

development/factory/post_type/_controller/AdminPageFramework_Link_PostType.php

Indentation +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
                 array( $this, '_replyToAddPostTypeQueryInEditPostLink' ), 
                 10, 
                 3 
-           );
+            );
         }        
     }
 

Spacing +9 added lines, -9 removed lines

@@ -22,12 +22,12 @@ 
      * Sets up hooks and properties.
      * 
      */
-    public function __construct( $oProp, $oMsg=null ) {
+    public function __construct( $oProp, $oMsg = null ) {
         
         parent::__construct( $oProp, $oMsg );
                        
         // For post type posts listing table page ( edit.php )
-        if ( isset( $_GET['post_type'] ) && $_GET['post_type'] == $this->oProp->sPostType ) {
+        if ( isset( $_GET[ 'post_type' ] ) && $_GET[ 'post_type' ] == $this->oProp->sPostType ) {
             add_action( 
                 'get_edit_post_link', 
                 array( $this, '_replyToAddPostTypeQueryInEditPostLink' ), 
@@ -53,7 +53,7 @@ 
         );
             
         // If the user explicitly sets an empty string to the label key, do not insert a link.
-        if ( ! $_sLinkLabel ) {
+        if ( !$_sLinkLabel ) {
             return $aLinks;
         }
 
@@ -61,13 +61,13 @@ 
         array_unshift(    
             $aLinks,
             // "<a href='" . esc_url( "edit.php?post_type={$this->oProp->sPostType}" ) . "'>" 
-            '<a ' . $this->getAttributes(
+            '<a '.$this->getAttributes(
                 array(
                     'href'      => esc_url( "edit.php?post_type={$this->oProp->sPostType}" ),
                     // 3.5.7+ Added for acceptance testing
                     'class'     => 'apf-plugin-title-action-link apf-admin-page',
                 )
-            ) . '>'             
+            ).'>'             
                 . $_sLinkLabel 
             . "</a>"
         ); 
@@ -83,9 +83,9 @@ 
     public function _replyToSetFooterInfo() {
 
         if ( 
-            ! $this->isPostDefinitionPage( $this->oProp->sPostType ) 
-            && ! $this->isPostListingPage( $this->oProp->sPostType ) 
-            && ! $this->isCustomTaxonomyPage( $this->oProp->sPostType )
+            !$this->isPostDefinitionPage( $this->oProp->sPostType ) 
+            && !$this->isPostListingPage( $this->oProp->sPostType ) 
+            && !$this->isCustomTaxonomyPage( $this->oProp->sPostType )
         ) {
             return;
         }
@@ -103,7 +103,7 @@ 
      * @remark      e.g. `http://.../wp-admin/post.php?post=180&action=edit` -> `http://.../wp-admin/post.php?post=180&action=edit&post_type=[...]`
      * @callback    filter      get_edit_post_link
      */  
-    public function _replyToAddPostTypeQueryInEditPostLink( $sURL, $iPostID=null, $sContext=null ) {
+    public function _replyToAddPostTypeQueryInEditPostLink( $sURL, $iPostID = null, $sContext = null ) {
         return add_query_arg( 
             array( 
                 'post'      => $iPostID, 

development/factory/post_type/_model/AdminPageFramework_Property_PostType.php

Spacing +2 added lines, -2 removed lines

@@ -74,7 +74,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. 
     );     
     
     /**
@@ -162,7 +162,7 @@ 
         parent::__construct( $oCaller, $sCallerPath, $sClassName, $sCapability, $sTextDomain, $sStructureType );
 
         // Set up activation/deactivation hooks.
-        if ( ! $sCallerPath ) {
+        if ( !$sCallerPath ) {
             return;
         }
         switch ( $this->_getCallerType( $sCallerPath ) ) {

development/factory/post_type/_view/AdminPageFramework_PageLoadInfo_PostType.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
         if ( in_array( $oProp->sClassName, self::$aClassNames ) )
             return self::$_oInstance;
         
-        self::$aClassNames[] = $oProp->sClassName;
+        self::$aClassNames[ ] = $oProp->sClassName;
         self::$_oInstance = new AdminPageFramework_PageLoadInfo_PostType( $oProp, $oMsg );
         
         return self::$_oInstance;
@@ -44,7 +44,7 @@ 
     public function _replyToSetPageLoadInfoInFooter() {
 
         // Some users sets $_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'] ) { return; }
+        if ( isset( $_GET[ 'page' ] ) && $_GET[ 'page' ] ) { return; }
     
         // For post type pages
         if ( 

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,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_PostType( $oProp, $oMsg );

development/factory/post_type/form/AdminPageFramework_Form_post_type.php

Braces +1 added lines, -1 removed lines

@@ -21,6 +21,6 @@
  * @extends     AdminPageFramework_Form
  * @internal
  */
-class AdminPageFramework_Form_post_type extends AdminPageFramework_Form {    
+class AdminPageFramework_Form_post_type extends AdminPageFramework_Form {
     public $sStructureType = 'post_type';
 }
\ No newline at end of file

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField.php

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 for creating fields in the taxonomy page (edit-tags.php).

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param       string          The text domain. Default: `admin-page-framework`.
      * @return      void
      */ 
-    function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    function __construct( $asTaxonomySlug, $sOptionKey = '', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
         
         if ( empty( $asTaxonomySlug ) ) { 
             // @todo trigger a PHP warning
@@ -54,7 +54,7 @@ 
         }
         
         // Properties 
-        $this->oProp                    = new AdminPageFramework_Property_TaxonomyField( 
+        $this->oProp = new AdminPageFramework_Property_TaxonomyField( 
             $this, 
             get_class( $this ), 
             $sCapability, 

Braces +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
      */ 
     function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
         
-        if ( empty( $asTaxonomySlug ) ) { 
+        if ( empty( $asTaxonomySlug ) ) {
             // @todo trigger a PHP warning
             return; 
         }

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Model.php

Spacing +12 added lines, -12 removed lines

@@ -54,8 +54,8 @@ 
         return $this->_getFilteredColumnsByFilterPrefix( 
             $this->oUtil->getAsArray( $aColumns ), 
             'columns_', 
-            isset( $_GET['taxonomy'] )  // in ajax, $_GET is not even set.
-                ? $_GET['taxonomy']
+            isset( $_GET[ 'taxonomy' ] )  // in ajax, $_GET is not even set.
+                ? $_GET[ 'taxonomy' ]
                 : ''
         );        
     }
@@ -72,8 +72,8 @@ 
         return $this->_getFilteredColumnsByFilterPrefix( 
             $this->oUtil->getAsArray( $aSortableColumns ), 
             'sortable_columns_', 
-            isset( $_GET['taxonomy'] )  // in ajax, $_GET is not even set.
-                ? $_GET['taxonomy']
+            isset( $_GET[ 'taxonomy' ] )  // in ajax, $_GET is not even set.
+                ? $_GET[ 'taxonomy' ]
                 : ''
         );
     }   
@@ -87,7 +87,7 @@ 
             if ( $sTaxonomy ) {
                 $aColumns = $this->oUtil->addAndApplyFilter(
                     $this, 
-                    "{$sFilterPrefix}{$_GET['taxonomy']}",
+                    "{$sFilterPrefix}{$_GET[ 'taxonomy' ]}",
                     $aColumns
                 );
             }
@@ -124,7 +124,7 @@ 
      * @since       DEVVER      No longer sets the value to `$this-oProp->aOptions` but to the form peoperty.
      * @internal
      */
-    protected function _setOptionArray( $iTermID=null, $sOptionKey ) {
+    protected function _setOptionArray( $iTermID = null, $sOptionKey ) {
         $this->oForm->aSavedData = $this->_getSavedFormData( 
             $iTermID, 
             $sOptionKey
@@ -140,7 +140,7 @@ 
             
             return $this->oUtil->addAndApplyFilter(
                 $this, // the caller factory object
-                'options_' . $this->oProp->sClassName,
+                'options_'.$this->oProp->sClassName,
                 $this->_getSavedTermFormData( $iTermID, $sOptionKey )
                 // @todo maybe pass the term id because the user will not know whihch form data is
             );
@@ -174,7 +174,7 @@ 
      */
     public function _replyToValidateOptions( $iTermID ) {
 
-        if ( ! $this->_shouldProceedValidation() ) {
+        if ( !$this->_shouldProceedValidation() ) {
             return;
         }              
 
@@ -183,7 +183,7 @@ 
         $_aSubmittedFormData    = $this->oForm->getSubmittedData( $_POST ); 
         $_aSubmittedFormData    = $this->oUtil->addAndApplyFilters( 
             $this, 
-            'validation_' . $this->oProp->sClassName, 
+            'validation_'.$this->oProp->sClassName, 
             call_user_func_array( 
                 array( $this, 'validate' ), // triggers __call()
                 array( $_aSubmittedFormData, $_aSavedFormData, $this )
@@ -194,7 +194,7 @@ 
         
         // @todo Examine whether it is appropriate to merge recursivly 
         // as some fields will have a problem such as select with multiple options.
-        $_aTaxonomyFormData[ $iTermID ]  = $this->oUtil->uniteArrays( 
+        $_aTaxonomyFormData[ $iTermID ] = $this->oUtil->uniteArrays( 
             $_aSubmittedFormData, 
             $_aSavedFormData 
         );
@@ -215,11 +215,11 @@ 
          */        
         private function _shouldProceedValidation() {
 
-            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) { 
+            if ( !isset( $_POST[ $this->oProp->sClassHash ] ) ) { 
             
                 return false;
             }
-            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
+            if ( !wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
                 return false;
             }        
             return true;

Braces +1 added lines, -1 removed lines

@@ -215,7 +215,7 @@
          */        
         private function _shouldProceedValidation() {
 
-            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) { 
+            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) {
             
                 return false;
             }

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Router.php

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
             return false; 
         }
         
-        if ( isset( $_GET['taxonomy'] ) && ! in_array( $_GET['taxonomy'], $this->oProp->aTaxonomySlugs ) ) {
+        if ( isset( $_GET[ 'taxonomy' ] ) && !in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {
             return false;
         }        
         
@@ -74,7 +74,7 @@ 
      */
     public function _replyToDetermineToLoad( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( !$this->_isInThePage() ) { 
             return; 
         }
 
@@ -87,7 +87,7 @@ 
             $this 
         );
                       
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
+        foreach ( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
             
             /* Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not */
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );

Braces +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
             return true;
         }    
         
-        if ( 'edit-tags.php' != $this->oProp->sPageNow ) { 
+        if ( 'edit-tags.php' != $this->oProp->sPageNow ) {
             return false; 
         }
         
@@ -74,7 +74,7 @@ 
      */
     public function _replyToDetermineToLoad( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
 
@@ -87,7 +87,7 @@ 
             $this 
         );
                       
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
+        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {
             
             /* Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not */
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );

development/factory/taxonomy_field/_controller/AdminPageFramework_Resource_TaxonomyField.php

Spacing +14 added lines, -14 removed lines

@@ -30,11 +30,11 @@ 
      * @remark      the $_deprecated parameter is just to avoid the PHP strict standards warning.
      * @internal
      */
-    public function _enqueueStyles( $aSRCs, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueStyles( $aSRCs, $aCustomArgs = array(), $_deprecated = null ) {
         
         $_aHandleIDs = array();
-        foreach( ( array ) $aSRCs as $_sSRC ) {
-            $_aHandleIDs[] = $this->_enqueueStyle( $_sSRC, $aCustomArgs );
+        foreach ( ( array ) $aSRCs as $_sSRC ) {
+            $_aHandleIDs[ ] = $this->_enqueueStyle( $_sSRC, $aCustomArgs );
         }
         return $_aHandleIDs;
         
@@ -58,7 +58,7 @@ 
      * @return      string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */    
-    public function _enqueueStyle( $sSRC, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueStyle( $sSRC, $aCustomArgs = array(), $_deprecated = null ) {
         
         $sSRC = trim( $sSRC );
         if ( empty( $sSRC ) ) { return ''; }        
@@ -73,13 +73,13 @@ 
             array(     
                 'sSRC'      => $sSRC,
                 'sType'     => 'style',
-                'handle_id' => 'style_' . $this->oProp->sClassName . '_' .  ( ++$this->oProp->iEnqueuedStyleIndex ),
+                'handle_id' => 'style_'.$this->oProp->sClassName.'_'.( ++$this->oProp->iEnqueuedStyleIndex ),
             ),
             self::$_aStructure_EnqueuingResources
         );
         
         // Store the attributes in another container by url.
-        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingStyles[ $_sSRCHash ]['handle_id'] ] = $this->oProp->aEnqueuingStyles[ $_sSRCHash ]['attributes'];
+        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingStyles[ $_sSRCHash ][ 'handle_id' ] ] = $this->oProp->aEnqueuingStyles[ $_sSRCHash ][ 'attributes' ];
         
         return $this->oProp->aEnqueuingStyles[ $_sSRCHash ][ 'handle_id' ];
         
@@ -91,11 +91,11 @@ 
      * @since       3.0.0
      * @remark      the $_deprecated parameter is just to avoid the PHP strict standards warning.
      */
-    public function _enqueueScripts( $aSRCs, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueScripts( $aSRCs, $aCustomArgs = array(), $_deprecated = null ) {
         
         $_aHandleIDs = array();
-        foreach( ( array ) $aSRCs as $_sSRC ) {
-            $_aHandleIDs[] = $this->_enqueueScript( $_sSRC, $aCustomArgs );
+        foreach ( ( array ) $aSRCs as $_sSRC ) {
+            $_aHandleIDs[ ] = $this->_enqueueScript( $_sSRC, $aCustomArgs );
         }
         return $_aHandleIDs;
         
@@ -120,7 +120,7 @@ 
      * @return      string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      * @internal
      */
-    public function _enqueueScript( $sSRC, $aCustomArgs=array(), $_deprecated=null ) {
+    public function _enqueueScript( $sSRC, $aCustomArgs = array(), $_deprecated = null ) {
         
         $sSRC       = trim( $sSRC );
         if ( empty( $sSRC ) ) { return ''; }
@@ -135,13 +135,13 @@ 
             array(     
                 'sSRC'      => $sSRC,
                 'sType'     => 'script',
-                'handle_id' => 'script_' . $this->oProp->sClassName . '_' .  ( ++$this->oProp->iEnqueuedScriptIndex ),
+                'handle_id' => 'script_'.$this->oProp->sClassName.'_'.( ++$this->oProp->iEnqueuedScriptIndex ),
             ),
             self::$_aStructure_EnqueuingResources
         );
         
         // Store the attributes in another container by url.
-        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingScripts[ $_sSRCHash ]['handle_id'] ] = $this->oProp->aEnqueuingScripts[ $_sSRCHash ]['attributes'];
+        $this->oProp->aResourceAttributes[ $this->oProp->aEnqueuingScripts[ $_sSRCHash ][ 'handle_id' ] ] = $this->oProp->aEnqueuingScripts[ $_sSRCHash ][ 'attributes' ];
         
         return $this->oProp->aEnqueuingScripts[ $_sSRCHash ][ 'handle_id' ];
         
@@ -153,7 +153,7 @@ 
      * @since       3.0.0
      * @internal
      */
-    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs=array() ) {
+    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs = array() ) {
         return $this->_enqueueStyle( $sSRC, $aCustomArgs );
     }
     /**
@@ -162,7 +162,7 @@ 
      * @since       3.0.0
      * @internal
      */    
-    public function _forceToEnqueueScript( $sSRC, $aCustomArgs=array() ) {
+    public function _forceToEnqueueScript( $sSRC, $aCustomArgs = array() ) {
         return $this->_enqueueScript( $sSRC, $aCustomArgs );
     }
     

development/factory/taxonomy_field/form/AdminPageFramework_Form_taxonomy_field.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
             '' // default value
         );    
     
-        if ( ! $this->canUserView( $this->sCapability ) ) {
+        if ( !$this->canUserView( $this->sCapability ) ) {
             return '';
         }