panda-cms 0.7.4 → 0.8.0

data/db/migrate/20240205223709_create_panda_cms_pages.rb

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

data/db/migrate/20240219213327_create_panda_cms_page_versions.rb

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

data/db/migrate/20240303002805_create_panda_cms_templates.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CreatePandaCMSTemplates < ActiveRecord::Migration[7.1]
   def change
     create_table :panda_cms_templates, id: :uuid do |t|
@@ -6,6 +8,7 @@ class CreatePandaCMSTemplates < ActiveRecord::Migration[7.1]
       t.timestamps
     end
 
-    add_reference :panda_cms_pages, :panda_cms_template, type: :uuid, foreign_key: {to_table: :panda_cms_templates}, null: false
+    add_reference :panda_cms_pages, :panda_cms_template, type: :uuid, foreign_key: {to_table: :panda_cms_templates},
+      null: false
   end
 end

data/db/migrate/20240303003434_create_panda_cms_template_versions.rb

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

data/db/migrate/20240303022441_create_panda_cms_blocks.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 class CreatePandaCMSBlocks < ActiveRecord::Migration[7.1]
   def change
-    create_enum :panda_cms_block_kind, ["plain_text", "rich_text", "image", "video", "audio", "file", "code", "iframe", "quote", "list", "table", "form"]
+    create_enum :panda_cms_block_kind,
+      %w[plain_text rich_text image video audio file code iframe quote list table form]
 
     create_table :panda_cms_blocks, id: :uuid do |t|
       t.enum :kind, enum_type: :panda_cms_block_kind, default: "plain_text", null: false

data/db/migrate/20240303024256_create_panda_cms_block_contents.rb

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

data/db/migrate/20240303024746_create_panda_cms_block_content_versions.rb

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

data/db/migrate/20240303233238_add_panda_cms_menu_table.rb

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

data/db/migrate/20240303234724_add_panda_cms_menu_item_table.rb

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

data/db/migrate/20240304134343_add_parent_id_to_panda_cms_pages.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddParentIdToPandaCMSPages < ActiveRecord::Migration[7.1]
   def change
     add_reference :panda_cms_pages, :parent, type: :uuid, foreign_key: {to_table: :panda_cms_pages}

data/db/migrate/20240315125411_add_status_to_panda_cms_pages.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 class AddStatusToPandaCMSPages < ActiveRecord::Migration[7.1]
   def change
-    unless column_exists?(:panda_cms_pages, :status)
-      create_enum :panda_cms_page_status, ["active", "draft", "hidden", "archived"]
-      add_column :panda_cms_pages, :status, :panda_cms_page_status, default: "active", null: false
-      add_index :panda_cms_pages, :status
-    end
+    return if column_exists?(:panda_cms_pages, :status)
+
+    create_enum :panda_cms_page_status, %w[active draft hidden archived]
+    add_column :panda_cms_pages, :status, :panda_cms_page_status, default: "active", null: false
+    add_index :panda_cms_pages, :status
   end
 end

data/db/migrate/20240315125421_add_nested_sets_to_panda_cms_pages.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddNestedSetsToPandaCMSPages < ActiveRecord::Migration[7.1]
   def self.up
     Panda::CMS::Page.where(parent_id: 0).update_all(parent_id: nil)

data/db/migrate/20240316212822_add_kind_to_panda_cms_menus.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 class AddKindToPandaCMSMenus < ActiveRecord::Migration[7.1]
   def change
-    create_enum :panda_cms_menu_kind, ["static", "auto"]
+    create_enum :panda_cms_menu_kind, %w[static auto]
     add_column :panda_cms_menus, :kind, :enum, enum_type: :panda_cms_menu_kind, default: "static", null: false
   end
 end

data/db/migrate/20240316221425_add_start_page_to_panda_cms_menus.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddStartPageToPandaCMSMenus < ActiveRecord::Migration[7.1]
   def change
     add_reference :panda_cms_menus, :start_page, type: :uuid, foreign_key: {to_table: :panda_cms_pages}

data/db/migrate/20240316230706_add_nested_to_panda_cms_menu_items.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddNestedToPandaCMSMenuItems < ActiveRecord::Migration[7.1]
   def change
     add_column :panda_cms_menu_items, :parent_id, :uuid

data/db/migrate/20240317010532_create_panda_cms_users.rb

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

data/db/migrate/20240317161534_add_max_uses_to_panda_cms_template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddMaxUsesToPandaCMSTemplate < ActiveRecord::Migration[7.1]
   def change
     add_column :panda_cms_templates, :max_uses, :integer, null: true, default: nil

data/db/migrate/20240317163053_reset_counter_cache_on_panda_cms_template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ResetCounterCacheOnPandaCMSTemplate < ActiveRecord::Migration[7.1]
   def change
     Panda::CMS::Template.find_each { |t| Panda::CMS::Template.reset_counters(t.id, :pages) }

data/db/migrate/20240317214827_create_panda_cms_redirects.rb

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

data/db/migrate/20240317230622_create_panda_cms_visits.rb

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

data/db/migrate/20240324205703_create_active_storage_tables.active_storage.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This migration comes from active_storage (originally 20170806125915)
 class CreateActiveStorageTables < ActiveRecord::Migration[7.0]
   def change
@@ -33,7 +35,8 @@ class CreateActiveStorageTables < ActiveRecord::Migration[7.0]
         t.datetime :created_at, null: false
       end
 
-      t.index [:record_type, :record_id, :name, :blob_id], name: :index_active_storage_attachments_uniqueness, unique: true
+      t.index %i[record_type record_id name blob_id], name: :index_active_storage_attachments_uniqueness,
+        unique: true
       t.foreign_key :active_storage_blobs, column: :blob_id
     end
 
@@ -41,7 +44,7 @@ class CreateActiveStorageTables < ActiveRecord::Migration[7.0]
       t.belongs_to :blob, null: false, index: false, type: foreign_key_type
       t.string :variation_digest, null: false
 
-      t.index [:blob_id, :variation_digest], name: :index_active_storage_variant_records_uniqueness, unique: true
+      t.index %i[blob_id variation_digest], name: :index_active_storage_variant_records_uniqueness, unique: true
       t.foreign_key :active_storage_blobs, column: :blob_id
     end
   end

data/db/migrate/20240408084718_default_panda_cms_users_admin_to_false.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class DefaultPandaCMSUsersAdminToFalse < ActiveRecord::Migration[7.1]
   def change
     change_column :panda_cms_users, :admin, :boolean, default: false

data/db/migrate/20240701225422_add_service_name_to_active_storage_blobs.active_storage.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: true
+
 # This migration comes from active_storage (originally 20190112182829)
 class AddServiceNameToActiveStorageBlobs < ActiveRecord::Migration[6.0]
   def up
     return unless table_exists?(:active_storage_blobs)
 
-    unless column_exists?(:active_storage_blobs, :service_name)
-      add_column :active_storage_blobs, :service_name, :string
+    return if column_exists?(:active_storage_blobs, :service_name)
 
-      if (configured_service = ActiveStorage::Blob.service.name)
-        ActiveStorage::Blob.unscoped.update_all(service_name: configured_service)
-      end
+    add_column :active_storage_blobs, :service_name, :string
 
-      change_column :active_storage_blobs, :service_name, :string, null: false
+    if (configured_service = ActiveStorage::Blob.service.name)
+      ActiveStorage::Blob.unscoped.update_all(service_name: configured_service)
     end
+
+    change_column :active_storage_blobs, :service_name, :string, null: false
   end
 
   def down

data/db/migrate/20240701225423_create_active_storage_variant_records.active_storage.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This migration comes from active_storage (originally 20191206030411)
 class CreateActiveStorageVariantRecords < ActiveRecord::Migration[6.0]
   def change

data/db/migrate/20240701225424_remove_not_null_on_active_storage_blobs_checksum.active_storage.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This migration comes from active_storage (originally 20211119233751)
 class RemoveNotNullOnActiveStorageBlobsChecksum < ActiveRecord::Migration[6.0]
   def change

data/db/migrate/20240804235210_create_panda_cms_forms.rb

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

data/db/migrate/20240805013612_create_panda_cms_form_submissions.rb

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

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,5 +1,9 @@
+# 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}
+    unless column_exists?(:panda_cms_posts, :author_id)
+      add_reference :panda_cms_posts, :author, type: :uuid, foreign_key: {to_table: :panda_core_users}
+    end
   end
 end

data/db/migrate/20250120235542_remove_paper_trail.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class RemovePaperTrail < ActiveRecord::Migration[7.1]
   def up
     version_tables = %w[
@@ -11,9 +13,7 @@ class RemovePaperTrail < ActiveRecord::Migration[7.1]
     ]
 
     version_tables.each do |table|
-      if table_exists?(table)
-        drop_table table
-      end
+      drop_table table if table_exists?(table)
     end
   end
 
@@ -36,7 +36,8 @@ class RemovePaperTrail < ActiveRecord::Migration[7.1]
       t.string :foreign_type
     end
     add_index :panda_cms_version_associations, [:version_id]
-    add_index :panda_cms_version_associations, [:foreign_key_name, :foreign_key_id, :foreign_type], name: "index_version_associations_on_foreign_key"
+    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|

data/db/migrate/20250126234001_create_panda_social_instagram_posts.rb

@@ -1,5 +1,9 @@
+# frozen_string_literal: true
+
 class CreatePandaSocialInstagramPosts < ActiveRecord::Migration[7.1]
   def change
+    return if table_exists?(:panda_social_instagram_posts)
+
     create_table :panda_social_instagram_posts, id: :uuid do |t|
       t.string :instagram_id, null: false
       t.text :caption

data/db/migrate/20250809231125_migrate_users_to_panda_core.rb

@@ -0,0 +1,111 @@
+class MigrateUsersToPandaCore < ActiveRecord::Migration[8.0]
+  def up
+    # First ensure panda_core_users table exists (from Core engine)
+    # It should already exist from the Core migration
+
+    # Migrate data from panda_cms_users to panda_core_users if needed
+    if table_exists?(:panda_cms_users) && table_exists?(:panda_core_users)
+      # Check if there's any data to migrate
+      cms_user_count = ActiveRecord::Base.connection.select_value("SELECT COUNT(*) FROM panda_cms_users")
+      return if cms_user_count == 0
+      # Copy all user data
+      execute <<-SQL
+        INSERT INTO panda_core_users (
+          id, name, email, image_url, is_admin, created_at, updated_at
+        )
+        SELECT 
+          id,
+          COALESCE(name, CONCAT(firstname, ' ', lastname), 'Unknown User'),
+          email,
+          image_url,
+          COALESCE(admin, false),
+          created_at,
+          updated_at
+        FROM panda_cms_users
+        WHERE NOT EXISTS (
+          SELECT 1 FROM panda_core_users WHERE panda_core_users.id = panda_cms_users.id
+        )
+      SQL
+
+      # Update foreign key references in other tables
+
+      # Posts author_id
+      if column_exists?(:panda_cms_posts, :author_id)
+        remove_foreign_key :panda_cms_posts, column: :author_id if foreign_key_exists?(:panda_cms_posts, :panda_cms_users, column: :author_id)
+        add_foreign_key :panda_cms_posts, :panda_core_users, column: :author_id, primary_key: :id
+      end
+
+      # Posts user_id (legacy column)
+      if column_exists?(:panda_cms_posts, :user_id)
+        remove_foreign_key :panda_cms_posts, column: :user_id if foreign_key_exists?(:panda_cms_posts, :panda_cms_users, column: :user_id)
+        add_foreign_key :panda_cms_posts, :panda_core_users, column: :user_id, primary_key: :id
+      end
+
+      # Visits user_id
+      if column_exists?(:panda_cms_visits, :user_id)
+        remove_foreign_key :panda_cms_visits, column: :user_id if foreign_key_exists?(:panda_cms_visits, :panda_cms_users, column: :user_id)
+        add_foreign_key :panda_cms_visits, :panda_core_users, column: :user_id, primary_key: :id
+      end
+
+      # Drop the old table
+      drop_table :panda_cms_users
+    end
+  end
+
+  def down
+    # Recreate panda_cms_users table
+    create_table :panda_cms_users, id: :uuid do |t|
+      t.string :name
+      t.string :firstname
+      t.string :lastname
+      t.string :email, null: false
+      t.string :image_url
+      t.boolean :admin, default: false
+      t.string :current_theme, default: "default"
+      t.timestamps
+    end
+
+    add_index :panda_cms_users, :email, unique: true
+
+    # Migrate data back
+    if table_exists?(:panda_core_users) && table_exists?(:panda_cms_users)
+      execute <<-SQL
+        INSERT INTO panda_cms_users (
+          id, firstname, lastname, name, email, image_url, admin, current_theme, created_at, updated_at
+        )
+        SELECT 
+          id,
+          split_part(name, ' ', 1),
+          CASE 
+            WHEN position(' ' IN name) > 0 
+            THEN substring(name FROM position(' ' IN name) + 1)
+            ELSE ''
+          END,
+          name,
+          email,
+          image_url,
+          is_admin,
+          'default',
+          created_at,
+          updated_at
+        FROM panda_core_users
+      SQL
+
+      # Restore foreign keys to panda_cms_users
+      if column_exists?(:panda_cms_posts, :author_id)
+        remove_foreign_key :panda_cms_posts, column: :author_id if foreign_key_exists?(:panda_cms_posts, column: :author_id)
+        add_foreign_key :panda_cms_posts, :panda_cms_users, column: :author_id, primary_key: :id
+      end
+
+      if column_exists?(:panda_cms_posts, :user_id)
+        remove_foreign_key :panda_cms_posts, column: :user_id if foreign_key_exists?(:panda_cms_posts, column: :user_id)
+        add_foreign_key :panda_cms_posts, :panda_cms_users, column: :user_id, primary_key: :id
+      end
+
+      if column_exists?(:panda_cms_visits, :user_id)
+        remove_foreign_key :panda_cms_visits, column: :user_id if foreign_key_exists?(:panda_cms_visits, column: :user_id)
+        add_foreign_key :panda_cms_visits, :panda_cms_users, column: :user_id, primary_key: :id
+      end
+    end
+  end
+end

data/db/migrate/20250811111000_make_post_user_references_nullable.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class MakePostUserReferencesNullable < ActiveRecord::Migration[8.0]
+  def change
+    # Make user_id and author_id nullable for posts
+    # This is needed because fixtures can't reference users from another gem
+    # Tests that need users will set them programmatically
+    change_column_null :panda_cms_posts, :user_id, true
+    change_column_null :panda_cms_posts, :author_id, true
+  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