Merge pull request #57 from schwuk/fix_56_url_not_found

Fixes #56 by not hardcoding the `subscription` URL

Author: schwuk

code_comments/htdocs/code-comments.js

@@ -326,6 +326,7 @@ var underscore = _.noConflict();
 		initialize: function(){
 			_.bindAll(this, "render");
 			this.model.listenTo(this.model, 'change', this.render);
+			this.model.url = $(this.$el).data('baseUrl') + '/subscription' + $(this.$el).data('path');
 			this.render();
 		},
 

code_comments/subscription.py

@@ -473,7 +473,7 @@ def process_request(self, req):
     def filter_stream(self, req, method, filename, stream, data):
         if re.match(r'^/(changeset|browser|attachment).*', req.path_info):
             filter = Transformer('//h1')
-            stream |= filter.before(self._subscription_button())
+            stream |= filter.before(self._subscription_button(req.path_info))
         return stream
 
     # Internal methods
@@ -505,10 +505,12 @@ def _do_PUT(self, req):
         req.send(json.dumps(subscription, cls=SubscriptionJSONEncoder),
                  'application/json')
 
-    def _subscription_button(self):
+    def _subscription_button(self, path):
         """
         Generates a (disabled) button to connect JavaScript to.
         """
         return tag.button('Subscribe', id_='subscribe', disabled=True,
                           title=('Code comment subscriptions require '
-                                 'JavaScript to be enabled'))
+                                 'JavaScript to be enabled'),
+                          data_base_url=self.env.project_url or self.env.abs_href(),
+                          data_path=path)