adding the debug mode option to toggle debug messages.

leocornus · Mar 1, 2013 · 1b470c0 · 1b470c0
1 parent 406ecb9
commit 1b470c0
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 16 deletions.
diff --git a/wp-trac-client/admin-manager.php b/wp-trac-client/admin-manager.php
@@ -21,7 +21,7 @@
 ?>
 
 <?php
-if ($DEBUG) {
+if (wptc_is_debug()) {
   global $wptc_db_version;
   //wptc_create_tables();
   $file = __FILE__;

diff --git a/wp-trac-client/admin-settings.php b/wp-trac-client/admin-settings.php
@@ -1,7 +1,6 @@
 <?php
 
 wp_enqueue_script("wp_ajax_response");
-$DEBUG = False;
 
 if (isset($_POST['wptc_settings_form_submit']) &&
     $_POST['wptc_settings_form_submit'] == 'Y') {
@@ -10,6 +9,7 @@
     update_site_option('wptc_rpcurl', $_POST['wptc_rpcurl']);
     update_site_option('wptc_username', $_POST['wptc_username']);
     update_site_option('wptc_password', $_POST['wptc_password']);
+    update_site_option('wptc_debug', $_POST['wptc_debug']);
 
     // show the message.
     echo '<div class="updated"><p><strong>Settings Updated</strong></p></div>';
@@ -65,6 +65,19 @@
                    value="<?php echo get_site_option('wptc_password'); ?>" size="58"/>
         </td>
       </tr>
+      <tr>
+        <th scope="row">Debug Mode: </th>
+        <td>
+          <input type="radio" name="wptc_debug" value="true"
+            <?php echo (get_site_option('wptc_debug') == 'true') ?
+                'checked' : '';?>
+          >On
+          <input type="radio" name="wptc_debug" value="false"
+            <?php echo (get_site_option('wptc_debug') == 'false') ?
+                'checked' : '';?>
+          >Off
+        </td>
+      </tr>
       <tr>
         <th scope="row"><input type="submit" name="saveSetting" class="button-primary" value="Save Settings" />
         </th>
@@ -105,7 +118,7 @@
     </tbody></table>
   </form>
 
-<?php if ($DEBUG) {
+<?php if (wptc_is_debug()) {
     // some testing code.
 ?>
     <h3>Some temp Testing:</h3>

diff --git a/wp-trac-client/admin-testing.php b/wp-trac-client/admin-testing.php
@@ -38,6 +38,7 @@
             name="wptc_function">
             <?php //TODO: generate on the fly  ?>
             <option></option>
+            <option>wptc_is_debug</option>
             <option>wptc_get_project</option>
             <option>wptc_get_projects</option>
             <option>wptc_get_project_mandv</option>

diff --git a/wp-trac-client/templates/page-ticket-details.php b/wp-trac-client/templates/page-ticket-details.php
@@ -70,8 +70,7 @@
 }
 // =========================================================
 // debug message...
-$DEBUG = False;
-if ($DEBUG) {
+if (wptc_is_debug()) {
     global $post, $current_blog;
     // dump the change log 
     //$ticket = wptc_get_ticket_actions($ticket_id);

diff --git a/wp-trac-client/wp-trac-client.php b/wp-trac-client/wp-trac-client.php
@@ -130,6 +130,12 @@ function get_wptc_client() {
     return $wptc_client;
 }
 
+function wptc_is_debug() {
+
+    $debug = get_site_option('wptc_debug');
+    return ($debug === 'true');
+}
+
 // we need a admin page on dashboard for configuration.
 add_action('network_admin_menu', 'wptc_admin_init');
 /**
@@ -153,22 +159,23 @@ function wptc_admin_init() {
                      'wp-trac-client/admin-settings.php'
                     );
     // the template management page.
-    add_submenu_page('wp-trac-client/admin-settings.php',
-                     'Trac Client Templates', 'Templates',
-                     'manage_options',
-                     'wp-trac-client/admin-templates.php'
-                    );
+    //add_submenu_page('wp-trac-client/admin-settings.php',
+    //                 'Trac Client Templates', 'Templates',
+    //                 'manage_options',
+    //                 'wp-trac-client/admin-templates.php'
+    //                );
     // some management work here.
     add_submenu_page('wp-trac-client/admin-settings.php', // parent slug.
                      'Trac Client Management', 'Project Management',
                      'manage_options', 
                      'wp-trac-client/admin-manager.php'
                     );
-    add_submenu_page('wp-trac-client/admin-settings.php', // parent slug.
-                     'Trac Client Testing', 'APIs Testing',
-                     'manage_options', 
-                     'wp-trac-client/admin-testing.php'
-                    );
-
+    if(wptc_is_debug()) {
+        add_submenu_page('wp-trac-client/admin-settings.php', // parent slug.
+                         'Trac Client Testing', 'APIs Testing',
+                         'manage_options', 
+                         'wp-trac-client/admin-testing.php'
+                        );
+    }
 }
 ?>