refactor: safe_urlメソッドをモデルからヘルパーに移動

- Dojoモデルのsafe_urlメソッドをApplicationHelperのsafe_dojo_urlに移動
- app/views/shared/_dojo.html.erbでsafe_dojo_url()ヘルパーを使用

Author: nacchan99

app/helpers/application_helper.rb

@@ -55,6 +55,17 @@ def sanitize_content(content)
     )
   end
 
+  def safe_dojo_url(dojo)
+    return '#' if dojo.url.blank?
+
+    begin
+      uri = URI.parse(dojo.url)
+      uri.scheme&.match?(/\Ahttps?\z/) ? dojo.url : '#'
+    rescue URI::InvalidURIError
+      '#'
+    end
+  end
+
   # 'inline_' プレフィックスがついたflashメッセージをビュー内で表示するヘルパー
   # inline_alert → alert, inline_warning → warning のように変換してBootstrapのCSSクラスを適用
   def render_inline_flash_messages

app/models/dojo.rb

@@ -90,19 +90,6 @@ def active_at?(date)
     created_at <= date && (inactivated_at.nil? || inactivated_at > date)
   end
 
-  # URLの安全性をチェックして、安全なURLを返す
-  def safe_url
-    return '#' if url.blank?
-  
-    # URI.parseがエラーになるか、HTTPスキームでない場合は'#'を返す
-    begin
-      uri = URI.parse(url)
-      uri.scheme&.match?(/\Ahttps?\z/) ? url : '#'
-    rescue URI::InvalidURIError
-      '#'
-    end
-  end
-  
   # 再活性化メソッド
   def reactivate!
     if inactivated_at.present?

app/views/shared/_dojo.html.erb

@@ -1,9 +1,9 @@
 <li class="dojo" id="<%= html_escape(dojo.name) %>">
   <header>
-    <%= link_to lazy_image_tag(dojo.logo, alt: html_escape("CoderDojo #{dojo.name}"), class: 'dojo-picture'), sanitize_url(dojo.safe_url),
+    <%= link_to lazy_image_tag(dojo.logo, alt: html_escape("CoderDojo #{dojo.name}"), class: 'dojo-picture'), safe_dojo_url(dojo),
               target: "_blank", rel: "external noopener noreferrer" %>
     <span class="dojo-name">
-      <%= link_to html_escape("#{dojo.name} (#{dojo.prefecture.name})"), sanitize_url(dojo.safe_url), target: "_blank", rel: "external noopener noreferrer" %>
+      <%= link_to html_escape("#{dojo.name} (#{dojo.prefecture.name})"), safe_dojo_url(dojo), target: "_blank", rel: "external noopener noreferrer" %>
       <% if not dojo.counter == 1 %>
         <span class="dojo-counter"
               data-original-title="道場数"