michaeluno / admin-page-framework

development/factory/admin_page/AdminPageFramework.php

Braces +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      * @param       string                  $sCapability    (optional) sets the overall access level to the admin pages created by the framework. The used capabilities are listed <a href="http://codex.wordpress.org/Roles_and_Capabilities">here</a>. The capability can be set per page, tab, setting section, setting field. Default: `manage_options`
      * @param       string                  $sTextDomain    (optional) the <a href="http://codex.wordpress.org/I18n_for_WordPress_Developers#Text_Domains" target="_blank">text domain</a> used for the framework's system messages. Default: admin-page-framework.
      */
-    public function __construct( $isOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ){
+    public function __construct( $isOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
                         
         if ( ! $this->_isInstantiatable() ) {
             return;

Spacing +5 added lines, -5 removed lines

@@ -67,9 +67,9 @@ 
      * @param       string                  $sCapability    (optional) sets the overall access level to the admin pages created by the framework. The used capabilities are listed <a href="http://codex.wordpress.org/Roles_and_Capabilities">here</a>. The capability can be set per page, tab, setting section, setting field. Default: `manage_options`
      * @param       string                  $sTextDomain    (optional) the <a href="http://codex.wordpress.org/I18n_for_WordPress_Developers#Text_Domains" target="_blank">text domain</a> used for the framework's system messages. Default: admin-page-framework.
      */
-    public function __construct( $isOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ){
+    public function __construct( $isOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
-        if ( ! $this->_isInstantiatable() ) {
+        if ( !$this->_isInstantiatable() ) {
             return;
         }
 
@@ -89,15 +89,15 @@ 
          */
         private function _getCallerPath( $sCallerPath ) {
 
-            if ( $sCallerPath  ) {
+            if ( $sCallerPath ) {
                 return trim( $sCallerPath );
             }
 
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return null;
             }
 
-            if ( ! isset( $GLOBALS[ 'pagenow' ] ) ) {
+            if ( !isset( $GLOBALS[ 'pagenow' ] ) ) {
                 return null;
             }
 

development/factory/admin_page/_model/delegate/AdminPageFramework_Model__FormSubmission_Base.php

Spacing +3 added lines, -3 removed lines

@@ -46,12 +46,12 @@
      * @return      null|string     Returns `null` if no value is found and the associated link url if found. 
      * Otherwise, the found value.
      */ 
-    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey='field_id' ) {    
+    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey = 'field_id' ) {    
 
-        foreach( $aPostElements as $_sInputID => $_aSubElements ) {
+        foreach ( $aPostElements as $_sInputID => $_aSubElements ) {
             
             // The 'name' key must be set.
-            if ( ! isset( $_aSubElements[ 'name' ] ) ) {
+            if ( !isset( $_aSubElements[ 'name' ] ) ) {
                 continue;
             }
             $_aNameKeys = explode( '|', $_aSubElements[ 'name' ] ); 

Braces +4 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
      * @return      null|string     Returns `null` if no value is found and the associated link url if found. 
      * Otherwise, the found value.
      */ 
-    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey='field_id' ) {    
+    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey='field_id' ) {
 
         foreach( $aPostElements as $_sInputID => $_aSubElements ) {
             
@@ -58,7 +58,8 @@ 
             $_aNameKeys = explode( '|', $_aSubElements[ 'name' ] ); 
             
             // If the element is not found, skip.
-            if ( null === $this->getElement( $_POST, $_aNameKeys, null ) ) {    // sanitization unnecessary
+            if ( null === $this->getElement( $_POST, $_aNameKeys, null ) ) {
+// sanitization unnecessary
                 continue;
             }
             
@@ -85,7 +86,7 @@ 
      */
     protected function _setSettingNoticeAfterValidation( $bIsInputEmtpy ) {
      
-        if ( $this->oFactory->hasSettingNotice() ) {     
+        if ( $this->oFactory->hasSettingNotice() ) {
             return;
         }
         $this->oFactory->setSettingNotice(  

development/factory/admin_page/_model/format/AdminPageFramework_Format_InPageTabs.php

Indentation +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
      */
     public function get() {
 
-         // Apply filters to modify the in-page tab array.
+            // Apply filters to modify the in-page tab array.
         $_aInPageTabs = $this->addAndApplyFilter(
             $this->oFactory,  // caller object
             "tabs_{$this->oFactory->oProp->sClassName}_{$this->sPageSlug}", // filter name

Spacing +3 added lines, -3 removed lines

@@ -53,14 +53,14 @@
 
          // Apply filters to modify the in-page tab array.
         $_aInPageTabs = $this->addAndApplyFilter(
-            $this->oFactory,  // caller object
+            $this->oFactory, // caller object
             "tabs_{$this->oFactory->oProp->sClassName}_{$this->sPageSlug}", // filter name
             $this->aInPageTabs     // filtering value
         );    
 
         // Added items may be missing necessary keys so format them
-        foreach( ( array ) $_aInPageTabs as $_sTabSlug => $_aInPageTab ) {
-            if ( ! is_array( $_aInPageTab ) ) {
+        foreach ( ( array ) $_aInPageTabs as $_sTabSlug => $_aInPageTab ) {
+            if ( !is_array( $_aInPageTab ) ) {
                 continue;
             }
             $_oFormatter = new AdminPageFramework_Format_InPageTab( 

development/factory/admin_page/_view/AdminPageFramework_View__PageMataBoxRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     public function render( $sContext ) {
                
         // If nothing is registered do not render even the container.
-        if ( ! $this->doesMetaBoxExist() ) {
+        if ( !$this->doesMetaBoxExist() ) {
             return;
         }
         

Indentation -1 removed lines

@@ -20,7 +20,6 @@
 
     /**
      * Renders a registered meta box.
-
      * @return      void
      * @param       string $sContext `side`, `normal`, or `advanced`.
      * @since       3.0.0

development/factory/network_admin_page/AdminPageFramework_NetworkAdmin.php

Braces +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      * @param       string      $sTextDomain        (optional) the <a href="http://codex.wordpress.org/I18n_for_WordPress_Developers#Text_Domains" target="_blank">text domain</a> used for the framework's system messages. Default: `admin-page-framework`.
      * @return      void        returns nothing.
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_network', $sTextDomain='admin-page-framework' ){
+    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_network', $sTextDomain='admin-page-framework' ) {
             
         if ( ! $this->_isInstantiatable() ) {
             return;

Spacing +7 added lines, -7 removed lines

@@ -39,12 +39,12 @@ 
     protected $_aBuiltInRootMenuSlugs = array(
         // All keys must be lower case to support case insensitive look-ups.
         'dashboard'     => 'index.php',
-        'sites'         => 'sites.php',         // not work
-        'themes'        => 'themes.php',        // not work
+        'sites'         => 'sites.php', // not work
+        'themes'        => 'themes.php', // not work
         'plugins'       => 'plugins.php',
         'users'         => 'users.php',
         'settings'      => 'settings.php',
-        'updates'       => 'update-core.php',   // does not work
+        'updates'       => 'update-core.php', // does not work
     );
 
     /**
@@ -65,15 +65,15 @@ 
      * @param       string      $sTextDomain        (optional) the <a href="http://codex.wordpress.org/I18n_for_WordPress_Developers#Text_Domains" target="_blank">text domain</a> used for the framework's system messages. Default: `admin-page-framework`.
      * @return      void        returns nothing.
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_network', $sTextDomain='admin-page-framework' ){
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_network', $sTextDomain = 'admin-page-framework' ) {
 
-        if ( ! $this->_isInstantiatable() ) {
+        if ( !$this->_isInstantiatable() ) {
             return;
         }
 
         $sCallerPath = $sCallerPath
             ? $sCallerPath
-            : AdminPageFramework_Utility::getCallerScriptPath( __FILE__ );     // this is important to attempt to find the caller script path here when separating the library into multiple files.
+            : AdminPageFramework_Utility::getCallerScriptPath( __FILE__ ); // this is important to attempt to find the caller script path here when separating the library into multiple files.
 
         parent::__construct( $sOptionKey, $sCallerPath, $sCapability, $sTextDomain );
 
@@ -157,7 +157,7 @@ 
      * @param       mixed       $vDefault     the default value that will be returned if nothing is stored.
      * @return      mixed       If the field ID is not specified.
      */
-    static public function getOption( $sOptionKey, $asKey=null , $vDefault=null ) {
+    static public function getOption( $sOptionKey, $asKey = null, $vDefault = null ) {
         return AdminPageFramework_WPUtility::getSiteOption( $sOptionKey, $asKey, $vDefault );
     }
 

include/class/admin/_abstract/AdminPageFrameworkLoader_AdminPage_RootBase.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         }       
         
         trigger_error( 
-            'Admin Page Framework - Loader: ' . ' : ' . sprintf( 
+            'Admin Page Framework - Loader: '.' : '.sprintf( 
                 __( 'The method is not defined: %1$s', 'admin-page-framework-loader' ),
                 $sMethodName 
             ), 

include/class/admin/admin-page-framework/AdminPageFrameworkLoader_AdminPageMetaBox_ExternalLinks.php

Spacing +2 added lines, -2 removed lines

@@ -24,12 +24,12 @@
      */
     public function content( $sContent ) {
         
-        $_aReplacements   = array(
+        $_aReplacements = array(
             '%PLUGIN_DIR_URL%'  => AdminPageFrameworkLoader_Registry::getPluginURL(),
             '%WP_ADMIN_URL%'    => admin_url(),
         );
         $_oWPReadmeParser = new AdminPageFramework_WPReadmeParser( 
-            AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/text/about.txt',
+            AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/text/about.txt',
             $_aReplacements
         );
         return ''

include/class/admin/admin-page-framework/help/AdminPageFrameworkLoader_AdminPage_Help_Debug.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     
     public function replyToDoTab( $oFactory ) {
         
-        echo "<h3>" . __( 'Saved Options', 'admin-page-framework-loader' ) . "</h3>";
+        echo "<h3>".__( 'Saved Options', 'admin-page-framework-loader' )."</h3>";
         $oFactory->oDebug->dump( $oFactory->oProp->aOptions );
         
     }

include/class/admin/dashboard/welcome/AdminPageFrameworkLoader_AdminPageWelcome_Welcome.php

Braces +3 added lines, -2 removed lines

@@ -226,8 +226,9 @@
 
             $_aContributors = json_decode( wp_remote_retrieve_body( $_mResponse ) );
 
-            if ( ! is_array( $_aContributors ) )
-                return array();
+            if ( ! is_array( $_aContributors ) ) {
+                            return array();
+            }
 
             set_transient( 'apfl_contributors', $_aContributors, 3600 );
 

Spacing +25 added lines, -25 removed lines

@@ -28,9 +28,9 @@ 
                 'tab_slug'          => $this->sTabSlug,
                 'section_id'        => $this->sTabSlug,
                 'section_tab_slug'  => 'welcome',
-                'title'             => __( "What's New", 'admin-page-framework-loader' ),   // '
+                'title'             => __( "What's New", 'admin-page-framework-loader' ), // '
                 'content'           => $this->_getReadmeContents(
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/text/about.txt',
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/text/about.txt',
                     '', // TOC title
                     array( 'New Features' ) // section
                 )
@@ -39,10 +39,10 @@ 
                 'tab_slug'          => $this->sTabSlug,
                 'section_tab_slug'  => $this->sTabSlug,
                 'section_id'        => 'getting_started',
-                'title'             => __( "Getting Started", 'admin-page-framework-loader' ),   // '
+                'title'             => __( "Getting Started", 'admin-page-framework-loader' ), // '
                 'content'           => $this->_getReadmeContents(
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/text/about.txt',
-                    "<h3>" . __( 'Contents', 'admin-page-framework-loader' ) . "</h3>",
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/text/about.txt',
+                    "<h3>".__( 'Contents', 'admin-page-framework-loader' )."</h3>",
                     array( 'Getting Started', 'Tutorials' )
                 ),
             ),
@@ -50,7 +50,7 @@ 
                 'tab_slug'          => $this->sTabSlug,
                 'section_tab_slug'  => $this->sTabSlug,
                 'section_id'        => 'change_log',
-                'title'             => __( "Change Log", 'admin-page-framework-loader' ),   // '
+                'title'             => __( "Change Log", 'admin-page-framework-loader' ), // '
                 'content'           => $this->_getChangeLog(),
             )
         );
@@ -86,13 +86,13 @@ 
             array(
                 'field_id'      => 'github_star',
                 'type'          => 'github',
-                'user_name'     => 'michaeluno',    // the GitHub account ID
-                'button_type'   => 'star',          // either of the followings: follow, star, watch, fork, issue
+                'user_name'     => 'michaeluno', // the GitHub account ID
+                'button_type'   => 'star', // either of the followings: follow, star, watch, fork, issue
                 'count'         => false,
                 'repository'    => 'admin-page-framework',
                 'size'          => 'mega',
                 'attributes'    => array(
-                    'data-text' => ' ' . AdminPageFramework_Registry::NAME . ' ' . AdminPageFramework_Registry::getVersion() . ' ',
+                    'data-text' => ' '.AdminPageFramework_Registry::NAME.' '.AdminPageFramework_Registry::getVersion().' ',
                     // 'data-icon' => 'octicon-mark-github',
                 ),
                 'description'   => __( 'Star the repository and get Involved!', 'admin-page-framework-loader' ),
@@ -132,26 +132,26 @@ 
          * @since       3.6.1
          * @return      void
          */
-        private function _getChangeLog( $sSection='Changelog' ) {
+        private function _getChangeLog( $sSection = 'Changelog' ) {
 
-            $_aReplacements   = array(
+            $_aReplacements = array(
                 '%PLUGIN_DIR_URL%'  => AdminPageFrameworkLoader_Registry::getPluginURL(),
                 '%WP_ADMIN_URL%'    => admin_url(),
             );
             $_oWPReadmeParser = new AdminPageFramework_WPReadmeParser(
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/readme.txt',
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/readme.txt',
                 $_aReplacements
             );
             $_sChangeLog = $_oWPReadmeParser->getSection( $sSection );
             $_oWPReadmeParser = new AdminPageFramework_WPReadmeParser(
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/changelog.md',
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/changelog.md',
                 $_aReplacements
             );
             $_sChangeLog .= $_oWPReadmeParser->getSection( $sSection );
 
             $_sChangeLog = $_sChangeLog
                 ? $_sChangeLog
-                : '<p>' . __( 'No valid changlog was found.', 'admin-page-framework-loader' ) . '</p>';
+                : '<p>'.__( 'No valid changlog was found.', 'admin-page-framework-loader' ).'</p>';
             return "<div class='changelog'>"
                 . $_sChangeLog
                 . "</div>";
@@ -182,18 +182,18 @@ 
                 return '';
             }
 
-            $_aOutput   = array();
+            $_aOutput = array();
             foreach ( $_aContributors as $_oContributor ) {
-                $_aOutput[] = '<li class="wp-person">';
-                $_aOutput[] .= sprintf( '<a href="%s" title="%s">',
-                    esc_url( 'https://github.com/' . $_oContributor->login ),
+                $_aOutput[ ] = '<li class="wp-person">';
+                $_aOutput[ ] .= sprintf( '<a href="%s" title="%s">',
+                    esc_url( 'https://github.com/'.$_oContributor->login ),
                     esc_html( sprintf( __( 'View %s', 'admin-page-framework-loader' ), $_oContributor->login ) )
                 );
-                $_aOutput[] .= sprintf( '<img src="%s" width="64" height="64" class="gravatar" alt="%s" />', esc_url( $_oContributor->avatar_url ), esc_html( $_oContributor->login ) );
-                $_aOutput[] .= '</a>';
-                $_aOutput[] .= sprintf( '<a class="web" href="%s">%s</a>', esc_url( 'https://github.com/' . $_oContributor->login ), esc_html( $_oContributor->login ) );
-                $_aOutput[] .= '</a>';
-                $_aOutput[] .= '</li>';
+                $_aOutput[ ] .= sprintf( '<img src="%s" width="64" height="64" class="gravatar" alt="%s" />', esc_url( $_oContributor->avatar_url ), esc_html( $_oContributor->login ) );
+                $_aOutput[ ] .= '</a>';
+                $_aOutput[ ] .= sprintf( '<a class="web" href="%s">%s</a>', esc_url( 'https://github.com/'.$_oContributor->login ), esc_html( $_oContributor->login ) );
+                $_aOutput[ ] .= '</a>';
+                $_aOutput[ ] .= '</li>';
             }
 
             return '<ul class="wp-people-group">'
@@ -217,7 +217,7 @@ 
                 return $_aContributors;
             }
 
-            $_mResponse = wp_remote_get( $sRepositoryURL . '/contributors', array( 'sslverify' => false ) );
+            $_mResponse = wp_remote_get( $sRepositoryURL.'/contributors', array( 'sslverify' => false ) );
 
             if ( is_wp_error( $_mResponse ) || 200 != wp_remote_retrieve_response_code( $_mResponse ) ) {
                 return array();
@@ -225,7 +225,7 @@ 
 
             $_aContributors = json_decode( wp_remote_retrieve_body( $_mResponse ) );
 
-            if ( ! is_array( $_aContributors ) )
+            if ( !is_array( $_aContributors ) )
                 return array();
 
             set_transient( 'apfl_contributors', $_aContributors, 3600 );