Merge pull request #3550 from techAPJ/patch-1

FIX: new-topic URL should survive login redirection

app/assets/javascripts/discourse/controllers/login.js.es6

@@ -78,9 +78,10 @@ export default DiscourseController.extend(ModalFunctionality, {
           var destinationUrl = $.cookie('destination_url');
           $hidden_login_form.find('input[name=username]').val(self.get('loginName'));
           $hidden_login_form.find('input[name=password]').val(self.get('loginPassword'));
-          if (self.get('loginRequired') && destinationUrl) {
+          if ((self.get('loginRequired') || $.cookie('shouldRedirectToUrl')) && destinationUrl) {
             // redirect client to the original URL
             $.cookie('destination_url', null);
+            $.cookie('shouldRedirectToUrl', null);
             $hidden_login_form.find('input[name=redirect]').val(destinationUrl);
           } else {
             $hidden_login_form.find('input[name=redirect]').val(window.location.href);

app/assets/javascripts/discourse/routes/new-topic.js.es6

@@ -13,6 +13,7 @@ export default Discourse.Route.extend({
       });
     } else {
       // User is not logged in
+      $.cookie('shouldRedirectToUrl', true);
       self.replaceWith('login');
     }
   }

app/controllers/static_controller.rb

@@ -72,6 +72,7 @@ class StaticController < ApplicationController
            uri.path !~ /\./
 
           destination = uri.path
+          destination = "#{uri.path}?#{uri.query}" if uri.path =~ /new-topic/
         end
       rescue URI::InvalidURIError
         # Do nothing if the URI is invalid