diff --git a/app/controllers/subscription_controller.rb b/app/controllers/subscription_controller.rb index 05c56aac7e..5a5e7fe611 100644 --- a/app/controllers/subscription_controller.rb +++ b/app/controllers/subscription_controller.rb @@ -119,11 +119,16 @@ def digest end def multiple_add - unless params[:names] + if !params[:names] || params[:names] == '' flash[:notice] = "Please enter tags for subscription in the url." redirect_to "/subscriptions" + "?_=" + Time.now.to_i.to_s + return + end + if params[:names].is_a? String + tag_list = params[:names].split(',') + else + tag_list = params[:names] end - tag_list = params[:names].split(',') # should be logged in to subscribe if current_user # assume tag, for now @@ -170,8 +175,8 @@ def multiple_add # user or node subscription end else - flash[:warning] = "You must be logged in to subscribe for email updates." - redirect_to "/login" + flash[:warning] = "You must be logged in to subscribe for email updates!" + redirect_to "/login?return_to=" + request.fullpath end end diff --git a/app/views/tag/_tags.html.erb b/app/views/tag/_tags.html.erb index 44c9ba2ce5..4aff9fdcdd 100644 --- a/app/views/tag/_tags.html.erb +++ b/app/views/tag/_tags.html.erb @@ -1,7 +1,6 @@