Add ability to CRUD Blocks independently of pages

This will allow platform managers to survey total list of blocks and their contents as well as create unattached blocks for use as dynamic content partials to be placed outside the contenxt of a BetterTogether::Page.

Prevent deletion of blocks that are attached to pages. They are managed from the page editor form.

Author: rsmithlal

app/controllers/better_together/content/blocks_controller.rb

@@ -0,0 +1,66 @@
+module BetterTogether
+  module Content
+    class BlocksController < ApplicationController
+      before_action :authenticate_user!
+      before_action :set_block, only: %i[show edit update destroy]
+      before_action only: %i[index], if: -> { Rails.env.development? } do
+        # Make sure that all BLock subclasses are loaded in dev to generate new block buttons
+        BetterTogether::Content::Block.load_all_subclasses
+      end
+
+      def index
+        @blocks = BetterTogether::Content::Block.includes(:pages).all
+      end
+
+      def create
+        @block = BetterTogether::Content::Block.new(block_params)
+
+        if @block.save
+          redirect_to content_block_path(@block), notice: 'Block was successfully created.'
+        else
+          render :new
+        end
+      end
+
+      def update
+        respond_to do |format|
+          if @block.update(block_params)
+            redirect_to edit_content_block_path(@block), notice: 'Block was successfully updated.'
+          else
+            format.turbo_stream do
+              render turbo_stream: turbo_stream.replace(helpers.dom_id(@block, 'form'), partial: 'form', locals: { block: @block } )
+            end
+          end
+        end
+      end
+
+      def new
+        @block = BetterTogether::Content::Block.new(type: params[:block_type])
+
+        respond_to do |format|
+          format.html
+        end
+      end
+
+      def destroy
+        @block.destroy unless @block.pages.any?
+
+        redirect_to content_blocks_path, notice: 'Block was sucessfully deleted'
+      end
+
+      private
+
+      def block_params
+        params.require(:block).permit(
+          :type, :media, :identifier,
+          *BetterTogether::Content::Block.localized_block_attributes,
+          *BetterTogether::Content::Block.storext_definitions.keys
+        )
+      end
+
+      def set_block
+        @block = BetterTogether::Content::Block.find(params[:id])
+      end
+    end
+  end
+end

app/controllers/better_together/content/page_blocks_controller.rb

@@ -4,10 +4,6 @@ class PageBlocksController < ApplicationController
       before_action :authenticate_user!
       before_action :set_page
 
-      def index
-        @blocks = @page.page_blocks.includes(:block)
-      end
-
       def new
         @page_block = @page.page_blocks.build
 

app/controllers/better_together/host_dashboard_controller.rb

@@ -12,6 +12,8 @@ def index # rubocop:todo Metrics/AbcSize, Metrics/MethodLength
       @resource_permissions = ResourcePermission.order(created_at: :desc).limit(3)
       @users = User.order(created_at: :desc).limit(3)
 
+      @content_blocks = Content::Block.order(created_at: :desc).limit(3)
+
       @geography_continents = Geography::Continent.order(created_at: :desc).limit(3)
       @geography_countries = Geography::Country.order(created_at: :desc).limit(3)
       @geography_states = Geography::State.order(created_at: :desc).limit(3)
@@ -27,6 +29,8 @@ def index # rubocop:todo Metrics/AbcSize, Metrics/MethodLength
       @resource_permission_count = ResourcePermission.count
       @user_count = User.count
 
+      @content_block_count = Content::Block.count
+
       @geography_continent_count = Geography::Continent.count
       @geography_country_count = Geography::Country.count
       @geography_state_count = Geography::State.count

app/helpers/better_together/content/blocks_helper.rb

@@ -3,7 +3,7 @@ module Content
     module BlocksHelper
       # Returns an array of acceptable image file types
       def acceptable_image_file_types
-        %w[image/jpeg image/png image/gif image/webp image/svg+xml]
+        BetterTogether::Content::Image::CONTENT_TYPES
       end
 
       # Helper to generate a unique temp_id for a model

app/models/better_together/content/block.rb

@@ -82,6 +82,11 @@ def self.localized_block_attributes
       def block_name
         self.class.block_name
       end
+
+      def to_s
+        "#{block_name} - #{persisted? ? (identifier.present? ? identifier : id.split('-').first) : 'new'}"
+      end
+      
     end
   end
 end

app/views/better_together/content/blocks/_associated_pages.html.erb

@@ -0,0 +1,8 @@
+<% if block.pages.any? %>
+  <div class="my-3">
+    <h5>Associated Pages</h5>
+    <%= block.pages.map { |page| link_to page }.join(', ').html_safe %>
+  </div>
+<% else %>
+  <p>No associated pages for this block.</p>
+<% end %>

app/views/better_together/content/blocks/_block_row.html.erb

@@ -0,0 +1,12 @@
+<%# app/views/better_together/content/blocks/_block_row.html.erb %>
+
+<tr>
+  <td><%= block %></td> <!-- Adjust to your actual title attribute if different -->
+  <td><%= block.class.name.demodulize %></td> <!-- Display block type without the namespace -->
+  <td><%= block.pages.map { |page| link_to page }.join(', ').html_safe %></td> <!-- Display block type without the namespace -->
+  <td>
+    <%= link_to 'Show', content_block_path(block), class: 'btn btn-sm btn-info' %>
+    <%= link_to 'Edit', edit_content_block_path(block), class: 'btn btn-sm btn-warning' %>
+    <%= link_to 'Delete', content_block_path(block), method: :delete, data: { confirm: 'Are you sure?', turbo_confirm: 'Are you sure?', turbo_method: :delete }, class: 'btn btn-sm btn-danger' unless block.pages.any? %>
+  </td>
+</tr>

app/views/better_together/content/blocks/_form.html.erb

@@ -0,0 +1,22 @@
+<%= form_with(model: block, url: block.persisted? ? content_block_path(block) : content_blocks_path, local: true, multipart: true, id: dom_id(block, 'form'), class: 'form', data: { controller: "form-validation" }) do |form| %>
+  <% if block.errors.any? %>
+    <div class="alert alert-danger">
+      <h4>Please correct the following errors:</h4>
+      <ul>
+        <% block.errors.full_messages.each do |msg| %>
+          <li><%= msg %></li>
+        <% end %>
+      </ul>
+    </div>
+  <% end %>
+
+  <!-- Add other block-specific fields here -->
+  <div class="mb-3">
+    <%= render partial: 'better_together/content/blocks/fields/block',
+             locals: { block: } %>
+  </div>
+
+  <div class="mb-3 text-end">
+    <%= form.submit 'Save Changes', class: 'btn btn-primary' %>
+  </div>
+<% end %>

app/views/better_together/content/blocks/edit.html.erb

@@ -0,0 +1,12 @@
+<div class="container mt-3">
+  <h1>Edit Block: <%= @block %></h1>
+
+  <%= render partial: 'associated_pages', locals: { block: @block } %>
+
+  <%= render partial: 'form', locals: { block: @block } %>
+
+  <div class="mt-3">
+    <%= link_to 'Back to Block List', content_blocks_path, class: 'btn btn-secondary me-2' %>
+    <%= link_to 'View Block', content_block_path(@block), class: 'btn btn-info' %>
+  </div>
+</div>

app/views/better_together/content/blocks/fields/_block.html.erb

@@ -1,4 +1,4 @@
-<%- scope = local_assigns[:scope] ? local_assigns[:scope] : block.block_name %>
+<%- scope = local_assigns[:scope] ? local_assigns[:scope] : BetterTogether::Content::Block.block_name %>
 <%- temp_id = block.persisted? ? block.id : SecureRandom.uuid %>
 
 <!-- Nested Block fields -->