Merge lp:~sinzui/launchpad/daily-build-form into lp:launchpad

[Curtis Hovey]

Hide the daily build form if it exists.

Pre-implementation: no one

When visiting a recipe page such as
https://code.qastaging.launchpad.net/~sinzui/+recipe/pocket-lint-daily I
see an error:
    Uncaught TypeError: Cannot call method 'addClass' of null
The browser just executed.
    Y.one('#request-daily-build-form').addClass('unseen')

The request-daily-build-form form is only rendered when the link is
available. This regression was introduced when I replaced the noscript
with a call to hide the form.

--------------------------------------------------------------------

RULES

    * Do not call addClass() if the form was not rendered.

QA

    * Visit https://code.qastaging.launchpad.net/~sinzui/+recipe/pocket-lint-daily
    * Verify there is not a TypeError when the page loads

LINT

    lib/lp/code/javascript/requestbuild_overlay.js
    lib/lp/code/javascript/tests/test_requestbuild_overlay.js

TEST

    ./bin/test -vvc --layer=YUITest lp.code

IMPLEMENTATION

Check that there is a node befor adding a class.
    lib/lp/code/javascript/requestbuild_overlay.js
    lib/lp/code/javascript/tests/test_requestbuild_overlay.js

[Richard Harding (rharding)]

Thanks, and appreciate the test to go with it there.

review: Approve (code)

=== modified file 'lib/lp/code/javascript/requestbuild_overlay.js'
--- lib/lp/code/javascript/requestbuild_overlay.js	2012-04-27 23:34:27 +0000
+++ lib/lp/code/javascript/requestbuild_overlay.js	2012-05-15 19:41:27 +0000
@@ -59,7 +59,10 @@
         }
         Y.lp.code.requestbuild_overlay.connect_requestbuilds();
     }
-    Y.one('#request-daily-build-form').addClass('unseen');
+    var form = Y.one('#request-daily-build-form');
+    if (form) {
+        form.addClass('unseen');
+    }
 };
 
 

=== modified file 'lib/lp/code/javascript/tests/test_requestbuild_overlay.js'
--- lib/lp/code/javascript/tests/test_requestbuild_overlay.js	2012-04-27 23:39:20 +0000
+++ lib/lp/code/javascript/tests/test_requestbuild_overlay.js	2012-05-15 19:41:27 +0000
@@ -264,6 +264,14 @@
             Y.one('#request-daily-build-form').hasClass('unseen'));
         Y.Assert.isFalse(
             Y.one('#request-daily-build').hasClass('unseen'));
+    },
+
+    test_hookUpDailyBuildsSchedule_without_form: function() {
+        var form = Y.one('#request-daily-build-form');
+        form.get('parentNode').removeChild(form);
+        module.hookUpDailyBuildsSchedule();
+        Y.Assert.isFalse(
+            Y.one('#request-daily-build').hasClass('unseen'));
     }
 }));