From d5f6d3c61b5ccee54f1ef58f7bfdbe1063fd09c8 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 3 Aug 2022 12:56:39 -0300
Subject: [PATCH] =?UTF-8?q?no=20compartir=20el=20uuid=20del=20post=20con?=
 =?UTF-8?q?=20su=20indexaci=C3=B3n?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

aunque los uuid son únicos, en el contexto de un sitio los posts son
únicos por cada sitio.  si tenemos sitios duplicados (por ejemplo un
sitio de testing), la indexación funciona pero va moviendo posts de un
panel a otro.

ahora los uuid de cada post se guardan por separado y las indexaciones
tienen sus propios uuids únicos.

por ahora no los estamos usando para nada, pero cuando tengamos
permalinks, los registros van a estar relacionados con sus posts
indexados.
---
 app/models/post/indexable.rb                  |  3 +--
 ...53308_indexed_posts_by_uuid_and_site_id.rb | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+), 2 deletions(-)
 create mode 100644 db/migrate/20220802153308_indexed_posts_by_uuid_and_site_id.rb

diff --git a/app/models/post/indexable.rb b/app/models/post/indexable.rb
index 7757e7f7..48dc6b0d 100644
--- a/app/models/post/indexable.rb
+++ b/app/models/post/indexable.rb
@@ -14,9 +14,8 @@ class Post
       #
       # @return [IndexedPost]
       def to_index
-        IndexedPost.find_or_create_by(id: uuid.value).tap do |indexed_post|
+        IndexedPost.find_or_create_by(post_id: uuid.value, site_id: site.id).tap do |indexed_post|
           indexed_post.layout = layout.name
-          indexed_post.site_id = site.id
           indexed_post.path = path.basename
           indexed_post.locale = locale.value
           indexed_post.dictionary = IndexedPost.to_dictionary(locale: locale.value)
diff --git a/db/migrate/20220802153308_indexed_posts_by_uuid_and_site_id.rb b/db/migrate/20220802153308_indexed_posts_by_uuid_and_site_id.rb
new file mode 100644
index 00000000..e6572ffb
--- /dev/null
+++ b/db/migrate/20220802153308_indexed_posts_by_uuid_and_site_id.rb
@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+# No podemos compartir el uuid entre indexed_posts y posts porque
+# podemos tener sitios duplicados.  Al menos hasta que los sitios de
+# testeo estén integrados en el panel vamos a tener que generar otros
+# UUID.
+class IndexedPostsByUuidAndSiteId < ActiveRecord::Migration[6.1]
+  def up
+    add_column :indexed_posts, :post_id, :uuid, index: true
+
+    IndexedPost.transaction do
+      ActiveRecord::Base.connection.execute('update indexed_posts set post_id = id where post_id is null')
+    end
+  end
+
+  def down
+    remove_column :indexed_posts, :post_id
+  end
+end