pg_rails 7.3.0 → 7.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 987f2b59fff584bde7004887e631c776fd0da367988e6ca6117726ff6b6de937
-  data.tar.gz: 41676e4fbcadad6e6a7a2b999368137c4d200fe83da5d7a1a300ed8d0a24a99f
+  metadata.gz: ad67165e1aad7f3b3e31dd236706d4b923eb2139740a095018257899b43928a7
+  data.tar.gz: f3d907a47d97cb7abbfb0e148bb2c2e9466a5255284f14f06c0363966e35e46b
 SHA512:
-  metadata.gz: 2cda455c10578a6c9b990e0f7c00317166c80d499ff82267a925912935f510b79195d1ad13caf9669af1f314da0dfeb23a7aa9782f3df56996b4a128af3b5535
-  data.tar.gz: f394f37f00f3f7dad3abcdfaab5697e1cf2079733a9e385df66101f8e5d1fe55e61f48d385ba9e6d855c9990b509c4a98a63c4efc1e1d9cb2c8535702c1c34cd
+  metadata.gz: 3d8482db6b3b8455c819d4db06db374e40785cec70a9737bfca652c2368bddad17ca7a2d3656e1632b48b0ebeb03c5142e83855d3d70cfd9d043de3e7d34609d
+  data.tar.gz: e6e2457ac2a84097209da469aa76dd2538e72c9da05ed383884c19fceddc071b5dc515a82c444c4f3487d7912ca79790bd6eb06f0eed0ae0c4caf48e8797884f

data/pg_associable/app/helpers/pg_associable/form_builder_methods.rb

@@ -16,6 +16,14 @@ module PgAssociable
     end
 
     def pg_associable(atributo, options = {})
+      # Si es new y tiene el nested asignado, no permito que se modifique
+      # porque de todos modos se pisaría en el create
+      if !object.persisted? &&
+         template.nested_record.present? &&
+         object.send(template.nested_key) == (template.nested_record.id)
+        options[:disabled] = true
+      end
+
       return input(atributo, options) if options[:disabled]
 
       collection, puede_crear = collection_pc(atributo, options)

data/pg_associable/app/helpers/pg_associable/helpers.rb

@@ -3,8 +3,8 @@ module PgAssociable
     MAX_RESULTS = 8
 
     def pg_respond_abrir_modal
-      content = FormModalComponent.new(@clase_modelo.new.decorate)
-                                  .render_in(view_context)
+      src = @clase_modelo.new.decorate.new_object_url
+      content = ModalContentComponent.new(src:).render_in(view_context)
       modal = AsociableModalComponent.new(modal_id: params[:id]).with_content(content)
       render turbo_stream: turbo_stream.append_all('body', modal)
     end

data/pg_associable/app/javascript/modal_controller.js

@@ -5,6 +5,7 @@ export default class extends Controller {
   static outlets = ['asociable']
 
   modalPuntero = null
+  history = []
 
   connect (e) {
     this.modalPuntero = new bootstrap.Modal(this.element)
@@ -15,22 +16,30 @@ export default class extends Controller {
     }
     this.modalPuntero.show()
 
+    this.element.addEventListener('turbo:frame-render', (ev) => {
+      if (ev.detail.fetchResponse.response.ok && ev.target.id === 'modal_content') {
+        this.history.push(ev.target.src)
+      }
+    })
+
     this.element.addEventListener('pg:record-created', (ev) => {
       const el = ev.data
       if (this.asociableOutlets.length > 0) {
         const newObject = JSON.parse(el.dataset.response)
         this.asociableOutlet.completarCampo(newObject)
         ev.stopPropagation()
+        this.remove()
+      } else {
+        this.back(ev)
       }
-      this.modalPuntero.hide()
     })
 
     this.element.addEventListener('pg:record-updated', (ev) => {
-      this.modalPuntero.hide()
+      this.back(ev)
     })
 
     this.element.addEventListener('pg:record-destroyed', (ev) => {
-      this.modalPuntero.hide()
+      this.remove()
     })
 
     document.addEventListener('turbo:before-cache', () => {
@@ -38,12 +47,65 @@ export default class extends Controller {
     }, { once: true })
   }
 
+  maximize (ev) {
+    const dialog = this.element.querySelector('.modal-dialog')
+    dialog.classList.toggle('modal-fullscreen')
+    const button = ev.currentTarget
+    const icon = button.querySelector('i')
+    icon.classList.toggle('bi-fullscreen')
+    icon.classList.toggle('bi-fullscreen-exit')
+
+    const tooltip = this.application.getControllerForElementAndIdentifier(button, 'tooltip')
+    if (tooltip) {
+      tooltip.hide()
+      if (icon.classList.contains('bi-fullscreen')) {
+        tooltip.setContent('Maximizar')
+      } else {
+        tooltip.setContent('Restaurar')
+      }
+    }
+  }
+
+  reloadTop () {
+    // FIXME: rename to main?
+    const topFrame = document.querySelector('#top')
+    if (topFrame.attributes.src) {
+      topFrame.reload()
+    } else {
+      topFrame.setAttribute('src', window.location)
+    }
+  }
+
+  back (ev) {
+    this.history.pop()
+    if (this.history.length > 0) {
+      const url = this.history[this.history.length - 1]
+      const frame = this.element.querySelector('#modal_content')
+      frame.src = url
+      frame.innerHTML = '<div style="min-height: 30em">Cargando...</div>'
+    } else {
+      this.modalPuntero.hide()
+    }
+    ev.stopPropagation()
+    this.reloadTop()
+  }
+
   openModal () {
     this.modalPuntero.show()
   }
 
+  remove () {
+    this.element.remove()
+  }
+
   disconnect (e) {
-    this.modalPuntero.hide()
+    // Antes, en lugar de quitar el modal-backdrop:
+    //        this.modalPuntero.hide()
+    // pero tiraba a veces error:
+    // TypeError: can't convert null to object, _isWithActiveTrigger
+    document.querySelectorAll('.modal-backdrop').forEach((el) => {
+      el.remove()
+    })
     document.dispatchEvent(new Event('hidden.bs.modal'))
     this.modalPuntero.dispose()
   }

data/pg_associable/spec/system/associable_spec.rb

@@ -3,16 +3,17 @@ require 'rails_helper'
 describe 'Associable' do
   let(:user) { create :user, :developer }
 
+  let(:path) { '/admin/cosas/new' }
+
   before do
     login_as user
-    driven_by :selenium
-    visit '/admin/cosas/new'
-    fill_in 'cosa_nombre', with: 'La cosa'
-    select 'Los', from: 'cosa_tipo'
-    find('.cosa_categoria_de_cosa input[type=text]').click
+    visit path
   end
 
   it do
+    fill_in 'cosa_nombre', with: 'La cosa'
+    select 'Los', from: 'cosa_tipo'
+    find('.cosa_categoria_de_cosa input[type=text]').click
     expect(page).to have_text :all, 'Nuevo'
     find('.cosa_categoria_de_cosa .list-group-item').click
     fill_in 'categoria_de_cosa_nombre', with: 'la categoría'
@@ -21,4 +22,26 @@ describe 'Associable' do
     click_on 'Cargar Coso'
     expect(page).to have_text 'Creado por'
   end
+
+  context 'cuando crea desde el nested' do
+    let!(:categ) { create :categoria_de_cosa }
+    let(:path) { "/admin/categoria_de_cosas/#{categ.hashid}/cosas/new" }
+
+    it do
+      input = find_by_id('cosa_categoria_de_cosa')
+      expect(input).to be_disabled
+    end
+  end
+
+  context 'cuando edita desde el nested' do
+    let!(:categ) { create :categoria_de_cosa }
+    let!(:cosa) { create :cosa, categoria_de_cosa: categ }
+    let(:path) { "/admin/categoria_de_cosas/#{categ.hashid}/cosas/#{cosa.id}/edit" }
+
+    it do
+      ele = find_by_id('cosa_categoria_de_cosa_id', visible: :all)
+      input = ele.sibling('input[type=text]')
+      expect(input).not_to be_disabled
+    end
+  end
 end

data/pg_engine/app/assets/stylesheets/pg_rails_b5.scss

@@ -9,6 +9,11 @@
   --bs-form-invalid-border-color: #b50000;
 }
 
+// Navs
+nav[aria-label=breadcrumb] a {
+  font-weight: bold;
+}
+
 // Toasts
 .toast.show {
   display: inline-block;

data/pg_engine/app/components/date_selector_component.rb

@@ -3,9 +3,9 @@ class DateSelectorComponent < ViewComponent::Base
     @field_id = field_id
 
     @types = [
-      ['Días corridos (L a D)', 'calendar_days'],
-      ['Días hábiles (L a V)', 'business_days'],
       ['Días hábiles no feriados', 'business_days_excluding_holidays'],
+      ['Días hábiles (L a V)', 'business_days'],
+      ['Días corridos (L a D)', 'calendar_days'],
       ['Semanas', 'weeks'],
       ['Meses', 'months']
     ]

data/pg_engine/app/components/modal_content_component.rb

@@ -1,28 +1,29 @@
 class ModalContentComponent < ViewComponent::Base
-  renders_one :actions
-  renders_one :body
-  renders_one :header
+  def initialize(src: nil)
+    @src = src
+    with_content(loading_html) if @src.present?
 
-  def before_render
-    controller.instance_variable_set(:@using_modal, true)
+    super
   end
 
-  erb_template <<~ERB
-    <%= helpers.turbo_frame_tag :modal_generic do %>
-      <div class="modal-header">
-        <%= header %>
-        <a class="btn-close" type="button" data-bs-dismiss="modal" aria-label="Close"></a>
+  def loading_html
+    <<~HTML.html_safe
+      <div class="text-center text-body-secondary fs-3" style="min-height: 15em">
+        Cargando...
       </div>
-      <div class="modal-body">
-        <div class="d-flex justify-content-around sticky-top">
-          <div class="flash position-relative w-100 d-flex justify-content-center">
-          </div>
-        </div>
-        <div class="float-end">
-          <%= actions %>
+    HTML
+  end
+
+  erb_template <<~ERB
+    <div class="modal-body">
+      <div class="d-flex justify-content-around sticky-top">
+        <div class="flash position-relative w-100 d-flex justify-content-center">
         </div>
-        <%= body %>
       </div>
-    <% end %>
+      <%= helpers.turbo_frame_tag :modal_content,
+                                  **{ src: @src, refresh: :morph }.compact do %>
+        <%= content %>
+      <% end %>
+    </div>
   ERB
 end

data/pg_engine/app/components/search_bar_component.html.slim

@@ -3,7 +3,7 @@
     .d-flex.align-items-center.px-3.py-2
       .px-2.d-none.d-sm-inline-block
         span.bi.bi-funnel-fill
-      = helpers.search_form_for @q, url: url_for, html: { 'data-turbo-action': :advance } do |f|
+      = helpers.search_form_for @q, url: url_for do |f|
         .row.g-1
           .col
             .row.g-1

data/pg_engine/app/controllers/admin/accounts_controller.rb

@@ -6,7 +6,7 @@ module Admin
   class AccountsController < AdminController
     include PgEngine::Resource
 
-    before_action { @clase_modelo = Account }
+    self.clase_modelo = Account
 
     before_action(only: :index) { authorize Account }
 

data/pg_engine/app/controllers/admin/email_logs_controller.rb

@@ -6,7 +6,7 @@ module Admin
   class EmailLogsController < AdminController
     include PgEngine::Resource
 
-    before_action { @clase_modelo = EmailLog }
+    self.clase_modelo = EmailLog
 
     before_action(only: :index) { authorize EmailLog }
 

data/pg_engine/app/controllers/admin/emails_controller.rb

@@ -6,7 +6,7 @@ module Admin
   class EmailsController < AdminController
     include PgEngine::Resource
 
-    before_action { @clase_modelo = Email }
+    self.clase_modelo = Email
 
     before_action(only: :index) { authorize Email }
 

data/pg_engine/app/controllers/admin/user_accounts_controller.rb

@@ -6,7 +6,7 @@ module Admin
   class UserAccountsController < AdminController
     include PgEngine::Resource
 
-    before_action { @clase_modelo = UserAccount }
+    self.clase_modelo = UserAccount
 
     before_action(only: :index) { authorize UserAccount }
 

data/pg_engine/app/controllers/admin/users_controller.rb

@@ -6,7 +6,7 @@ module Admin
   class UsersController < AdminController
     include PgEngine::Resource
 
-    before_action { @clase_modelo = User }
+    self.clase_modelo = User
 
     before_action(only: :index) { authorize User }