michaeluno / admin-page-framework

development/factory/post_type/AdminPageFramework_PostType_Model.php

Spacing +9 added lines, -9 removed lines

@@ -36,13 +36,13 @@ 
         
         // Properties
         $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.
             // 'categories' => $this->oMsg->get( 'categories' ), // Categories the post belongs to. 
             // 'tags' => $this->oMsg->get( 'tags' ),        // Tags for the post. 
             '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. 
         );     
                             
         if ( $this->_isInThePage() ) :
@@ -115,7 +115,7 @@ 
         echo $this->oUtil->addAndApplyFilter( 
             $this, 
             "cell_{$this->oProp->sPostType}_{$sColumnKey}", 
-            '',  // cell output
+            '', // cell output
             $iPostID 
         );
     }    
@@ -156,7 +156,7 @@ 
      */
     public function _replyToRegisterTaxonomies() {
 
-        foreach( $this->oProp->aTaxonomies as $_sTaxonomySlug => $_aArgs ) {
+        foreach ( $this->oProp->aTaxonomies as $_sTaxonomySlug => $_aArgs ) {
             $this->_registerTaxonomy( 
                 $_sTaxonomySlug,  
                 is_array( $this->oProp->aTaxonomyObjectTypes[ $_sTaxonomySlug ] ) 
@@ -175,8 +175,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,
@@ -193,7 +193,7 @@ 
      */
     public function _replyToRemoveTexonomySubmenuPages() {
     
-        foreach( $this->oProp->aTaxonomyRemoveSubmenuPages as $sSubmenuPageSlug => $sTopLevelPageSlug ) {
+        foreach ( $this->oProp->aTaxonomyRemoveSubmenuPages as $sSubmenuPageSlug => $sTopLevelPageSlug ) {
             
             remove_submenu_page( $sTopLevelPageSlug, $sSubmenuPageSlug );
             

Braces +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
  * @subpackage      PostType
  * @internal
  */
-abstract class AdminPageFramework_PostType_Model extends AdminPageFramework_PostType_Router {    
+abstract class AdminPageFramework_PostType_Model extends AdminPageFramework_PostType_Router {
 
     /**
      * Sets up hooks and properties.
@@ -111,7 +111,7 @@ 
      * @callback    filter      manage_{post type slug}_posts_custom_column
      * @return      string
      */
-    public function _replyToPrintColumnCell( $sColumnKey, $iPostID ) {                 
+    public function _replyToPrintColumnCell( $sColumnKey, $iPostID ) {
         echo $this->oUtil->addAndApplyFilter( 
             $this, 
             "cell_{$this->oProp->sPostType}_{$sColumnKey}", 
@@ -127,10 +127,10 @@ 
      */
     public function _replyToDisableAutoSave() {
         
-        if ( $this->oProp->bEnableAutoSave ) { 
+        if ( $this->oProp->bEnableAutoSave ) {
             return; 
         }
-        if ( $this->oProp->sPostType != get_post_type() ) { 
+        if ( $this->oProp->sPostType != get_post_type() ) {
             return; 
         }
         wp_dequeue_script( 'autosave' );

development/factory/post_type/AdminPageFramework_PostType_Router.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     public function _isInThePage() {
         
         // If it's not in one of the post type's pages
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
@@ -38,7 +38,7 @@ 
             return true;
         }        
         
-        if ( ! in_array( $this->oProp->sPageNow, array( 'edit.php', 'edit-tags.php', 'post.php', 'post-new.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'edit.php', 'edit-tags.php', 'post.php', 'post-new.php' ) ) ) {
             return false;
         }
                 

Braces +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  * @subpackage      PostType
  * @internal
  */
-abstract class AdminPageFramework_PostType_Router extends AdminPageFramework_Factory {    
+abstract class AdminPageFramework_PostType_Router extends AdminPageFramework_Factory {
   
     /**
      * Determines whether the currently loaded page is of the post type page.

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