pg_rails 7.6.41 → 7.6.43

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 64c0c348910ec6e2795eebf4add22342e4f99c1e0a190a9a8f37299b542c5804
-  data.tar.gz: d128c7801a01c1beb02c52c3033b90128a6a33e2a93737972f4475a18e7d0112
+  metadata.gz: a32df0403ce2d6750bef9c081329cf0b64d50170aa9419784c69196781c43a03
+  data.tar.gz: c14290d74b1c8e382dfeacdf0d6d507116d5abbd8f48196e77b2893199a212a8
 SHA512:
-  metadata.gz: 1561850b970f4c782facbdca1169edda0264795e3f67c7be1525a08bd4bb6405b2b43e57b54ba5d0e24d62423185b205f3e9491b98559da444515aa9e79d6a10
-  data.tar.gz: e57dfba1fbbe3adbbcf8019807bfd5cdddc3fc7a97a2081e381ab369ebf7b8c35cda2fcd97fa22af64c030140c5c51caa4fb257444fa4d662dbc4d2a85a6449c
+  metadata.gz: bd4228c52176a156318f9f25fffdbdfa3095192d7a18e85b9adb4c70f5b14ef71317e7a6fdfb3ac014cbd614f1a08c3c3486f1a75825d5f8c7584503e638aef7
+  data.tar.gz: d47e45800ff171164c443b924f6f7b7d40347ef57aaf7c15419aa378f1c3ddf1dc00642cfffe3661e0cd3ac2f299903124a092e5603450e645c940642556981f

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

@@ -21,7 +21,7 @@ module PgAssociable
       # porque de todos modos se pisaría en el create
       if !object.persisted? &&
          template.nested_record.present? &&
-         object.send(atributo) == (template.nested_record)
+         object.send(atributo) == template.nested_record
         options[:disabled] = true
       end
 

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

@@ -13,7 +13,6 @@ module Admin
       end
     end
 
-    # rubocop:disable Metrics/MethodLength
     def create
       @simple_user_notifier = SimpleUserNotifier.new(modelo_params)
       # @simple_user_notifier.message.save!

data/pg_engine/app/controllers/concerns/pg_engine/resource.rb

@@ -1,6 +1,5 @@
 module PgEngine
   module Resource
-    # rubocop:disable Metrics/PerceivedComplexity
     def self.included(clazz)
       class << clazz
         # This is a per class variable, all subclasses of clazz inherit it
@@ -86,7 +85,6 @@ module PgEngine
 
       clazz.layout :set_layout
     end
-    # rubocop:enable Metrics/PerceivedComplexity
 
     def column_options_for(_object, _attribute)
       { class: 'text-nowrap' }
@@ -293,7 +291,14 @@ module PgEngine
     protected
 
     def discard_undiscard(method)
+      object = instancia_modelo
       if instancia_modelo.send(method)
+        if method == :discard
+          ActiveSupport::Notifications.instrument("record_discarded.pg_engine", object)
+        else
+          ActiveSupport::Notifications.instrument("record_restored.pg_engine", object)
+        end
+
         if accepts_turbo_stream?
           body = <<~HTML.html_safe
             <pg-event data-event-name="pg:record-updated" data-reload="true" data-turbo-temporary>
@@ -382,7 +387,9 @@ module PgEngine
 
     def pg_respond_update
       object = instancia_modelo
+      # Maybe lock! to ensure consistency of audits
       if (@saved = object.save)
+        ActiveSupport::Notifications.instrument("record_updated.pg_engine", object)
         respond_to do |format|
           format.html do
             if params[:inline_attribute].present?
@@ -419,6 +426,7 @@ module PgEngine
     def pg_respond_create
       object = instancia_modelo
       if (@saved = object.save)
+        ActiveSupport::Notifications.instrument("record_created.pg_engine", object)
         if in_modal?
           body = <<~HTML.html_safe
             <pg-event data-event-name="pg:record-created" data-turbo-temporary
@@ -472,9 +480,9 @@ module PgEngine
       end
     end
 
-    # rubocop:disable Metrics/PerceivedComplexity
     def pg_respond_destroy(model, land_on = nil)
       if destroy_model(model)
+        ActiveSupport::Notifications.instrument("record_destroyed.pg_engine", model)
         # TODO!!: rename to main
         if turbo_frame? && current_turbo_frame != 'top'
           body = <<~HTML.html_safe
@@ -509,7 +517,6 @@ module PgEngine
         redirect_back(fallback_location: root_path, status: 303)
       end
     end
-    # rubocop:enable Metrics/PerceivedComplexity
 
     def destroy_model(model)
       @error_message = I18n.t('pg_engine.resource_not_destroyed', model: model.class)
@@ -581,12 +588,12 @@ module PgEngine
       if action_name == 'new'
         params.permit(atributos_permitidos)
       else
-        params.require(nombre_modelo).permit(atributos_permitidos)
+        params.require(clase_modelo.model_name.param_key).permit(atributos_permitidos)
       end
     end
 
     def nombre_modelo
-      clase_modelo.name.underscore
+      clase_modelo.model_name.element
     end
 
     def filtros_y_policy(campos, dflt_sort = nil, archived: false)

data/pg_engine/app/controllers/concerns/pg_engine/tenant_helper.rb

@@ -2,7 +2,6 @@
 
 module PgEngine
   module TenantHelper
-    # rubocop:disable Metrics/AbcSize
     def set_tenant_from_params_or_fail!
       if ActsAsTenant.current_tenant.present?
         return unless Rails.env.test?

data/pg_engine/app/controllers/pg_engine/health_controller.rb

@@ -11,11 +11,19 @@ module PgEngine
       check_postgres
       check_websocket
       check_ssl
+      # FIXME: make configurable
+      check_good_job unless ENV.fetch("HEALTH_CHECK_SKIP_GOOD_JOB", nil) == "1"
       render_up
     end
 
     private
 
+    def check_good_job
+      return if GoodJob::Process.active.count.positive?
+
+      raise PgEngine::Error, 'good_job is down'
+    end
+
     def check_postgres
       return if User.count.is_a? Integer
 
@@ -28,7 +36,6 @@ module PgEngine
       raise PgEngine::Error, 'redis is down'
     end
 
-    # rubocop:disable Metrics/MethodLength
     def check_websocket
       result = nil
       begin

data/pg_engine/app/decorators/pg_engine/base_record_decorator.rb

@@ -196,7 +196,7 @@ module PgEngine
     end
 
     def target_new
-      mod_name_sing = object.class.model_name.singular.to_sym
+      mod_name_sing = object.class.model_name.singular_route_key.to_sym
       [:new, pg_namespace, nested_record, mod_name_sing]
     end
 

data/pg_engine/app/helpers/pg_engine/form_helper.rb

@@ -2,7 +2,6 @@
 
 module PgEngine
   module FormHelper
-    # rubocop:disable Metrics/CyclomaticComplexity
     def pg_form_for(object, *args, &block_passed)
       resource = object
       if object.is_a? PgEngine::BaseRecordDecorator

data/pg_engine/lib/pg_engine/active_job_extensions.rb

@@ -8,7 +8,7 @@ module PgEngine
     end
 
     def deserialize(job_data)
-      ::Current.app_name = job_data.delete('app_name')
+      ::Current.app_name = job_data.delete('app_name')&.to_sym
       ::Current.user = User.find(job_data.delete('user_id')) if job_data['user_id'].present?
 
       super

data/pg_engine/lib/pg_engine/mailgun/log_sync.rb

@@ -7,7 +7,7 @@ require 'json'
 module PgEngine
   module Mailgun
     class LogSync
-      def self.download # rubocop:disable Metrics/AbcSize
+      def self.download
         key = Rails.application.credentials.dig(:mailgun, :api_key)
         mg_client = ::Mailgun::Client.new(key)
         items = []

data/pg_engine/lib/pg_engine/navigator.rb

@@ -1,6 +1,5 @@
 module PgEngine
   class Navigator
-    # rubocop:disable Metrics/MethodLength
     def configure(navbar)
       # *****************************************************
 

data/pg_engine/lib/pg_engine/test/dummy_brand.rb

@@ -1,7 +1,7 @@
 module PgEngine
   module Test
     class DummyBrand < PgEngine::SiteBrand
-      def initialize(include_all: false) # rubocop:disable Metrics/MethodLength
+      def initialize(skip_default_url_options: Rails.env.test?)
         super()
 
         @default_site_brand = :factura
@@ -48,7 +48,7 @@ module PgEngine
           }
         }
 
-        return unless Rails.env.development? || include_all
+        return if skip_default_url_options
 
         @options.merge!({
                           default_url_options: {

data/pg_engine/spec/mailers/pg_engine/base_mailer_spec.rb

@@ -27,7 +27,7 @@ describe PgEngine::BaseMailer do
     # rubocop:disable Style/GlobalVars
     before do
       $site_brand_before = PgEngine.site_brand
-      PgEngine.site_brand = PgEngine::Test::DummyBrand.new(include_all: true)
+      PgEngine.site_brand = PgEngine::Test::DummyBrand.new(skip_default_url_options: false)
     end
 
     after do

data/pg_engine/spec/requests/base_controller_requests_spec.rb

@@ -42,7 +42,7 @@ describe 'Base requests' do
         { 'ACCEPT' => 'text/html' }
       end
 
-      include_examples 'manda el status correcto'
+      it_behaves_like 'manda el status correcto'
 
       it 'no manda el flash' do
         subject
@@ -60,7 +60,7 @@ describe 'Base requests' do
         { 'ACCEPT' => 'text/vnd.turbo-stream.html' }
       end
 
-      include_examples 'manda el status correcto'
+      it_behaves_like 'manda el status correcto'
 
       it 'manda el flash' do
         subject
@@ -73,7 +73,7 @@ describe 'Base requests' do
         { 'ACCEPT' => 'application/json' }
       end
 
-      include_examples 'manda el status correcto'
+      it_behaves_like 'manda el status correcto'
     end
   end
 

data/pg_engine/spec/system/signup_spec.rb

@@ -6,7 +6,7 @@ describe 'Al Registrarse' do
   include ActiveJob::TestHelper
 
   find_scroll = proc do |selector, options = {}|
-    elem = find(selector, **options.merge(visible: :all))
+    elem = find(selector, **options, visible: :all)
     script = 'arguments[0].scrollIntoView({ behavior: "instant", block: "start", inline: "nearest" });'
     page.execute_script(script, elem)
     sleep 0.5

data/pg_layout/app/lib/navbar.rb

@@ -31,6 +31,14 @@ class Navbar
     @configured = true
   end
 
+  def topbar_signed_in_links
+    return [] if @user.blank?
+
+    configure
+
+    bar('topbar.signed_in')
+  end
+
   def sidebar
     configure
 
@@ -43,13 +51,13 @@ class Navbar
     bar_data = @yaml_data[key]
     return [] if bar_data.blank?
 
-    # rubocop:disable Style/MultilineBlockChain:
+    # rubocop:disable Style/MultilineBlockChain
     orig_idx = 0
     bar_data.map do |item|
       orig_idx += 1
       evaluate_node(item, orig_idx)
     end.sort_by { |a| [a[:priority], a[:orig_idx]] }
-    # rubocop:enable Style/MultilineBlockChain:
+    # rubocop:enable Style/MultilineBlockChain
   end
 
   def evaluate_node(item, orig_idx = nil)

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

@@ -33,6 +33,8 @@ html
     meta name="view-transition" content="same-origin"
     meta name="cable-history-timestamp" content="#{Time.now.to_i}"
 
+    / Para qué era esto? Tenía que ver con la cache, no sé si del browser o de nginx, o de Rails
+    / Pero no sé si sigue siendo necesario. Se usa en las páginas estáticas de legales y history
     - unless @skip_csrf_meta_tags
       = csrf_meta_tags
 

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

@@ -60,5 +60,12 @@ ul.navbar-nav.gap-3.align-items-center class="gap-#{@breakpoint_navbar_expand}-0
       li = link_to Account.nombre_plural, users_accounts_path(tid: nil), class: 'dropdown-item'
       - if Current.user.developer?
         li = link_to 'Admin', admin_users_path, class: 'dropdown-item'
+
+      - if @navbar.present?
+        - @navbar.topbar_signed_in_links.each do |entry|
+          - next if @navbar.hide_entry?(entry)
+          li
+            a href=entry[:path] class="dropdown-item #{@navbar.active_entry?(entry, request) ? 'active' : ''} #{entry[:attributes]}"
+              = entry[:title]
       li = link_to 'Cerrar sesión', destroy_user_session_path(tid: nil),
                    'data-turbo-method': :delete, class: 'dropdown-item'

data/pg_rails/lib/version.rb

@@ -2,6 +2,6 @@
 
 # :nocov:
 module PgRails
-  VERSION = '7.6.41'
+  VERSION = '7.6.43'
 end
 # :nocov:

data/pg_scaffold/lib/generators/pg_factory_bot/model/model_generator.rb

@@ -82,7 +82,7 @@ module PgFactoryBot
           'Faker::Lorem.sentence'
         elsif attribute.type == :date
           'Faker::Date.backward'
-        elsif attribute.type == :float || attribute.type == :decimal
+        elsif %i[float decimal].include?(attribute.type)
           'Faker::Number.decimal(l_digits: 3, r_digits: 2)'
         elsif attribute.type == :integer
           'rand(1..9999)'

data/pg_scaffold/lib/generators/pg_rspec/scaffold/scaffold_generator.rb

@@ -15,8 +15,8 @@ module PgRspec
       class_option :trackeo_de_usuarios, type: :boolean, default: true,
                                          desc: 'Genera campos creado_por y actualizado_por.'
 
-      class_option :controller_specs, type: :boolean, default: true
-      class_option :request_specs,    type: :boolean, default: false
+      class_option :controller_specs, type: :boolean, default: false
+      class_option :request_specs,    type: :boolean, default: true
       class_option :view_specs,       type: :boolean, default: false
 
       # remove_hook_for :integration_tool, as: :integration

data/pg_scaffold/spec/generators_spec.rb

@@ -33,9 +33,8 @@ describe 'Generators', type: :generator do
     it do
       run_generator(['users/modelo', 'bla:integer'])
 
-      my_assert_file 'spec/controllers/users/modelos_controller_spec.rb' do |content|
-        expect(content).to match(/routing/)
-        expect(content).to match(/sign_in/)
+      my_assert_file 'spec/requests/users/modelos_spec.rb' do |content|
+        expect(content).to match(/valid_attributes/)
       end
     end
   end

metadata

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