From 0e010e804ca78e5c9192af4dd637164af8551f56 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 17:58:14 -0300
Subject: [PATCH 01/14] fix: el valor original viene del documento y se cambia
 una sola vez

---
 app/models/metadata_template.rb | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/app/models/metadata_template.rb b/app/models/metadata_template.rb
index 823443d2..89fd8315 100644
--- a/app/models/metadata_template.rb
+++ b/app/models/metadata_template.rb
@@ -38,18 +38,10 @@ MetadataTemplate = Struct.new(:site, :document, :name, :label, :type,
     "#{cache_key}-#{cache_version}"
   end
 
-  # XXX: Deberíamos sanitizar durante la asignación?
-  def value=(new_value)
-    @value_was = value
-    self[:value] = new_value
-  end
-
   # Siempre obtener el valor actual y solo obtenerlo del documento una
   # vez.
   def value_was
-    return @value_was if instance_variable_defined? '@value_was'
-
-    @value_was = document_value
+    @value_was ||= document_value.nil? ? default_value : document_value
   end
 
   def changed?

From 9fb3601d9274c67202797a6dfefe35cc11a1a413 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 18:04:19 -0300
Subject: [PATCH 02/14] fix: cambiar el valor una sola vez

---
 app/models/metadata_template.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/models/metadata_template.rb b/app/models/metadata_template.rb
index 89fd8315..03d42339 100644
--- a/app/models/metadata_template.rb
+++ b/app/models/metadata_template.rb
@@ -161,7 +161,7 @@ MetadataTemplate = Struct.new(:site, :document, :name, :label, :type,
   # once => el campo solo se puede modificar si estaba vacío
   def writable?
     case layout.metadata.dig(name, 'writable')
-    when 'once' then value.blank?
+    when 'once' then value_was.blank?
     else true
     end
   end

From 42e9d5fd6612887d106a917e72731874da487800 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 18:05:01 -0300
Subject: [PATCH 03/14] feat: poder asignar atributos sin guardar cambios

---
 app/models/post.rb | 39 +++++++++++++++++++++++++++++----------
 1 file changed, 29 insertions(+), 10 deletions(-)

diff --git a/app/models/post.rb b/app/models/post.rb
index 8885897f..60382baf 100644
--- a/app/models/post.rb
+++ b/app/models/post.rb
@@ -15,6 +15,9 @@ class Post
   PUBLIC_ATTRIBUTES = %i[lang date uuid created_at].freeze
   ATTR_SUFFIXES = %w[? =].freeze
 
+  class PostError < StandardError; end
+  class UnknownAttributeError < PostError; end
+
   attr_reader :attributes, :errors, :layout, :site, :document
 
   # TODO: Modificar el historial de Git con callbacks en lugar de
@@ -50,12 +53,33 @@ class Post
     @errors = {}
     @metadata = {}
 
-    # Inicializar valores
+    # Leer el documento si existe
+    # @todo Asignar todos los valores a self[:value] luego de leer
+    document&.read! unless new?
+
+    # Inicializar valores o modificar los que vengan del documento
+    assignable_attributes = args.slice(*attributes)
+    assign_attributes(assignable_attributes) if assignable_attributes.present?
+  end
+
+  # Asignar atributos, ignorando atributos que no se pueden modificar
+  # o inexistentes
+  #
+  # @param attrs [Hash]
+  def assign_attributes(attrs)
+    attrs = attrs.transform_keys(&:to_sym)
+
     attributes.each do |attr|
-      public_send(attr)&.value = args[attr] if args.key?(attr)
+      self[attr].value = attrs[attr] if attrs.key?(attr) && self[attr].writable?
     end
 
-    document.read! unless new?
+    unknown_attrs = attrs.keys.map(&:to_sym) - attributes
+
+    if unknown_attrs.present?
+      raise UnknownAttributeError, "Unknown attribute(s) #{unknown_attrs.map(&:to_s).join(', ')} for Post"
+    end
+
+    nil
   end
 
   def inspect
@@ -165,8 +189,7 @@ class Post
     # Limpiar el nombre del atributo, para que todos los ayudantes
     # reciban el método en limpio
     unless attribute? name
-      raise NoMethodError, I18n.t('exceptions.post.no_method',
-                                  method: name)
+      raise UnknownAttributeError, I18n.t('exceptions.post.no_method', method: name)
     end
 
     define_singleton_method(name) do
@@ -386,11 +409,7 @@ class Post
   end
 
   def update_attributes(hashable)
-    hashable.to_hash.each do |attr, value|
-      next unless self[attr].writable?
-
-      self[attr].value = value
-    end
+    assign_attributes(hashable)
 
     save
   end

From ce3ff1bdab28f0e0b19079c19c7374150f7d1bb1 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 18:06:56 -0300
Subject: [PATCH 04/14] refactor: asignar atributos

---
 app/services/post_service.rb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/services/post_service.rb b/app/services/post_service.rb
index 4631a9a4..5770a717 100644
--- a/app/services/post_service.rb
+++ b/app/services/post_service.rb
@@ -10,13 +10,14 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
     self.post = site.posts(lang: locale)
                     .build(layout: layout)
     post.usuaries << usuarie
-    params[:post][:draft] = true if site.invitade? usuarie
+    post.draft.value = true if site.invitade? usuarie
+    post.assign_attributes(post_params)
 
     params.require(:post).permit(:slug).tap do |p|
       post.slug.value = p[:slug] if p[:slug].present?
     end
 
-    commit(action: :created, add: update_related_posts) if post.update(post_params)
+    commit(action: :created, add: update_related_posts) if post.save
 
     update_site_license!
 

From 363e2d8385dcc8f822f7a39045522c8c8dfb29ad Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 18:07:42 -0300
Subject: [PATCH 05/14] refactor: llevar un staging de archivos a agregar

---
 app/services/post_service.rb | 25 ++++++++++++++++++++-----
 1 file changed, 20 insertions(+), 5 deletions(-)

diff --git a/app/services/post_service.rb b/app/services/post_service.rb
index 5770a717..20256d68 100644
--- a/app/services/post_service.rb
+++ b/app/services/post_service.rb
@@ -17,7 +17,9 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
       post.slug.value = p[:slug] if p[:slug].present?
     end
 
-    commit(action: :created, add: update_related_posts) if post.save
+    update_related_posts
+
+    commit(action: :created, add: files) if post.save
 
     update_site_license!
 
@@ -35,7 +37,7 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
     # Los artículos anónimos siempre son borradores
     params[:draft] = true
 
-    commit(action: :created, add: [post.path.absolute]) if post.update(anon_post_params)
+    commit(action: :created, add: files) if post.update(anon_post_params)
     post
   end
 
@@ -48,9 +50,11 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
       rm = []
       rm << post.path.value_was if post.path.changed?
 
+      update_related_posts
+
       # Es importante que el artículo se guarde primero y luego los
       # relacionados.
-      commit(action: :updated, add: update_related_posts, rm: rm)
+      commit(action: :updated, add: files, rm: rm)
 
       update_site_license!
     end
@@ -97,6 +101,15 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
 
   private
 
+  # Una lista de archivos a modificar
+  #
+  # @return [Set]
+  def files
+    @files ||= Set.new.tap do |f|
+      f << post.path.absolute
+    end
+  end
+
   def commit(action:, add: [], rm: [])
     site.repository.commit(add: add,
                            rm: rm,
@@ -147,8 +160,10 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
     end
 
     posts.map do |p|
-      p.path.absolute if p.save(validate: false)
-    end.compact << post.path.absolute
+      next unless p.save(validate: false)
+
+      files << p.path.absolute
+    end
   end
 
   # Si les usuaries modifican o crean una licencia, considerarla

From 5e2bddf7cc49fbd8519698d871eeeea09ae5b7fd Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 18:09:05 -0300
Subject: [PATCH 06/14] =?UTF-8?q?feat:=20crear=20art=C3=ADculos=20anidados?=
 =?UTF-8?q?=20#15066?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/models/metadata_template.rb |  4 ++++
 app/models/post.rb              |  7 +++++++
 app/services/post_service.rb    | 20 +++++++++++++++++++-
 3 files changed, 30 insertions(+), 1 deletion(-)

diff --git a/app/models/metadata_template.rb b/app/models/metadata_template.rb
index 03d42339..5af68099 100644
--- a/app/models/metadata_template.rb
+++ b/app/models/metadata_template.rb
@@ -12,6 +12,10 @@ MetadataTemplate = Struct.new(:site, :document, :name, :label, :type,
     false
   end
 
+  def nested?
+    false
+  end
+
   def inspect
     "#<#{self.class} site=#{site.name.inspect} post=#{post.id.inspect} value=#{value.inspect}>"
   end
diff --git a/app/models/post.rb b/app/models/post.rb
index 60382baf..953598a9 100644
--- a/app/models/post.rb
+++ b/app/models/post.rb
@@ -423,6 +423,13 @@ class Post
     @usuaries ||= document_usuaries.empty? ? [] : Usuarie.where(id: document_usuaries).to_a
   end
 
+  # Todos los atributos anidados
+  #
+  # @return [Array<Symbol>]
+  def nested_attributes
+    @nested_attributes ||= attributes.map { |a| self[a] }.select(&:nested?).map(&:name)
+  end
+
   private
 
   # Levanta un error si al construir el artículo no pasamos un atributo.
diff --git a/app/services/post_service.rb b/app/services/post_service.rb
index 20256d68..08a46ae7 100644
--- a/app/services/post_service.rb
+++ b/app/services/post_service.rb
@@ -17,6 +17,8 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
       post.slug.value = p[:slug] if p[:slug].present?
     end
 
+    # Crea los posts anidados
+    create_nested_posts! post, params[:post]
     update_related_posts
 
     commit(action: :created, add: files) if post.save
@@ -122,7 +124,7 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
 
   # Solo permitir cambiar estos atributos de cada articulo
   def post_params
-    params.require(:post).permit(post.params)
+    @post_params ||= params.require(:post).permit(post.params).to_h
   end
 
   # Eliminar metadatos internos
@@ -173,4 +175,20 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
       site.update licencia: Licencia.find_by_icons('custom')
     end
   end
+
+  # Encuentra todos los posts anidados y los crea o modifica
+  def create_nested_posts!(post, params)
+    post.nested_attributes.each do |nested_attribute|
+     nested_metadata = post[nested_attribute]
+     # @todo find_or_initialize
+     nested_post = site.posts(lang: post.lang.value).build(layout: nested_metadata.nested)
+     nested_params = params.require(nested_attribute).permit(nested_post.params).to_hash
+
+     # Completa la relación 1:1
+     nested_params[nested_metadata.inverse.to_s] = post.uuid.value
+     post[nested_attribute].value = nested_post.uuid.value
+
+     files << nested_post.path.absolute if nested_post.update(nested_params)
+    end
+  end
 end

From 0cf2ec04ba9b54c4e74001923003dc308a60ca04 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Wed, 17 Apr 2024 18:09:55 -0300
Subject: [PATCH 07/14] =?UTF-8?q?feat:=20poder=20anidar=20un=20art=C3=ADcu?=
 =?UTF-8?q?lo=20#15066?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/models/metadata_has_one_nested.rb         | 11 +++++++++++
 app/views/posts/_attributes.haml              | 16 ++++++++++++++++
 app/views/posts/_attributes_nested.haml       | 19 +++++++++++++++++++
 app/views/posts/_form.haml                    | 11 +----------
 .../posts/attribute_ro/_has_one_nested.haml   |  6 ++++++
 .../posts/attributes/_has_one_nested.haml     |  6 ++++++
 6 files changed, 59 insertions(+), 10 deletions(-)
 create mode 100644 app/models/metadata_has_one_nested.rb
 create mode 100644 app/views/posts/_attributes.haml
 create mode 100644 app/views/posts/_attributes_nested.haml
 create mode 100644 app/views/posts/attribute_ro/_has_one_nested.haml
 create mode 100644 app/views/posts/attributes/_has_one_nested.haml

diff --git a/app/models/metadata_has_one_nested.rb b/app/models/metadata_has_one_nested.rb
new file mode 100644
index 00000000..ed509013
--- /dev/null
+++ b/app/models/metadata_has_one_nested.rb
@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class MetadataHasOneNested < MetadataHasOne
+  def nested
+    @nested ||= layout.metadata.dig(name, 'nested')
+  end
+
+  def nested?
+    true
+  end
+end
diff --git a/app/views/posts/_attributes.haml b/app/views/posts/_attributes.haml
new file mode 100644
index 00000000..ed958d08
--- /dev/null
+++ b/app/views/posts/_attributes.haml
@@ -0,0 +1,16 @@
+-#
+  @param base [String]
+  @param locale [String]
+  @param post [Post]
+  @param site [Site]
+  @param dir [String]
+- post.attributes.each do |attribute|
+  - metadata = post[attribute]
+  - type = metadata.type
+
+  - cache [metadata, I18n.locale] do
+    = render("posts/attributes/#{type}",
+            base: base, post: post, attribute: attribute,
+            metadata: metadata, site: site,
+            dir: dir, locale: locale,
+            autofocus: (post.attributes.first == attribute))
diff --git a/app/views/posts/_attributes_nested.haml b/app/views/posts/_attributes_nested.haml
new file mode 100644
index 00000000..33996aa3
--- /dev/null
+++ b/app/views/posts/_attributes_nested.haml
@@ -0,0 +1,19 @@
+-#
+  @param inverse [Symbol]
+  @param base [String]
+  @param locale [String]
+  @param post [Post]
+  @param site [Site]
+  @param dir [String]
+- post.attributes.each do |attribute|
+  - next if attribute == :date
+  - next if attribute == :draft
+  - next if attribute == inverse
+  - metadata = post[attribute]
+  - type = metadata.type
+
+  - cache [metadata, I18n.locale] do
+    = render "posts/attributes/#{type}",
+      base: base, post: post, attribute: attribute,
+      metadata: metadata, site: site,
+      dir: dir, locale: locale, autofocus: false
diff --git a/app/views/posts/_form.haml b/app/views/posts/_form.haml
index 7de0ea79..92bee939 100644
--- a/app/views/posts/_form.haml
+++ b/app/views/posts/_form.haml
@@ -41,16 +41,7 @@
   = hidden_field_tag 'post[layout]', post.layout.name
 
   -# Dibuja cada atributo
-  - post.attributes.each do |attribute|
-    - metadata = post[attribute]
-    - type = metadata.type
-
-    - cache [metadata, I18n.locale] do
-      = render("posts/attributes/#{type}",
-              base: 'post', post: post, attribute: attribute,
-              metadata: metadata, site: site,
-              dir: dir, locale: @locale,
-              autofocus: (post.attributes.first == attribute))
+  = render 'posts/attributes', site: site, post: post, dir: dir, base: 'post', locale: @locale
 
   -# Botones de guardado
   = render 'posts/submit', site: site, post: post
diff --git a/app/views/posts/attribute_ro/_has_one_nested.haml b/app/views/posts/attribute_ro/_has_one_nested.haml
new file mode 100644
index 00000000..425e659e
--- /dev/null
+++ b/app/views/posts/attribute_ro/_has_one_nested.haml
@@ -0,0 +1,6 @@
+%tr{ id: attribute }
+  %th= post_label_t(attribute, post: post)
+  %td{ dir: dir, lang: locale }
+    - p = metadata.has_one
+    - if p
+      = link_to p.title.value, site_post_path(site, p.id)
diff --git a/app/views/posts/attributes/_has_one_nested.haml b/app/views/posts/attributes/_has_one_nested.haml
new file mode 100644
index 00000000..e98bff47
--- /dev/null
+++ b/app/views/posts/attributes/_has_one_nested.haml
@@ -0,0 +1,6 @@
+- new_post = site.posts(lang: locale).build(layout: metadata.nested)
+- base = "#{base}[#{metadata.name}]"
+
+.form-group
+  = render 'layouts/details', id: metadata.nested, summary: site.layouts[metadata.nested].humanized_name do
+    = render 'posts/attributes_nested', site: site, post: new_post, dir: dir, base: base, locale: locale, inverse: metadata.inverse

From 04c0dffb26c639bd15ed1c934f2b3de587eea370 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Fri, 19 Apr 2024 15:00:48 -0300
Subject: [PATCH 08/14] fix: ignorar has_one_nested

---
 app/models/metadata_has_one_nested.rb | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/models/metadata_has_one_nested.rb b/app/models/metadata_has_one_nested.rb
index ed509013..2f3e8e02 100644
--- a/app/models/metadata_has_one_nested.rb
+++ b/app/models/metadata_has_one_nested.rb
@@ -8,4 +8,9 @@ class MetadataHasOneNested < MetadataHasOne
   def nested?
     true
   end
+
+  # No tener conflictos con related
+  def related_methods
+    @related_methods ||= [].freeze
+  end
 end

From b05e0356c17623c83967110111249619c313176a Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Fri, 19 Apr 2024 15:01:16 -0300
Subject: [PATCH 09/14] fix: incluir el post en la cache

---
 app/views/posts/_attributes_nested.haml | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/app/views/posts/_attributes_nested.haml b/app/views/posts/_attributes_nested.haml
index 33996aa3..83bcc51c 100644
--- a/app/views/posts/_attributes_nested.haml
+++ b/app/views/posts/_attributes_nested.haml
@@ -10,10 +10,9 @@
   - next if attribute == :draft
   - next if attribute == inverse
   - metadata = post[attribute]
-  - type = metadata.type
 
-  - cache [metadata, I18n.locale] do
-    = render "posts/attributes/#{type}",
+  - cache [post, metadata, I18n.locale] do
+    = render "posts/attributes/#{metadata.type}",
       base: base, post: post, attribute: attribute,
       metadata: metadata, site: site,
       dir: dir, locale: locale, autofocus: false

From bcf3925d9bb87089a14047f7b4a6f13897bd57af Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Fri, 19 Apr 2024 15:06:17 -0300
Subject: [PATCH 10/14] feat: poder modificar posts anidados

---
 app/services/post_service.rb                    | 3 ++-
 app/views/posts/attributes/_has_one_nested.haml | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/services/post_service.rb b/app/services/post_service.rb
index 08a46ae7..90aa9401 100644
--- a/app/services/post_service.rb
+++ b/app/services/post_service.rb
@@ -52,6 +52,7 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
       rm = []
       rm << post.path.value_was if post.path.changed?
 
+      create_nested_posts! post, params[:post]
       update_related_posts
 
       # Es importante que el artículo se guarde primero y luego los
@@ -181,7 +182,7 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
     post.nested_attributes.each do |nested_attribute|
      nested_metadata = post[nested_attribute]
      # @todo find_or_initialize
-     nested_post = site.posts(lang: post.lang.value).build(layout: nested_metadata.nested)
+     nested_post = nested_metadata.has_one || site.posts(lang: post.lang.value).build(layout: nested_metadata.nested)
      nested_params = params.require(nested_attribute).permit(nested_post.params).to_hash
 
      # Completa la relación 1:1
diff --git a/app/views/posts/attributes/_has_one_nested.haml b/app/views/posts/attributes/_has_one_nested.haml
index e98bff47..4aabf386 100644
--- a/app/views/posts/attributes/_has_one_nested.haml
+++ b/app/views/posts/attributes/_has_one_nested.haml
@@ -1,6 +1,6 @@
-- new_post = site.posts(lang: locale).build(layout: metadata.nested)
+- nested_post = metadata.has_one || site.posts(lang: locale).build(layout: metadata.nested)
 - base = "#{base}[#{metadata.name}]"
 
 .form-group
   = render 'layouts/details', id: metadata.nested, summary: site.layouts[metadata.nested].humanized_name do
-    = render 'posts/attributes_nested', site: site, post: new_post, dir: dir, base: base, locale: locale, inverse: metadata.inverse
+    = render 'posts/attributes_nested', site: site, post: nested_post, dir: dir, base: base, locale: locale, inverse: metadata.inverse

From 382f0545d128cb277a9d89b31fc88052bf1ae7fa Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Fri, 19 Apr 2024 15:06:56 -0300
Subject: [PATCH 11/14] feat: mostrar detalles como desplegable

---
 app/views/layouts/_details.haml               | 17 ++++++++++
 app/views/posts/_table.haml                   | 34 +++++++++++++++++++
 .../posts/attribute_ro/_has_one_nested.haml   | 10 +++---
 app/views/posts/show.haml                     | 28 ++-------------
 4 files changed, 58 insertions(+), 31 deletions(-)
 create mode 100644 app/views/layouts/_details.haml
 create mode 100644 app/views/posts/_table.haml

diff --git a/app/views/layouts/_details.haml b/app/views/layouts/_details.haml
new file mode 100644
index 00000000..9f8a4658
--- /dev/null
+++ b/app/views/layouts/_details.haml
@@ -0,0 +1,17 @@
+-#
+  Detail Cola de Moderación
+
+  @param :id [String] El ID opcional sirve para mantener el historial de
+  cuál estaba abierto y recuperarlo al cargar la página
+  @param :summary [String] El resumen
+  @param :summary_class [String] Clases para el summary
+
+- local_assigns[:summary_class] ||= 'h3'
+- local_assigns[:details_class] ||= 'py-2'
+
+%details.details{ id: local_assigns[:id], class: local_assigns[:details_class], data: { controller: 'details', action: 'toggle->details#store' } }
+  %summary.d-flex.flex-row.align-items-center.justify-content-between{ class: local_assigns[:summary_class] }
+    %span= summary
+    %span.hide-when-open &#x25B6;
+    %span.show-when-open &#x25BC;
+  = yield
diff --git a/app/views/posts/_table.haml b/app/views/posts/_table.haml
new file mode 100644
index 00000000..ff35aace
--- /dev/null
+++ b/app/views/posts/_table.haml
@@ -0,0 +1,34 @@
+-#
+  Muestra una tabla con todos los atributos de un post
+
+  @param site [Site]
+  @param locale [Symbol]
+  @param dir [String]
+  @param post [Post]
+  @param title [String]
+
+%table.table.table-condensed
+  %thead
+    %tr
+      %th.text-center{ colspan: 2 }= title
+  %tbody
+    - post.attributes.each do |attr|
+      - metadata = post[attr]
+      - next unless metadata.front_matter?
+
+      - cache [post, metadata, I18n.locale] do
+        = render("posts/attribute_ro/#{metadata.type}",
+                 post: post, attribute: attr,
+                 metadata: metadata,
+                 site: site,
+                 locale: locale,
+                 dir: dir)
+
+-# Mostrar todo lo que no va en el front_matter (el contenido)
+- post.attributes.each do |attr|
+  - metadata = post[attr]
+  - next if metadata.front_matter?
+
+  - cache [post, metadata, I18n.locale] do
+    %section.content.pb-3{ id: attr, dir: dir }
+      = metadata.to_s.html_safe
diff --git a/app/views/posts/attribute_ro/_has_one_nested.haml b/app/views/posts/attribute_ro/_has_one_nested.haml
index 425e659e..1c89474e 100644
--- a/app/views/posts/attribute_ro/_has_one_nested.haml
+++ b/app/views/posts/attribute_ro/_has_one_nested.haml
@@ -1,6 +1,6 @@
 %tr{ id: attribute }
-  %th= post_label_t(attribute, post: post)
-  %td{ dir: dir, lang: locale }
-    - p = metadata.has_one
-    - if p
-      = link_to p.title.value, site_post_path(site, p.id)
+  %td{ dir: dir, lang: locale, colspan: 2 }
+    - if (p = metadata.has_one)
+      = render 'layouts/details', details_class: '', summary_class: 'font-weight-bold', summary: post_label_t(attribute, post: post) do
+        .mt-3
+          = render 'posts/table', site: site, post: p, dir: dir, locale: locale, title: p.layout.humanized_name
diff --git a/app/views/posts/show.haml b/app/views/posts/show.haml
index 10900d67..6798c41c 100644
--- a/app/views/posts/show.haml
+++ b/app/views/posts/show.haml
@@ -1,4 +1,5 @@
 - dir = @site.data.dig(params[:locale], 'dir')
+
 .row.justify-content-center
   .col-md-8
     %article.content.table-responsive-md
@@ -6,29 +7,4 @@
         edit_site_post_path(@site, @post.id),
         class: 'btn btn-secondary btn-block'
 
-      %table.table.table-condensed
-        %thead
-          %tr
-            %th.text-center{ colspan: 2 }= t('.front_matter')
-        %tbody
-          - @post.attributes.each do |attr|
-            - metadata = @post[attr]
-            - next unless metadata.front_matter?
-
-            - cache [metadata, I18n.locale] do
-              = render("posts/attribute_ro/#{metadata.type}",
-                       post: @post, attribute: attr,
-                       metadata: metadata,
-                       site: @site,
-                       tags: all_html_tags,
-                       locale: @locale,
-                       dir: dir)
-
-      -# Mostrar todo lo que no va en el front_matter (el contenido)
-      - @post.attributes.each do |attr|
-        - metadata = @post[attr]
-        - next if metadata.front_matter?
-
-        - cache [metadata, I18n.locale] do
-          %section.content.pb-3{ id: attr, dir: dir }
-            = @post.public_send(attr).to_s.html_safe
+      = render 'table', dir: dir, site: @site, locale: @locale, post: @post, title: t('.front_matter')

From 9d902fdd499764e9dbd1e25e625aa3b38b0ae0c4 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Sat, 20 Apr 2024 11:24:09 -0300
Subject: [PATCH 12/14] fix: es necesario NoMethodError para poder hacer
 flatten luego

closes #16100
---
 app/models/post.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/models/post.rb b/app/models/post.rb
index 953598a9..327df3e2 100644
--- a/app/models/post.rb
+++ b/app/models/post.rb
@@ -189,7 +189,7 @@ class Post
     # Limpiar el nombre del atributo, para que todos los ayudantes
     # reciban el método en limpio
     unless attribute? name
-      raise UnknownAttributeError, I18n.t('exceptions.post.no_method', method: name)
+      raise NoMethodError, I18n.t('exceptions.post.no_method', method: name)
     end
 
     define_singleton_method(name) do

From 88c9399ba5ee4054c792ddac899ec9f3823e0946 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Tue, 23 Apr 2024 14:09:26 -0300
Subject: [PATCH 13/14] fix: no guardar hashwithindifferentaccess

---
 app/models/metadata_file.rb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/models/metadata_file.rb b/app/models/metadata_file.rb
index 3ac89c9b..61aea897 100644
--- a/app/models/metadata_file.rb
+++ b/app/models/metadata_file.rb
@@ -49,6 +49,8 @@ class MetadataFile < MetadataTemplate
       value['path'] = relative_destination_path_with_filename.to_s if static_file
     end
 
+    self[:value] = self[:value].to_h
+
     true
   end
 

From b3ed65b340dff909001ccd699944d6ee8c424e93 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Tue, 23 Apr 2024 14:11:58 -0300
Subject: [PATCH 14/14] fix: primero guardar para que se actualicen las
 relaciones

---
 app/services/post_service.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/services/post_service.rb b/app/services/post_service.rb
index 90aa9401..84f58dad 100644
--- a/app/services/post_service.rb
+++ b/app/services/post_service.rb
@@ -19,9 +19,10 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
 
     # Crea los posts anidados
     create_nested_posts! post, params[:post]
+    post.save
     update_related_posts
 
-    commit(action: :created, add: files) if post.save
+    commit(action: :created, add: files)
 
     update_site_license!