Create CommunityLinks.

Links must be https.  Add validate_url ruby gem to validate the urls.

Add not null on link site and url.

Add relevant FK.

Author: openbrian

Gemfile

@@ -67,6 +67,7 @@ gem "rails_param"
 gem "rinku", ">= 2.0.6", :require => "rails_rinku"
 gem "strong_migrations", "< 2.0.0"
 gem "validates_email_format_of", ">= 1.5.1"
+gem "validate_url"
 
 # Native OSM extensions
 gem "quad_tile", "~> 1.0.1"

Gemfile.lock

@@ -583,6 +583,9 @@ GEM
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.5.0)
     uri (0.13.0)
+    validate_url (1.0.15)
+      activemodel (>= 3.0.0)
+      public_suffix
     validates_email_format_of (1.8.2)
       i18n (>= 0.8.0)
       simpleidn
@@ -703,6 +706,7 @@ DEPENDENCIES
   terser
   turbo-rails
   unicode-display_width
+  validate_url
   validates_email_format_of (>= 1.5.1)
   vendorer
   webmock

app/abilities/ability.rb

@@ -18,6 +18,7 @@ def initialize(user)
       can [:index, :feed, :show], Changeset
       can :index, ChangesetComment
       can [:index, :show], Community
+      can [:index], CommunityLink
       can [:confirm, :confirm_resend, :confirm_email], :confirmation
       can [:index, :rss, :show], DiaryEntry
       can :index, DiaryComment
@@ -50,6 +51,7 @@ def initialize(user)
         can [:new, :create, :reply, :show, :inbox, :outbox, :muted, :mark, :unmute, :destroy], Message
         can [:create, :new], Community
         can [:edit, :update], Community, { :organizer_id => user.id }
+        can [:edit, :create, :destroy, :new, :update], CommunityLink, { :community => { :organizer_id => user.id } }
         can [:close, :reopen], Note
         can [:show, :edit, :update], :preference
         can [:edit, :update], :profile

app/assets/stylesheets/communities.scss

@@ -9,10 +9,4 @@
   label {
     font-weight: bold;
   }
-  ul {
-    display: inline-block;
-  }
-  ul > li {
-    display: inline-block;
-  }
 }

app/controllers/community_links_controller.rb

@@ -0,0 +1,61 @@
+class CommunityLinksController < ApplicationController
+  layout "site"
+  before_action :authorize_web
+
+  before_action :set_link, :only => [:destroy, :edit, :update]
+
+  load_and_authorize_resource :except => [:create, :new]
+  authorize_resource
+
+  def index
+    @community = Community.friendly.find(params[:community_id])
+    @links = @community.community_links
+  end
+
+  def new
+    return "missing parameter community_id" unless params.key?(:community_id)
+
+    @community = Community.friendly.find(params[:community_id])
+    @title = t ".title"
+    @link = CommunityLink.new
+    @link.community_id = params[:community_id]
+  end
+
+  def edit; end
+
+  def create
+    @community = Community.friendly.find(params[:community_id])
+    @link = @community.community_links.build(link_params)
+    if @link.save
+      response.set_header("link_id", @link.id) # for testing
+      redirect_to @link.community, :notice => t(".success")
+    else
+      render "new"
+    end
+  end
+
+  def update
+    if @link.update(link_params)
+      redirect_to @link.community, :notice => t(".success")
+    else
+      flash.now[:alert] = t(".failure")
+      render :edit
+    end
+  end
+
+  def destroy
+    community_id = @link.community_id
+    @link.delete
+    redirect_to community_path(community_id)
+  end
+
+  private
+
+  def set_link
+    @link = CommunityLink.find(params[:id])
+  end
+
+  def link_params
+    params.require(:community_link).permit(:community_id, :text, :url)
+  end
+end

app/models/community.rb

@@ -35,6 +35,7 @@ class Community < ApplicationRecord
   friendly_id :name, :use => :slugged
 
   belongs_to :organizer, :class_name => "User"
+  has_many :community_links
 
   validates :name, :presence => true, :length => 1..255, :characters => true
   validates :description, :presence => true, :length => 1..1023, :characters => true

app/models/community_link.rb

@@ -0,0 +1,25 @@
+# == Schema Information
+#
+# Table name: community_links
+#
+#  id           :bigint(8)        not null, primary key
+#  community_id :bigint(8)        not null
+#  text         :string           not null
+#  url          :string           not null
+#  created_at   :datetime         not null
+#  updated_at   :datetime         not null
+#
+# Indexes
+#
+#  index_community_links_on_community_id  (community_id)
+#
+# Foreign Keys
+#
+#  fk_rails_...  (community_id => communities.id)
+#
+
+class CommunityLink < ApplicationRecord
+  belongs_to :community
+  validates :text, :presence => true, :length => 1..255, :characters => true
+  validates :url, :presence => true, :length => 1..255, :url => { :schemes => ["https"] }
+end

app/views/communities/show.html.erb

@@ -29,6 +29,22 @@
       <p>
         <%= auto_link @community.description %>
       </p>
+      <div class="d-flex">
+        <label>
+          <% if current_user == @community.organizer %>
+            <%= link_to t(".links"), community_community_links_path(@community) %>
+          <% else %>
+            <%= t(".links") %>
+          <% end %>
+        </label>
+        <ul class="ps-2 breadcrumb">
+          <% @community.community_links.each do |link| %>
+            <li class="breadcrumb-item">
+              <a href="<%= link.url %>"><%= link.site %></a>
+            </li>
+          <% end %>
+        </ul>
+      </div>
       <div>
         <label><%= t(".organizer") %></label>
         <%= link_to @community.organizer.display_name, user_path(@community.organizer) %>

app/views/community_links/_form.html.erb

@@ -0,0 +1,5 @@
+<%= bootstrap_form_for [@community, @link] do |form| %>
+  <%= form.text_field :text %>
+  <%= form.text_field :url %>
+  <%= form.primary %>
+<% end %>

app/views/community_links/edit.html.erb

@@ -0,0 +1,3 @@
+<h1><%= t(".edit_community_link") %></h1>
+
+<%= render "form", :link => @link %>