This project is archived and is in readonly mode.
ActiveSupport::OrderedHash::[] does not work identically to ::Hash::[] in ruby 1.8.7
Reported by brewski | June 24th, 2009 @ 07:09 PM | in 2.x
ActiveSupport::OrderedHash::[] does not work identically to ::Hash::[] in ruby 1.8.7. There are two cases where OrderedHash::[] differs from the ruby implementation of Hash::[].
-
OrderedHash::[] does not properly initialize the hash when it receives an array of arrays containing the key value pairs.
-
OrderedHash::[] does not raise an ArgumentError when it receives an odd number of arguments.
Attached is a patch that addresses these issues.
Comments and changes to this ticket
-
Repository July 2nd, 2009 @ 08:46 PM
- State changed from new to resolved
(from [944f4fc7d2ee8fc54008dfda1014983334427edc]) Fixed ActiveSupport::OrderedHash::[] work identically to ::Hash::[] in ruby 1.8.7 [#2832 state:resolved]
Signed-off-by: Yehuda Katz + Carl Lerche ykatz+clerche@engineyard.com
http://github.com/rails/rails/commit/944f4fc7d2ee8fc54008dfda101498... -
Repository July 2nd, 2009 @ 08:46 PM
(from [d03689971758b905fa0087bc93cf474a9d0585f5]) Fixed ActiveSupport::OrderedHash::[] work identically to ::Hash::[] in ruby 1.8.7 [#2832 state:resolved]
Signed-off-by: Yehuda Katz + Carl Lerche ykatz+clerche@engineyard.com
http://github.com/rails/rails/commit/d03689971758b905fa0087bc93cf47...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Attachments
Tags
Referenced by
- 2832 ActiveSupport::OrderedHash::[] does not work identically to ::Hash::[] in ruby 1.8.7 (from [944f4fc7d2ee8fc54008dfda1014983334427edc]) Fixed A...
- 2832 ActiveSupport::OrderedHash::[] does not work identically to ::Hash::[] in ruby 1.8.7 (from [d03689971758b905fa0087bc93cf474a9d0585f5]) Fixed A...