From: toyomoyo <toyomoyo>
Date: Tue, 21 Aug 2007 04:12:03 +0000 (+0000)
Subject: changed subscription/unsubscription to links if javascript is enabled, and button... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=5080d619a3cb4799393096e641d4ef9a3aaf2b88;p=moodle.git

changed subscription/unsubscription to links if javascript is enabled, and button if javascript is disabled
---

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index 49a5dc5df4..36a48f3aeb 100644
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -3255,8 +3255,18 @@ function forum_get_subscribe_link($forum, $context, $messages = array(), $cantac
             $linktext = $messages['unsubscribed'];
             $linktitle = get_string('subscribestart', 'forum');
         }
-        return print_single_button($CFG->wwwroot . '/mod/forum/subscribe.php?id=' . $forum->id,
+
+        $link = '<script type="text/javascript">';
+        $link .= 'document.getElementById("subscriptionlink").innerHTML = "<a title=\"' . $linktitle . '\" href=\"' . $CFG->wwwroot .
+           '/mod/forum/subscribe.php?id=' . $forum->id . '\">' . $linktext . '</a>";';
+        $link .= '</script>';
+        // use <noscript> to print button in case javascript is not enabled
+        $link .= '<noscript>';
+        $link .= print_single_button($CFG->wwwroot . '/mod/forum/subscribe.php?id=' . $forum->id,
                 '', $linktext, 'post', '_self', true, $linktitle);
+        $link .= '</noscript>';
+
+        return $link;  
     }    
 }
 
diff --git a/mod/forum/view.php b/mod/forum/view.php
index e18e7af855..e08505bcdd 100644
--- a/mod/forum/view.php
+++ b/mod/forum/view.php
@@ -155,7 +155,7 @@
                 echo '<span class="helplink">'.$streveryonecannowchoose.'</span>';
             }
 
-            echo '<br /><span class="helplink">', forum_get_subscribe_link($forum, $context, 
+            echo '<br /><span class="helplink" id="subscriptionlink">', forum_get_subscribe_link($forum, $context, 
                     array('forcesubscribed' => '', 'cantsubscribe' => '')), '</span>';
         }