Hashed_Pasword?

J

Jason James

I am having a problem with my hashed_password.
I am following the lessons in the apress beginning rails book.
Here is my issue:

def password_required?
hashed_password.blank? || !password.blank?
end

Ruby is calling this method out as a problem.

Error message from the web server:
undefined local variable or method `hashed_password' for #<User id: nil,
login: "", email: "", hashed_passwd: nil>

I am stuck!
 
T

Tim Hunter

Jason said:
I am having a problem with my hashed_password.
I am following the lessons in the apress beginning rails book.
Here is my issue:

def password_required?
hashed_password.blank? || !password.blank?
end

Ruby is calling this method out as a problem.

Error message from the web server:
undefined local variable or method `hashed_password' for #<User id: nil,
login: "", email: "", hashed_passwd: nil>

I am stuck!

Is the variable named hashed_password or hashed_passwd?
 
J

Jason James

Tim said:
Is the variable named hashed_password or hashed_passwd?

Here is the entire section of code :
def self.authenticate(login, password)
user = find_by_login(login)
return user if user && user.authenticated?(password)
end

def authenticated?(password)
hashed_password == encrypt(password)
end

protected
def encrypt_new_password
return if password.blank?
self.hashed_password = encrypt(password)
end

def password_required?
hashed_password.blank? || !password.blank?
end

def encrypt(string)
Digest::SHA1.hexdigest(string)
end
 
J

Jason James

Jason said:
Here is the entire section of code :
def self.authenticate(login, password)
user = find_by_login(login)
return user if user && user.authenticated?(password)
end

def authenticated?(password)
hashed_password == encrypt(password)
end

protected
def encrypt_new_password
return if password.blank?
self.hashed_password = encrypt(password)
end

def password_required?
hashed_password.blank? || !password.blank?
end

def encrypt(string)
Digest::SHA1.hexdigest(string)
end

The web server is calling out line 47 which is hashed_password.blank? ||
!password.blank?

I think the hashed_password.blank? is my problem but I do not know why.

the variable is hashed_password.
 
H

Hassan Schroeder

Error message from the web server:
undefined local variable or method `hashed_password' for #<User id: nil,
login: "", email: "", hashed_passwd: nil>

It appears your model has a field called 'hashed_passwd', which is
not the same as 'hashed_password'. So change one of 'em :)
 
M

Michael Malone

Um, shouldn't you be storing the result of the hash into hashed_password?
Also, if this is part of a User class (as suggested by the output
earlier) then you probably want to use @ -scoped variables, not
method-local.
def Klass
some_variable = value
end

is NOT the same as:
def Klass
def initialize
@some_variable = value
end
end

That seems the most likely problem to me. If this doesn't solve your
problem, could you post the point at which you initialise/declare your
variables?

Cheers,
Michael


=======================================================================
This email, including any attachments, is only for the intended
addressee. It is subject to copyright, is confidential and may be
the subject of legal or other privilege, none of which is waived or
lost by reason of this transmission.
If the receiver is not the intended addressee, please accept our
apologies, notify us by return, delete all copies and perform no
other act on the email.
Unfortunately, we cannot warrant that the email has not been
altered or corrupted during transmission.
=======================================================================
 
R

Rick DeNatale

[Note: parts of this message were removed to make it a legal post.]

Here is the entire section of code :
Um, shouldn't you be storing the result of the hash into hashed_password?
Also, if this is part of a User class (as suggested by the output earlier)
then you probably want to use @ -scoped variables, not method-local.
def Klass
some_variable = value
end

is NOT the same as:
def Klass
def initialize
@some_variable = value
end
end

That seems the most likely problem to me. If this doesn't solve your
problem, could you post the point at which you initialise/declare your
variables?
No, you should be aware that this is pretty obviously an ActiveRecord model
object in a Rails app. The accessor methods for database fields are
generated automagically from the database schema.

It's also probably using either the restful_authentication plugin, or it's
older brother acts_as_authenticated, both of which normally use
crypted_password, instead of hashed_password for the field name, although I
beleive that this can be overridden when the authentication code is
generated.

I'm pretty sure that Hassan Schroeder has the right diagnosis.
undefined local variable or method `hashed_password' for #<User id: nil,
login: "", email: "", hashed_passwd: nil>

This indicates that the database column is named hashed_passwd NOT
hashed_password, so ActiveRecord isn't providing a method with the name
hashed_password.

This is the kind of question which would probably be answered much more
quickly, and with less confusion on the rails mailing list.

--
Rick DeNatale

Blog: http://talklikeaduck.denhaven2.com/
Twitter: http://twitter.com/RickDeNatale
WWR: http://www.workingwithrails.com/person/9021-rick-denatale
LinkedIn: http://www.linkedin.com/in/rickdenatale
 
J

Jason James

Rick said:
No, you should be aware that this is pretty obviously an ActiveRecord
model
object in a Rails app. The accessor methods for database fields are
generated automagically from the database schema.

It's also probably using either the restful_authentication plugin, or
it's
older brother acts_as_authenticated, both of which normally use
crypted_password, instead of hashed_password for the field name,
although I
beleive that this can be overridden when the authentication code is
generated.

I'm pretty sure that Hassan Schroeder has the right diagnosis.
undefined local variable or method `hashed_password' for #<User id: nil,
login: "", email: "", hashed_passwd: nil>

This indicates that the database column is named hashed_passwd NOT
hashed_password, so ActiveRecord isn't providing a method with the name
hashed_password.

This is the kind of question which would probably be answered much more
quickly, and with less confusion on the rails mailing list.

--
Rick DeNatale

Blog: http://talklikeaduck.denhaven2.com/
Twitter: http://twitter.com/RickDeNatale
WWR: http://www.workingwithrails.com/person/9021-rick-denatale
LinkedIn: http://www.linkedin.com/in/rickdenatale
Rick,
Thanks for the resources. I will use the mailing list next time.
You were right - I figured out before seeing your post. I submitted the
new user request and was able to look at the error messages from the
web server which led me to take a second look at the database.

Sure enough there was a field spelling error as you pointed out.
Thanks again!
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

Forum statistics

Threads
474,177
Messages
2,570,952
Members
47,506
Latest member
tomiy16522

Latest Threads

Top