# Review: Misc ActiveRecord bug fixes

**URL:** <https://www.ruby-forum.com/t/review-misc-activerecord-bug-fixes/171141>\
**Category:** IronRuby\
**Created:** [June 26, 2009, 11:28pm UTC](https://www.ruby-forum.com/t/review-misc-activerecord-bug-fixes/171141 "2009-06-26T23:28:37Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![Shri\_B](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/s/898d66/32.png) [@Shri\_B](https://www.ruby-forum.com/u/Shri_B)\
**Post date:** [June 26, 2009, 11:28pm UTC](https://www.ruby-forum.com/t/review-misc-activerecord-bug-fixes/171141/1 "2009-06-26T23:28:37Z")

</div>

Thanks to Ray and Joshua for pair debugging this morning!

> <https://github.com/shri-zz/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029>
>
> (timeObj == "string") should return nil. It was throwing TypeError because metho…d overload resolution was finding System.DateTime.op\_Equality. Throwing a TypeError is ok for .NET types with strongly typed op\_Equality, but not for those types that represent Ruby types.
> Fix typo in multi-line regex in YAML engine causing incorrect parsing of Time yaml strings

Module#autoload call should be ignored if the constant is defined.  
Without this, a class definition was being lost if autoload was called  
after the class definition was executed  
(timeObj == “string”) should return nil. It was throwing TypeError  
because method overload resolution was finding  
System.DateTime.op\_Equality. Throwing a TypeError is ok for .NET types  
with strongly typed op\_Equality, but not for those types that represent  
Ruby types.  
Fix typo in multi-line regex in YAML engine causing incorrect parsing of  
Time yaml strings

- 

Merlin/External.LCA\_RESTRICTED/Languages/IronRuby/Yaml/IronRuby.Libraries.Yaml/Engine/BaseConstructor.cs[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-0](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-0)  
\*  
Merlin/External.LCA\_RESTRICTED/Languages/IronRuby/mspec/rubyspec/core/module/autoload\_spec.rb[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-1](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-1)  
\*  
Merlin/External.LCA\_RESTRICTED/Languages/IronRuby/mspec/rubyspec/core/time/eql\_spec.rb[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-2](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-2)  
\*  
Merlin/External.LCA\_RESTRICTED/Languages/IronRuby/mspec/rubyspec/core/time/equal\_value\_spec.rb[http://github.com/shri/ironruby/tree/226ceebdd8e64fc86042e626b9de1e96b4226029/Merlin/External.LCA\_RESTRICTED/Languages/IronRuby/mspec/rubyspec/core/time/equal\_value\_spec.rb](http://github.com/shri/ironruby/tree/226ceebdd8e64fc86042e626b9de1e96b4226029/Merlin/External.LCA_RESTRICTED/Languages/IronRuby/mspec/rubyspec/core/time/equal_value_spec.rb)  
\*  
Merlin/Main/Languages/Ruby/Libraries.LCA\_RESTRICTED/Builtins/KernelOps.cs[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-4](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-4)  
\*  
Merlin/Main/Languages/Ruby/Libraries.LCA\_RESTRICTED/Builtins/TimeOps.cs[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-5](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-5)  
\*  
Merlin/Main/Languages/Ruby/Libraries.LCA\_RESTRICTED/Initializers.Generated.cs[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-6](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-6)  
\*  
Merlin/Main/Languages/Ruby/Ruby/Builtins/RubyModule.cs[http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-7](http://github.com/shri/ironruby/commit/226ceebdd8e64fc86042e626b9de1e96b4226029#diff-7)  
Thanks,  
Shri