JumpstartLab/curriculum — pull request #34 by verdi327 (2012-01-20 17:41:51+00:00)
Title: JSAttend

Great tut, but had a handful of minor errors that I corrected.

Here is an overview of the changes I made.

Original: Refactoring is the an important part of programming
Should be: Refactoring is an important part of programming

Original
If it’s 11 digits long
Is the first number a 1?
If so, cut it off the leading 1
If not, it’s junk
If not, this number is junk.

Should be
If it’s 11 digits long
Is the first number a 1?
If so, cut off the leading 1
If not, it’s junk

Need to add line about installing "sunlight" gem before you can just call "require sunlight"
Loop makes no reference to adding clean_zipcodes(line[:zipcode]) as the parameter

the method calls for the leg object are incorrect. They need to be "firstname" and "lastname" not first_name and last_name

method names for create letter method are wrong. Listed as "firstname" & "lastname" need to be first_name and last_name

method names in create letters are wrong again. Listed as "#{first_name}" and "#{last_name}". Both need to include line before them otherwise they are returned as undefined local variables

Method name for rank_times is listed initially as time_stats

hours[hour] = hours[hour] + 1 --- needs to have the to_i method added to [hour] to make it work

Inside of the state_data method there is a syntax error under the else statement. It should read state_data[state] = state_data[state] + 1 instead of [:state]