panda-cms 0.7.3 → 0.7.5

data/db/migrate/20240805121123_create_panda_cms_posts.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CreatePandaCMSPosts < ActiveRecord::Migration[7.1]
   def change
     create_table :panda_cms_post_tags, id: :uuid do |t|
@@ -20,7 +22,7 @@ class CreatePandaCMSPosts < ActiveRecord::Migration[7.1]
       t.index :slug, unique: true
     end
 
-    create_enum :panda_cms_post_status, ["active", "draft", "hidden", "archived"]
+    create_enum :panda_cms_post_status, %w[active draft hidden archived]
     add_column :panda_cms_posts, :status, :panda_cms_post_status, default: "draft", null: false
     add_index :panda_cms_posts, :status
   end

data/db/migrate/20240805123104_create_panda_cms_post_versions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CreatePandaCMSPostVersions < ActiveRecord::Migration[7.1]
   def change
     create_table :panda_cms_post_versions, id: :uuid do |t|

data/db/migrate/20240806112735_fix_panda_cms_visits_column_names.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class FixPandaCMSVisitsColumnNames < ActiveRecord::Migration[7.1]
   def change
     change_table :panda_cms_visits do |t|

data/db/migrate/20240806204412_add_completion_path_to_panda_cms_forms.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddCompletionPathToPandaCMSForms < ActiveRecord::Migration[7.1]
   def change
     add_column :panda_cms_forms, :completion_path, :string

data/db/migrate/20240820081917_change_form_submissions_to_submission_count.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ChangeFormSubmissionsToSubmissionCount < ActiveRecord::Migration[7.2]
   def change
     rename_column :panda_cms_forms, :submissions, :submission_count

data/db/migrate/20240923234535_add_depth_to_panda_cms_menus.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 class AddDepthToPandaCMSMenus < ActiveRecord::Migration[7.2]
   def change
     add_column :panda_cms_menus, :depth, :integer, null: true, default: nil
 
     homepage = Panda::CMS::Page.find_by(path: "/")
-    if homepage
-      main_menu = Panda::CMS::Menu.find_by(start_page_id: homepage.id, kind: :auto)
-      main_menu&.update(depth: 2)
-    end
+    return unless homepage
+
+    main_menu = Panda::CMS::Menu.find_by(start_page_id: homepage.id, kind: :auto)
+    main_menu&.update(depth: 2)
   end
 end

data/db/migrate/20241031205109_add_cached_content_to_panda_cms_block_contents.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddCachedContentToPandaCMSBlockContents < ActiveRecord::Migration[7.2]
   def change
     add_column :panda_cms_block_contents, :cached_content, :jsonb

data/db/migrate/20241119214548_convert_post_content_to_editor_js.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ConvertPostContentToEditorJs < ActiveRecord::Migration[7.1]
   def up
     Panda::CMS::Post.find_each do |post|

data/db/migrate/20241120000419_remove_post_tag_references.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class RemovePostTagReferences < ActiveRecord::Migration[8.0]
   def up
     remove_reference :panda_cms_posts, :post_tag, foreign_key: {to_table: :panda_cms_post_tags}, type: :uuid

data/db/migrate/20241120110943_add_editor_js_to_posts.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddEditorJsToPosts < ActiveRecord::Migration[7.0]
   def up
     # First, add a temporary column

data/db/migrate/20241120113859_add_cached_content_to_panda_cms_posts.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddCachedContentToPandaCMSPosts < ActiveRecord::Migration[8.0]
   def change
     add_column :panda_cms_posts, :cached_content, :text

data/db/migrate/20241123234140_remove_post_tag_id_from_posts.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class RemovePostTagIdFromPosts < ActiveRecord::Migration[8.0]
   def change
     remove_column :panda_cms_posts, :post_tag_id if column_exists?(:panda_cms_posts, :post_tag_id)

data/db/migrate/20250106223303_add_author_id_to_panda_cms_posts.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddAuthorIdToPandaCMSPosts < ActiveRecord::Migration[8.0]
   def change
     add_reference :panda_cms_posts, :author, type: :uuid, foreign_key: {to_table: :panda_cms_users}

data/db/migrate/20250120235542_remove_paper_trail.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+class RemovePaperTrail < ActiveRecord::Migration[7.1]
+  def up
+    version_tables = %w[
+      panda_cms_versions
+      panda_cms_version_associations
+      panda_cms_template_versions
+      panda_cms_page_versions
+      panda_cms_block_content_versions
+      panda_cms_post_versions
+      action_text_rich_text_versions
+    ]
+
+    version_tables.each do |table|
+      drop_table table if table_exists?(table)
+    end
+  end
+
+  def down
+    create_table :panda_cms_versions do |t|
+      t.string :item_type, null: false
+      t.uuid :item_id, null: false
+      t.string :event, null: false
+      t.string :whodunnit
+      t.jsonb :object
+      t.jsonb :object_changes
+      t.datetime :created_at
+    end
+    add_index :panda_cms_versions, %i[item_type item_id]
+
+    create_table :panda_cms_version_associations do |t|
+      t.integer :version_id
+      t.string :foreign_key_name, null: false
+      t.integer :foreign_key_id
+      t.string :foreign_type
+    end
+    add_index :panda_cms_version_associations, [:version_id]
+    add_index :panda_cms_version_associations, %i[foreign_key_name foreign_key_id foreign_type],
+      name: "index_version_associations_on_foreign_key"
+
+    # Model-specific version tables
+    %w[template page block_content post action_text_rich_text].each do |model|
+      create_table "panda_cms_#{model}_versions" do |t|
+        t.string :item_type, null: false
+        t.uuid :item_id, null: false
+        t.string :event, null: false
+        t.string :whodunnit
+        t.jsonb :object
+        t.jsonb :object_changes
+        t.datetime :created_at
+      end
+      add_index "panda_cms_#{model}_versions", %i[item_type item_id]
+    end
+  end
+end

data/db/migrate/20250126234001_create_panda_social_instagram_posts.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+class CreatePandaSocialInstagramPosts < ActiveRecord::Migration[7.1]
+  def change
+    create_table :panda_social_instagram_posts, id: :uuid do |t|
+      t.string :instagram_id, null: false
+      t.text :caption
+      t.datetime :posted_at, null: false
+      t.string :permalink
+      t.timestamps
+
+      t.index :instagram_id, unique: true
+      t.index :posted_at
+    end
+  end
+end

data/db/migrate/20250504221812_add_current_theme_to_panda_cms_users.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddCurrentThemeToPandaCMSUsers < ActiveRecord::Migration[8.0]
+  def change
+    add_column :panda_cms_users, :current_theme, :string, default: "default"
+  end
+end

data/db/seeds.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 generator = Panda::CMS::DemoSiteGenerator.new
 generator.create_templates
 generator.create_pages

data/lib/generators/panda/cms/install_generator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Generators
   module Panda
     module CMS

data/lib/panda/cms/asset_loader.rb

@@ -0,0 +1,390 @@
+# frozen_string_literal: true
+
+module Panda
+  module CMS
+    # AssetLoader handles loading compiled assets from GitHub releases
+    # Falls back to local development assets when GitHub assets unavailable
+    class AssetLoader
+      class << self
+        # Generate HTML tags for loading Panda CMS assets
+        def asset_tags(options = {})
+          if use_github_assets?
+            github_asset_tags(options)
+          else
+            development_asset_tags(options)
+          end
+        end
+
+        # Get the JavaScript asset URL
+        def javascript_url
+          if use_github_assets?
+            github_javascript_url
+          else
+            development_javascript_url
+          end
+        end
+
+        # Get the CSS asset URL (if exists)
+        def css_url
+          if use_github_assets?
+            github_css_url
+          else
+            development_css_url
+          end
+        end
+
+        # Check if GitHub-hosted assets should be used
+        def use_github_assets?
+          # Use GitHub assets in production or when explicitly enabled
+          Rails.env.production? ||
+            ENV["PANDA_CMS_USE_GITHUB_ASSETS"] == "true" ||
+            !development_assets_available? ||
+            ((Rails.env.test? || in_test_environment?) && compiled_assets_available?)
+        end
+
+        # Download assets from GitHub to local cache
+        def ensure_assets_available!
+          return if development_assets_available? && !use_github_assets?
+
+          cache_dir = local_cache_directory
+          version = `git rev-parse --short HEAD`.strip
+
+          # Check if we already have cached assets for this version
+          if cached_assets_exist?(version)
+            Rails.logger.info "[Panda CMS] Using cached assets #{version}"
+            return
+          end
+
+          Rails.logger.info "[Panda CMS] Downloading assets #{version} from GitHub..."
+          download_github_assets(version, cache_dir)
+        end
+
+        private
+
+        def github_asset_tags(options = {})
+          version = asset_version
+          base_url = github_base_url(version)
+
+          tags = []
+
+          # JavaScript tag with integrity check
+          js_url = "#{base_url}panda-cms-#{version}.js"
+          integrity = asset_integrity(version, "panda-cms-#{version}.js")
+
+          js_attrs = {
+            src: js_url
+          }
+          # In CI environment, don't use defer to ensure immediate execution
+          js_attrs[:defer] = true unless ENV["GITHUB_ACTIONS"] == "true"
+          # Standalone bundles should NOT use type="module" - they're regular scripts
+          # Only use type="module" for importmap/ES module assets
+          if !js_url.include?("panda-cms-assets")
+            js_attrs[:type] = "module"
+          end
+          js_attrs[:integrity] = integrity if integrity
+          js_attrs[:crossorigin] = "anonymous" if integrity
+
+          tags << content_tag(:script, "", js_attrs)
+
+          # CSS tag if CSS bundle exists
+          css_url = "#{base_url}panda-cms-#{version}.css"
+          if github_asset_exists?(version, "panda-cms-#{version}.css")
+            css_integrity = asset_integrity(version, "panda-cms-#{version}.css")
+
+            css_attrs = {
+              rel: "stylesheet",
+              href: css_url
+            }
+            css_attrs[:integrity] = css_integrity if css_integrity
+            css_attrs[:crossorigin] = "anonymous" if css_integrity
+
+            tags << tag(:link, css_attrs)
+          end
+
+          tags.join("\n").html_safe
+        end
+
+        def development_asset_tags(options = {})
+          # In test environment with CI, always use compiled assets
+          if (Rails.env.test? || ENV["CI"].present?) && compiled_assets_available?
+            # Use the same logic as GitHub assets but with local paths
+            version = asset_version
+            js_url = "/panda-cms-assets/panda-cms-#{version}.js"
+            css_url = "/panda-cms-assets/panda-cms-#{version}.css"
+            
+            tags = []
+            
+            # JavaScript tag
+            tags << content_tag(:script, "", {
+              src: js_url,
+              defer: true
+            })
+            
+            # CSS tag if exists
+            if cached_asset_exists?(css_url)
+              tags << tag(:link, {
+                rel: "stylesheet",
+                href: css_url
+              })
+            end
+            
+            tags.join("\n").html_safe
+          else
+            # In development, just use a simple script tag
+            # The view will handle importmap tags separately
+            content_tag(:script, "", {
+              src: development_javascript_url,
+              type: "module",
+              defer: true
+            })
+          end
+        end
+
+        def github_javascript_url
+          version = asset_version
+          # In test environment with local compiled assets, use local URL
+          if Rails.env.test? && compiled_assets_available?
+            "/panda-cms-assets/panda-cms-#{version}.js"
+          else
+            "#{github_base_url(version)}panda-cms-#{version}.js"
+          end
+        end
+
+        def github_css_url
+          version = asset_version
+          # In test environment with local compiled assets, use local URL
+          if Rails.env.test? && compiled_assets_available?
+            "/panda-cms-assets/panda-cms-#{version}.css"
+          else
+            "#{github_base_url(version)}panda-cms-#{version}.css"
+          end
+        end
+
+        def development_javascript_url
+          # Try cached assets first, then importmap
+          version = asset_version
+          # Try root level first (standalone bundle), then versioned directory
+          root_path = "/panda-cms-assets/panda-cms-#{version}.js"
+          versioned_path = "/panda-cms-assets/#{version}/panda-cms-#{version}.js"
+
+          if cached_asset_exists?(root_path)
+            root_path
+          elsif cached_asset_exists?(versioned_path)
+            versioned_path
+          else
+            # Fallback to importmap or engine asset
+            "/assets/panda/cms/controllers/index.js"
+          end
+        end
+
+        def development_css_url
+          version = asset_version
+          # Try versioned directory first, then root level
+          versioned_path = "/panda-cms-assets/#{version}/panda-cms-#{version}.css"
+          root_path = "/panda-cms-assets/panda-cms-#{version}.css"
+
+          if cached_asset_exists?(versioned_path)
+            versioned_path
+          elsif cached_asset_exists?(root_path)
+            root_path
+          else
+            nil # No CSS in development mode typically
+          end
+        end
+
+        def github_base_url(version)
+          # In test environment with compiled assets, use local URLs
+          if (Rails.env.test? || in_test_environment?) && compiled_assets_available?
+            "/panda-cms-assets/"
+          else
+            "https://github.com/tastybamboo/panda-cms/releases/download/#{version}/"
+          end
+        end
+
+        def asset_version
+          # In test environment, use VERSION constant for consistency with compiled assets
+          # In other environments, use git SHA for dynamic versioning
+          # Also check for test environment indicators since Rails.env might be development in specs
+          if Rails.env.test? || ENV["CI"].present? || in_test_environment?
+            Panda::CMS::VERSION
+          else
+            `git rev-parse --short HEAD`.strip
+          end
+        end
+
+        def in_test_environment?
+          # Check if we're running specs even if Rails.env is development
+          defined?(RSpec) && RSpec.respond_to?(:configuration)
+        end
+
+        def compiled_assets_available?
+          # Check if compiled assets exist in test location
+          version = asset_version
+          js_file = Rails.public_path.join("panda-cms-assets", "panda-cms-#{version}.js")
+          css_file = Rails.public_path.join("panda-cms-assets", "panda-cms-#{version}.css")
+          js_file.exist? && css_file.exist?
+        end
+
+        def development_assets_available?
+          # Check if local development assets exist (importmap, etc.)
+          importmap_available? || engine_assets_available?
+        end
+
+        def importmap_available?
+          return false unless defined?(Rails.application.importmap)
+
+          begin
+            # Rails 8+ uses a different API for accessing importmap entries
+            if Rails.application.importmap.respond_to?(:to_json)
+              importmap_json = JSON.parse(Rails.application.importmap.to_json)
+              importmap_json.any? { |name, _| name.include?("panda") }
+            elsif Rails.application.importmap.respond_to?(:entries)
+              Rails.application.importmap.entries.any? { |entry| entry.name.include?("panda") }
+            else
+              false
+            end
+          rescue => e
+            Rails.logger.debug "[Panda CMS] Could not check importmap: #{e.message}"
+            false
+          end
+        end
+
+        def engine_assets_available?
+          # Check if engine's JavaScript files are available
+          engine_js_path = Rails.root.join("..", "..", "app", "javascript", "panda", "cms", "controllers", "index.js")
+          File.exist?(engine_js_path)
+        end
+
+        def cached_assets_exist?(version)
+          cache_dir = local_cache_directory.join(version)
+          cache_dir.exist? && cache_dir.join("panda-cms-#{version}.js").exist?
+        end
+
+        def cached_asset_exists?(path)
+          Rails.public_path.join(path.delete_prefix("/")).exist?
+        end
+
+        def local_cache_directory
+          Rails.public_path.join("panda-cms-assets")
+        end
+
+        def download_github_assets(version, cache_dir)
+          require "net/http"
+          require "uri"
+          require "json"
+
+          version_dir = cache_dir.join(version)
+          FileUtils.mkdir_p(version_dir)
+
+          # Download manifest first
+          manifest_url = "#{github_base_url(version)}manifest.json"
+
+          begin
+            manifest_response = fetch_url(manifest_url)
+            if manifest_response.success?
+              manifest = JSON.parse(manifest_response.body)
+
+              # Download each file
+              manifest["files"].each do |file_info|
+                filename = file_info["filename"]
+                file_url = "#{github_base_url(version)}#{filename}"
+
+                Rails.logger.debug "[Panda CMS] Downloading #{filename}..."
+
+                file_response = fetch_url(file_url)
+                if file_response.success?
+                  File.write(version_dir.join(filename), file_response.body)
+                  Rails.logger.debug "[Panda CMS] Downloaded #{filename}"
+                else
+                  Rails.logger.warn "[Panda CMS] Failed to download #{filename}: #{file_response.code}"
+                end
+              end
+
+              # Save manifest
+              File.write(version_dir.join("manifest.json"), manifest_response.body)
+              Rails.logger.info "[Panda CMS] Assets cached locally"
+            else
+              Rails.logger.warn "[Panda CMS] Failed to download manifest: #{manifest_response.code}"
+            end
+          rescue => e
+            Rails.logger.error "[Panda CMS] Error downloading assets: #{e.message}"
+            # Fall back to development mode
+          end
+        end
+
+        def fetch_url(url)
+          uri = URI(url)
+          http = Net::HTTP.new(uri.host, uri.port)
+          http.use_ssl = (uri.scheme == "https")
+          http.open_timeout = 10
+          http.read_timeout = 30
+
+          request = Net::HTTP::Get.new(uri)
+          request["User-Agent"] = "Panda-CMS/#{`git rev-parse --short HEAD`}"
+
+          response = http.request(request)
+
+          OpenStruct.new(
+            success?: response.code.to_i == 200,
+            code: response.code,
+            body: response.body
+          )
+        rescue => e
+          Rails.logger.error "[Panda CMS] Network error: #{e.message}"
+          OpenStruct.new(success?: false, code: "error", body: nil)
+        end
+
+        def asset_integrity(version, filename)
+          # Try to get integrity from cached manifest
+          manifest_path = local_cache_directory.join(version, "manifest.json")
+          return nil unless manifest_path.exist?
+
+          begin
+            manifest = JSON.parse(File.read(manifest_path))
+            file_info = manifest["files"].find { |f| f["filename"] == filename }
+            return nil unless file_info
+
+            "sha256-#{Base64.strict_encode64([file_info["sha256"]].pack("H*"))}"
+          rescue => e
+            Rails.logger.warn "[Panda CMS] Error reading asset integrity: #{e.message}"
+            nil
+          end
+        end
+
+        def github_asset_exists?(version, filename)
+          # Quick check - assume it exists for now
+          # Could be enhanced to do HEAD request
+          true
+        end
+
+        def content_tag(name, content, options = {})
+          if defined?(ActionView::Helpers::TagHelper)
+            # Create a view context to render the tag
+            view_context = ActionView::Base.new(ActionView::LookupContext.new([]), {}, nil)
+            view_context.content_tag(name, content, options)
+          else
+            # Fallback implementation
+            attrs = options.map { |k, v| %(#{k}="#{v}") }.join(" ")
+            if content.present?
+              "<#{name} #{attrs}>#{content}</#{name}>"
+            else
+              "<#{name} #{attrs}></#{name}>"
+            end
+          end
+        end
+
+        def tag(name, options = {})
+          if defined?(ActionView::Helpers::TagHelper)
+            # Create a view context to render the tag
+            view_context = ActionView::Base.new(ActionView::LookupContext.new([]), {}, nil)
+            view_context.tag(name, options)
+          else
+            # Fallback implementation
+            attrs = options.map { |k, v| %(#{k}="#{v}") }.join(" ")
+            "<#{name} #{attrs}>"
+          end
+        end
+      end
+    end
+  end
+end

data/lib/panda/cms/bulk_editor.rb

@@ -1,4 +1,5 @@
-require "htmlentities"
+# frozen_string_literal: true
+
 require "json"
 
 module Panda
@@ -77,11 +78,12 @@ module Panda
 
             if current_data.dig("pages", path, "contents", key).nil?
               raise "Unknown page 1" if page.nil?
+
               block = Panda::CMS::Block.find_or_create_by(key: key, template: page.template) do |block_meta|
                 block_meta.name = key.titleize
               end
 
-              if !block
+              unless block
                 debug[:error] << "Error creating block '#{key.titleize}' on page '#{page.title}'"
                 next
               end
@@ -104,6 +106,7 @@ module Panda
             elsif content != current_data["pages"][path]["contents"][key]["content"]
               # Content has changed
               raise "Unknown page 2" if page.nil?
+
               block = Panda::CMS::Block.find_by(key: key, template: page.template)
               if Panda::CMS::BlockContent.find_by(page: page, block: block)&.update(content: content)
                 debug[:success] << "Updated '#{key.titleize}' content on page '#{page.title}'"
@@ -143,7 +146,8 @@ module Panda
         end
 
         # TODO: Eventually set the position of the block in the template, and then order from there rather than the name?
-        Panda::CMS::BlockContent.includes(:block, page: [:template]).order("panda_cms_pages.lft ASC, panda_cms_blocks.key ASC").each do |block_content|
+        Panda::CMS::BlockContent.includes(:block,
+          page: [:template]).order("panda_cms_pages.lft ASC, panda_cms_blocks.key ASC").each do |block_content|
           item = data["pages"][block_content.page.path] ||= {}
           item["title"] = block_content.page.title
           item["template"] = block_content.page.template.name

data/lib/panda/cms/demo_site_generator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class DemoSiteGenerator
@@ -34,10 +36,31 @@ module Panda
       #
       # @return [Hash] A hash containing the created pages
       def create_pages
-        @pages[:home] = Panda::CMS::Page.find_or_create_by({path: "/", title: "Home", template: @templates[:homepage]})
-        @pages[:about] = Panda::CMS::Page.find_or_create_by({path: "/about", title: "About", template: @templates[:page], parent: @pages[:home]})
-        @pages[:not_found] = Panda::CMS::Page.find_or_create_by({path: "/404", title: "Page Not Found", template: @templates[:page], parent: @pages[:home], status: "hidden"})
-        @pages[:internal_error] = Panda::CMS::Page.find_or_create_by({path: "/500", title: "Internal Server Error", template: @templates[:page], parent: @pages[:home], status: "hidden"})
+        @pages[:home] = Panda::CMS::Page.find_or_create_by({
+          path: "/",
+          title: "Home",
+          panda_cms_template_id: @templates[:homepage].id
+        })
+        @pages[:about] = Panda::CMS::Page.find_or_create_by({
+          path: "/about",
+          title: "About",
+          panda_cms_template_id: @templates[:page].id,
+          parent: @pages[:home]
+        })
+        @pages[:not_found] = Panda::CMS::Page.find_or_create_by({
+          path: "/404",
+          title: "Page Not Found",
+          panda_cms_template_id: @templates[:page].id,
+          parent: @pages[:home],
+          status: "hidden"
+        })
+        @pages[:internal_error] = Panda::CMS::Page.find_or_create_by({
+          path: "/500",
+          title: "Internal Server Error",
+          panda_cms_template_id: @templates[:page].id,
+          parent: @pages[:home],
+          status: "hidden"
+        })
 
         Panda::CMS::Page.reset_column_information
         Panda::CMS::Page.rebuild!