Project

General

Profile

Actions

Bug #16914

closed

Hash.new with `default_proc` returns this `default_proc` instead of calling it sometimes

Added by beauraF (Florent Beaurain) almost 4 years ago. Updated almost 4 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 2.7.1p83 (2020-03-31 revision a0c7c23c9c) [x86_64-darwin19]
[ruby-core:98523]

Description

Hello,

First of all, I want to apologize because this bug report will not be realized at all in good shape. I encounter this problem on a rails application currently in production, but I am unable to reproduce it. I failed to provide you a script to reproduce. I'm very sorry. But.. maybe it will ring a bell to someone.


So, I have a hash, built this way:

messages
  .each_with_object(Hash.new { [] }) { ... }
  .transform_values { ... }

{
  [:appointment, {:import_identifier=>"TVER111111114^ORBIS"}]=>#<ExternalSyncMessage id: 82, ... >,
  [:appointment, {:import_identifier=>"TVER111111115^ORBIS"}]=>#<ExternalSyncMessage id: 84, ... >,
  [:appointment, {:import_identifier=>"TVER111111116^ORBIS"}]=>#<ExternalSyncMessage id: 85, ... >, 
  [:appointment, {:id=>"30"}]=>#<ExternalSyncMessage id: 86, ... >,
  [:appointment, {:import_identifier=>"TVER111111111^ORBIS"}]=>#<ExternalSyncMessage id: 79, ... >,
  [:appointment, {:import_identifier=>"TVER111111112^ORBIS"}]=>#<ExternalSyncMessage id: 80, ... >,
}

Here is the result in ruby 2.6.6p146 (2020-03-31 revision 67876) [x86_64-darwin19] when I try to access it:

hash[[:appointment, {:import_identifier=>"TVER111111114^ORBIS"}]]
=> #<ExternalSyncMessage id: 82, ... >

hash[[:appointment, {:import_identifier=>"TVER111111117^ORBIS"}]]
=> nil

Here is the result in ruby 2.7.1p83 (2020-03-31 revision a0c7c23c9c) [x86_64-darwin19] when I try to access it:

hash[[:appointment, {:import_identifier=>"TVER111111114^ORBIS"}]]
=> #<ExternalSyncMessage id: 82, ... >

hash[[:appointment, {:import_identifier=>"TVER111111117^ORBIS"}]]
=> #<Proc:0x00007fdc59a6e310 /Users/beauraf/Code/doctolib/doctolib/app/services/sync/farming_support/retry_inbound_messages.rb:75>

hash[[:appointment, {:import_identifier=>"TVER111111117^ORBIS"}]].call
=> []

note: retry_inbound_messages.rb:75 => .each_with_object(Hash.new { [] }) { ... }


So, if I'm right, the result on ruby 2.6, was not the good one but acceptable one on my side. On ruby 2.7, we're close to the good result, but we have to call the proc manually, which causes a lot of worries on my side.

At your entire disposal, if I can help in any way.


Related issues 1 (0 open1 closed)

Is duplicate of Ruby master - Bug #16498: Hash#transform_values in 2.7.0 sets new hash's default to old hash's default_procClosedActions
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0