michaeluno / admin-page-framework

development/factory/admin_page/AdminPageFramework_Controller_Menu.php

Doc Comments +3 added lines

@@ -48,6 +48,8 @@ 
      *  <li>a base64-encoded SVG using a data URI, which will be colored to match the color scheme. This should begin with 'data:image/svg+xml;base64,'.</li>
      * </ul>
      * @param       string      (optional) the position number that is passed to the <var>$position</var> parameter of the <a href="http://codex.wordpress.org/Function_Reference/add_menu_page">add_menu_page()</a> function.
+     * @param string $sIcon16x16
+     * @param integer $iMenuPosition
      * @return      void
      */
     public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16=null, $iMenuPosition=null ) {
@@ -70,6 +72,7 @@ 
          * 
          * @since       2.0.0
          * @internal
+         * @param string $sMenuLabel
          * @return      void|string Returns the associated slug string, if true.
          */ 
         private function _isBuiltInMenuItem( $sMenuLabel ) {

Indentation +91 added lines, -91 removed lines

@@ -104,39 +104,39 @@ 
     }
     
     /**
-    * Adds sub-menu items on the left sidebar menu of the administration panel. 
-    * 
-    * It supports pages and links. Each of them has the specific array structure.
-    * 
-    * <h4>Example</h4>
-    * <code>$this->addSubMenuItems(
-    *       array(
-    *           'title'         => 'Various Form Fields',
-    *           'page_slug'     => 'first_page',
-    *           'screen_icon'   => 'options-general',
-    *       ),
-    *       array(
-    *           'title'         => 'Manage Options',
-    *           'page_slug'     => 'second_page',
-    *           'screen_icon'   => 'link-manager',
-    *       ),
-    *       array(
-    *           'title'         => 'Google',
-    *           'href'          => 'http://www.google.com',    
-    *           'show_page_heading_tab' => false, // this removes the title from the page heading tabs.
-    *       )
-    * );</code>
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public.
-    * @remark       The sub menu page slug should be unique because add_submenu_page() can add one callback per page slug.
-    * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
-    * @param        array       $aSubMenuItem1      a first sub-menu array. A sub-menu array can be a link or a page. For the specifications of the array structures and its arguments, refer to the parameter section of the `addSubMenuItem()` method.
-    * @param        array       $aSubMenuItem2      (optional) a second sub-menu array.
-    * @param        array       $_and_more          (optional) a third and add items as many as necessary with next parameters.
-    * @access       public
-    * @return       void
-    */     
+     * Adds sub-menu items on the left sidebar menu of the administration panel. 
+     * 
+     * It supports pages and links. Each of them has the specific array structure.
+     * 
+     * <h4>Example</h4>
+     * <code>$this->addSubMenuItems(
+     *       array(
+     *           'title'         => 'Various Form Fields',
+     *           'page_slug'     => 'first_page',
+     *           'screen_icon'   => 'options-general',
+     *       ),
+     *       array(
+     *           'title'         => 'Manage Options',
+     *           'page_slug'     => 'second_page',
+     *           'screen_icon'   => 'link-manager',
+     *       ),
+     *       array(
+     *           'title'         => 'Google',
+     *           'href'          => 'http://www.google.com',    
+     *           'show_page_heading_tab' => false, // this removes the title from the page heading tabs.
+     *       )
+     * );</code>
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public.
+     * @remark       The sub menu page slug should be unique because add_submenu_page() can add one callback per page slug.
+     * @remark       Accepts variadic parameters; the number of accepted parameters are not limited to three.
+     * @param        array       $aSubMenuItem1      a first sub-menu array. A sub-menu array can be a link or a page. For the specifications of the array structures and its arguments, refer to the parameter section of the `addSubMenuItem()` method.
+     * @param        array       $aSubMenuItem2      (optional) a second sub-menu array.
+     * @param        array       $_and_more          (optional) a third and add items as many as necessary with next parameters.
+     * @access       public
+     * @return       void
+     */     
     public function addSubMenuItems( $aSubMenuItem1, $aSubMenuItem2=null, $_and_more=null ) {
         foreach ( func_get_args() as $_aSubMenuItem ) {
             $this->addSubMenuItem( $_aSubMenuItem );     
@@ -144,50 +144,50 @@ 
     }
     
     /**
-    * Adds the given sub-menu item on the left sidebar menu of the administration panel.
-    * 
-    * It supports pages and links. Each of them has the specific array structure.
-    * 
-    * <h4>Example</h4>
-    * <code>$this->addSubMenuItem(
-    *       array(
-    *           'title'         => 'Read Me',
-    *           'menu_title'    => 'About'
-    *           'page_slug'     => 'my_plugin_readme',
-    *       )
-    * );</code>
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public.
-    * @remark       The sub menu page slug should be unique because add_submenu_page() can add one callback per page slug.
-    * @param        array       a sub-menu array. It can be a page or a link. The array structures are as follows:
-    * <h4>Sub-menu Page Array</h4>
-    * <ul>
-    *   <li>**title** - (string) the page title of the page.</li>
-    *   <li>**page_slug** - (string) the page slug of the page. Non-alphabetical characters should not be used including dots(.) and hyphens(-).</li>
-    *   <li>**screen_icon** - (optional, string) either the ID selector name from the following list or the icon URL. The size of the icon should be 32 by 32 in pixel. This is for WordPress 3.7.x or below.
-    *       <pre>edit, post, index, media, upload, link-manager, link, link-category, edit-pages, page, edit-comments, themes, plugins, users, profile, user-edit, tools, admin, options-general, ms-admin, generic</pre>
-    *       <p>( Notes: the `generic` icon is available WordPress version 3.5 or above.)</p> 
-    *   </li>
-    *   <li>**capability** - (optional, string) the access level to the created admin pages defined <a href="http://codex.wordpress.org/Roles_and_Capabilities">here</a>. If not set, the overall capability assigned in the class constructor, which is `manage_options` by default, will be used.</li>
-    *   <li>**order** - (optional, integer) the order number of the page. The lager the number is, the lower the position it is placed in the menu.</li>
-    *   <li>**show_page_heading_tab** - (optional, boolean) if this is set to false, the page title won't be displayed in the page heading tab. Default: `true`.</li>
-    *   <li>**show_in_menu** - (optional) If this is set to false, the page title won't be displayed in the sidebar menu while the page is still accessible. Default: true.</li>
-    *   <li>**page_title** - (optional) [3.3.0+] When the page title differs from the menu title, use this argument.</li>
-    *   <li>**menu_title** - (optional) [3.3.0+] When the menu title differs from the menu title, use this argument.</li>
-    * </ul>
-    * <h4>Sub-menu Link Array</h4>
-    * <ul>
-    *   <li>**title** - (string) the link title.</li>
-    *   <li>**href** - (string) the URL of the target link.</li>
-    *   <li>**capability** - (optional, string) the access level to show the item, defined <a href="http://codex.wordpress.org/Roles_and_Capabilities">here</a>. If not set, the overall capability assigned in the class constructor, which is `manage_options` by default, will be used.</li>
-    *   <li>**order** - (optional, integer) the order number of the page. The lager the number is, the lower the position it is placed in the menu.</li>
-    *   <li>**show_page_heading_tab** - (optional, boolean) if this is set to false, the page title won't be displayed in the page heading tab. Default: `true`.</li>
-    *   <li>**show_in_menu** - (optional) If this is set to false, the page title won't be displayed in the sidebar menu while the page is still accessible. Default: true.</li>
-    * </ul>
-    * @access       public
-    * @return       void
-    */    
+     * Adds the given sub-menu item on the left sidebar menu of the administration panel.
+     * 
+     * It supports pages and links. Each of them has the specific array structure.
+     * 
+     * <h4>Example</h4>
+     * <code>$this->addSubMenuItem(
+     *       array(
+     *           'title'         => 'Read Me',
+     *           'menu_title'    => 'About'
+     *           'page_slug'     => 'my_plugin_readme',
+     *       )
+     * );</code>
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public.
+     * @remark       The sub menu page slug should be unique because add_submenu_page() can add one callback per page slug.
+     * @param        array       a sub-menu array. It can be a page or a link. The array structures are as follows:
+     * <h4>Sub-menu Page Array</h4>
+     * <ul>
+     *   <li>**title** - (string) the page title of the page.</li>
+     *   <li>**page_slug** - (string) the page slug of the page. Non-alphabetical characters should not be used including dots(.) and hyphens(-).</li>
+     *   <li>**screen_icon** - (optional, string) either the ID selector name from the following list or the icon URL. The size of the icon should be 32 by 32 in pixel. This is for WordPress 3.7.x or below.
+     *       <pre>edit, post, index, media, upload, link-manager, link, link-category, edit-pages, page, edit-comments, themes, plugins, users, profile, user-edit, tools, admin, options-general, ms-admin, generic</pre>
+     *       <p>( Notes: the `generic` icon is available WordPress version 3.5 or above.)</p> 
+     *   </li>
+     *   <li>**capability** - (optional, string) the access level to the created admin pages defined <a href="http://codex.wordpress.org/Roles_and_Capabilities">here</a>. If not set, the overall capability assigned in the class constructor, which is `manage_options` by default, will be used.</li>
+     *   <li>**order** - (optional, integer) the order number of the page. The lager the number is, the lower the position it is placed in the menu.</li>
+     *   <li>**show_page_heading_tab** - (optional, boolean) if this is set to false, the page title won't be displayed in the page heading tab. Default: `true`.</li>
+     *   <li>**show_in_menu** - (optional) If this is set to false, the page title won't be displayed in the sidebar menu while the page is still accessible. Default: true.</li>
+     *   <li>**page_title** - (optional) [3.3.0+] When the page title differs from the menu title, use this argument.</li>
+     *   <li>**menu_title** - (optional) [3.3.0+] When the menu title differs from the menu title, use this argument.</li>
+     * </ul>
+     * <h4>Sub-menu Link Array</h4>
+     * <ul>
+     *   <li>**title** - (string) the link title.</li>
+     *   <li>**href** - (string) the URL of the target link.</li>
+     *   <li>**capability** - (optional, string) the access level to show the item, defined <a href="http://codex.wordpress.org/Roles_and_Capabilities">here</a>. If not set, the overall capability assigned in the class constructor, which is `manage_options` by default, will be used.</li>
+     *   <li>**order** - (optional, integer) the order number of the page. The lager the number is, the lower the position it is placed in the menu.</li>
+     *   <li>**show_page_heading_tab** - (optional, boolean) if this is set to false, the page title won't be displayed in the page heading tab. Default: `true`.</li>
+     *   <li>**show_in_menu** - (optional) If this is set to false, the page title won't be displayed in the sidebar menu while the page is still accessible. Default: true.</li>
+     * </ul>
+     * @access       public
+     * @return       void
+     */    
     public function addSubMenuItem( array $aSubMenuItem ) {
         if ( isset( $aSubMenuItem[ 'href' ] ) ) {
             $this->addSubMenuLink( $aSubMenuItem );
@@ -197,20 +197,20 @@ 
     }
 
     /**
-    * Adds the given link into the menu on the left sidebar of the administration panel.
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public from protected.
-    * @since        3.5.0       Changed the scope to public as it was still protected.
-    * @param        string      the menu title.
-    * @param        string      the URL linked to the menu.
-    * @param        string      (optional) the <a href="http://codex.wordpress.org/Roles_and_Capabilities" target="_blank">access level</a>.
-    * @param        string      (optional) the order number. The larger it is, the lower the position it gets.
-    * @param        string      (optional) if set to false, the menu title will not be listed in the tab navigation menu at the top of the page.
-    * @access       public
-    * @return       void
-    * @internal
-    */    
+     * Adds the given link into the menu on the left sidebar of the administration panel.
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public from protected.
+     * @since        3.5.0       Changed the scope to public as it was still protected.
+     * @param        string      the menu title.
+     * @param        string      the URL linked to the menu.
+     * @param        string      (optional) the <a href="http://codex.wordpress.org/Roles_and_Capabilities" target="_blank">access level</a>.
+     * @param        string      (optional) the order number. The larger it is, the lower the position it gets.
+     * @param        string      (optional) if set to false, the menu title will not be listed in the tab navigation menu at the top of the page.
+     * @access       public
+     * @return       void
+     * @internal
+     */    
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.

Spacing +9 added lines, -9 removed lines

@@ -50,7 +50,7 @@ 
      * @param       string      (optional) the position number that is passed to the <var>$position</var> parameter of the <a href="http://codex.wordpress.org/Function_Reference/add_menu_page">add_menu_page()</a> function.
      * @return      void
      */
-    public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16=null, $iMenuPosition=null ) {
+    public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16 = null, $iMenuPosition = null ) {
 
         $sRootMenuLabel = trim( $sRootMenuLabel );
         $_sSlug         = $this->_isBuiltInMenuItem( $sRootMenuLabel ); // if true, this method returns the slug
@@ -98,8 +98,8 @@ 
      */ 
     public function setRootMenuPageBySlug( $sRootMenuSlug ) {
         
-        $this->oProp->aRootMenu['sPageSlug']    = $sRootMenuSlug; // do not sanitize the slug here because post types includes a question mark.
-        $this->oProp->aRootMenu['fCreateRoot']  = false; // indicates to use an existing menu item. 
+        $this->oProp->aRootMenu[ 'sPageSlug' ]    = $sRootMenuSlug; // do not sanitize the slug here because post types includes a question mark.
+        $this->oProp->aRootMenu[ 'fCreateRoot' ]  = false; // indicates to use an existing menu item. 
         
     }
     
@@ -137,7 +137,7 @@ 
     * @access       public
     * @return       void
     */     
-    public function addSubMenuItems( $aSubMenuItem1, $aSubMenuItem2=null, $_and_more=null ) {
+    public function addSubMenuItems( $aSubMenuItem1, $aSubMenuItem2 = null, $_and_more = null ) {
         foreach ( func_get_args() as $_aSubMenuItem ) {
             $this->addSubMenuItem( $_aSubMenuItem );     
         }
@@ -214,16 +214,16 @@ 
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.
-        if ( ! isset( $aSubMenuLink['href'], $aSubMenuLink['title'] ) ) { 
+        if ( !isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
             return; 
         }
         
         // If the set URL is not valid, return.
-        if ( ! filter_var( $aSubMenuLink['href'], FILTER_VALIDATE_URL ) ) { 
+        if ( !filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
             return; 
         }
 
-        $_oFormatter   = new AdminPageFramework_Format_SubMenuLink( 
+        $_oFormatter = new AdminPageFramework_Format_SubMenuLink( 
             $aSubMenuLink, 
             $this 
         );
@@ -312,11 +312,11 @@ 
      */ 
     public function addSubMenuPage( array $aSubMenuPage ) {
 
-        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
+        if ( !isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
             return; 
         }
             
-        $_oFormatter   = new AdminPageFramework_Format_SubMenuPage( 
+        $_oFormatter = new AdminPageFramework_Format_SubMenuPage( 
             $aSubMenuPage,
             $this 
         );

Braces +3 added lines, -3 removed lines

@@ -215,12 +215,12 @@ 
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.
-        if ( ! isset( $aSubMenuLink['href'], $aSubMenuLink['title'] ) ) { 
+        if ( ! isset( $aSubMenuLink['href'], $aSubMenuLink['title'] ) ) {
             return; 
         }
         
         // If the set URL is not valid, return.
-        if ( ! filter_var( $aSubMenuLink['href'], FILTER_VALIDATE_URL ) ) { 
+        if ( ! filter_var( $aSubMenuLink['href'], FILTER_VALIDATE_URL ) ) {
             return; 
         }
 
@@ -313,7 +313,7 @@ 
      */ 
     public function addSubMenuPage( array $aSubMenuPage ) {
 
-        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
+        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) {
             return; 
         }
             

development/factory/admin_page/AdminPageFramework_Controller_Page.php

Doc Comments +4 added lines, -36 removed lines

@@ -52,42 +52,6 @@ 
      * @since       2.0.0
      * @since       3.0.0     Changed the scope to public. Added page slug target support. 
      * @since       3.3.1       Moved from `AdminPageFramework_Page`.
-     * @param       array     $aTab1 The in-page tab array.
-     * <h4>In-Page Tab Array</h4>
-     * <ul>
-     *     <li>**page_slug** - (string) the page slug that the tab belongs to.</li>
-     *     <li>**tab_slug** -  (string) the tab slug. Non-alphabetical characters should not be used including dots(.) and hyphens(-).</li>
-     *     <li>**title** - (string) the title of the tab.</li>
-     *     <li>**order** - (optional, integer) the order number of the tab. The lager the number is, the lower the position it is placed in the menu.</li>
-     *     <li>**show_in_page_tab** - (optional, boolean) default: `true`. If this is set to `false`, the tab title will not be displayed in the tab navigation menu; however, it is still accessible from the direct URL.</li>
-     *     <li>**parent_tab_slug** - (optional, string) this needs to be set if the above `show_in_page_tab` argument is `false` so that the parent tab will be emphasized as active when the hidden page is accessed.</li>
-     *     <li>**url** - [3.5.0+] (optional, string) If this is set, the link url of the navigation tab will be this url. Use this to create link only tab.</li>
-     *     <li>**capability** - [3.6.0+] (optional, string) The capability level.</li>
-     *     <li>**if** - [3.6.0+] (optional, boolean) Whether the tab should be displayed or not.</li>
-     *     <li>style - [3.6.3+] (optional) (string|array) The path or url of a stylesheet which gets loaded in the head tag. Or inline CSS rules.
-     * When custom arguments need to be set such as whether it should be inserted in the footer, set an array holding the following arguments.
-     *          <ul>
-     *              <li>src - (required, string) the source url or path.</li>    
-     *              <li>handle_id - (optional, string) The handle ID of the stylesheet.</li>    
-     *              <li>dependencies - (optional, array) The dependency array.</li>    
-     *              <li>version - (optional, string) The stylesheet version number.</li>    
-     *              <li>media - (optional, string) the description of the field which is inserted into the after the input field tag.</li>    
-     *          </ul>
-     *     </li>
-     *     <li>script - [3.6.3+] (optional) (string|array) The path or url of a JavaScript script which gets loaded in the head tag. Or an inline JavaScript script.
-     * When custom arguments need to be set such as whether it should be inserted in the footer, set an array holding the following arguments.
-     *          <ul>
-     *              <li>src - (required, string) the source url or path.</li>    
-     *              <li>handle_id - (optional, string) The handle ID of the stylesheet.</li>    
-     *              <li>dependencies - (optional, array) The dependency array.</li>    
-     *              <li>version - (optional, string) The stylesheet version number.</li>    
-     *              <li>translation - (optional, array) an array holding translation key-value pairs.</li>    
-     *          </ul>
-     *     </li>
-     * </ul>
-     * @param       array       $aTab2      Another in-page tab array.
-     * @param       array       $_and_more  (optional) Add in-page tab arrays as many as necessary to the next parameters.
-     * @param       string      $sPageSlug  (optional) If the passed parameter item is a string, it will be stored as the target page slug so that it will be applied to the next passed tab arrays as the page_slug element.
      * @remark      Accepts variadic parameters; the number of accepted parameters are not limited to three.
      * @remark      In-page tabs are different from page-heading tabs which is automatically added with page titles.  
      * @return      void
@@ -277,6 +241,10 @@ 
         /**
          * Sets a page property.
          * @since       3.6.0
+         * @param string $sPropertyName
+         * @param string $sPropertyKey
+         * @param boolean|string $mValue
+         * @param string $sPageSlug
          * @return      void
          */
         private function _setPageProperty( $sPropertyName, $sPropertyKey, $mValue, $sPageSlug ) {

Spacing +14 added lines, -14 removed lines

@@ -93,7 +93,7 @@ 
      * @return      void
      */             
     public function addInPageTabs( /* $aTab1, $aTab2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $asTab ) { 
+        foreach ( func_get_args() as $asTab ) { 
             $this->addInPageTab( $asTab ); 
         }
     }
@@ -115,7 +115,7 @@ 
         
         // Target page slug - will be applied when no page slug is specified.
         static $__sTargetPageSlug; 
-        if ( ! is_array( $asInPageTab ) ) {
+        if ( !is_array( $asInPageTab ) ) {
             $__sTargetPageSlug = is_string( $asInPageTab ) 
                 ? $asInPageTab 
                 : $__sTargetPageSlug; // set the target page slug
@@ -130,23 +130,23 @@ 
         );
         
         // Set the target page slug for next calls
-        $__sTargetPageSlug  = $aInPageTab[ 'page_slug' ]; 
+        $__sTargetPageSlug = $aInPageTab[ 'page_slug' ]; 
 
         // Required keys
-        if ( ! isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) { 
+        if ( !isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) { 
             return; 
         }
         
         // Count - the number of added in-page tabs.
-        $_aElements         = $this->oUtil->getElement(
+        $_aElements = $this->oUtil->getElement(
             $this->oProp->aInPageTabs,
             $aInPageTab[ 'page_slug' ],
             array()
         );
-        $_iCountElement      = count( $_aElements );
+        $_iCountElement = count( $_aElements );
 
         // Pre-format
-        $aInPageTab         = array( 
+        $aInPageTab = array( 
             'page_slug' => $this->oUtil->sanitizeSlug( $aInPageTab[ 'page_slug' ] ),
             'tab_slug'  => $this->oUtil->sanitizeSlug( $aInPageTab[ 'tab_slug' ] ),
             'order'     => $this->oUtil->getAOrB(
@@ -174,7 +174,7 @@ 
      * @param       boolean     $bShow If false, the page title will not be displayed.
      * @return      void
      */ 
-    public function setPageTitleVisibility( $bShow=true, $sPageSlug='' ) {
+    public function setPageTitleVisibility( $bShow = true, $sPageSlug = '' ) {
         $this->_setPageProperty( 
             'bShowPageTitle', 
             'show_page_title', 
@@ -198,7 +198,7 @@ 
      * @remark      Page-heading tabs and in-page tabs are different. The former displays page titles and the latter displays tab titles.
      * @remark      If the second parameter is omitted, it sets the default value.
      */ 
-    public function setPageHeadingTabsVisibility( $bShow=true, $sPageSlug='' ) {
+    public function setPageHeadingTabsVisibility( $bShow = true, $sPageSlug = '' ) {
         $this->_setPageProperty( 
             'bShowPageHeadingTabs', 
             'show_page_heading_tabs', 
@@ -219,7 +219,7 @@ 
      * @param       string      $sPageSlug  The page to apply the visibility setting. If not set, it applies to all the pages.
      * @remark      If the second parameter is omitted, it sets the default value.
      */
-    public function setInPageTabsVisibility( $bShow=true, $sPageSlug='' ) {
+    public function setInPageTabsVisibility( $bShow = true, $sPageSlug = '' ) {
         $this->_setPageProperty( 
             'bShowInPageTabs', 
             'show_in_page_tabs', 
@@ -242,7 +242,7 @@ 
      * @param       string      $sPageSlug      The page slug that applies the setting.    
      * @remark      If the second parameter is omitted, it sets the default value.
      */     
-    public function setInPageTabTag( $sTag='h3', $sPageSlug='' ) {      
+    public function setInPageTabTag( $sTag = 'h3', $sPageSlug = '' ) {      
         $this->_setPageProperty( 
             'sInPageTabTag', 
             'in_page_tab_tag', 
@@ -265,9 +265,9 @@ 
      * @param       string      $sPageSlug  The page slug that applies the setting.    
      * @remark      If the second parameter is omitted, it sets the default value.
      */
-    public function setPageHeadingTabTag( $sTag='h2', $sPageSlug='' ) {
+    public function setPageHeadingTabTag( $sTag = 'h2', $sPageSlug = '' ) {
         $this->_setPageProperty( 
-            'sPageHeadingTabTag',   // property name
+            'sPageHeadingTabTag', // property name
             'page_heading_tab_tag', // property key
             $sTag, // value
             $sPageSlug  // page slug
@@ -289,7 +289,7 @@ 
             
             $this->oProp->{$sPropertyName} = $mValue;
        
-            foreach( $this->oProp->aPages as &$_aPage ) {
+            foreach ( $this->oProp->aPages as &$_aPage ) {
                 $_aPage[ $sPropertyKey ] = $mValue;
 
             }                        

Braces +4 added lines, -4 removed lines

@@ -93,7 +93,7 @@ 
      * @return      void
      */             
     public function addInPageTabs( /* $aTab1, $aTab2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $asTab ) { 
+        foreach( func_get_args() as $asTab ) {
             $this->addInPageTab( $asTab ); 
         }
     }
@@ -111,7 +111,7 @@ 
      * @remark      In-page tabs are different from page-heading tabs which are automatically added with page titles.
      * @return      void
      */         
-    public function addInPageTab( $asInPageTab ) {    
+    public function addInPageTab( $asInPageTab ) {
         
         // Target page slug - will be applied when no page slug is specified.
         static $__sTargetPageSlug; 
@@ -133,7 +133,7 @@ 
         $__sTargetPageSlug  = $aInPageTab[ 'page_slug' ]; 
 
         // Required keys
-        if ( ! isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) { 
+        if ( ! isset( $aInPageTab[ 'page_slug' ], $aInPageTab[ 'tab_slug' ] ) ) {
             return; 
         }
         
@@ -242,7 +242,7 @@ 
      * @param       string      $sPageSlug      The page slug that applies the setting.    
      * @remark      If the second parameter is omitted, it sets the default value.
      */     
-    public function setInPageTabTag( $sTag='h3', $sPageSlug='' ) {      
+    public function setInPageTabTag( $sTag='h3', $sPageSlug='' ) {
         $this->_setPageProperty( 
             'sInPageTabTag', 
             'in_page_tab_tag', 

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 to handle importing options.

development/factory/admin_page/AdminPageFramework_Model_Menu.php

Doc Comments +1 added lines

@@ -24,6 +24,7 @@
      * Registers necessary callbacks and sets up properties.
      * 
      * @internal
+     * @param string $sCallerPath
      */
     public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
         

Braces +4 added lines, -4 removed lines

@@ -126,7 +126,7 @@ 
 
             foreach ( $this->oProp->aPages as $_aPage ) {
                 
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) { 
+                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
                     continue; 
                 }
                 $this->oProp->sDefaultPageSlug = $_aPage[ 'page_slug' ];
@@ -263,11 +263,11 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
                  * @return      void
                  */
-                private function _removePageSubmenuItem( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ){
+                private function _removePageSubmenuItem( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) {
                     
                     foreach( ( array ) $GLOBALS['submenu'][ $sMenuSlug ] as $_iIndex => $_aSubMenu ) {
                       
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) { 
+                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
                             continue; 
                         }
                                                
@@ -282,7 +282,7 @@ 
                             $sPageTitle,
                             $sPageSlug,
                         );
-                        if ( $_aA !== $_aB ) { 
+                        if ( $_aA !== $_aB ) {
                             continue;
                         }
 

Spacing +46 added lines, -46 removed lines

@@ -25,7 +25,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 );
         
@@ -126,7 +126,7 @@ 
 
             foreach ( $this->oProp->aPages as $_aPage ) {
                 
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) { 
+                if ( !isset( $_aPage[ 'page_slug' ] ) ) { 
                     continue; 
                 }
                 $this->oProp->sDefaultPageSlug = $_aPage[ 'page_slug' ];
@@ -144,12 +144,12 @@ 
          */ 
         private function _registerRootMenuPage() {
             $this->oProp->aRootMenu[ '_page_hook' ] = add_menu_page(  
-                $this->oProp->sClassName,                 // Page title - will be invisible anyway
-                $this->oProp->aRootMenu[ 'sTitle' ],      // Menu title - should be the root page title.
-                $this->oProp->sCapability,                // Capability - access right
-                $this->oProp->aRootMenu[ 'sPageSlug' ],   // Menu ID 
-                '',                                       // Page content displaying function - the root page will be removed so no need to register a function.
-                $this->oProp->aRootMenu[ 'sIcon16x16' ],  // icon path
+                $this->oProp->sClassName, // Page title - will be invisible anyway
+                $this->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title.
+                $this->oProp->sCapability, // Capability - access right
+                $this->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID 
+                '', // Page content displaying function - the root page will be removed so no need to register a function.
+                $this->oProp->aRootMenu[ 'sIcon16x16' ], // icon path
                 $this->oUtil->getElement( 
                     $this->oProp->aRootMenu, 
                     'iPosition', 
@@ -173,32 +173,32 @@ 
          */ 
         private function _registerSubMenuItem( array $aArgs ) {
 
-            if ( ! current_user_can( $aArgs['capability'] ) ) {
+            if ( !current_user_can( $aArgs[ 'capability' ] ) ) {
                 return '';
             }
                  
-            $_sRootPageSlug = $this->oProp->aRootMenu['sPageSlug'];
+            $_sRootPageSlug = $this->oProp->aRootMenu[ 'sPageSlug' ];
             $_sMenuSlug     = plugin_basename( $_sRootPageSlug ); // to be compatible with add_submenu_page()
             
-            switch( $aArgs['type'] ) {
+            switch ( $aArgs[ 'type' ] ) {
                 case 'page':
                     // it's possible that the page_slug key is not set if the user uses a method like setPageHeadingTabsVisibility() prior to addSubMenuItam().
                     return $this->_addPageSubmenuItem(
                         $_sRootPageSlug,
                         $_sMenuSlug,
-                        $aArgs['page_slug'],
-                        $this->oUtil->getElement( $aArgs, 'page_title', $aArgs['title'] ),
-                        $this->oUtil->getElement( $aArgs, 'menu_title', $aArgs['title'] ),
-                        $aArgs['capability'],
-                        $aArgs['show_in_menu']
+                        $aArgs[ 'page_slug' ],
+                        $this->oUtil->getElement( $aArgs, 'page_title', $aArgs[ 'title' ] ),
+                        $this->oUtil->getElement( $aArgs, 'menu_title', $aArgs[ 'title' ] ),
+                        $aArgs[ 'capability' ],
+                        $aArgs[ 'show_in_menu' ]
                     );
                 case 'link':
                     return $this->_addLinkSubmenuItem( 
                         $_sMenuSlug, 
-                        $aArgs['title'], 
-                        $aArgs['capability'],
-                        $aArgs['href'],
-                        $aArgs['show_in_menu']
+                        $aArgs[ 'title' ], 
+                        $aArgs[ 'capability' ],
+                        $aArgs[ 'href' ],
+                        $aArgs[ 'show_in_menu' ]
                     );
                 default:
                     return '';
@@ -214,43 +214,43 @@ 
              */
             private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu ) {
                 
-                if ( ! $sPageSlug ) {
+                if ( !$sPageSlug ) {
                     return '';
                 }
                 $_sPageHook = add_submenu_page( 
-                    $sRootPageSlug,         // the root (parent) page slug
-                    $sPageTitle,            // page title
-                    $sMenuTitle,            // menu title
-                    $sCapability,           // capability
-                    $sPageSlug,             // menu slug
+                    $sRootPageSlug, // the root (parent) page slug
+                    $sPageTitle, // page title
+                    $sMenuTitle, // menu title
+                    $sCapability, // capability
+                    $sPageSlug, // menu slug
                     // In admin.php ( line 149 of WordPress v3.6.1 ), do_action($page_hook) ( where $page_hook is $_sPageHook )
                     // will be executed and it triggers the __call() magic method with the method name of "md5 class hash + _page_ + this page slug".
-                    array( $this, $this->oProp->sClassHash . '_page_' . $sPageSlug )
+                    array( $this, $this->oProp->sClassHash.'_page_'.$sPageSlug )
                 );     
                 
                 // Ensure only it is added one time per page slug.
-                if ( ! isset( $this->oProp->aPageHooks[ $_sPageHook ] ) ) {
+                if ( !isset( $this->oProp->aPageHooks[ $_sPageHook ] ) ) {
                     // 3.4.1+ Give a lower priority as the page meta box class also hooks the current_screen to register form elements.
                     // When the validation callback is triggered, their form registration should be done already. So this hook should be loaded later than them.
-                    add_action( 'current_screen' , array( $this, "load_pre_" . $sPageSlug ), 20 );
+                    add_action( 'current_screen', array( $this, "load_pre_".$sPageSlug ), 20 );
                     
                     // 3.6.3+
                     // It is possible that an in-page tab is added during the above hooks and the current page is the default tab without the tab GET query key in the url. 
                     // Set a low priority because the user may add in-page tabs in their callback method of this action hook.
-                    add_action( "load_" . $sPageSlug, array( $this, '_replyToFinalizeInPageTabs' ), 9999 );
+                    add_action( "load_".$sPageSlug, array( $this, '_replyToFinalizeInPageTabs' ), 9999 );
                     
                     // 3.6.3+
-                    add_action( "load_after_" . $sPageSlug, array( $this, '_replyToEnqueuePageAssets' ) );
+                    add_action( "load_after_".$sPageSlug, array( $this, '_replyToEnqueuePageAssets' ) );
                     
                 }
                 $this->oProp->aPageHooks[ $sPageSlug ] = $this->oUtil->getAOrB(
                     is_network_admin(),
-                    $_sPageHook . '-network',
+                    $_sPageHook.'-network',
                     $_sPageHook
                 );
                 
                 // If the visibility option is false, remove the one just added from the sub-menu array
-                if ( ! $bShowInMenu ) {
+                if ( !$bShowInMenu ) {
                     $this->_removePageSubmenuItem( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug );
                 }                
                 return $_sPageHook;
@@ -263,11 +263,11 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
                  * @return      void
                  */
-                private function _removePageSubmenuItem( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ){
+                private function _removePageSubmenuItem( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) {
                     
-                    foreach( ( array ) $GLOBALS['submenu'][ $sMenuSlug ] as $_iIndex => $_aSubMenu ) {
+                    foreach ( ( array ) $GLOBALS[ 'submenu' ][ $sMenuSlug ] as $_iIndex => $_aSubMenu ) {
                       
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) { 
+                        if ( !isset( $_aSubMenu[ 3 ] ) ) { 
                             continue; 
                         }
                                                
@@ -318,15 +318,15 @@ 
                         
                         // If it is in the network admin area, do not remove the menu; otherwise, it gets not accessible. 
                         if ( is_network_admin() ) {
-                            unset( $GLOBALS['submenu'][ $sMenuSlug ][ $_iIndex ] );
+                            unset( $GLOBALS[ 'submenu' ][ $sMenuSlug ][ $_iIndex ] );
                             return;
                         } 
                         
                         if ( 
-                            ! isset( $_GET['page'] ) 
-                            || isset( $_GET['page'] ) && $sPageSlug != $_GET['page'] 
+                            !isset( $_GET[ 'page' ] ) 
+                            || isset( $_GET[ 'page' ] ) && $sPageSlug != $_GET[ 'page' ] 
                         ) {
-                            unset( $GLOBALS['submenu'][ $sMenuSlug ][ $_iIndex ] );
+                            unset( $GLOBALS[ 'submenu' ][ $sMenuSlug ][ $_iIndex ] );
                         }                        
                         
                     }
@@ -339,13 +339,13 @@ 
              * @return      void
              */
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu ) {
-                if ( ! $bShowInMenu ) {
+                if ( !$bShowInMenu ) {
                     return;
                 }
-                if ( ! isset( $GLOBALS['submenu'][ $sMenuSlug ] ) ) {
-                    $GLOBALS['submenu'][ $sMenuSlug ] = array();
+                if ( !isset( $GLOBALS[ 'submenu' ][ $sMenuSlug ] ) ) {
+                    $GLOBALS[ 'submenu' ][ $sMenuSlug ] = array();
                 }
-                $GLOBALS['submenu'][ $sMenuSlug ][] = array ( 
+                $GLOBALS[ 'submenu' ][ $sMenuSlug ][ ] = array( 
                     $sTitle, 
                     $sCapability, 
                     $sHref,
@@ -360,8 +360,8 @@ 
          * @internal
          */
         public function _replyToFixPageTitleForHiddenPages( $sAdminTitle, $sPageTitle ) {
-            if ( isset( $_GET['page'], $this->oProp->aHiddenPages[ $_GET['page'] ] ) ) {
-                return $this->oProp->aHiddenPages[ $_GET['page'] ] . $sAdminTitle;
+            if ( isset( $_GET[ 'page' ], $this->oProp->aHiddenPages[ $_GET[ 'page' ] ] ) ) {
+                return $this->oProp->aHiddenPages[ $_GET[ 'page' ] ].$sAdminTitle;
             }    
             return $sAdminTitle;     
         }  

development/factory/admin_page/form/AdminPageFramework_Form_admin_page.php

Doc Comments +1 added lines

@@ -455,6 +455,7 @@
      * @internal
      * @since       3.5.3
      * @since       DEVVER      Moved from `AdminPageFramework_FormDefinition_Page`.
+     * @param string $sTabSlug
      * @return      boolean
      */
     private function _isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) {

Spacing +19 added lines, -19 removed lines

@@ -52,16 +52,16 @@ 
     public function getPageOnlyOptions( $aOptions, $sPageSlug ) {
 
         $_aStoredOptionsOfThePage = array();
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // Check the section
-            if ( ! $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
+            if ( !$this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
                 continue;
             }
 
             // At this point, the element belongs the given page slug as the section is of the given page slug's.
             $this->_setPageOnlyOptions( 
-                $_aStoredOptionsOfThePage,  // by reference - gets updated in the method.
+                $_aStoredOptionsOfThePage, // by reference - gets updated in the method.
                 $aOptions, 
                 $_aSubSectionsOrFields, 
                 $sPageSlug,
@@ -81,7 +81,7 @@ 
          * @internal
          */
         private function _setPageOnlyOptions( array &$_aStoredOptionsOfThePage, array $aOptions, array $_aSubSectionsOrFields, $sPageSlug, $_sSectionID ) {
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
                 
                 // If it's a sub-section array,
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -153,7 +153,7 @@ 
     public function getOtherPageOptions( $aOptions, $sPageSlug ) {
 
         $_aStoredOptionsNotOfThePage = array();
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // Check the section
             if ( $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
@@ -182,7 +182,7 @@ 
          * @internal
          */
         private function _setOtherPageOptions( array &$_aStoredOptionsNotOfThePage, array $aOptions, array $_aSubSectionsOrFields, $sPageSlug ) {
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
 
                 // It's a sub-section array. 
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -245,7 +245,7 @@ 
     public function getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug ) {
 
         $_aStoredOptionsNotOfTheTab = array();
-        foreach( $this->aFieldsets as $_sSectionPath => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionPath => $_aSubSectionsOrFields ) {
                         
             // If the section is of the given page and the given tab, skip.
             if ( $this->_isThisSectionSetToThisTab( $_sSectionPath, $sPageSlug, $sTabSlug ) ) {
@@ -276,7 +276,7 @@ 
         private function _setOtherTabOptions( array &$_aStoredOptionsNotOfTheTab, array $aOptions, array $_aSubSectionsOrFields, $sSectionPath ) {
             
             // At this point, the passed element belongs to the other tabs since the section of the given tab is skipped.
-            foreach ( $_aSubSectionsOrFields as $_isSubSectionIndexOrFieldID => $_aSubSectionOrField  ) {
+            foreach ( $_aSubSectionsOrFields as $_isSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
                 
                 // If it's a sub section
                 if ( $this->isNumericInteger( $_isSubSectionIndexOrFieldID ) ) {
@@ -333,7 +333,7 @@ 
      * @since       DEVVER      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {     
+    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug = '' ) {     
         $_aOtherTabOptions = $this->getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug );
         return $this->invertCastArrayContents( $aOptions, $_aOtherTabOptions );     
     }
@@ -348,17 +348,17 @@ 
      * @since       DEVVER      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug='' ) {
+    public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug = '' ) {
         
         $_aStoredOptionsOfTheTab = array();
-        if ( ! $sTabSlug ) { 
+        if ( !$sTabSlug ) { 
             return $_aStoredOptionsOfTheTab; 
         }
         
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
              
             // Check the section
-            if ( ! $this->_isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) ) {
+            if ( !$this->_isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) ) {
                 continue;
             }
             
@@ -384,7 +384,7 @@ 
          */
         private function _setTabOnlyOptions( array &$_aStoredOptionsOfTheTab, array $aOptions, array $_aSubSectionsOrFields, $_sSectionID ) {
             
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
                                 
                 // if it's a sub-section array.
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -441,7 +441,7 @@ 
      */
     private function _isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) {
         
-        if ( ! isset( $this->aSectionsets[ $_sSectionID ][ 'page_slug' ] ) ) {
+        if ( !isset( $this->aSectionsets[ $_sSectionID ][ 'page_slug' ] ) ) {
             return false;
         }
         return ( 
@@ -459,10 +459,10 @@ 
      */
     private function _isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) {
         
-        if ( ! $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
+        if ( !$this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
             return false;
         }
-        if ( ! isset( $this->aSectionsets[ $_sSectionID ][ 'tab_slug' ] ) ) {
+        if ( !isset( $this->aSectionsets[ $_sSectionID ][ 'tab_slug' ] ) ) {
             return false;
         }
         return (
@@ -479,9 +479,9 @@ 
      */
     private function _setOptionValue( &$aSubject, $asDimensionalPath, $aOptions ) {
         $_aDimensionalPath = $this->getAsArray( $asDimensionalPath );
-        $_mValue     = $this->getElement(
+        $_mValue = $this->getElement(
             $aOptions,
-            $_aDimensionalPath,    // as of DEVVER, it can be an array or string
+            $_aDimensionalPath, // as of DEVVER, it can be an array or string
             null
         );
         if ( isset( $_mValue ) ) {

Braces +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
                 );
                 // @todo Examine whether this check can be removed 
                 // as the section that hods this field is already checked above outside this loop.
-                if ( $sPageSlug !== $_aFieldset[ 'page_slug' ] ) { 
+                if ( $sPageSlug !== $_aFieldset[ 'page_slug' ] ) {
                     continue; 
                 }        
                 
@@ -333,7 +333,7 @@ 
      * @since       DEVVER      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {     
+    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {
         $_aOtherTabOptions = $this->getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug );
         return $this->invertCastArrayContents( $aOptions, $_aOtherTabOptions );     
     }
@@ -351,7 +351,7 @@ 
     public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug='' ) {
         
         $_aStoredOptionsOfTheTab = array();
-        if ( ! $sTabSlug ) { 
+        if ( ! $sTabSlug ) {
             return $_aStoredOptionsOfTheTab; 
         }
         

development/factory/meta_box/_controller/AdminPageFramework_HelpPane_MetaBox.php

Doc Comments +1 added lines

@@ -43,6 +43,7 @@
      * @since       2.1.0
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the <em>replyToRegisterHelpTabTextForMetaBox()</em> method.
      * @internal
+     * @param string $sHTMLContent
      */ 
     public function _addHelpText( $sHTMLContent, $sHTMLSidebarContent="" ) {
         

Spacing +9 added lines, -9 removed lines

@@ -44,10 +44,10 @@ 
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the <em>replyToRegisterHelpTabTextForMetaBox()</em> method.
      * @internal
      */ 
-    public function _addHelpText( $sHTMLContent, $sHTMLSidebarContent="" ) {
+    public function _addHelpText( $sHTMLContent, $sHTMLSidebarContent = "" ) {
         
-        $this->oProp->aHelpTabText[]        = "<div class='contextual-help-description'>" . $sHTMLContent . "</div>";
-        $this->oProp->aHelpTabTextSide[]    = "<div class='contextual-help-description'>" . $sHTMLSidebarContent . "</div>";
+        $this->oProp->aHelpTabText[ ]        = "<div class='contextual-help-description'>".$sHTMLContent."</div>";
+        $this->oProp->aHelpTabTextSide[ ]    = "<div class='contextual-help-description'>".$sHTMLSidebarContent."</div>";
         
     }
     
@@ -62,9 +62,9 @@ 
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the `replyToRegisterHelpTabTextForMetaBox()` method.
      * @internal
      */     
-    public function _addHelpTextForFormFields( $sFieldTitle, $sHelpText, $sHelpTextSidebar="" ) {
+    public function _addHelpTextForFormFields( $sFieldTitle, $sHelpText, $sHelpTextSidebar = "" ) {
         $this->_addHelpText(
-            "<span class='contextual-help-tab-title'>" . $sFieldTitle . "</span> - " . PHP_EOL
+            "<span class='contextual-help-tab-title'>".$sFieldTitle."</span> - ".PHP_EOL
                 . $sHelpText,     
             $sHelpTextSidebar
         );     
@@ -81,7 +81,7 @@ 
     public function _replyToRegisterHelpTabTextForMetaBox() {
 
         // Check if the currently loaded page is of meta box page.
-        if ( ! $this->_isInThePage() ) { return false; }
+        if ( !$this->_isInThePage() ) { return false; }
 
         $this->_setHelpTab(     // defined in the base class.
             $this->oProp->sMetaBoxID, 
@@ -100,11 +100,11 @@ 
      */
     protected function _isInThePage() {
         
-        if ( ! $this->oProp->bIsAdmin ) { return false; }
-        if ( ! in_array( $this->oProp->sPageNow, array( 'post.php', 'post-new.php' ) ) ) {
+        if ( !$this->oProp->bIsAdmin ) { return false; }
+        if ( !in_array( $this->oProp->sPageNow, array( 'post.php', 'post-new.php' ) ) ) {
             return false;
         }
-        if ( ! in_array( $this->oUtil->getCurrentPostType(), $this->oProp->aPostTypes ) ) {
+        if ( !in_array( $this->oUtil->getCurrentPostType(), $this->oProp->aPostTypes ) ) {
             return false;     
         }    
         return true;

development/factory/page_meta_box/_model/AdminPageFramework_Property_MetaBox_Page.php

Doc Comments +4 added lines

@@ -53,6 +53,10 @@
      */
     public $_sFormRegistrationHook = 'admin_enqueue_scripts';
     
+    /**
+     * @param AdminPageFramework_PageMetaBox_Model $oCaller
+     * @param string $sClassName
+     */
     function __construct( $oCaller, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='page_meta_box' ) {     
         
         // this must be done after the menu class finishes building the menu with the _replyToBuildMenu() method.

Indentation +1 added lines, -1 removed lines

@@ -170,7 +170,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 isset( $_GET[ 'page' ] ) 
             ? $_GET[ 'page' ] 

Spacing +9 added lines, -9 removed lines

@@ -53,7 +53,7 @@ 
      */
     public $_sFormRegistrationHook = 'admin_enqueue_scripts';
     
-    function __construct( $oCaller, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='page_meta_box' ) {     
+    function __construct( $oCaller, $sClassName, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework', $sStructureType = 'page_meta_box' ) {     
         
         // this must be done after the menu class finishes building the menu with the _replyToBuildMenu() method.
         add_action( 'admin_menu', array( $this, '_replyToSetUpProperties' ), 100 ); 
@@ -91,12 +91,12 @@ 
      */
     public function _replyToSetUpProperties() {
         
-        if ( ! isset( $_GET[ 'page' ] ) ) { 
+        if ( !isset( $_GET[ 'page' ] ) ) { 
             return; 
         }
                 
         $this->oAdminPage = $this->_getOwnerObjectOfPage( $_GET[ 'page' ] );
-        if ( ! $this->oAdminPage ) { 
+        if ( !$this->oAdminPage ) { 
             return; 
         }
         
@@ -127,7 +127,7 @@ 
      */
     public function _getScreenIDOfPage( $sPageSlug ) {
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
-            ? $_oAdminPage->oProp->aPages[ $sPageSlug ][ '_page_hook' ] . ( is_network_admin() ? '-network' : '' )
+            ? $_oAdminPage->oProp->aPages[ $sPageSlug ][ '_page_hook' ].( is_network_admin() ? '-network' : '' )
             : '';
     }    
     
@@ -137,7 +137,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 = '' ) {    
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
             ? $_oAdminPage->oProp->isPageAdded( $sPageSlug )
             : false;
@@ -153,7 +153,7 @@ 
     public function isCurrentTab( $sTabSlug ) {
         
         $_sCurrentPageSlug = $this->getElement( $_GET, 'page' );
-        if ( ! $_sCurrentPageSlug ) { 
+        if ( !$_sCurrentPageSlug ) { 
             return false; 
         }
         $_sCurrentTabSlug = $this->getElement( 
@@ -203,7 +203,7 @@ 
      */         
     public function getDefaultInPageTab( $sPageSlug ) {
     
-        if ( ! $sPageSlug ) { 
+        if ( !$sPageSlug ) { 
             return ''; 
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -218,7 +218,7 @@ 
      */
     public function getOptionKey( $sPageSlug ) {
         
-        if ( ! $sPageSlug ) { 
+        if ( !$sPageSlug ) { 
             return ''; 
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -244,7 +244,7 @@ 
             $GLOBALS,
             array( 'aAdminPageFramework', 'aPageClasses' )
         );                 
-        foreach( $_aPageClasses as $_oAdminPage ) {
+        foreach ( $_aPageClasses as $_oAdminPage ) {
             if ( $_oAdminPage->oProp->isPageAdded( $sPageSlug ) ) {
                 return $_oAdminPage;
             }

Braces +8 added lines, -8 removed lines

@@ -53,11 +53,11 @@ 
      */
     public $_sFormRegistrationHook = 'admin_enqueue_scripts';
     
-    function __construct( $oCaller, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='page_meta_box' ) {     
+    function __construct( $oCaller, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='page_meta_box' ) {
         
         // this must be done after the menu class finishes building the menu with the _replyToBuildMenu() method.
         add_action( 'admin_menu', array( $this, '_replyToSetUpProperties' ), 100 ); 
-        if ( is_network_admin() ) { 
+        if ( is_network_admin() ) {
             add_action( 'network_admin_menu', array( $this, '_replyToSetUpProperties' ), 100 ); 
         }     
         
@@ -91,12 +91,12 @@ 
      */
     public function _replyToSetUpProperties() {
         
-        if ( ! isset( $_GET[ 'page' ] ) ) { 
+        if ( ! isset( $_GET[ 'page' ] ) ) {
             return; 
         }
                 
         $this->oAdminPage = $this->_getOwnerObjectOfPage( $_GET[ 'page' ] );
-        if ( ! $this->oAdminPage ) { 
+        if ( ! $this->oAdminPage ) {
             return; 
         }
         
@@ -137,7 +137,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='' ) {
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
             ? $_oAdminPage->oProp->isPageAdded( $sPageSlug )
             : false;
@@ -153,7 +153,7 @@ 
     public function isCurrentTab( $sTabSlug ) {
         
         $_sCurrentPageSlug = $this->getElement( $_GET, 'page' );
-        if ( ! $_sCurrentPageSlug ) { 
+        if ( ! $_sCurrentPageSlug ) {
             return false; 
         }
         $_sCurrentTabSlug = $this->getElement( 
@@ -203,7 +203,7 @@ 
      */         
     public function getDefaultInPageTab( $sPageSlug ) {
     
-        if ( ! $sPageSlug ) { 
+        if ( ! $sPageSlug ) {
             return ''; 
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )
@@ -218,7 +218,7 @@ 
      */
     public function getOptionKey( $sPageSlug ) {
         
-        if ( ! $sPageSlug ) { 
+        if ( ! $sPageSlug ) {
             return ''; 
         }
         return ( $_oAdminPage = $this->_getOwnerObjectOfPage( $sPageSlug ) )

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Model.php

Doc Comments +2 added lines

@@ -32,6 +32,8 @@
      * Sets up properties and hooks.
      * 
      * @since       3.0.4
+     * @param string $sMetaBoxID
+     * @param string $sTitle
      */
     public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {     
                 

Indentation +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
         );        
     }    
     
-       /**
+        /**
          * Extracts meta box form fields options array from the given options array of an admin page.
          * 
          * @since       3.5.6

Spacing +23 added lines, -23 removed lines

@@ -33,10 +33,10 @@ 
      * 
      * @since       3.0.4
      */
-    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' ) {     
                 
         // The property object needs to be done first before the parent constructor.
-        $this->oProp             = new AdminPageFramework_Property_MetaBox_Page( 
+        $this->oProp = new AdminPageFramework_Property_MetaBox_Page( 
             $this, 
             get_class( $this ), 
             $sCapability, 
@@ -81,11 +81,11 @@ 
     protected function _setUpValidationHooks( $oScreen ) {
 
         // Validation hooks
-        foreach( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
+        foreach ( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
             
             if ( is_string( $_asTabArrayOrPageSlug ) ) {     
                 $_sPageSlug = $_asTabArrayOrPageSlug;
-                add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 );  // 3.4.1+
+                add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 ); // 3.4.1+
                 add_filter( "validation_{$_sPageSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
                 add_filter( "options_update_status_{$_sPageSlug}", array( $this, '_replyToModifyOptionsUpdateStatus' ) );
                 continue;
@@ -94,7 +94,7 @@ 
             // At this point, the array key is the page slug. It means the user specified the tab(s).
             $_sPageSlug = $_sIndexOrPageSlug;
             $_aTabs     = $_asTabArrayOrPageSlug;
-            foreach( $_aTabs as $_sTabSlug ) {
+            foreach ( $_aTabs as $_sTabSlug ) {
                 add_filter( "validation_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
                 add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 ); // 3.4.1+
                 add_filter( "options_update_status_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToModifyOptionsUpdateStatus' ) );
@@ -113,9 +113,9 @@ 
      * @return      void
      * @callback    action      add_meta_boxes
      */ 
-    public function _replyToAddMetaBox( $sPageHook='' ) {
-        foreach( $this->oProp->aPageSlugs as $sKey => $_asPage ) {
-            if ( is_string( $_asPage ) )  {
+    public function _replyToAddMetaBox( $sPageHook = '' ) {
+        foreach ( $this->oProp->aPageSlugs as $sKey => $_asPage ) {
+            if ( is_string( $_asPage ) ) {
                 $this->_addMetaBox( $_asPage );
                 continue;
             }            
@@ -130,8 +130,8 @@ 
          * @return      void
          */
         private function _addMetaBoxes( $sPageSlug, $asPage ) {
-            foreach( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
-                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) { 
+            foreach ( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
+                if ( !$this->oProp->isCurrentTab( $_sTabSlug ) ) { 
                     continue; 
                 }
                 $this->_addMetaBox( $sPageSlug );
@@ -147,12 +147,12 @@ 
          */
         private function _addMetaBox( $sPageSlug ) {
             add_meta_box( 
-                $this->oProp->sMetaBoxID,                       // id
-                $this->oProp->sTitle,                           // title
+                $this->oProp->sMetaBoxID, // id
+                $this->oProp->sTitle, // title
                 array( $this, '_replyToPrintMetaBoxContents' ), // callback
                 $this->oProp->_getScreenIDOfPage( $sPageSlug ), // screen ID
-                $this->oProp->sContext,                         // context
-                $this->oProp->sPriority,                        // priority
+                $this->oProp->sContext, // context
+                $this->oProp->sPriority, // priority
                 null                                            // argument (deprecated)
             );
         }
@@ -200,17 +200,17 @@ 
         
         $_aNewMetaBoxInputs      = $this->oForm->getSubmittedData( $_POST );
         $_aOldMetaBoxInputs      = $this->oUtil->castArrayContents( 
-            $this->oForm->getDataStructureFromAddedFieldsets(),   // model
+            $this->oForm->getDataStructureFromAddedFieldsets(), // model
             $aOldPageOptions        // data source
         );
 
         // Apply filters - third party scripts will have access to the input.
         $_aNewMetaBoxInputsRaw   = $_aNewMetaBoxInputs; // copy one for validation errors.
         $_aNewMetaBoxInputs      = call_user_func_array( 
-            array( $this, 'validate' ),     // triggers __call()
+            array( $this, 'validate' ), // triggers __call()
             array( $_aNewMetaBoxInputs, $_aOldMetaBoxInputs, $this, $aSubmitInfo ) 
         ); // 3.5.3+
-        $_aNewMetaBoxInputs      = $this->oUtil->addAndApplyFilters( 
+        $_aNewMetaBoxInputs = $this->oUtil->addAndApplyFilters( 
             $this, 
             "validation_{$this->oProp->sClassName}", 
             $_aNewMetaBoxInputs, 
@@ -245,7 +245,7 @@ 
      */
     public function _replyToModifyOptionsUpdateStatus( $aStatus ) {
         
-        if ( ! $this->hasFieldError() ) {
+        if ( !$this->hasFieldError() ) {
             return $aStatus;
         }
         return array( 
@@ -268,11 +268,11 @@ 
     public function _replyToGetSavedFormData() {
         $_aPageOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->oProp->oAdminPage->oProp->aOptions  
         );
         return $this->oUtil->castArrayContents( 
-            $this->oForm->getDataStructureFromAddedFieldsets(),   // model
+            $this->oForm->getDataStructureFromAddedFieldsets(), // model
             $_aPageOptions        // data source
         );        
     }    
@@ -288,9 +288,9 @@ 
         private function _getPageMetaBoxOptionsFromPageOptions( array $aPageOptions, array $aFieldsets ) {
      
             $_aOptions = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
-                if ( '_default' === $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aField ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+                if ( '_default' === $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aField ) {
                         if ( array_key_exists( $_aField[ 'field_id' ], $aPageOptions ) ) {
                             $_aOptions[ $_aField[ 'field_id' ] ] = $aPageOptions[ $_aField[ 'field_id' ] ];
                         }

Braces +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
      * 
      * @since       3.0.4
      */
-    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' ) {
                 
         // The property object needs to be done first before the parent constructor.
         $this->oProp             = new AdminPageFramework_Property_MetaBox_Page( 
@@ -83,7 +83,7 @@ 
         // Validation hooks
         foreach( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
             
-            if ( is_string( $_asTabArrayOrPageSlug ) ) {     
+            if ( is_string( $_asTabArrayOrPageSlug ) ) {
                 $_sPageSlug = $_asTabArrayOrPageSlug;
                 add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 );  // 3.4.1+
                 add_filter( "validation_{$_sPageSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
@@ -115,7 +115,7 @@ 
      */ 
     public function _replyToAddMetaBox( $sPageHook='' ) {
         foreach( $this->oProp->aPageSlugs as $sKey => $_asPage ) {
-            if ( is_string( $_asPage ) )  {
+            if ( is_string( $_asPage ) ) {
                 $this->_addMetaBox( $_asPage );
                 continue;
             }            
@@ -131,7 +131,7 @@ 
          */
         private function _addMetaBoxes( $sPageSlug, $asPage ) {
             foreach( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
-                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) { 
+                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) {
                     continue; 
                 }
                 $this->_addMetaBox( $sPageSlug );

development/factory/post_type/AdminPageFramework_PostType_View.php

Doc Comments +1 added lines

@@ -112,6 +112,7 @@
              * 
              * @internal
              * @since       3.5.10
+             * @param string $sMenuKey
              * @return      void
              */
             private function _removeAddNewSidebarSubMenu( $sMenuKey, $sPostTypeSlug ) {

Spacing +23 added lines, -23 removed lines

@@ -76,7 +76,7 @@ 
             $this->_removeAddNewSidebarSubMenu(
                 is_string( $_bsShowInMenu )
                     ? $_bsShowInMenu
-                    : 'edit.php?post_type=' . $this->oProp->sPostType,
+                    : 'edit.php?post_type='.$this->oProp->sPostType,
                 $this->oProp->sPostType
             );
             
@@ -117,20 +117,20 @@ 
             private function _removeAddNewSidebarSubMenu( $sMenuKey, $sPostTypeSlug ) {
 
                 // Remove the default post type menu item.
-                if ( ! isset( $GLOBALS['submenu'][ $sMenuKey ] ) ) {
+                if ( !isset( $GLOBALS[ 'submenu' ][ $sMenuKey ] ) ) {
                     // logged-in users of an insufficient access level don't have the menu to be registered.
                     return; 
                 } 
                 
-                foreach ( $GLOBALS['submenu'][ $sMenuKey ] as $_iIndex => $_aSubMenu ) {
+                foreach ( $GLOBALS[ 'submenu' ][ $sMenuKey ] as $_iIndex => $_aSubMenu ) {
                                 
-                    if ( ! isset( $_aSubMenu[ 2 ] ) ) { 
+                    if ( !isset( $_aSubMenu[ 2 ] ) ) { 
                         continue; 
                     }
                     
                     // Remove the default Add New entry.
-                    if ( 'post-new.php?post_type=' . $sPostTypeSlug === $_aSubMenu[ 2 ] ) {
-                        unset( $GLOBALS['submenu'][ $sMenuKey ][ $_iIndex ] );
+                    if ( 'post-new.php?post_type='.$sPostTypeSlug === $_aSubMenu[ 2 ] ) {
+                        unset( $GLOBALS[ 'submenu' ][ $sMenuKey ][ $_iIndex ] );
                         continue;
                     }
                     
@@ -145,13 +145,13 @@ 
      */ 
     public function _replyToAddAuthorTableFilter() {
         
-        if ( ! $this->oProp->bEnableAuthorTableFileter ) { 
+        if ( !$this->oProp->bEnableAuthorTableFileter ) { 
             return; 
         }
         
         if ( 
-            ! ( isset( $_GET['post_type'] ) && post_type_exists( $_GET['post_type'] ) 
-            && in_array( strtolower( $_GET['post_type'] ), array( $this->oProp->sPostType ) ) ) 
+            !( isset( $_GET[ 'post_type' ] ) && post_type_exists( $_GET[ 'post_type' ] ) 
+            && in_array( strtolower( $_GET[ 'post_type' ] ), array( $this->oProp->sPostType ) ) ) 
         ) {
             return;
         }
@@ -161,7 +161,7 @@ 
                 'show_option_all'   => $this->oMsg->get( 'show_all_authors' ),
                 'show_option_none'  => false,
                 'name'              => 'author',
-                'selected'          => ! empty( $_GET[ 'author' ] ) 
+                'selected'          => !empty( $_GET[ 'author' ] ) 
                     ? $_GET[ 'author' ] 
                     : 0,
                 'include_selected'  => false,
@@ -187,9 +187,9 @@ 
             return;
         }
         
-        foreach ( get_object_taxonomies( $GLOBALS['typenow'] ) as $_sTaxonomySulg ) {
+        foreach ( get_object_taxonomies( $GLOBALS[ 'typenow' ] ) as $_sTaxonomySulg ) {
             
-            if ( ! in_array( $_sTaxonomySulg, $this->oProp->aTaxonomyTableFilters ) ) {
+            if ( !in_array( $_sTaxonomySulg, $this->oProp->aTaxonomyTableFilters ) ) {
                 continue;
             }
             
@@ -203,7 +203,7 @@ 
             // Echo the drop down list based on the passed array argument.
             wp_dropdown_categories( 
                 array(
-                    'show_option_all'   => $this->oMsg->get( 'show_all' ) . ' ' . $_oTaxonomy->label,
+                    'show_option_all'   => $this->oMsg->get( 'show_all' ).' '.$_oTaxonomy->label,
                     'taxonomy'          => $_sTaxonomySulg,
                     'name'              => $_oTaxonomy->name,
                     'orderby'           => 'name',
@@ -223,24 +223,24 @@ 
      * 
      * @internal
      */
-    public function _replyToGetTableFilterQueryForTaxonomies( $oQuery=null ) {
+    public function _replyToGetTableFilterQueryForTaxonomies( $oQuery = null ) {
         
         if ( 'edit.php' != $this->oProp->sPageNow ) { 
             return $oQuery; 
         }
         
-        if ( ! isset( $GLOBALS['typenow'] ) ) { 
+        if ( !isset( $GLOBALS[ 'typenow' ] ) ) { 
             return $oQuery; 
         }
 
-        foreach ( get_object_taxonomies( $GLOBALS['typenow'] ) as $sTaxonomySlug ) {
+        foreach ( get_object_taxonomies( $GLOBALS[ 'typenow' ] ) as $sTaxonomySlug ) {
             
-            if ( ! in_array( $sTaxonomySlug, $this->oProp->aTaxonomyTableFilters ) ) { 
+            if ( !in_array( $sTaxonomySlug, $this->oProp->aTaxonomyTableFilters ) ) { 
                 continue; 
             }
             
             $sVar = &$oQuery->query_vars[ $sTaxonomySlug ];
-            if ( ! isset( $sVar ) ) { 
+            if ( !isset( $sVar ) ) { 
                 continue; 
             }
             
@@ -268,14 +268,14 @@ 
         }
 
         // If the screen icon url is specified
-        if ( isset( $this->oProp->aPostTypeArgs['screen_icon'] ) && $this->oProp->aPostTypeArgs['screen_icon'] ) {
-            $this->oProp->sStyle .= $this->_getStylesForPostTypeScreenIcon( $this->oProp->aPostTypeArgs['screen_icon'] );
+        if ( isset( $this->oProp->aPostTypeArgs[ 'screen_icon' ] ) && $this->oProp->aPostTypeArgs[ 'screen_icon' ] ) {
+            $this->oProp->sStyle .= $this->_getStylesForPostTypeScreenIcon( $this->oProp->aPostTypeArgs[ 'screen_icon' ] );
         }
             
         $this->oProp->sStyle = $this->oUtil->addAndApplyFilters( $this, "style_{$this->oProp->sClassName}", $this->oProp->sStyle );
         
         // Print out the filtered styles.
-        if ( ! empty( $this->oProp->sStyle ) ) {
+        if ( !empty( $this->oProp->sStyle ) ) {
             echo "<style type='text/css' id='admin-page-framework-style-post-type'>" 
                 . $this->oProp->sStyle
                 . "</style>";     
@@ -326,10 +326,10 @@ 
      */
     public function _replyToFilterPostTypeContent( $sContent ) {
         
-        if ( ! is_singular() ) {
+        if ( !is_singular() ) {
             return $sContent;
         }
-        if ( ! is_main_query() ) {
+        if ( !is_main_query() ) {
             return $sContent;
         }
         global $post;

Braces +9 added lines, -9 removed lines

@@ -17,7 +17,7 @@ 
  * @package     AdminPageFramework
  * @subpackage  PostType
  */
-abstract class AdminPageFramework_PostType_View extends AdminPageFramework_PostType_Model {    
+abstract class AdminPageFramework_PostType_View extends AdminPageFramework_PostType_Model {
 
     /**
      * Sets up hooks.
@@ -29,7 +29,7 @@ 
         
         parent::__construct( $oProp );
                         
-        if ( $this->_isInThePage() ) {     
+        if ( $this->_isInThePage() ) {
     
             // Table filters
             add_action( 'restrict_manage_posts', array( $this, '_replyToAddAuthorTableFilter' ) );
@@ -124,7 +124,7 @@ 
                 
                 foreach ( $GLOBALS['submenu'][ $sMenuKey ] as $_iIndex => $_aSubMenu ) {
                                 
-                    if ( ! isset( $_aSubMenu[ 2 ] ) ) { 
+                    if ( ! isset( $_aSubMenu[ 2 ] ) ) {
                         continue; 
                     }
                     
@@ -145,7 +145,7 @@ 
      */ 
     public function _replyToAddAuthorTableFilter() {
         
-        if ( ! $this->oProp->bEnableAuthorTableFileter ) { 
+        if ( ! $this->oProp->bEnableAuthorTableFileter ) {
             return; 
         }
         
@@ -177,7 +177,7 @@ 
      */ 
     public function _replyToAddTaxonomyTableFilter() {
         
-        if ( $GLOBALS[ 'typenow' ] != $this->oProp->sPostType ) { 
+        if ( $GLOBALS[ 'typenow' ] != $this->oProp->sPostType ) {
             return; 
         }
         
@@ -225,22 +225,22 @@ 
      */
     public function _replyToGetTableFilterQueryForTaxonomies( $oQuery=null ) {
         
-        if ( 'edit.php' != $this->oProp->sPageNow ) { 
+        if ( 'edit.php' != $this->oProp->sPageNow ) {
             return $oQuery; 
         }
         
-        if ( ! isset( $GLOBALS['typenow'] ) ) { 
+        if ( ! isset( $GLOBALS['typenow'] ) ) {
             return $oQuery; 
         }
 
         foreach ( get_object_taxonomies( $GLOBALS['typenow'] ) as $sTaxonomySlug ) {
             
-            if ( ! in_array( $sTaxonomySlug, $this->oProp->aTaxonomyTableFilters ) ) { 
+            if ( ! in_array( $sTaxonomySlug, $this->oProp->aTaxonomyTableFilters ) ) {
                 continue; 
             }
             
             $sVar = &$oQuery->query_vars[ $sTaxonomySlug ];
-            if ( ! isset( $sVar ) ) { 
+            if ( ! isset( $sVar ) ) {
                 continue; 
             }
             

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_View.php

Doc Comments +1 added lines

@@ -107,6 +107,7 @@
          * 
          * @since       3.0.0
          * @internal
+         * @param boolean $bRenderTableRow
          * @return      string
          */
         private function _getFieldsOutput( $iTermID, $bRenderTableRow ) {

Indentation +1 added lines, -1 removed lines

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

Spacing +8 added lines, -8 removed lines

@@ -48,7 +48,7 @@ 
             '',
             "[{$_sKey}]"
         );        
-        return $_aField['field_id'] . $_sKey; 
+        return $_aField[ 'field_id' ].$_sKey; 
         
     }
     /**
@@ -66,7 +66,7 @@ 
             '',
             "|{$_sKey}"
         );
-        return "{$_aField['field_id']}{$_sKey}";
+        return "{$_aField[ 'field_id' ]}{$_sKey}";
     }
 
    /**
@@ -114,7 +114,7 @@ 
             $_aOutput = array();
             
             // Set nonce.           
-            $_aOutput[] = wp_nonce_field( 
+            $_aOutput[ ] = wp_nonce_field( 
                 $this->oProp->sClassHash, 
                 $this->oProp->sClassHash, 
                 true, 
@@ -125,17 +125,17 @@ 
             $this->_setOptionArray( $iTermID, $this->oProp->sOptionKey );
             
             // Get the field outputs
-            $_aOutput[] = $this->oForm->get();
+            $_aOutput[ ] = $this->oForm->get();
             
             // 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;
                        
@@ -154,8 +154,8 @@ 
     public function _replyToPrintColumnCell( $vValue, $sColumnSlug, $sTermID ) {
         
         $_sCellHTML = '';
-        if ( isset( $_GET['taxonomy'] ) && $_GET['taxonomy'] ) {
-            $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$_GET['taxonomy']}", $vValue, $sColumnSlug, $sTermID );
+        if ( isset( $_GET[ 'taxonomy' ] ) && $_GET[ 'taxonomy' ] ) {
+            $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$_GET[ 'taxonomy' ]}", $vValue, $sColumnSlug, $sTermID );
         }
         $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$this->oProp->sClassName}", $_sCellHTML, $sColumnSlug, $sTermID );
         $_sCellHTML = $this->oUtil->addAndApplyFilter( $this, "cell_{$this->oProp->sClassName}_{$sColumnSlug}", $_sCellHTML, $sTermID ); // 3.0.2+