On Nov 25, 2008, at 12:34 PM, Peter Jaros wrote:

On Tue, Nov 25, 2008 at 2:04 PM, s.ross <[EMAIL PROTECTED]> wrote:
In Rails, the primary key, by default 'id', is used all over the
place. However, Ruby now deprecates the use of constructs like:

@post = Post.find(:first)
@post_id = @post.id

I think you've got the wrong end of the stick there.  Object#id (which
returns a unique id for the Ruby object) was deprecated in favor of
Object#object_id, mostly because ActiveRecord overrides it to return
the primary key.  #id seemed in retrospect too common to be reserved
for an implementation detail.  You *should* use model.id to get the
primary key.

If you're getting that warning, you're calling #id on something that
doesn't decend from ActiveRecord::Base.  #id is one of the few things
that nil responds to (and issues the deprecation warning); maybe there
are no Posts in your DB?  Try checking the value of @post.

Peter

Thanks Peter, Nick, Pat and Mark. I think I have a better handle on this now. I'm up-migrating what is appearing to me to be an inconsistent codebase to Rails 2.2 and the issue of indexing versus member access came up. It appears I was incorrect in my assessment that using a hash index would be the safer way to go. Thankfully we have a reasonable number of specs and they will help us find and fix a many of these issues.

Steve
_______________________________________________
rspec-users mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/rspec-users

Reply via email to