Fix flash message for a non-existent tags page

Currently, the application redirects the user to 'all items' screen for the hub, if a tags page does not exist. The solution is to use flash instead of flash.now, as the latter is only used for rendering. This issue was raised in berkmancenter#50
adl1995 · Mar 26, 2018 · 8e6bc5a · 8e6bc5a
1 parent da8a744
commit 8e6bc5a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/controllers/tags_controller.rb b/app/controllers/tags_controller.rb
@@ -127,7 +127,7 @@ def load_tag_from_name
              ActsAsTaggableOn::Tag.find_by(id: params[:id])
            end
     unless @tag
-      flash.now[:error] = "We're sorry, but '#{params[:name]}' is not a tag for '#{@hub.title}'"
+      flash[:error] = "We're sorry, but '#{params[:id]}' is not a tag for '#{@hub.title}'"
 
       redirect_to hub_path(@hub)
     end