use current_provider for topic when user is not an admin

Author: dmitrytrager

Procfile.dev

@@ -0,0 +1 @@
+web: bin/rails server -p 3000

app/controllers/topics_controller.rb

@@ -52,7 +52,14 @@ def other_available_providers
   end
 
   def topic_params
-    params.require(:topic).permit(:title, :description, :uid, :language_id, :provider_id, documents: [])
+    params
+      .require(:topic)
+      .permit(:title, :description, :uid, :language_id, :provider_id, documents: []).tap do |perm_params|
+        if perm_params["provider_id"].present?
+          perm_params["provider_id"] = Current.user.providers.find(perm_params["provider_id"]).id
+          perm_params["provider_id"] = current_provider.id if current_provider && !Current.user.is_admin?
+        end
+      end
   end
 
   def search_params

app/views/topics/_form.html.erb

@@ -16,10 +16,12 @@
           <%= f.label :language %>
           <%= f.collection_select :language_id, Language.all, :id, :name, { prompt: "Select Language" }, class: "form-select" %>
         </div>
-        <div class="form-group">
-          <%= f.label :provider %>
-          <%= f.collection_select :provider_id, Provider.all, :id, :name, { prompt: "Select Provider" }, class: "form-select" %>
-        </div>
+        <% if Current.user.is_admin? %>
+          <div class="form-group">
+            <%= f.label :provider %>
+            <%= f.collection_select :provider_id, Provider.all, :id, :name, { prompt: "Select Provider" }, class: "form-select" %>
+          </div>
+        <% end %>
         <div class="form-group">
           <%= f.label :description %>
           <%= f.text_area :description, class: "form-control", placeholder: "Descripion" %>

app/views/topics/index.html.erb

@@ -8,7 +8,7 @@
         <div class="card">
           <div class="card-header d-flex justify-content-between align-items-center">
             <h2 class="card-title"><%= topics_title %></h2>
-            <%= link_to new_topic_path, class: "btn btn-primary" do %>
+            <%= link_to new_topic_path, class: "btn btn-primary", data: { turbo: false } do %>
               <i class="bi bi-plus"></i> Add New Topic
             <% end %>
           </div>

bin/server

@@ -1,3 +1,3 @@
 #!/usr/bin/env sh
 
-overmind start -r all
+overmind start -r all -f Procfile.dev

spec/requests/topics/create_spec.rb

@@ -7,7 +7,10 @@
     let(:language) { create(:language) }
     let(:topic_params) { attributes_for(:topic, provider_id: provider.id, language_id: language.id) }
 
-    before { sign_in(user) }
+    before do
+      provider.users << user
+      sign_in(user)
+    end
 
     it "creates a Topic" do
       post topics_url, params: { topic: topic_params }
@@ -18,5 +21,35 @@
       expect(topic.description).to eq("details")
       expect(topic.state).to eq("active")
     end
+
+    context "when current provider is set" do
+      let(:current_provider) { create(:provider) }
+
+      before do
+        current_provider.users << user
+        cookies = ActionDispatch::Request.new(Rails.application.env_config).cookie_jar
+        cookies.signed[:current_provider_id] = current_provider.id
+      end
+
+      it "creates a Topic for the current provider" do
+        post topics_url, params: { topic: topic_params }
+
+        expect(response).to redirect_to(topics_url)
+        topic = Topic.last
+        expect(topic.provider).to eq(current_provider)
+      end
+
+      context "when user is an admin" do
+        before { user.update(is_admin: true) }
+
+        it "creates a Topic for the selected provider" do
+          post topics_url, params: { topic: topic_params }
+
+          expect(response).to redirect_to(topics_url)
+          topic = Topic.last
+          expect(topic.provider).to eq(provider)
+        end
+      end
+    end
   end
 end