Refactor site#welcome to use abilities instead of require_user
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 10 Oct 2018 09:49:45 +0000 (11:49 +0200)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 10 Oct 2018 09:49:45 +0000 (11:49 +0200)
app/controllers/site_controller.rb
app/models/ability.rb

index 2fa9125..4b960e4 100644 (file)
@@ -103,9 +103,7 @@ class SiteController < ApplicationController
     @locale = params[:copyright_locale] || I18n.locale
   end
 
-  def welcome
-    require_user
-  end
+  def welcome; end
 
   def help; end
 
index 3919bde..3ff4665 100644 (file)
@@ -5,7 +5,7 @@ class Ability
 
   def initialize(user)
     can :index, :site
-    can [:permalink, :edit, :help, :fixthemap, :offline, :export, :about, :preview, :copyright, :key, :id, :welcome], :site
+    can [:permalink, :edit, :help, :fixthemap, :offline, :export, :about, :preview, :copyright, :key, :id], :site
 
     can [:index, :rss, :show, :comments], DiaryEntry
 
@@ -13,7 +13,7 @@ class Ability
          :search_geonames, :search_osm_nominatim_reverse, :search_geonames_reverse], :geocoder
 
     if user
-      can :weclome, :site
+      can :welcome, :site
 
       can [:create, :edit, :comment, :subscribe, :unsubscribe], DiaryEntry