Merge branch 'issue-16509' of 0xacab.org:sutty/sutty into panel.testing.sutty.nl

2024-11-24 20:46:22 +00:00 · 2024-06-14 16:35:41 -03:00 · 2024-06-14 16:35:41 -03:00 · a8d4ca2cfa
commit a8d4ca2cfa
parent edf4cd90ad 7f6730c1e6
53 changed files with 566 additions and 409 deletions
--- a/app/assets/stylesheets/application.scss
+++ b/app/assets/stylesheets/application.scss
@ -670,3 +670,20 @@ a.black {
  }
 }

+.break-all {
+  word-break: break-all;
+}
+
+.details-agregar {
+  @extend .d-flex;
+  @extend .border;
+  @extend .border-magenta;
+  @extend .justify-content-between;
+  @extend .align-items-center;
+  @extend .w-100;
+  @extend .mb-3;
+}
+
+.btn-sm-queries {
+  @include button-size($btn-padding-y-sm, $btn-padding-x-sm, $btn-font-size-sm, $btn-line-height, $btn-border-radius-sm);
+}
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@ -86,9 +86,7 @@ class ApplicationController < ActionController::Base
  end

  def site
-    @site ||= find_site.tap do |s|
-      s.reindex_changes!
-    end
+    @site ||= find_site.tap(&:reindex_changes!)
  end

  protected
--- a/app/controllers/stats_controller.rb
+++ b/app/controllers/stats_controller.rb
@ -175,7 +175,7 @@ class StatsController < ApplicationController
      locale: I18n.locale,
      empty: I18n.t('stats.index.empty', **please_return_at),
      loading: I18n.t('stats.index.loading'),
-      html: %(<div id="%{id}" class="d-flex align-items-center justify-content-center" style="height: %{height}; width: %{width};">%{loading}</div>)
+      html: %(<div id="%<id>s" class="d-flex align-items-center justify-content-center" style="height: %<height>s; width: %<width>s;">%<loading>s</div>)
    }
  end

--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@ -30,7 +30,6 @@ module ApplicationHelper
      if k == key
        case v
        when Array then [k, v - [value]]
-        else nil
        end
      else
        [k, v]
--- a/app/models/deploy_distributed_press.rb
+++ b/app/models/deploy_distributed_press.rb
@ -17,7 +17,7 @@ class DeployDistributedPress < Deploy
  before_create :create_remote_site!
  before_destroy :delete_remote_site!

-  DEPENDENCIES = %i[deploy_local]
+  DEPENDENCIES = %i[deploy_local].freeze

  # Actualiza la información y luego envía los cambios
  #
@ -32,9 +32,7 @@ class DeployDistributedPress < Deploy
    create_remote_site! if remote_site_id.blank?
    save

-    if remote_site_id.blank?
-      raise DeployJob::DeployException, 'El sitio no se creó en Distributed Press'
-    end
+    raise DeployJob::DeployException, 'El sitio no se creó en Distributed Press' if remote_site_id.blank?

    site_client.tap do |c|
      stdout = Thread.new(publisher.logger_out) do |io|
@ -54,7 +52,7 @@ class DeployDistributedPress < Deploy
      end

      if status
-        self.remote_info[:distributed_press] = c.show(publishing_site).to_h
+        remote_info[:distributed_press] = c.show(publishing_site).to_h
        save
      end

@ -123,7 +121,10 @@ class DeployDistributedPress < Deploy
  #
  # @return [DistributedPressPublisher::V1::Schemas::NewSite]
  def create_site
-    DistributedPress::V1::Schemas::NewSite.new.call(domain: hostname, protocols: { http: true, ipfs: true, hyper: true })
+    DistributedPress::V1::Schemas::NewSite.new.call(domain: hostname,
+                                                    protocols: {
+                                                      http: true, ipfs: true, hyper: true
+                                                    })
  end

  # Crea el sitio en la instancia con el hostname especificado
--- a/app/models/deploy_local.rb
+++ b/app/models/deploy_local.rb
@ -86,7 +86,7 @@ class DeployLocal < Deploy
      'AIRBRAKE_PROJECT_ID' => site.id.to_s,
      'AIRBRAKE_PROJECT_KEY' => site.airbrake_api_key,
      'YARN_CACHE_FOLDER' => yarn_cache_dir,
-      'GEMS_SOURCE' => ENV['GEMS_SOURCE']
+      'GEMS_SOURCE' => ENV.fetch('GEMS_SOURCE', nil)
    }
  end

--- a/app/models/deploy_rsync.rb
+++ b/app/models/deploy_rsync.rb
@ -5,7 +5,7 @@
 class DeployRsync < Deploy
  store :values, accessors: %i[hostname destination host_keys], coder: JSON

-  DEPENDENCIES = %i[deploy_local deploy_zip]
+  DEPENDENCIES = %i[deploy_local deploy_zip].freeze

  def deploy(output: false)
    ssh? && rsync(output: output)
@ -39,6 +39,7 @@ class DeployRsync < Deploy
  # @return [Boolean]
  def ssh?
    return true if destination.start_with? 'rsync://'
+
    user, host = user_host
    ssh_available = false

@ -66,7 +67,7 @@ class DeployRsync < Deploy
    {
      'HOME' => home_dir,
      'PATH' => '/usr/bin',
-      'LANG' => ENV['LANG']
+      'LANG' => ENV.fetch('LANG', nil)
    }
  end

@ -92,7 +93,8 @@ class DeployRsync < Deploy
  #
  # @return [Boolean]
  def rsync(output: false)
-    run %(rsync -aviH --delete-after --timeout=5 #{Shellwords.escape source}/ #{Shellwords.escape destination}/), output: output
+    run %(rsync -aviH --delete-after --timeout=5 #{Shellwords.escape source}/ #{Shellwords.escape destination}/),
+        output: output
  end

  # El origen es el destino de la compilación
--- a/app/models/indexed_post.rb
+++ b/app/models/indexed_post.rb
@ -38,15 +38,15 @@ class IndexedPost < ApplicationRecord
  #
  # @param :attribute [String,Symbol]
  # @return [Array]
-  scope :everything_of, ->(attribute) do
+  scope :everything_of, lambda { |attribute|
    where('front_matter ? :attribute', attribute: attribute)
      .pluck(
        Arel.sql(
-          ActiveRecord::Base::sanitize_sql(['front_matter -> :attribute', attribute: attribute])
+          ActiveRecord::Base.sanitize_sql(['front_matter -> :attribute', { attribute: attribute }])
        )
      )
      .flatten.uniq
-  end
+  }

  validates_presence_of :layout, :path, :locale

--- a/app/models/metadata_belongs_to.rb
+++ b/app/models/metadata_belongs_to.rb
@ -29,6 +29,6 @@ class MetadataBelongsTo < MetadataRelatedPosts
  private

  def sanitize(uuid)
-    uuid.to_s.gsub(/[^a-f0-9\-]/i, '')
+    uuid.to_s.gsub(/[^a-f0-9-]/i, '')
  end
 end
--- a/app/models/metadata_content.rb
+++ b/app/models/metadata_content.rb
@ -53,18 +53,15 @@ class MetadataContent < MetadataTemplate

    # Eliminar elementos sin src y comprobar su origen
    html.css(elements).each do |element|
-      begin
      raise URI::Error unless element['src'].present?

      uri = URI element['src']

      # No permitimos recursos externos, solo si sabemos cuales son
      # los recursos locales
-        if Rails.application.config.hosts.present?
-          unless Rails.application.config.hosts.include?(uri.hostname)
+      if Rails.application.config.hosts.present? && !Rails.application.config.hosts.include?(uri.hostname)
        raise URI::Error
      end
-        end

      element['src'] = convert_src_to_internal_path uri

@ -72,7 +69,6 @@ class MetadataContent < MetadataTemplate
    rescue URI::Error
      element.remove
    end
-    end

    # Eliminar figure sin contenido
    html.css('figure').each do |figure|
@ -101,11 +97,10 @@ class MetadataContent < MetadataTemplate
  # @param style [String]
  # @return [String]
  def sanitize_style(style)
-    style.split(';').reduce({}) do |style_hash, style_string|
+    style.split(';').each_with_object({}) do |style_string, style_hash|
      key, value = style_string.split(':', 2)

      style_hash[key] ||= value
-      style_hash
    end.slice(*allowed_styles).map do |style_pair|
      style_pair.join(':')
    end.join(';')
--- a/app/models/metadata_related_posts.rb
+++ b/app/models/metadata_related_posts.rb
@ -10,7 +10,8 @@ class MetadataRelatedPosts < MetadataArray
  def values
    @values ||= posts.pluck(:title, :created_at, :layout, :post_id).to_h do |row|
      row.tap do |value|
-        value[0] = "#{value[0]} #{value.delete_at(1).strftime('%F')} (#{site.layouts[value.delete_at(1)].humanized_name})"
+        value[0] =
+          "#{value[0]} #{value.delete_at(1).strftime('%F')} (#{site.layouts[value.delete_at(1)].humanized_name})"
      end
    end
  end
@ -46,7 +47,7 @@ class MetadataRelatedPosts < MetadataArray

  def sanitize(uuid)
    super(uuid.map do |u|
-      u.to_s.gsub(/[^a-f0-9\-]/i, '')
+      u.to_s.gsub(/[^a-f0-9-]/i, '')
    end)
  end
 end
--- a/app/models/post.rb
+++ b/app/models/post.rb
@ -132,6 +132,7 @@ class Post
        src = element.attributes['src']

        next unless src&.value&.start_with? 'public/'
+
        file = MetadataFile.new(site: site, post: self, document: document, layout: layout)
        file.value['path'] = src.value

@ -250,7 +251,8 @@ class Post

  # La fecha de creación inmodificable del post
  def created_at
-    @metadata[:created_at] ||= MetadataCreatedAt.new(document: document, site: site, layout: layout, name: :created_at, type: :created_at, post: self, required: true)
+    @metadata[:created_at] ||= MetadataCreatedAt.new(document: document, site: site, layout: layout, name: :created_at,
+                                                     type: :created_at, post: self, required: true)
  end

  # Detecta si es un atributo válido o no, a partir de la tabla de la
--- a/app/models/post/indexable.rb
+++ b/app/models/post/indexable.rb
@ -6,7 +6,6 @@ class Post
    extend ActiveSupport::Concern

    included do
-
      # @return [IndexedPost,nil]
      def indexed_post
        site.indexed_posts.find_by_post_id(uuid.value)
--- a/app/models/site.rb
+++ b/app/models/site.rb
@ -419,7 +419,6 @@ class Site < ApplicationRecord
    FileUtils.rm_rf path
  end

-
  # Sincroniza algunos atributos del sitio con su configuración y
  # guarda los cambios
  #
--- a/app/models/site/index.rb
+++ b/app/models/site/index.rb
@ -104,8 +104,8 @@ class Site
        indexable_posts.select do |delta|
          MODIFIED_STATUSES.include? delta.status
        end.each do |delta|
-          locale, _ = locale_and_path_from(delta.new_file[:path])
-          full_path = File.join(self.path, delta.new_file[:path])
+          locale, = locale_and_path_from(delta.new_file[:path])
+          full_path = File.join(path, delta.new_file[:path])

          Post.build(path: full_path, site: self, layout: Post.find_layout(full_path), locale: locale).index!
        end
--- a/app/models/site/repository.rb
+++ b/app/models/site/repository.rb
@ -148,7 +148,7 @@ class Site
    # @param :rm [Array] Archivos a eliminar
    # @param :usuarie [Usuarie] Quién hace el commit
    # @param :message [String] Mensaje
-    def commit(add: [], rm: [], usuarie:, message:)
+    def commit(usuarie:, message:, add: [], rm: [])
      # Cargar el árbol actual
      rugged.index.read_tree rugged.head.target.tree

@ -182,7 +182,7 @@ class Site
    #
    # @return [Boolean]
    def gc
-      git_sh("git", "gc")
+      git_sh('git', 'gc')
    end

    # Pushea cambios al repositorio remoto
@ -196,8 +196,8 @@ class Site

    # Hace limpieza de LFS
    def lfs_cleanup
-      git_sh("git", "lfs", "prune")
-      git_sh("git", "lfs", "dedup")
+      git_sh('git', 'lfs', 'prune')
+      git_sh('git', 'lfs', 'dedup')
    end

    private
@ -257,7 +257,7 @@ class Site
    # @param :args [Array]
    # @return [Boolean]
    def git_sh(*args)
-      env = { 'PATH' => '/usr/bin', 'LANG' => ENV['LANG'], 'HOME' => path }
+      env = { 'PATH' => '/usr/bin', 'LANG' => ENV.fetch('LANG', nil), 'HOME' => path }

      r = nil
      Open3.popen2e(env, *args, unsetenv_others: true, chdir: path) do |_, _, t|
--- a/app/services/post_service.rb
+++ b/app/services/post_service.rb
@ -145,10 +145,10 @@ PostService = Struct.new(:site, :usuarie, :post, :params, keyword_init: true) do
  # Si les usuaries modifican o crean una licencia, considerarla
  # personalizada en el panel.
  def update_site_license!
-    if site.usuarie?(usuarie) && post.layout.name == :license && !site.licencia.custom?
+    return unless site.usuarie?(usuarie) && post.layout.name == :license && !site.licencia.custom?
+
    site.update licencia: Licencia.find_by_icons('custom')
  end
-  end

  # @return [Set<String>]
  def associated_posts_to_save
--- a/app/views/bootstrap/_custom_checkbox_for_field.haml
+++ b/app/views/bootstrap/_custom_checkbox_for_field.haml
@ -2,5 +2,9 @@
 - id = "#{field.object_name}_#{name}"
 - name = "#{field.object_name}[#{name}]"

-= render 'bootstrap/custom_checkbox', id: id, name: name, content: content, required: local_assigns[:required], value: "1" do
+= render 'bootstrap/custom_checkbox', id: id,
+                                      name: name,
+                                      content: content,
+                                      required: local_assigns[:required],
+                                      value: '1' do
  = yield
--- a/app/views/build_stats/index.haml
+++ b/app/views/build_stats/index.haml
@ -1,6 +1,6 @@
 %main.row
  %aside.menu.col-12.col-lg-3
-    = render 'sites/header', site: @site
+    = render 'sites/header', site: @site, filter_params: @filter_params
  .col
    %h1= t('.title')

@ -14,7 +14,10 @@
          - row[:urls].each do |url|
            %tr
              %th{ scope: 'row' }= row[:title]
-              %td= link_to_if (url.present? && url.scheme.present?), url.to_s, url.to_s, class: 'word-break-all'
+              %td= link_to_if (url.present? && url.scheme.present?), 
+                              url.to_s,
+                              url.to_s,
+                              class: 'word-break-all'
              %td
                %time{ datetime: row[:seconds][:machine] }= row[:seconds][:human]
              %td= row[:size]
--- a/app/views/devise/confirmations/new.haml
+++ b/app/views/devise/confirmations/new.haml
@ -12,7 +12,6 @@
               as: resource_name,
               url: confirmation_path(resource_name),
               html: { method: :post }) do |f|
-
      :ruby
        value = if resource.pending_reconfirmation?
          resource.unconfirmed_email
--- a/app/views/devise/mailer/invitation_instructions.html.haml
+++ b/app/views/devise/mailer/invitation_instructions.html.haml
@ -10,7 +10,9 @@

 - if @resource.needs_invitation_link?
  %p= link_to t('devise.mailer.invitation_instructions.accept'),
-    accept_invitation_url(@resource, invitation_token: @token, change_locale_to: @resource.lang)
+              accept_invitation_url(@resource,
+                                    invitation_token: @token,
+                                    change_locale_to: @resource.lang)

  - if @resource.invitation_due_at
    %p= t('devise.mailer.invitation_instructions.accept_until',
@ -19,6 +21,9 @@

  %p= t('devise.mailer.invitation_instructions.ignore')
 - elsif !@resource.confirmed? && @resource.confirmation_token
-  = confirmation_url(@resource, confirmation_token: @resource.confirmation_token, change_locale_to: @resource.lang)
+  = confirmation_url(@resource,
+                     confirmation_token: @resource.confirmation_token,
+                     change_locale_to: @resource.lang)
 - else
-  %p= link_to t('devise.mailer.invitation_instructions.sign_in'), root_url(change_locale_to: @resource.lang)
+  %p= link_to t('devise.mailer.invitation_instructions.sign_in'),
+              root_url(change_locale_to: @resource.lang)
--- a/app/views/devise/passwords/edit.haml
+++ b/app/views/devise/passwords/edit.haml
@ -12,7 +12,6 @@
    = form_for(resource, as: resource_name,
                         url: password_path(resource_name),
                         html: { method: :put }) do |f|
-
      = f.hidden_field :reset_password_token

      .form-group
--- a/app/views/devise/registrations/edit.haml
+++ b/app/views/devise/registrations/edit.haml
@ -12,7 +12,6 @@
               as: resource_name,
               url: registration_path(resource_name),
               html: { method: :put }) do |f|
-
      .form-group
        = f.label :email
        = f.email_field :email, autofocus: true, autocomplete: 'email',
@ -31,7 +30,8 @@
      .form-group
        = f.label :password
        = f.password_field :password, autocomplete: 'new-password',
-          class: 'form-control', aria: { describedby: 'password-help' }
+                                      class: 'form-control',
+                                      aria: { describedby: 'password-help' }
        %small.text-muted.form-text#password-help
          = t('.leave_blank_if_you_don_t_want_to_change_it')
          - if @minimum_password_length
--- a/app/views/devise/registrations/new.haml
+++ b/app/views/devise/registrations/new.haml
@ -11,7 +11,6 @@
    = form_for(resource,
               as: resource_name,
               url: registration_path(resource_name, params: { locale: params[:locale] })) do |f|
-
      .form-group
        = f.label :email, class: 'sr-only'
        = f.email_field :email, autofocus: true, autocomplete: 'email',
@ -23,7 +22,8 @@
      .form-group
        = f.label :password, class: 'sr-only'
        = f.password_field :password, autocomplete: 'new-password',
-          class: 'form-control', min: @minimum_password_length,
+                                      class: 'form-control',
+                                      min: @minimum_password_length,
                                      aria: { describedby: 'minimum-password-length' },
                                      placeholder: t(password)
        - if @minimum_password_length
@ -48,7 +48,12 @@
          - content = t(".#{field}.label")
          - href = t(".#{field}.href", default: '')
          - help_content = t(".#{field}.help")
-          = render 'bootstrap/custom_checkbox', id: id, name: name, content: content, required: required, value: "1" do
+          = render 'bootstrap/custom_checkbox',
+                   id: id,
+                   name: name,
+                   content: content,
+                   required: required,
+                   value: '1' do
            - if href.present?
              = link_to help_content, href, target: '_blank', rel: 'noopener'
            - else
--- a/app/views/devise/sessions/new.haml
+++ b/app/views/devise/sessions/new.haml
@ -28,8 +28,10 @@
                           placeholder: t('login.password')
      - if devise_mapping.rememberable?
        .form-group
-          = render 'bootstrap/custom_checkbox_for_field', field: f, name: :remember_me do
-            = t('login.remember_me', remember_for: distance_of_time_in_words(Usuarie.remember_for))
+          = render 'bootstrap/custom_checkbox_for_field',
+                   field: f, name: :remember_me do
+            = t('login.remember_me',
+                remember_for: distance_of_time_in_words(Usuarie.remember_for))
      .actions
        = f.submit t('.sign_in'),
                   class: 'btn btn-secondary btn-lg btn-block'
--- a/app/views/layouts/_breadcrumb.haml
+++ b/app/views/layouts/_breadcrumb.haml
@ -1,7 +1,8 @@
 %nav.navbar.flex-md-nowrap.px-0
  %a.navbar-brand.order-0{ href: '/' }
    = inline_svg_tag 'sutty.svg', class: 'black', aria: true,
-      title: t('svg.sutty.title'), desc: t('svg.sutty.desc')
+                                  title: t('svg.sutty.title'),
+                                  desc: t('svg.sutty.desc')

  - if breadcrumbs?
    %nav.flex-grow-1.order-2.order-md-1{ aria: { label: t('.title') } }
@ -13,11 +14,11 @@
            - else
              = link_to crumb.name, crumb.url, class: 'line-clamp-1'

-  - if @current_usuarie || current_usuarie
+  - if usuarie || current_usuarie
    %ul.navbar-nav.order-1.order-md-2
-      - if @site&.tienda?
+      - if site&.tienda?
        %li.nav-item
-          = link_to t('.tienda'), @site.tienda_url,
+          = link_to t('.tienda'), site.tienda_url,
                    role: 'button', class: 'btn btn-secondary'

      %li.nav-item
@ -31,5 +32,7 @@
    - params.permit!
    - I18n.available_locales.each do |locale|
      - next if locale == I18n.locale
+
      %li.nav-item
-        = link_to t("switch_locale.#{locale}"), params.to_h.merge(change_locale_to: locale)
+        = link_to t("switch_locale.#{locale}"),
+                  params.to_h.merge(change_locale_to: locale)
--- a/app/views/layouts/_details.haml
+++ b/app/views/layouts/_details.haml
@ -7,11 +7,13 @@
  @param :summary_class [String] Clases para el summary

 - local_assigns[:summary_class] ||= 'h3'
- local_assigns[:closed] ||= '&#x25B6'.html_safe;
- local_assigns[:open] ||= '&#x25BC'.html_safe;
+- local_assigns[:closed] ||= '&#x25B6'.html_safe
+- local_assigns[:open] ||= '&#x25BC'.html_safe

-
-%details.details.py-2{ id: local_assigns[:id], data: { controller: 'details', action: 'toggle->details#store' }, class: local_assigns[:details_class] }
+%details.details{ id: local_assigns[:id],
+                       data: { controller: 'details',
+                               action: 'toggle->details#store' },
+                       class: local_assigns[:details_class] }
  %summary.d-flex.flex-row.align-items-center.justify-content-between{ class: local_assigns[:summary_class] }
    %span= summary
    %span.hide-when-open{ class: local_assigns[:open_class] }= local_assigns[:closed]
--- a/app/views/layouts/application.html.haml
+++ b/app/views/layouts/application.html.haml
@ -13,16 +13,27 @@

    %script{ type: 'text/javascript', src: '/env.js' }
    = csrf_meta_tags
-    = stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track': 'reload'
-    = stylesheet_link_tag 'dark', rel: 'alternate stylesheet', media: 'all', 'data-turbolinks-track': 'reload', title: t('dark')
-    = javascript_pack_tag 'application', 'data-turbolinks-track': 'reload', defer: true
-    = stylesheet_pack_tag 'application', 'data-turbolinks-track': 'reload'
-    = favicon_link_tag 'sutty_cuadrada.png', rel: 'apple-touch-icon', type: 'image/png'
+    = stylesheet_link_tag 'application',
+                          media: 'all',
+                          'data-turbolinks-track': 'reload'
+    = stylesheet_link_tag 'dark',
+                          rel: 'alternate stylesheet',
+                          media: 'all',
+                          'data-turbolinks-track': 'reload',
+                          title: t('dark')
+    = javascript_pack_tag 'application',
+                          'data-turbolinks-track': 'reload',
+                          defer: true
+    = stylesheet_pack_tag 'application',
+                          'data-turbolinks-track': 'reload'
+    = favicon_link_tag 'sutty_cuadrada.png',
+                       rel: 'apple-touch-icon',
+                       type: 'image/png'
    = render 'layouts/link_rel_alternate'

  %body{ class: yield(:body) }
    .container-fluid#sutty
-      = render 'layouts/breadcrumb'
+      = render 'layouts/breadcrumb', usuarie: @current_usuarie, site: @site
      = render 'layouts/flash'

      = yield
--- a/app/views/posts/attribute_ro/_related_posts.haml
+++ b/app/views/posts/attribute_ro/_related_posts.haml
@ -2,9 +2,11 @@
  %th= post_label_t(attribute, post: post)
  %td
    %ul{ dir: dir, lang: locale }
-      - site.indexed_posts.where(locale: post.lang.value, post_id: metadata.value).find_each do |p|
+      - site.indexed_posts.where(locale: post.lang.value,
+                                 post_id: metadata.value).find_each do |p|
        -#
          XXX: Ignorar todos los posts no encontrados (ej: fueron
          borrados o el uuid cambió)
        - next unless p
+
        %li= link_to p.title, site_post_path(site, p.path)
--- a/app/views/posts/attributes/_draft.haml
+++ b/app/views/posts/attributes/_draft.haml
--- a/app/views/posts/attributes/_file.haml
+++ b/app/views/posts/attributes/_file.haml
@ -20,16 +20,26 @@
    -# Los archivos requeridos solo se pueden reemplazar
    - unless metadata.required
      .custom-control.custom-switch
-        = check_box_tag "#{base}[#{attribute}][path]", '', false, id: "#{base}_#{attribute}_destroy", class: 'custom-control-input'
-        = label_tag "#{base}_#{attribute}_destroy", t('posts.attributes.file.destroy'), class: 'custom-control-label'
+        = check_box_tag "#{base}[#{attribute}][path]",
+                        '',
+                        false,
+                        id: "#{base}_#{attribute}_destroy",
+                        class: 'custom-control-input'
+        = label_tag "#{base}_#{attribute}_destroy",
+                    t('posts.attributes.file.destroy'),
+                    class: 'custom-control-label'

  .custom-file
    = file_field(*field_name_for(base, attribute, :path),
-      **field_options(attribute, metadata, required: (metadata.required && !metadata.path?)),
+                 **field_options(attribute,
+                                 metadata,
+                                 required: (metadata.required && !metadata.path?)),
                 class: "custom-file-input #{invalid(post, attribute)}",
-      data: { target: 'file-preview.input', action: 'file-preview#update' })
+                 data: { target: 'file-preview.input',
+                         action: 'file-preview#update' })
    = label_tag "#{base}_#{attribute}_path",
-      post_label_t(attribute, :path, post: post), class: 'custom-file-label'
+                post_label_t(attribute, :path, post: post),
+                class: 'custom-file-label'
    = render 'posts/attribute_feedback',
             post: post, attribute: [attribute, :path], metadata: metadata

--- a/app/views/posts/attributes/_image.haml
+++ b/app/views/posts/attributes/_image.haml
@ -11,8 +11,14 @@
    -# Las imágenes requeridas solo se pueden reemplazar
    - unless metadata.required
      .custom-control.custom-switch
-        = check_box_tag "#{base}[#{attribute}][path]", '', false, id: "#{base}_#{attribute}_destroy", class: 'custom-control-input'
-        = label_tag "#{base}_#{attribute}_destroy", t('posts.attributes.image.destroy'), class: 'custom-control-label'
+        = check_box_tag "#{base}[#{attribute}][path]",
+                        '',
+                        false,
+                        id: "#{base}_#{attribute}_destroy",
+                        class: 'custom-control-input'
+        = label_tag "#{base}_#{attribute}_destroy",
+                    t('posts.attributes.image.destroy'),
+                    class: 'custom-control-label'
  - else
    = image_tag '',
                alt: metadata.value['description'],
@ -21,12 +27,16 @@

  .custom-file
    = file_field(*field_name_for(base, attribute, :path),
-      **field_options(attribute, metadata, required: (metadata.required && !metadata.path?)),
+                 **field_options(attribute,
+                                 metadata,
+                                 required: (metadata.required && !metadata.path?)),
                 class: "custom-file-input #{invalid(post, attribute)}",
                 accept: ActiveStorage.web_image_content_types.join(','),
-      data: { target: 'file-preview.input', action: 'file-preview#update' })
+                 data: { target: 'file-preview.input',
+                         action: 'file-preview#update' })
    = label_tag "#{base}_#{attribute}_path",
-      post_label_t(attribute, :path, post: post), class: 'custom-file-label'
+                post_label_t(attribute, :path, post: post),
+                class: 'custom-file-label'
    = render 'posts/attribute_feedback',
             post: post, attribute: [attribute, :path], metadata: metadata

--- a/app/views/posts/attributes/_locales.haml
+++ b/app/views/posts/attributes/_locales.haml
@ -15,4 +15,5 @@

        = select_tag("#{plain_field_name_for(base, attribute)}[]",
                     options_for_select(metadata.values[locale], value),
-          **field_options(attribute, metadata), include_blank: t('.empty'))
+                     **field_options(attribute, metadata),
+                     include_blank: t('.empty'))
--- a/app/views/posts/index.haml
+++ b/app/views/posts/index.haml
@ -8,48 +8,62 @@
 %main.row
  %aside.menu.col-lg-3
    .mb-3
-      = render 'sites/header', site: @site
+      = render 'sites/header', site: @site, filter_params: @filter_params
      = render 'sites/status', site: @site
      = render 'sites/build', site: @site, class: 'btn-block'
      = render 'sites/moderation_queue', site: @site, class: 'btn-block'

    = render 'layouts/details', summary: t('posts.filters.title') do
-      
      %form{ method: :get }
-        .border.border-magenta.p-1
+        .border.border-magenta.p-3
          - @filter_params.each do |param, values|
            - next if param == :layout
+
            - [values].flatten.each do |value|
-              %input{ type: 'hidden', name: values.is_a?(Array) ? "#{param}[]" : param, value: value }
+              %input{ type: 'hidden',
+                      name: values.is_a?(Array) ? "#{param}[]" : param,
+                      value: value }
          %legend.font-weight-bold.m-0.h6= 'Tipo de contenido'
          - @site.schema_organization.each do |key, _|
            .custom-control.custom-checkbox
              - schema = @site.layouts[key]
-              = render 'schemas/filter', site: @site, key: key, schema: schema, filter: @filter_params
-        %button.btn.btn-secondary.mt-3{ type: 'submit' }= t('posts.filters.submit')
+              = render 'schemas/filter', site: @site,
+                                         key: key,
+                                         schema: schema,
+                                         filter: @filter_params
+        %button.btn.btn-secondary.mt-2.mb-4{ type: 'submit' }= t('posts.filters.submit')
    = render 'layouts/details',
             summary: t('posts.new'),
-      summary_class: "h4 magenta font-weight-bold m-0 px-2", 
-      details_class: "d-flex border border-magenta justify-content-between align-items-center w-100 mb-3", 
-      open: "+", closed: "+", 
-      open_class: "h1 magenta font-weight-bold m-0", 
-      closed_class: "h1 magenta font-weight-bold m-0" do
+             summary_class: 'h4 magenta font-weight-bold m-0 px-2 py-2',
+             details_class: 'details-agregar',
+             open: '+', closed: '+',
+             open_class: 'h1 magenta font-weight-bold m-0',
+             closed_class: 'h1 magenta font-weight-bold m-0' do
      %table.table-sm.w-100
        %tbody
          - @site.schema_organization.each do |schema, _|
            - schema = @site.layouts[schema]
            - next if schema.hidden?
-            = render 'schemas/row', site: @site, schema: schema, filter: @filter_params

+            = render 'schemas/row', site: @site,
+                                    schema: schema,
+                                    filter: @filter_params

    - if policy(@site_stat).index?
-      = link_to t('stats.index.title'), site_stats_path(@site), class: 'btn btn-secondary'
+      = link_to t('stats.index.title'),
+                site_stats_path(@site),
+                class: 'btn btn-secondary'

    - if policy(@site).edit?
-      = link_to t('sites.edit.btn', site: @site.title), edit_site_path(@site), class: 'btn btn-secondary'
+      = link_to t('sites.edit.btn', site: @site.title),
+                edit_site_path(@site),
+                class: 'btn btn-secondary'

    - if policy(@site).private?
-      = link_to t('sites.private'), '../private/' + @site.name, class: 'btn btn-secondary', target: '_blank', rel: 'noopener'
+      = link_to t('sites.private'), "../private/#{@site.name}",
+                class: 'btn btn-secondary',
+                target: '_blank',
+                rel: 'noopener'

    - if policy(SiteUsuarie.new(@site, current_usuarie)).index?
      = render 'layouts/btn_with_tooltip',
@ -61,9 +75,13 @@
    - if @site.design.credits
      = render 'bootstrap/alert' do
        = sanitize_markdown @site.design.credits
-        = link_to t('sites.donations.text'), t('sites.donations.url'), class: 'btn btn-secondary'
+        = link_to t('sites.donations.text'),
+                  t('sites.donations.url'),
+                  class: 'btn btn-secondary'
        - if @site.design.designer_url
-          = link_to t('sites.designer_url'), @site.design.designer_url, class: 'btn btn-secondary'
+          = link_to t('sites.designer_url'),
+                    @site.design.designer_url,
+                    class: 'btn btn-secondary'

  %section.col
    .d-flex.justify-content-between.align-items-center.pl-2-plus.pr-2-plus.mb-2
@ -71,7 +89,8 @@
      - if @site.locales.size > 1
        %nav#locales
          - @site.locales.each do |locale|
-            = link_to @site.data.dig(locale.to_s, 'locale') || locale, site_posts_path(@site, **@filter_params.merge(locale: locale)),
+            = link_to @site.data.dig(locale.to_s, 'locale') || locale,
+                      site_posts_path(@site, **@filter_params.merge(locale: locale)),
                      class: "mr-2 mt-2 mb-2 #{locale == @locale ? 'active font-weight-bold' : ''}"

    - if @posts.empty?
@ -86,7 +105,8 @@
                .d-flex.flex-row.justify-content-between
                  %div
                    - if reorder_allowed
-                      = submit_tag t('posts.reorder.submit'), class: 'btn btn-secondary'
+                      = submit_tag t('posts.reorder.submit'),
+                                   class: 'btn btn-secondary'
                      %button.btn.btn-secondary{ data: { action: 'reorder#unselect' } }
                        = t('posts.reorder.unselect')
                        %span.badge{ data: { target: 'reorder.counter' } } 0
@ -94,12 +114,18 @@
                      %button.btn.btn-secondary{ data: { action: 'reorder#down' } }= t('posts.reorder.down')
                      %button.btn.btn-secondary{ data: { action: 'reorder#top' } }= t('posts.reorder.top')
                      %button.btn.btn-secondary{ data: { action: 'reorder#bottom' } }= t('posts.reorder.bottom')
-                      %input{ type: 'hidden', name: 'post[lang]', value: @locale }
+                      %input{ type: 'hidden',
+                              name: 'post[lang]',
+                              value: @locale }

                  - if @site.pagination
                    %div
-                      = link_to_prev_page @posts, t('posts.prev'), class: 'btn btn-secondary'
-                      = link_to_next_page @posts, t('posts.next'), class: 'btn btn-secondary'
+                      = link_to_prev_page @posts,
+                                          t('posts.prev'),
+                                          class: 'btn btn-secondary'
+                      = link_to_next_page @posts,
+                                          t('posts.next'),
+                                          class: 'btn btn-secondary'
          %tbody
            - dir = @site.data.dig(params[:locale], 'dir')
            - size = @posts.size
@ -107,14 +133,17 @@
              -#
                TODO: Solo les usuaries cachean porque tenemos que separar
                les botones por permisos.
-              - begin
+
              - cache_if @usuarie, [post, I18n.locale] do
                - checkbox_id = "checkbox-#{post.post_id}"
                %tr{ id: post.post_id, data: reorder_target }
                  - if reorder_allowed
                    %td
                      .custom-control.custom-checkbox
-                          %input.custom-control-input{ id: checkbox_id, type: 'checkbox', autocomplete: 'off', data: { action: 'reorder#select' } }
+                        %input.custom-control-input{ id: checkbox_id,
+                                                     type: 'checkbox',
+                                                     autocomplete: 'off',
+                                                     data: { action: 'reorder#select' } }
                        %label.custom-control-label{ for: checkbox_id }
                          %span.sr-only= t('posts.reorder.select')
                      -# Orden más alto es mayor prioridad
@ -128,7 +157,8 @@
                      %span.badge.badge-primary= I18n.t('posts.attributes.draft.label')
                    %br
                    %small
-                        = link_to @site.layouts[post.layout].humanized_name, site_posts_path(@site, **@filter_params.merge(layout: [post.layout]))
+                      = link_to @site.layouts[post.layout].humanized_name,
+                                site_posts_path(@site, **@filter_params.merge(layout: [post.layout]))
                      - post.front_matter['categories']&.each do |category|
                        = link_to site_posts_path(@site, **@filter_params.merge(category: category)) do
                          %span{ lang: post.locale, dir: dir }= category
@ -141,14 +171,23 @@
                  %td.text-nowrap
                    .d-flex.flex-row.align-items-start
                      - if @usuarie || policy(post).edit?
-                          = link_to t('posts.edit_post'), edit_site_post_path(@site, post.path), class: 'btn btn-secondary'
+                        = link_to t('posts.edit_post'),
+                                  edit_site_post_path(@site, post.path),
+                                  class: 'btn btn-secondary'
                      - if @usuarie || policy(post).destroy?
-                          = link_to t('posts.destroy'), site_post_path(@site, post.path), class: 'btn btn-secondary', method: :delete, data: { confirm: t('posts.confirm_destroy') }
+                        = link_to t('posts.destroy'),
+                                  site_post_path(@site, post.path),
+                                  class: 'btn btn-secondary',
+                                  method: :delete,
+                                  data: { confirm: t('posts.confirm_destroy') }
              -#
                Rescatar cualquier error en un post, notificarlo e
                ignorar su renderización.
              - rescue ActionView::Template::Error => e
-                - ExceptionNotifier.notify_exception(e.cause, data: { site: @site.name, post: @post.path.absolute, usuarie: current_usuarie.id })
+                - ExceptionNotifier.notify_exception(e.cause,
+                                                     data: { site: @site.name,
+                                                             post: @post.path.absolute,
+                                                             usuarie: current_usuarie.id })

 #footnotes{ hidden: true }
  - @filter_params.each do |param, value|
--- a/app/views/schemas/_add.haml
+++ b/app/views/schemas/_add.haml
@ -1 +1,3 @@
-= link_to t(schema.humanized_name), new_site_post_path(site, layout: schema.value), class: 'stretched-link black text-decoration-none'
+= link_to t(schema.humanized_name),
+          new_site_post_path(site, layout: schema.value),
+          class: 'stretched-link black text-decoration-none'
--- a/app/views/schemas/_filter.haml
+++ b/app/views/schemas/_filter.haml
@ -1,10 +1,18 @@
 %div
-  %input.custom-control-input.magenta{ type: 'checkbox', id: schema, name: "layout[]", class: "", value: schema.name, checked: @filter_params[:layout]&.include?(key.to_s) }
+  %input.custom-control-input.magenta{ type: 'checkbox',
+                                       id: schema,
+                                       name: 'layout[]',
+                                       class: '',
+                                       value: schema.name,
+                                       checked: filter[:layout]&.include?(key.to_s) }
  %label.custom-control-label.font-weight-normal{ for: schema }= schema.humanized_name

- 
 -# XXX: Solo un nivel de recursividad
 - unless local_assigns[:parent_schema]
  - schema.schemas.each do |s|
    - next if s.hidden?
-    = render 'schemas/filter', schema: s, key: s.name, site: site, filter: filter
+
+    = render 'schemas/filter', schema: s,
+                               key: s.name,
+                               site: site,
+                               filter: filter
--- a/app/views/schemas/_row.haml
+++ b/app/views/schemas/_row.haml
@ -1,6 +1,5 @@
 %tr.border-top.border-magenta
-  
-  %th.font-weight-normal.w-100.position-relative{ scope: 'row' }
+  %th.font-weight-normal.w-100.position-relative.px-3.py-2{ scope: 'row' }
    - if local_assigns[:parent_schema]
      %span.text-muted &mdash;
    = render 'schemas/add', schema: schema, **local_assigns
@ -9,5 +8,8 @@
 - unless local_assigns[:parent_schema]
  - schema.schemas.each do |s|
    - next if s.hidden?
-    = render 'schemas/row', schema: s, site: site, filter: filter, parent_schema: schema

+    = render 'schemas/row', schema: s,
+                            site: site,
+                            filter: filter,
+                            parent_schema: schema
--- a/app/views/sites/_form.haml
+++ b/app/views/sites/_form.haml
@ -8,7 +8,7 @@
      - site.errors.messages.each_pair do |attr, error|
        - attr = attr.to_s
        - error.each do |e|
-          %li= link_to t('activerecord.attributes.site.' + attr) + ' ' + e, '#' + attr
+          %li= link_to "#{t("activerecord.attributes.site.#{attr}")} #{e}", "##{attr}"

 = form_for site, html: { class: form_class(site) } do |f|
  - unless site.persisted?
@ -83,6 +83,7 @@
    %p.lead= t('.help.licencia')
    - Licencia.all.find_each do |licencia|
      - next if licencia.custom? && site.licencia != licencia
+
      .row.license
        .col
          .media.mt-1
@ -93,13 +94,18 @@
                = f.radio_button :licencia_id, licencia.id,
                                 checked: licencia.id == site.licencia_id,
                                 required: true, class: 'custom-control-input'
-                = f.label "licencia_id_#{licencia.id}", class: 'custom-control-label' do
+                = f.label "licencia_id_#{licencia.id}",
+                          class: 'custom-control-label' do
                  = licencia.name
              = sanitize_markdown licencia.description,
                                  tags: %w[p a strong em ul ol li h1 h2 h3 h4 h5 h6]

              - unless licencia.custom?
-                = link_to t('.licencia.url'), licencia.url, target: '_blank', class: 'btn btn-secondary', rel: 'noopener'
+                = link_to t('.licencia.url'),
+                          licencia.url,
+                          target: '_blank',
+                          class: 'btn btn-secondary',
+                          rel: 'noopener'

          %hr/

--- a/app/views/sites/_header.haml
+++ b/app/views/sites/_header.haml
@ -2,26 +2,34 @@
  %h1= site.title
  %p.lead= site.description
  %form.mb-3{ action: site_posts_path }
-    - @filter_params.each do |param, values|
+    - filter_params.each do |param, values|
      - next if param == :q
+
      - [values].flatten.each do |value|
-        %input{ type: 'hidden', name: values.is_a?(Array) ? "#{param}[]" : param, value: value }
-    .form-group.flex-grow-0.m-0
+        %input{ type: 'hidden',
+                name: values.is_a?(Array) ? "#{param}[]" : param,
+                value: value }
+    .form-group.flex-grow-0.m-0.mb-1
      %label.h3{ for: 'q' }= t('posts.index.search')
      .input-group
-        %input#q.form-control.border.border-magenta.border-right-0{ type: 'search', name: 'q', value: @filter_params[:q] }
+        %input.form-control.border.border-magenta.border-right-0#q{ type: 'search',
+                                                                    name: 'q',
+                                                                    value: filter_params[:q] }
        .input-group-append
          %span.input-group-text.background-white.magenta.border.border-magenta.border-top.border-left-0.border-right.border-bottom
            %i.fa.fa-fw.fa-search
      %input.sr-only{ type: 'submit' }
-    - @filter_params.each do |param, values|
+      
+    - filter_params.each do |param, values|
+      .d-flex.flex-wrap
        - [values].flatten.each do |value|
-        = link_to site_posts_path(@site, **filter_params_by(@filter_params, param, value)),
-          class: 'btn btn-secondary btn-sm',
-          title: t('posts.remove_filter_help', filter: value),
+          - humanized_name = site.layouts[value.to_sym].humanized_name 
+          = link_to site_posts_path(site, **filter_params_by(filter_params, param, value)),
+                    class: 'btn btn-secondary btn-sm-queries',
+                    title: humanized_name,
                    aria: { labelledby: "help-filter-#{param}" } do
            - if param == :layout
-            = @site.layouts[value.to_sym].humanized_name
+              %span= humanized_name
            - else
              = value
-          &times;
+            %span &times;
--- a/app/views/sites/index.haml
+++ b/app/views/sites/index.haml
@ -15,6 +15,7 @@
      %tbody
        - @sites.each do |site|
          - next unless site.jekyll?
+
          %tr
            %td
              %h2
--- a/app/views/stats/index.haml
+++ b/app/views/stats/index.haml
@ -11,10 +11,19 @@

    %form.mb-5.form-inline{ method: 'get' }
      - Stat::INTERVALS.each do |interval|
-        = link_to t(".#{interval}"), site_stats_path(interval: interval, urls: params[:urls], period_start: params[:period_start].to_date.try(:"beginning_of_#{interval}").to_date, period_end: params[:period_end]), class: "mb-0 btn #{@interval == interval ? 'btn-primary active' : 'btn-secondary' }"
+        = link_to t(".#{interval}"),
+                  site_stats_path(interval: interval,
+                                  urls: params[:urls],
+                                  period_start: params[:period_start].to_date.try(:"beginning_of_#{interval}").to_date,
+                                  period_end: params[:period_end]),
+                  class: "mb-0 btn #{@interval == interval ? 'btn-primary active' : 'btn-secondary'}"

-      %input.form-control{ type: 'date', name: :period_start, value: params[:period_start] }
-      %input.form-control{ type: 'date', name: :period_end, value: params[:period_end] }
+      %input.form-control{ type: 'date',
+                           name: :period_start,
+                           value: params[:period_start] }
+      %input.form-control{ type: 'date',
+                           name: :period_end,
+                           value: params[:period_end] }
      %button.btn.btn-secondary.mb-0{ type: 'submit' }= t('.filter')

    .mb-5
@ -22,25 +31,38 @@
      %p.lead= t('.host.description')
      = line_chart site_stats_host_path(@chart_params), **@chart_options

-    #custom-urls.mb-5
+    .mb-5#custom-urls
      %h2= t('.urls.title')
      %p.lead= t('.urls.description')
      %form{ method: 'get', action: '#custom-urls' }
-        %input{ type: 'hidden', name: 'interval', value: @interval }
-        %input{ type: 'hidden', name: 'period_start', value: params[:period_start] }
-        %input{ type: 'hidden', name: 'period_end', value: params[:period_end] }
+        %input{ type: 'hidden',
+                name: 'interval',
+                value: @interval }
+        %input{ type: 'hidden',
+                name: 'period_start',
+                value: params[:period_start] }
+        %input{ type: 'hidden',
+                name: 'period_end',
+                value: params[:period_end] }
        .form-group
          %label{ for: 'urls' }= t('.urls.label')
-          %textarea#urls.form-control{ name: 'urls', autocomplete: 'on', required: true, rows: @normalized_urls.size + 1, aria_describedby: 'help-urls' }= @normalized_urls.join("\n")
-          %small#help-urls.feedback.form-text.text-muted= t('.urls.help')
+          %textarea.form-control#urls{ name: 'urls',
+                                       autocomplete: 'on',
+                                       required: true,
+                                       rows: @normalized_urls.size + 1,
+                                       aria_describedby: 'help-urls' }= @normalized_urls.join("\n")
+          %small.feedback.form-text.text-muted#help-urls= t('.urls.help')
        .form-group
          %button.btn.btn-secondary{ type: 'submit' }= t('.urls.submit')
      - if @normalized_urls.present?
-        = line_chart site_stats_uris_path(urls: @normalized_urls, **@chart_params), **@chart_options
+        = line_chart site_stats_uris_path(urls: @normalized_urls,
+                                          **@chart_params),
+                     **@chart_options

    .row.mb-5.row-cols-1.row-cols-lg-2
      - @columns.each_pair do |column, values|
        - next if values.blank?
+
        .col.mb-5
          %h2= t(".columns.#{column}.title")
          %p.lead= t(".columns.#{column}.description")
@ -57,7 +79,8 @@
            %tbody
              - values.each_pair do |col, val|
                %tr
-                  %th{ scope: 'row', style: 'word-break: break-all' }= col.blank? ? t(".columns.#{column}.empty") : col
+                  %th.break-all{ scope: 'row' }
+                    = col.blank? ? t(".columns.#{column}.empty") : col
                  %td= val
    .mb-5
      %h2= t('.resources.title')
@ -67,4 +90,5 @@
      .mb-5
        %h3= t(".resources.#{resource}.title")
        %p.lead= t(".resources.#{resource}.description")
-        = line_chart site_stats_resources_path(resource: resource, **@chart_params), **@chart_options.merge(StatsController::EXTRA_OPTIONS[resource])
+        = line_chart site_stats_resources_path(resource: resource, **@chart_params),
+                     **@chart_options.merge(StatsController::EXTRA_OPTIONS[resource])
--- a/app/views/usuaries/invite.haml
+++ b/app/views/usuaries/invite.haml
@ -9,8 +9,8 @@
      .form-group
        = f.label :invitaciones do
          = t('.invitaciones')
-          %small.text-muted.form-text= t('.help.invitaciones',
-            invite_as: invite_as)
+          %small.text-muted.form-text
+            = t('.help.invitaciones', invite_as: invite_as)
        = f.text_area :invitaciones, class: 'form-control'
      .form-group
        = f.submit t('.submit'), class: 'btn btn-secondary'
--- a/config/environments/test.rb
+++ b/config/environments/test.rb
@ -46,7 +46,7 @@ Rails.application.configure do
  # The :test delivery method accumulates sent emails in the
  # ActionMailer::Base.deliveries array.
  config.action_mailer.delivery_method = :test
-  config.action_mailer.default_options = { from: ENV['DEFAULT_FROM'] }
+  config.action_mailer.default_options = { from: ENV.fetch('DEFAULT_FROM', nil) }
  config.action_mailer.default_url_options = { host: 'localhost',
                                               port: 3000 }

@ -54,8 +54,8 @@ Rails.application.configure do
                        error_grouping: true,
                        email: {
                          email_prefix: '',
-                          sender_address: ENV['DEFAULT_FROM'],
-                          exception_recipients: ENV['EXCEPTION_TO'],
+                          sender_address: ENV.fetch('DEFAULT_FROM', nil),
+                          exception_recipients: ENV.fetch('EXCEPTION_TO', nil),
                          normalize_subject: true
                        }

--- a/test/controllers/posts_controller_test.rb
+++ b/test/controllers/posts_controller_test.rb
@ -167,9 +167,7 @@ class PostsControllerTest < ActionDispatch::IntegrationTest
  end

  test 'si hay algún error se recupera' do
-    File.open(File.join(@site.path, '_es', "#{Date.today}-#{SecureRandom.hex}.markdown"), 'w') do |f|
-      f.write ''
-    end
+    File.write(File.join(@site.path, '_es', "#{Date.today}-#{SecureRandom.hex}.markdown"), '')

    get site_posts_url(@site), headers: @authorization
  end