refactor(views): extract shared Attributes, ActionsButtons, and ErrorPage

DRY up duplicated update_attributes into a Views::Attributes module
shared by BasicLayout and BasicComponent. Extract inline actions_buttons
methods from Index and Show into a reusable ActionsButtons component.
Consolidate three identical error page view_templates into an ErrorPage
base class. Also adds nil guard on field access in Index and Show
table/record iteration.

Author: blocknotes

lib/tiny_admin/views/actions/index.rb

@@ -75,6 +75,8 @@ def table_body
                 attributes = prepare_record.call(record)
                 attributes.each do |key, value|
                   field = fields[key]
+                  next unless field
+
                   td(class: "field-value-#{field.name} field-value-type-#{field.type}") {
                     render TinyAdmin.settings.components[:field_value].new(field, value, record: record)
                   }
@@ -110,16 +112,9 @@ def table_body
         end
 
         def actions_buttons
-          ul(class: "nav justify-content-end") {
-            (actions || {}).each do |action, action_class|
-              li(class: "nav-item mx-1") {
-                href = TinyAdmin.route_for(slug, action: action)
-                a(href: href, class: "nav-link btn btn-outline-secondary") {
-                  action_class.respond_to?(:title) ? action_class.title : action
-                }
-              }
-            end
-          }
+          buttons = TinyAdmin::Views::Components::ActionsButtons.new
+          buttons.update_attributes(actions: actions, slug: slug)
+          render buttons
         end
       end
     end

lib/tiny_admin/views/actions/show.rb

@@ -25,10 +25,10 @@ def view_template
 
               prepare_record.call(record).each do |key, value|
                 field = fields[key]
+                next unless field
+
                 div(class: "field-#{field.name} row lh-lg") {
-                  if field
-                    div(class: "field-header col-2") { field.options[:header] || field.title }
-                  end
+                  div(class: "field-header col-2") { field.options[:header] || field.title }
                   div(class: "field-value col-10") {
                     render TinyAdmin.settings.components[:field_value].new(field, value, record: record)
                   }
@@ -43,16 +43,9 @@ def view_template
         private
 
         def actions_buttons
-          ul(class: "nav justify-content-end") {
-            (actions || {}).each do |action, action_class|
-              li(class: "nav-item mx-1") {
-                href = TinyAdmin.route_for(slug, reference: reference, action: action)
-                a(href: href, class: "nav-link btn btn-outline-secondary") {
-                  action_class.respond_to?(:title) ? action_class.title : action
-                }
-              }
-            end
-          }
+          buttons = TinyAdmin::Views::Components::ActionsButtons.new
+          buttons.update_attributes(actions: actions, slug: slug, reference: reference)
+          render buttons
         end
       end
     end

lib/tiny_admin/views/attributes.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module TinyAdmin
+  module Views
+    module Attributes
+      def update_attributes(attributes)
+        attributes.each do |key, value|
+          send("#{key}=", value)
+        end
+      end
+    end
+  end
+end

lib/tiny_admin/views/basic_layout.rb

@@ -3,19 +3,14 @@
 module TinyAdmin
   module Views
     class BasicLayout < Phlex::HTML
+      include Attributes
       include Utils
 
       attr_accessor :content, :params, :widgets
 
       def label_for(value, options: [])
         TinyAdmin.settings.helper_class.label_for(value, options: options)
       end
-
-      def update_attributes(attributes)
-        attributes.each do |key, value|
-          send("#{key}=", value)
-        end
-      end
     end
   end
 end

lib/tiny_admin/views/components/actions_buttons.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module TinyAdmin
+  module Views
+    module Components
+      class ActionsButtons < BasicComponent
+        attr_accessor :actions, :slug, :reference
+
+        def view_template
+          ul(class: "nav justify-content-end") {
+            (actions || {}).each do |action, action_class|
+              li(class: "nav-item mx-1") {
+                href = TinyAdmin.route_for(slug, reference: reference, action: action)
+                a(href: href, class: "nav-link btn btn-outline-secondary") {
+                  action_class.respond_to?(:title) ? action_class.title : action
+                }
+              }
+            end
+          }
+        end
+      end
+    end
+  end
+end

lib/tiny_admin/views/components/basic_component.rb

@@ -4,11 +4,7 @@ module TinyAdmin
   module Views
     module Components
       class BasicComponent < Phlex::HTML
-        def update_attributes(attributes)
-          attributes.each do |key, value|
-            send("#{key}=", value)
-          end
-        end
+        include Attributes
       end
     end
   end

lib/tiny_admin/views/pages/error_page.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module TinyAdmin
+  module Views
+    module Pages
+      class ErrorPage < DefaultLayout
+        def view_template
+          super do
+            div(class: css_class) {
+              h1(class: "title") { title }
+            }
+          end
+        end
+
+        private
+
+        def css_class
+          self.class.name.split("::").last
+              .gsub(/([A-Z])/, '_\1')
+              .sub(/^_/, "")
+              .downcase
+        end
+      end
+    end
+  end
+end

lib/tiny_admin/views/pages/page_not_allowed.rb

@@ -3,15 +3,7 @@
 module TinyAdmin
   module Views
     module Pages
-      class PageNotAllowed < DefaultLayout
-        def view_template
-          super do
-            div(class: "page_not_allowed") {
-              h1(class: "title") { title }
-            }
-          end
-        end
-
+      class PageNotAllowed < ErrorPage
         def title
           "Page not allowed"
         end

lib/tiny_admin/views/pages/page_not_found.rb

@@ -3,15 +3,7 @@
 module TinyAdmin
   module Views
     module Pages
-      class PageNotFound < DefaultLayout
-        def view_template
-          super do
-            div(class: "page_not_found") {
-              h1(class: "title") { title }
-            }
-          end
-        end
-
+      class PageNotFound < ErrorPage
         def title
           "Page not found"
         end

lib/tiny_admin/views/pages/record_not_found.rb

@@ -3,15 +3,7 @@
 module TinyAdmin
   module Views
     module Pages
-      class RecordNotFound < DefaultLayout
-        def view_template
-          super do
-            div(class: "record_not_found") {
-              h1(class: "title") { title }
-            }
-          end
-        end
-
+      class RecordNotFound < ErrorPage
         def title
           "Record not found"
         end