# Code Review: ActiveRecord fix

**URL:** <https://www.ruby-forum.com/t/code-review-activerecord-fix/183396>\
**Category:** IronRuby\
**Created:** [February 13, 2010, 12:03am UTC](https://www.ruby-forum.com/t/code-review-activerecord-fix/183396 "2010-02-13T00:03:12Z")\
**Posts on this page:** 2\
**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:** [February 13, 2010, 12:03am UTC](https://www.ruby-forum.com/t/code-review-activerecord-fix/183396/1 "2010-02-13T00:03:12Z")

</div>

tfpt review “/shelveset:ar;REDMOND\sborde”

Comment :

remove\_const was not returning the old value in some cases

---

<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:** [February 13, 2010, 2:26am UTC](https://www.ruby-forum.com/t/code-review-activerecord-fix/183396/2 "2010-02-13T02:26:00Z")

</div>

Looks good.

Tomas

From: Shri B.  
Sent: Friday, February 12, 2010 3:02 PM  
To: IronRuby External Code R.  
Cc: removed\_email\_address@domain.invalid  
Subject: Code Review: ActiveRecord fix

tfpt review “/shelveset:ar;REDMOND\sborde”

Comment :

remove\_const was not returning the old value in some cases