Move snippet controller to share namespace

Author: rossta

app/controllers/snippets_controller.rb renamed to app/controllers/share/snippets_controller.rb

@@ -1,4 +1,4 @@
-class SnippetsController < ApplicationController
+class Share::SnippetsController < ApplicationController
   before_action :feature_enabled!
 
   # GET /snippets
@@ -32,7 +32,7 @@ def create
     @snippet = Snippet.new(snippet_params)
 
     if @snippet.save
-      redirect_to @snippet, notice: "Snippet was successfully created."
+      redirect_to share_snippet_url(@snippet), notice: "Snippet was successfully created."
     else
       render :new, status: :unprocessable_entity
     end
@@ -43,7 +43,7 @@ def update
     @snippet = Snippet.find(params[:id])
     if @snippet.update(snippet_params)
       @snippet.attach_screenshot_from_base64(params[:screenshot]) if params[:screenshot]
-      redirect_to @snippet, notice: "Snippet was successfully updated.", status: :see_other
+      redirect_to share_snippet_url(@snippet), notice: "Snippet was successfully updated.", status: :see_other
     else
       render :edit, status: :unprocessable_entity
     end
@@ -53,7 +53,7 @@ def update
   def destroy
     @snippet = Snippet.find(params[:id])
     @snippet.destroy!
-    redirect_to snippets_url, notice: "Snippet was successfully destroyed.", status: :see_other
+    redirect_to share_snippets_url, notice: "Snippet was successfully destroyed.", status: :see_other
   end
 
   private

app/views/snippets/_snippet.html.erb renamed to app/views/share/snippets/_snippet.html.erb

@@ -0,0 +1,11 @@
+<%= render Pages::Header.new(title: "Edit snippet") %>
+<div class="section-content container py-gap">
+  <%= render Share::Snippets::Form.new(snippet: @snippet) %>
+
+  <br>
+
+  <div>
+    <%= link_to "Show this snippet", share_snippet_path(@snippet) %> |
+    <%= link_to "Back to snippets", share_snippets_path %>
+  </div>
+</div>

app/views/share/snippets/edit.html.erb

@@ -1,4 +1,4 @@
-class Snippets::Form < ApplicationComponent
+class Share::Snippets::Form < ApplicationComponent
   include Phlex::Rails::Helpers::DOMID
   include Phlex::Rails::Helpers::FormWith
   include Phlex::Rails::Helpers::TurboFrameTag
@@ -12,7 +12,7 @@ def initialize(snippet:)
 
   def view_template
     form_with(
-      model: snippet,
+      model: [:share, snippet],
       class: "grid-content",
       data: {
         controller: "snippet-preview snippet-screenshot",
@@ -117,9 +117,9 @@ def language_select_options
 
   def form_path
     if snippet.persisted?
-      edit_snippet_path(snippet)
+      edit_share_snippet_path(snippet)
     else
-      new_snippet_path
+      new_share_snippet_path
     end
   end
 end

app/views/snippets/form.rb renamed to app/views/share/snippets/form.rb

@@ -1,11 +1,11 @@
 <%= render Pages::Header.new(title: "Snippets") %>
 <div id="snippets" class="section-content container py-gap">
   <div class="flex">
-    <%= link_to "New snippet", new_snippet_path, class: "button primary" %>
+    <%= link_to "New snippet", new_share_snippet_path, class: "button primary" %>
   </div>
 
   <% @snippets.each do |snippet| %>
-    <%= link_to snippet, class: "block" do %>
+    <%= link_to share_snippet_path(snippet), class: "block" do %>
       <% if snippet.screenshot.attached? %>
         <%= image_tag snippet.screenshot %>
       <% else %>

app/views/snippets/index.html.erb renamed to app/views/share/snippets/index.html.erb

@@ -1,10 +1,10 @@
 <%= render Pages::Header.new(title: "New Snippet") %>
 <div class="section-content container py-gap">
-  <%= render Snippets::Form.new(snippet: @snippet) %>
+  <%= render Share::Snippets::Form.new(snippet: @snippet) %>
 
   <br>
 
   <div>
-    <%= link_to "Back to snippets", snippets_path %>
+    <%= link_to "Back to snippets", share_snippets_path %>
   </div>
 </div>

app/views/snippets/new.html.erb renamed to app/views/share/snippets/new.html.erb

@@ -5,9 +5,9 @@
   <%= image_tag @snippet.screenshot if @snippet.screenshot.attached? %>
 
   <div>
-    <%= link_to "Edit this snippet", edit_snippet_path(@snippet) %> |
-    <%= link_to "Back to snippets", snippets_path %>
+    <%= link_to "Edit this snippet", edit_share_snippet_path(@snippet) %> |
+    <%= link_to "Back to snippets", share_snippets_path %>
 
-    <%= button_to "Destroy this snippet", @snippet, method: :delete %>
+    <%= button_to "Destroy this snippet", share_snippet_path(@snippet), method: :delete %>
   </div>
 </div>

app/views/snippets/show.html.erb renamed to app/views/share/snippets/show.html.erb

@@ -10,7 +10,12 @@
   sitepress_root controller: :site
   sitepress_pages controller: :site
 
-  resources :snippets
+  namespace :share do
+    resources :snippets
+    # resources :snippets do
+    #   resource :screenshot, only: [:new, :create], as: :screenshot
+    # end
+  end
 
   resources :newsletters, only: [:index, :show]
 

app/views/snippets/edit.html.erb

@@ -6,7 +6,7 @@
       Flipper.enable(:snippets, login_as_user)
 
       FactoryBot.create(:snippet)
-      get snippets_url
+      get share_snippets_url
       expect(response).to be_successful
     end
   end
@@ -16,15 +16,15 @@
       Flipper.enable(:snippets, login_as_user)
 
       snippet = FactoryBot.create(:snippet)
-      get snippet_url(snippet)
+      get share_snippet_url(snippet)
       expect(response).to be_successful
     end
   end
 
   describe "GET /new" do
     it "renders a successful response" do
       Flipper.enable(:snippets, login_as_user)
-      get new_snippet_url
+      get new_share_snippet_url
       expect(response).to be_successful
     end
   end
@@ -33,7 +33,7 @@
     it "renders a successful response" do
       Flipper.enable(:snippets, login_as_user)
       snippet = FactoryBot.create(:snippet)
-      get edit_snippet_url(snippet)
+      get edit_share_snippet_url(snippet)
       expect(response).to be_successful
     end
   end
@@ -46,13 +46,13 @@
 
       it "creates a new Snippet" do
         expect {
-          post snippets_url, params: {snippet: {source: "puts \"Hello!\"", language: "ruby"}}
+          post share_snippets_url, params: {snippet: {source: "puts \"Hello!\"", language: "ruby"}}
         }.to change(Snippet, :count).by(1)
       end
 
       it "redirects to the created snippet" do
-        post snippets_url, params: {snippet: {source: "puts \"Hello!\"", language: "ruby"}}
-        expect(response).to redirect_to(snippet_url(Snippet.last))
+        post share_snippets_url, params: {snippet: {source: "puts \"Hello!\"", language: "ruby"}}
+        expect(response).to redirect_to(share_snippet_url(Snippet.last))
       end
     end
 
@@ -63,12 +63,12 @@
 
       it "does not create a new Snippet" do
         expect {
-          post snippets_url, params: {snippet: {}}
+          post share_snippets_url, params: {snippet: {}}
         }.to change(Snippet, :count).by(0)
       end
 
       it "renders a response with 422 status (i.e. to display the 'new' template)" do
-        post snippets_url, params: {snippet: {}}
+        post share_snippets_url, params: {snippet: {}}
         expect(response).to have_http_status(:unprocessable_entity)
       end
     end
@@ -82,16 +82,16 @@
 
       it "updates the requested snippet" do
         snippet = FactoryBot.create(:snippet)
-        patch snippet_url(snippet), params: {snippet: {source: "puts \"Goodbye!\""}}
+        patch share_snippet_url(snippet), params: {snippet: {source: "puts \"Goodbye!\""}}
         snippet.reload
         expect(snippet.source).to eq("puts \"Goodbye!\"")
       end
 
       it "redirects to the snippet" do
         snippet = FactoryBot.create(:snippet)
-        patch snippet_url(snippet), params: {snippet: {source: "puts \"Goodbye!\""}}
+        patch share_snippet_url(snippet), params: {snippet: {source: "puts \"Goodbye!\""}}
         snippet.reload
-        expect(response).to redirect_to(snippet_url(snippet))
+        expect(response).to redirect_to(share_snippet_url(snippet))
       end
     end
 
@@ -102,7 +102,7 @@
 
       it "renders a response with 422 status (i.e. to display the 'edit' template)" do
         snippet = FactoryBot.create(:snippet)
-        patch snippet_url(snippet), params: {snippet: {language: "does_not_exist"}}
+        patch share_snippet_url(snippet), params: {snippet: {language: "does_not_exist"}}
         expect(response).to have_http_status(:unprocessable_entity)
       end
     end
@@ -113,15 +113,15 @@
       Flipper.enable(:snippets, login_as_user)
       snippet = FactoryBot.create(:snippet)
       expect {
-        delete snippet_url(snippet)
+        delete share_snippet_url(snippet)
       }.to change(Snippet, :count).by(-1)
     end
 
     it "redirects to the snippets list" do
       Flipper.enable(:snippets, login_as_user)
       snippet = FactoryBot.create(:snippet)
-      delete snippet_url(snippet)
-      expect(response).to redirect_to(snippets_url)
+      delete share_snippet_url(snippet)
+      expect(response).to redirect_to(share_snippets_url)
     end
   end
 end