pg_rails 7.6.37 → 7.6.39

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d78d6754e8cd67e2b293e26fa01585fefdefb68fd030a3ad00a420119ecdcdff
-  data.tar.gz: 62cf672c47ff40c4bb45d58880643955450d7db7069ebd4df92cffce4283bba8
+  metadata.gz: 2de59469aa32c2ef73f030ea848179b8941f67a1eb948b72b018cf2d9e7b1b95
+  data.tar.gz: 4311903fb7f15aa7b070e2ac36ff530bc6e4a7750ac011a7843d34a85921483e
 SHA512:
-  metadata.gz: 7443db791d97a49a3755d4922c855202522c4a1a980b4c66f44fa45733131827f5c88cc059a88ad2f21495d21e3f226d9fe39ad8af2d420bf2a28ee05f177924
-  data.tar.gz: 870d9f4006606617247fecd04e93a832ecd13da8d844e0f1086cc4dfdd9da735fd6508167a4e8e29f658ebb5763867a0e68c662d86a889c019f0d1f20d246851
+  metadata.gz: 37288f9d151624cb72e6c7567e6fdb875d91b5945baf3d66e045c640fa500d3ff3f1a90cbd6bd76aa898562c0c6b8f9d09aceddd287975f456af068b3310cee3
+  data.tar.gz: 63d8f3e1d03d521db97cae73cde5908342bbcd9dc44913289f516ee0c932b63fcdb6e626ed6a575556f04b651315293f7315b5d0e1ed69471f750dfa14fc2375

data/pg_associable/app/javascript/asociable_controller.tsx

@@ -174,7 +174,9 @@ export default class extends Controller {
 
   mostrarError () {
     if (this.element.querySelector('.resultados .spinner-border')) {
+      // TODO: rastrear causa, probablemente sea por websocket desconectado.
       Rollbar.error('Time out de asociable.js')
+
       // TODO: link a contacto
       this.subWrapper.innerHTML = renderToStaticMarkup(
         <div className="resultados" tabIndex={-1}>

data/pg_associable/app/javascript/modal_controller.js

@@ -39,14 +39,17 @@ export default class extends Controller {
 
     this.element.addEventListener('pg:record-updated', (ev) => {
       if (ev.data.dataset.inline) {
+        // Is an inline edit submit
         this.reloadTop()
         ev.stopPropagation()
       } else {
         if (ev.data.dataset.reload) {
+          // Must reload modal. Ie: on discard/undiscard
           this.reload()
           this.reloadTop()
           ev.stopPropagation()
         } else {
+          // Is a traditional form submit
           this.back(ev)
         }
       }

data/pg_engine/app/overrides/activestorage_direct_uploads.rb

@@ -4,6 +4,7 @@
 ActiveStorage::DirectUploadsController.class_eval do
   alias_method :old_blob_args, :blob_args
 
+  # FIXME: testear
   def blob_args
     old_blob_args.merge(service_name: :local)
   end

data/pg_engine/app/views/pg_engine/base/download.xlsx.axlsx

@@ -2,6 +2,7 @@
 
 wb = xlsx_package.workbook
 
+# FIXME: add colspaned header with model name
 title = wb.styles.add_style(
   b: true, sz: 13, bg_color: '1a172f', fg_color: 'ebebeb'
   # alignment: { horizontal: :center, vertical: :center }

data/pg_engine/app/views/pg_engine/base/index.html.slim

@@ -30,6 +30,9 @@
           i.bi.bi-pencil.lh-1
           = t('pg_engine.base.index.bulk_edit.link', model: clase_modelo)
 
+/ if turbo stream? && Current.account.present?
+/   = turbo_stream_from Current.account, @clase_modelo.model_name.plural
+
 - if action_name == 'archived'
   .text-center.p-3.text-warning-emphasis.border-bottom
     div

data/pg_engine/lib/pg_engine/utils/pg_logger.rb

@@ -55,7 +55,7 @@ module PgEngine
       def notify_all(mensaje, type)
         send_to_logger(mensaje, type)
         send_to_rollbar(mensaje, type)
-        send_to_stdout(mensaje, type) if ENV.fetch('LOG_TO_STDOUT', nil)
+        send_to_stdout(mensaje, type) if ENV.fetch('LOG_TO_STDOUT', nil) || (Rails.env.test? && type == :debug)
         save_internal(mensaje, type) if Rails.env.test?
         nil
       end
@@ -110,7 +110,7 @@ module PgEngine
       end
 
       def rainbow_wrap(mensaje, type)
-        Rainbow(mensaje).bold.send(color_for(type))
+        mensaje.split("\n").map { Rainbow(_1).bold.send(color_for(type)) }.join("\n")
       end
 
       def color_for(type)

data/pg_layout/app/assets/stylesheets/sidebar.scss

@@ -47,10 +47,22 @@ $chevron-color: 200,200,200,.5;
   // background-color: #d2f4ea;
 }
 
-.pg--nav-button.active {
+.navbar__nav-button.active {
   --bs-link-color-rgb: white!important;
   background-color: tint-color($primary, 17%);
 }
+
+.navbar__nav-button[aria-expanded=true] {
+  font-weight: bold;
+}
+
+// Child entries
+#sidebar .collapse-inner {
+  background-color: tint-color($primary, 5%);
+  padding-bottom: 0.1em;
+  padding-top: 0.1em;
+}
+
 // Los small-items están deprecados
 #sidebar {
   &.opened {
@@ -100,7 +112,7 @@ $chevron-color: 200,200,200,.5;
   }
 }
 
-.pg--nav-button {
+.navbar__nav-button {
   padding: 0.4em 1.7em;
   width: 100%;
   text-align: right;
@@ -109,11 +121,11 @@ $chevron-color: 200,200,200,.5;
 .offcanvas-body {
   padding: 0!important;
 }
-.pg--nav-button:hover {
+.navbar__nav-button:hover {
   background-color: tint-color($primary, 20%);
 }
 
-.offcanvas .pg--nav-button {
+.offcanvas .navbar__nav-button {
   padding: 0.7em 1.7em;
   text-align: right;
 }

data/pg_layout/app/lib/navbar.rb

@@ -43,31 +43,40 @@ class Navbar
     bar_data = @yaml_data[key]
     return [] if bar_data.blank?
 
-    # rubocop:disable Security/Eval
     # rubocop:disable Style/MultilineBlockChain:
     orig_idx = 0
     bar_data.map do |item|
       orig_idx += 1
-      {
-        title: item['name'],
-        attributes: item['attributes']&.html_safe,
-        path: eval(item['path']),
-        show: item['policy'] ? eval(item['policy']) : true,
-        priority: item['priority'] || 999_999,
-        orig_idx:
-      }
+      evaluate_node(item, orig_idx)
     end.sort_by { |a| [a[:priority], a[:orig_idx]] }
-    # rubocop:enable Security/Eval
     # rubocop:enable Style/MultilineBlockChain:
   end
 
+  def evaluate_node(item, orig_idx = nil)
+    # rubocop:disable Security/Eval
+    aux = {
+      title: item['name'],
+      attributes: item['attributes']&.html_safe,
+      path: item['path'].present? ? eval(item['path']) : nil,
+      show: item['policy'] ? eval(item['policy']) : true,
+      priority: item['priority'] || 999_999,
+      orig_idx:
+    }
+    # rubocop:enable Security/Eval
+
+    if item[:children].present?
+      aux[:children] = item[:children].map { |it| evaluate_node(it) }
+    end
+
+    aux
+  end
+
   def all_children_hidden?(entry)
     entry[:children].all? { |child| child[:show] == false }
   end
 
   def any_children_active?(entry, request)
     entry[:children].any? { |child| active_entry?(child, request) }
-    true
     # TODO: quitar
   end
 

data/pg_layout/app/views/layouts/pg_layout/containerized.html.slim

@@ -1,5 +1,5 @@
 - content_for :content do
-  - klass = modal_targeted? ? '' : (yield(:container_class) || 'container-fluid')
+  - klass = modal_targeted? ? '' : (yield(:container_class).presence || 'container-fluid')
   div class="pt-3 #{klass}" style=yield(:container_style)
     = content_for?(:containerized_content) ? yield(:containerized_content) : yield
 

data/pg_layout/app/views/pg_layout/_sidebar.html.slim

@@ -6,5 +6,23 @@ div id="sidebar" class="#{@navbar_opened_class} flex-shrink-0 d-none d-#{@breakp
       - @navbar.sidebar.each do |entry|
         - next if @navbar.hide_entry?(entry)
         li.mb-1
-          a href=entry[:path] class="d-inline-flex text-decoration-none pg--nav-button #{@navbar.active_entry?(entry, request) ? 'active' : ''} #{entry[:attributes]}"
-            = entry[:title]
+          - if entry[:children].present?
+            - entry_id = "nav-entry-#{entry[:orig_idx]}"
+            - active = @navbar.active_entry?(entry, request)
+            a class="d-inline-flex text-decoration-none navbar__nav-button #{ active ? '' : 'collapsed'} #{entry[:attributes]}" data-bs-toggle="collapse" href="##{entry_id}" role="button" aria-expanded="#{ active ? 'true' : 'false'}" aria-controls=entry_id
+              = entry[:title]
+              - if active
+                span.bi.bi-chevron-down.ms-1 style="font-size: 0.7em"
+              - else
+                span.bi.bi-chevron-right.ms-1 style="font-size: 0.7em"
+
+            .collapse data-turbo-temporary=('true' unless @navbar.active_entry?(entry, request)) class="#{ 'show' if @navbar.active_entry?(entry, request)}" id=entry_id
+              .collapse-inner
+                - entry[:children].each do |child|
+                  - next if @navbar.hide_entry?(child)
+                  .mb-1
+                    a href=child[:path] class="d-inline-flex text-decoration-none navbar__nav-button #{@navbar.active_entry?(child, request) ? 'active' : ''} #{child[:attributes]}"
+                      = child[:title]
+          - else
+            a href=entry[:path] class="d-inline-flex text-decoration-none navbar__nav-button #{@navbar.active_entry?(entry, request) ? 'active' : ''} #{entry[:attributes]}"
+              = entry[:title]

data/pg_layout/app/views/pg_layout/_sidebar_mobile.html.slim

@@ -10,7 +10,7 @@
         - @navbar.sidebar.each do |entry|
           - next if @navbar.hide_entry?(entry)
           li
-            a href=entry[:path] class="pg--nav-button pe-4 text-light #{@navbar.active_entry?(entry, request) ? 'active' : ''} #{entry[:attributes]}"
+            a href=entry[:path] class="navbar__nav-button pe-4 text-light #{@navbar.active_entry?(entry, request) ? 'active' : ''} #{entry[:attributes]}"
               = entry[:title]
 css:
   .offcanvas a { text-decoration: none; }

data/pg_rails/lib/pg_rails/capybara_support.rb

@@ -50,3 +50,10 @@ Capybara.register_driver :selenium_chrome_headless_iphone,
 Capybara.register_driver :selenium_chrome_debugger, &chrome_driver_gen(headless: false, debugger: true)
 
 Capybara.javascript_driver = ENV.fetch('DRIVER') { 'selenium_chrome_notebook' }.to_sym
+
+RSpec.configure do |config|
+  config.before(:each, type: :system) do
+    driven_by ENV.fetch('DRIVER', :selenium_chrome_headless_iphone).to_sym,
+              using: ENV.fetch('BROWSER', :headless_chrome).to_sym
+  end
+end

data/pg_rails/lib/version.rb

@@ -2,6 +2,6 @@
 
 # :nocov:
 module PgRails
-  VERSION = '7.6.37'
+  VERSION = '7.6.39'
 end
 # :nocov:

data/pg_scaffold/lib/generators/pg_rails/system_spec/templates/system_spec.rb

@@ -8,12 +8,15 @@ require 'rails_helper'
 #
 # DRIVER=selenium BROWSER=firefox rspec
 describe '<%= name %>' do
+  # include ActionView::RecordIdentifier
+
   subject(:visitar) do
-    visit 'some url'
+    path = tpath("/u/t/some_tenanted_path", query_string: false)
+    visit path
   end
 
-  let(:logged_user) { create :user }
-  let(:account) { ActsAsTenant.current_tenant }
+  let(:logged_user) { create :user, :owner }
+  # let(:account) { ActsAsTenant.current_tenant }
 
   before do
     login_as logged_user

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pg_rails
 version: !ruby/object:Gem::Version
-  version: 7.6.37
+  version: 7.6.39
 platform: ruby
 authors:
 - Martín Rosso
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2025-04-02 00:00:00.000000000 Z
+date: 2025-04-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails