require login to write a review (bug 871871)

hearth/media/js/ratings.js

@@ -102,7 +102,8 @@ define('ratings',
             // If the user isn't logged in, prompt them to do so.
             if (!user.logged_in()) {
                 login.login().done(function() {
-                    addReview(e, $senderEl);
+                    // TODO: This should be more graceful than a page reload.
+                    require('views').reload();
                 });
                 return;
             }

hearth/media/js/views/app.js

@@ -3,6 +3,8 @@ define('views/app',
     function(caps, l10n, utils, requests, _, urls, z, nunjucks, overflow) {
     'use strict';
 
+    var login = require('login');
+
     z.page.on('click', '#product-rating-status .toggle', utils._pd(function() {
         // Show/hide scary content-rating disclaimers to developers.
         $(this).closest('.toggle').siblings('div').toggleClass('hidden');
@@ -58,6 +60,14 @@ define('views/app',
             }
         }).onload('ratings', function() {
             var reviews = $('.detail .reviews li');
+            if (!require('user').logged_in()) {
+                $('#add-review').text(gettext('Sign in to Review')).on('click', function(e) {
+                    e.preventDefault();
+                    e.stopPropagation();
+                    login.login();
+                });
+            }
+
             if (reviews.length < 3) return;
 
             for (var i = 0; i < reviews.length - 2; i += 2) {

hearth/media/js/views/app/ratings.js

@@ -1,11 +1,20 @@
-define('views/app/ratings', ['l10n', 'urls'], function(l10n, urls) {
+define('views/app/ratings', ['l10n', 'urls', 'user'], function(l10n, urls, user) {
 
     var gettext = l10n.gettext;
 
     return function(builder, args) {
         var slug = args[0];
+
         builder.start('ratings/main.html', {
             'slug': slug
+        }).done(function() {
+            if (!user.logged_in()) {
+                $('#write-review').text(gettext('Sign in to Review')).on('click', function(e) {
+                    e.preventDefault();
+                    e.stopPropagation();
+                    require('login').login();
+                });
+            }
         });
 
         builder.z('type', 'leaf');

hearth/media/js/views/app/ratings/add.js

@@ -37,6 +37,7 @@ define('views/app/ratings/add',
         });
 
         builder.z('type', 'leaf');
+        builder.z('reload_on_login', true);
         builder.z('parent', urls.reverse('app/ratings', [slug]));
         builder.z('title', gettext('Write a Review'));
     };

hearth/media/js/views/app/ratings/edit.js

@@ -47,6 +47,7 @@ define('views/app/ratings/edit',
         // have even gotten to this page in their current state anyway.
         if (!user.logged_in()) {
             z.page.trigger('navigate', urls.reverse('app', [slug]));
+            require('views').reload();
             return;
         }