michaeluno / admin-page-framework

development/factory/admin_page/AdminPageFramework_Model_Form.php

Spacing +17 added lines, -17 removed lines

@@ -58,7 +58,7 @@ 
      * @since       3.3.0
      * @since       3.3.1       Moved from `AdminPageFramework_Setting_Base`.
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         parent::__construct( $sOptionKey, $sCallerPath, $sCapability, $sTextDomain );
 
@@ -66,7 +66,7 @@ 
 //        if ( $this->oProp->bIsAdminAjax ) {
 //            return;
 //        }
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
 
@@ -116,7 +116,7 @@ 
             'section_title' => null,
             'section_id'    => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
 
@@ -134,7 +134,7 @@ 
                 'help_tab_id'               => $_sRootSectionID,
                 'help_tab_content'          => "<span class='contextual-help-tab-title'>"
                         . $aFieldset[ 'title' ]
-                    . "</span> - " . PHP_EOL
+                    . "</span> - ".PHP_EOL
                     . $aFieldset[ 'help' ],
                 'help_tab_sidebar_content'  => $aFieldset[ 'help_aside' ]
                     ? $aFieldset[ 'help_aside' ]
@@ -168,7 +168,7 @@ 
         public function _registerHelpPaneItemsOfFormSections( $aSectionsets ) {
 // @todo Test if help pane item gets displayed
 
-            foreach( $aSectionsets as $_aSectionset ) {
+            foreach ( $aSectionsets as $_aSectionset ) {
 // @todo check capability and conditions
                 $_aSectionset = $_aSectionset + array(
                     'help'          => null,
@@ -202,13 +202,13 @@ 
      */
     public function _replyToDetermineSectionsetVisibility( $bVisible, $aSectionset ) {
 
-        if ( ! current_user_can( $aSectionset[ 'capability' ] ) ) {
+        if ( !current_user_can( $aSectionset[ 'capability' ] ) ) {
             return false;
         }
-        if ( ! $aSectionset[ 'if' ] ) {
+        if ( !$aSectionset[ 'if' ] ) {
             return false;
         }
-        if ( ! $this->_isSectionOfCurrentPage( $aSectionset ) ) {
+        if ( !$this->_isSectionOfCurrentPage( $aSectionset ) ) {
             return false;
         }
         return $bVisible;
@@ -226,15 +226,15 @@ 
         private function _isSectionOfCurrentPage( array $aSectionset ) {
 
             // Make sure the value type is string so that when the page_slug is not set, it won't match.
-            $_sCurrentPageSlug  = ( string ) $this->oProp->getCurrentPageSlug();
+            $_sCurrentPageSlug = ( string ) $this->oProp->getCurrentPageSlug();
 
             // Make sure if it's in the loading page.
-            if ( $aSectionset[ 'page_slug' ] !== $_sCurrentPageSlug  ) {
+            if ( $aSectionset[ 'page_slug' ] !== $_sCurrentPageSlug ) {
                 return false;
             }
 
             // If no tag is specified, the user wants to display the section regardless of the tab.
-            if ( ! $aSectionset[ 'tab_slug' ] ) {
+            if ( !$aSectionset[ 'tab_slug' ] ) {
                 return true;
             }
 
@@ -250,7 +250,7 @@ 
      */
     public function _replyToDetermineFieldsetVisibility( $bVisible, $aFieldset ) {
 
-        $_sCurrentPageSlug  = $this->oProp->getCurrentPageSlug();
+        $_sCurrentPageSlug = $this->oProp->getCurrentPageSlug();
 
         // If the specified field does not exist, do nothing.
         if ( $aFieldset[ 'page_slug' ] !== $_sCurrentPageSlug ) {
@@ -280,7 +280,7 @@ 
             array( $_sSectionPath, 'page_slug' ),
             $this->oProp->getCurrentPageSlugIfAdded()
         );
-        $aFieldset[ 'tab_slug' ]        = $this->oUtil->getElement(
+        $aFieldset[ 'tab_slug' ] = $this->oUtil->getElement(
             $aSectionsets,
             array( $_sSectionPath, 'tab_slug' ),
             $this->oProp->getCurrentInPageTabSlugIfAdded()
@@ -291,11 +291,11 @@ 
             $aSectionsets,
             $_sSectionPath
         );
-        $aFieldset[ 'section_title' ]   = $this->oUtil->getElement(
+        $aFieldset[ 'section_title' ] = $this->oUtil->getElement(
             $_aSectionset,
             'title'
         );
-        $aFieldset[ 'capability' ]   = $aFieldset[ 'capability' ]
+        $aFieldset[ 'capability' ] = $aFieldset[ 'capability' ]
             ? $aFieldset[ 'capability' ]
             : $this->_replyToGetCapabilityForForm(
                 $this->oUtil->getElement( $_aSectionset, 'capability' ),
@@ -345,13 +345,13 @@ 
 
             // Find the capability of the parent section if nested.
             if ( 0 < $aSectionset[ '_nested_depth' ] ) {
-                $_aSectionPath         = $aSectionset[ '_section_path_array' ];
+                $_aSectionPath = $aSectionset[ '_section_path_array' ];
                 array_pop( $_aSectionPath ); // remove the last element
                 $_sParentCapability = $this->oUtil->getElement(
                     $this->oForm->aSectionsets,
                     array_merge( $_aSectionPath, array( 'capability' ) )
                 );
-                if( $_sParentCapability ) {
+                if ( $_sParentCapability ) {
                     return $_sParentCapability;
                 }
             }

Braces +2 added lines, -1 removed lines

@@ -74,7 +74,8 @@
         // new AdminPageFramework_Model__FormEmailHandler( $this );
 
         // Checking the GET and POST methods.
-        if ( isset( $_REQUEST[ 'apf_remote_request_test' ] ) && '_testing' === $_REQUEST[ 'apf_remote_request_test' ] ) {   // sanitization unnecessary
+        if ( isset( $_REQUEST[ 'apf_remote_request_test' ] ) && '_testing' === $_REQUEST[ 'apf_remote_request_test' ] ) {
+// sanitization unnecessary
             exit( 'OK' );
         }
 

development/factory/admin_page/AdminPageFramework_Model_Menu.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      *
      * @internal
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         parent::__construct( $sOptionKey, $sCallerPath, $sCapability, $sTextDomain );
 

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer__PageHeadingTabs.php

Spacing +12 added lines, -12 removed lines

@@ -45,7 +45,7 @@ 
         $_aPage = $this->oFactory->oProp->aPages[ $this->sPageSlug ];
 
         // If the page title is disabled, return an empty string.
-        if ( ! $_aPage[ 'show_page_title' ] ) {
+        if ( !$_aPage[ 'show_page_title' ] ) {
             return "";
         }
 
@@ -64,10 +64,10 @@ 
          */
         private function _getOutput( $aPage, $sTag ) {
 
-            $sTag  = $this->_getPageHeadingTabTag( $sTag, $aPage );
+            $sTag = $this->_getPageHeadingTabTag( $sTag, $aPage );
 
             // If the page heading tab visibility is disabled, or only one page is registered, return the title.
-            if ( ! $aPage[ 'show_page_heading_tabs' ] || count( $this->oFactory->oProp->aPages ) == 1 ) {
+            if ( !$aPage[ 'show_page_heading_tabs' ] || count( $this->oFactory->oProp->aPages ) == 1 ) {
                 return "<{$sTag}>"
                         . $aPage[ 'title' ]
                     . "</{$sTag}>";
@@ -105,9 +105,9 @@ 
             private function _getPageHeadingtabNavigationBar( array $aPages, $sTag, $sCurrentPageSlug ) {
 
                 $_oTabBar = new AdminPageFramework_TabNavigationBar(
-                    $aPages,     // tab items
+                    $aPages, // tab items
                     $sCurrentPageSlug, // active tab slug
-                    $sTag,       // container tag
+                    $sTag, // container tag
                     array(       // container attributes
                         // 'class' => '...',
                     ),
@@ -130,8 +130,8 @@ 
                  * @since           3.5.10
                  * @since       3.6.3       Moved from `AdminPageFramework_Page_View`.
                  */
-                public function _replyToFormatNavigationTabItem_PageHeadingTab( $aSubPage, $aStructure, $aPages, $aArguments=array() ) {
-                    switch( $aSubPage['type'] ) {
+                public function _replyToFormatNavigationTabItem_PageHeadingTab( $aSubPage, $aStructure, $aPages, $aArguments = array() ) {
+                    switch ( $aSubPage[ 'type' ] ) {
                         case 'link':
                             return $this->_getFormattedPageHeadingtabNavigationBarLinkItem( $aSubPage, $aStructure );
                         default:
@@ -148,10 +148,10 @@ 
                      */
                     private function _getFormattedPageHeadingtabNavigationBarPageItem( array $aSubPage, $aStructure ) {
 
-                        if ( ! isset( $aSubPage[ 'page_slug' ] ) ) {
+                        if ( !isset( $aSubPage[ 'page_slug' ] ) ) {
                             return array();
                         }
-                        if ( ! $aSubPage[ 'show_page_heading_tab' ] ) {
+                        if ( !$aSubPage[ 'show_page_heading_tab' ] ) {
                             return array();
                         }
                         return array(
@@ -182,10 +182,10 @@ 
                      */
                     private function _getFormattedPageHeadingtabNavigationBarLinkItem( array $aSubPage, $aStructure ) {
 
-                        if ( ! isset( $aSubPage[ 'href' ] ) ) {
+                        if ( !isset( $aSubPage[ 'href' ] ) ) {
                             return array();
                         }
-                        if ( ! $aSubPage[ 'show_page_heading_tab' ] ) {
+                        if ( !$aSubPage[ 'show_page_heading_tab' ] ) {
                             return array();
                         }
                         $aSubPage = array(
@@ -197,7 +197,7 @@ 
                             + array( 'class' => null )
                             + $aStructure;
 
-                        $aSubPage[ 'class' ] = trim( $aSubPage[ 'class' ] . ' link' );
+                        $aSubPage[ 'class' ] = trim( $aSubPage[ 'class' ].' link' );
                         return $aSubPage;
                     }
 

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer__ScreenIcon.php

Spacing +12 added lines, -12 removed lines

@@ -40,7 +40,7 @@ 
      */
     public function get() {
 
-        if ( ! $this->_isScreenIconVisible() ) {
+        if ( !$this->_isScreenIconVisible() ) {
             return '';
         }
         return $this->_getScreenIcon( $this->sPageSlug );
@@ -52,7 +52,7 @@ 
          */
         private function _isScreenIconVisible() {
 
-            $_bShowPageTitle        = $this->getElement(
+            $_bShowPageTitle = $this->getElement(
                 $this->oFactory->oProp->aPages,
                 array( $this->sPageSlug, 'show_page_title' )
             );
@@ -130,7 +130,7 @@ 
                 array( $sPageSlug, 'href_icon_32x32' ),
                 ''
             );
-            if ( ! $_sScreenIconPath ) {
+            if ( !$_sScreenIconPath ) {
                 return;
             }
             $_sScreenIconPath = $this->getResolvedSRC(
@@ -140,7 +140,7 @@ 
             $_aAttributes = array(
                 'style'    => $this->getInlineCSS(
                     array(
-                        'background-image' => "url('" . esc_url( $_sScreenIconPath ) . "')"
+                        'background-image' => "url('".esc_url( $_sScreenIconPath )."')"
                     )
                 )
             );
@@ -164,12 +164,12 @@ 
                 array( $sPageSlug, 'screen_icon_id' ),
                 ''
             );
-            if ( ! $_sScreenIconID ) {
+            if ( !$_sScreenIconID ) {
                 return;
             }
 
             $_aAttributes = array(
-                'id'       => "icon-" . $_sScreenIconID,
+                'id'       => "icon-".$_sScreenIconID,
             );
 
             // Go to the catch clause.
@@ -192,16 +192,16 @@ 
                 'class'    => $this->getClassAttribute(
                     $this->getAOrB(
                         empty( $_sIconIDAttribute ) && $_oScreen->post_type,
-                        sanitize_html_class( 'icon32-posts-' . $_oScreen->post_type ),
+                        sanitize_html_class( 'icon32-posts-'.$_oScreen->post_type ),
                         ''
                     ),
                     $this->getAOrB(
                         empty( $_sIconIDAttribute ) || $_sIconIDAttribute == $this->oFactory->oProp->sClassName,
-                        'generic',  // the default value
+                        'generic', // the default value
                         ''
                     )
                 ),
-                'id'       => "icon-" . $_sIconIDAttribute,
+                'id'       => "icon-".$_sIconIDAttribute,
             );
             return $this->_getScreenIconByAttributes( $_aAttributes );
 
@@ -215,7 +215,7 @@ 
              */
             private function _getScreenIDAttribute( $oScreen ) {
 
-                if ( ! empty( $oScreen->parent_base ) ) {
+                if ( !empty( $oScreen->parent_base ) ) {
                     return $oScreen->parent_base;
                 }
                 if ( 'page' === $oScreen->post_type ) {
@@ -236,10 +236,10 @@ 
             private function _getScreenIconByAttributes( array $aAttributes ) {
 
                 $aAttributes[ 'class' ] = $this->getClassAttribute(
-                    'icon32',   // required for a screen icon container element.
+                    'icon32', // required for a screen icon container element.
                     $this->getElement( $aAttributes, 'class' )
                 );
-                return "<div " . $this->getAttributes( $aAttributes ) . ">"
+                return "<div ".$this->getAttributes( $aAttributes ).">"
                         . "<br />"
                     . "</div>";
 

development/factory/page_meta_box/form/AdminPageFramework_Form_page_meta_box.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
     public function construct() {
 
         add_filter(
-            'options_' . $this->aArguments[ 'caller_id' ],
+            'options_'.$this->aArguments[ 'caller_id' ],
             array( $this, '_replyToSanitizeSavedFormData' ),
             5   //  high priority as it must be done earlier
         );
@@ -53,7 +53,7 @@ 
 
         // Extract the meta box field form data (options) from the page form data (options).
         return $this->castArrayContents(
-            $this->getDataStructureFromAddedFieldsets(),    // form data structure generate from fieldsets
+            $this->getDataStructureFromAddedFieldsets(), // form data structure generate from fieldsets
             $aSavedFormData
         );
 

development/factory/page_meta_box/AdminPageFramework_MetaBox_Page.php

Spacing +2 added lines, -2 removed lines

@@ -23,12 +23,12 @@
      * @since       3.0.0
      * @deprecated  3.7.0      Use     `AdminPageFramework_PageMetaBox` instead.
      */
-    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs = array(), $sContext = 'normal', $sPriority = 'default', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         parent::__construct( $sMetaBoxID, $sTitle, $asPageSlugs, $sContext, $sPriority, $sCapability, $sTextDomain );
 
         $this->oUtil->showDeprecationNotice(
-            'The class, ' . __CLASS__ . ',', // deprecated item
+            'The class, '.__CLASS__.',', // deprecated item
             'AdminPageFramework_PageMetaBox' // alternative
         );
 

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_View.php

Indentation +1 added lines, -1 removed lines

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

Spacing +6 added lines, -6 removed lines

@@ -47,7 +47,7 @@ 
             '',
             "[{$_sKey}]"
         );
-        return $_aField['field_id'] . $_sKey;
+        return $_aField[ 'field_id' ].$_sKey;
 
     }
     /**
@@ -65,7 +65,7 @@ 
             '',
             "|{$_sKey}"
         );
-        return "{$_aField['field_id']}{$_sKey}";
+        return "{$_aField[ 'field_id' ]}{$_sKey}";
     }
 
    /**
@@ -115,7 +115,7 @@ 
             $_aOutput = array();
 
             // Set nonce.
-            $_aOutput[] = wp_nonce_field(
+            $_aOutput[ ] = wp_nonce_field(
                 $this->oProp->sClassHash,
                 $this->oProp->sClassHash,
                 true,
@@ -126,17 +126,17 @@ 
             $this->_setOptionArray( $iTermID, $this->oProp->sOptionKey );
 
             // Get the field outputs
-            $_aOutput[] = $this->oForm->get( $bRenderTableRow );
+            $_aOutput[ ] = $this->oForm->get( $bRenderTableRow );
 
             // Filter the output
             $_sOutput = $this->oUtil->addAndApplyFilters(
                 $this,
-                'content_' . $this->oProp->sClassName,
+                'content_'.$this->oProp->sClassName,
                 $this->content( implode( PHP_EOL, $_aOutput ) )
             );
 
             // Do action
-            $this->oUtil->addAndDoActions( $this, 'do_' . $this->oProp->sClassName, $this );
+            $this->oUtil->addAndDoActions( $this, 'do_'.$this->oProp->sClassName, $this );
 
             return $_sOutput;
 

development/factory/widget/AdminPageFramework_Widget_Controller.php

Indentation +16 added lines, -16 removed lines

@@ -19,22 +19,22 @@
 abstract class AdminPageFramework_Widget_Controller extends AdminPageFramework_Widget_View {
 
     /**
-    * The method for necessary set-ups.
-    *
-    * <h4>Example</h4>
-    * <code>
-    *   public function setUp() {
-    *       $this->setArguments(
-    *           array(
-    *               'description'   =>  __( 'This is a sample widget with built-in field types created by Admin Page Framework.', 'admin-page-framework-demo' ),
-    *           )
-    *       );
-    *   }
-    * </code>
-    *
-    * @abstract
-    * @since        3.2.0
-    */
+     * The method for necessary set-ups.
+     *
+     * <h4>Example</h4>
+     * <code>
+     *   public function setUp() {
+     *       $this->setArguments(
+     *           array(
+     *               'description'   =>  __( 'This is a sample widget with built-in field types created by Admin Page Framework.', 'admin-page-framework-demo' ),
+     *           )
+     *       );
+     *   }
+     * </code>
+     *
+     * @abstract
+     * @since        3.2.0
+     */
     public function setUp() {}
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
      * @see         https://codex.wordpress.org/Function_Reference/wp_register_sidebar_widget
      * @return      void
      */
-    protected function setArguments( array $aArguments=array() ) {
+    protected function setArguments( array $aArguments = array() ) {
         $this->oProp->aWidgetArguments = $aArguments;
     }
 

development/factory/widget/AdminPageFramework_Widget_Model.php

Spacing +3 added lines, -3 removed lines

@@ -36,9 +36,9 @@ 
 
         if ( $this->oProp->bIsAdmin ) {
             add_filter(
-                'validation_' . $this->oProp->sClassName,
+                'validation_'.$this->oProp->sClassName,
                 array( $this, '_replyToSortInputs' ),
-                1,  // set a high priority
+                1, // set a high priority
                 3   // number of parameters
             );
         }
@@ -108,7 +108,7 @@ 
      */
     public function _replyToRegisterWidget() {
 
-        if ( ! is_object( $GLOBALS[ 'wp_widget_factory' ] ) ) {
+        if ( !is_object( $GLOBALS[ 'wp_widget_factory' ] ) ) {
             return;
         }