Merge lp:~henninge/launchpad/bug-753387-ajax-bug-subscription into lp:launchpad

[Henning Eggers]

[r=danilo][bug=753387] Fix js bug when subscribing w/o structural bug subscriptions feature.

= Summary =

Even without the new "mute" feature enabled (part of structural bug
subscriptions), subscribing and unscribing tried to update the "mute
subscription" node. This failed because that node does not exist if
the feature is not enabled.

== Proposed fix ==

Check if the node exists before doing stuff with it.

== Pre-implementation notes ==

None, all my own thinking.

== Implementation details ==

The check had only to be added in two places although the function that
eventually fails (update_mute_after_subscription_change) is called more
often. Those other call sites are only reached when the feature is
enabled and therefore when the node is actually on the page.
The two affected call sites can easily be spotted because they are the
only ones that call get_mute_subscription directly before.

== Tests ==

None, this is transitional anyway.

== Demo and Q/A ==

Open any bug when the feature is not enabled, open a javascript
console. Click on "Subscribe" and watch that you *don't* see an
error message on the console.
The second callsite is in "Subscribe someone else" but is only
triggered when the user has permissions to also unsubscribe that
other person. I could not (easily) figure out how to get into that
position.

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/bugs/javascript/bugtask_index_portlets.js

[Henning Eggers (henninge)]

I wonder what would be the correct way to trigger a removal of the conditions once the feature flag is removed. Any hints? Add an XXX?

[Данило Шеган (danilo)]

Looks good. As discussed on IRC, I'd only move the mute check inside update_mute_after_subscription_change, but you did say you considered it. So, your call if you want to do that.

review: Approve

=== modified file 'lib/lp/bugs/javascript/bugtask_index_portlets.js'
--- lib/lp/bugs/javascript/bugtask_index_portlets.js	2011-05-18 18:34:19 +0000
+++ lib/lp/bugs/javascript/bugtask_index_portlets.js	2011-05-25 08:49:27 +0000
@@ -768,8 +768,10 @@
                 }
 
                 add_user_name_link(subscription);
-                var mute_subscription = get_mute_subscription();
-                update_mute_after_subscription_change(mute_subscription);
+                if (namespace.use_advanced_subscriptions) {
+                    var mute_subscription = get_mute_subscription();
+                    update_mute_after_subscription_change(mute_subscription);
+                }
 
                 // Only when the link is added to the page, indicate success.
                 Y.on('contentready', function() {
@@ -1392,10 +1394,11 @@
                             if (team_member) {
                                 subscription.set('can_be_unsubscribed', true);
                                 add_temp_user_name(subscription);
-                                var mute_subscription;
-                                mute_subscription = get_mute_subscription();
-                                update_mute_after_subscription_change(
-                                    mute_subscription);
+                                if (namespace.use_advanced_subscriptions) {
+                                    var mute_subscr = get_mute_subscription();
+                                    update_mute_after_subscription_change(
+                                        mute_subscr);
+                                }
                             } else {
                                 subscription.set(
                                    'can_be_unsubscribed', false);