eliminated empty helpers, fixed README

gnocode · Jul 24, 2011 · 608be95 · 608be95
1 parent 3c4ebf5
commit 608be95
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 12 deletions.
diff --git a/README.markdown b/README.markdown
@@ -10,6 +10,8 @@ This project was created by the generous volunteers who participated in the seco
 
 Also, giving credit where credit is due: [Prtlnd](http://www.prtlnd.com) | [Des Mob](http://www.desmob.com)
 
+
+
 Development
 ===========
 Just follow these steps to get started on submitting pull requests!
@@ -19,4 +21,8 @@ Just follow these steps to get started on submitting pull requests!
   * run "bundle install"
   * run "rake db:setup"
   * run "rails server" and double check everything is working for you.
+  * run "rake db:test:prepare" to setup the test database
+  * in a separate terminal window, run "guard" to setup the testing suite
   * Change and submit a pull request back to us!  Also, please try to have a corresponding test to accompany whatever change you make if you can.  Otherwise we have to do it, and that makes us sad :(
+
+Also, there are folks at Ruby Bayou every Tuesday at 7pm who are also probably hacking around on this.  Feel free to come by and join if you'd like to help but aren't sure where / how to start!
diff --git a/app/helpers/categories_helper.rb b/app/helpers/categories_helper.rb
diff --git a/app/helpers/missions_helper.rb b/app/helpers/missions_helper.rb
diff --git a/app/helpers/neighborhoods_helper.rb b/app/helpers/neighborhoods_helper.rb
diff --git a/app/helpers/password_helper.rb b/app/helpers/password_helper.rb
diff --git a/app/helpers/tags_helper.rb b/app/helpers/tags_helper.rb
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -22,7 +22,7 @@ class User < ActiveRecord::Base
   validates :missions, :length => { :minimum => Constants::MinimumMissions, :message => "You must have at least one mission to be listed."}
   validate :avatar_size
 
-  before_save :fix_urls
+  before_save :format_urls
 
   accepts_nested_attributes_for :missions, :reject_if => proc {|attributes| attributes['statement'].blank? }
 
@@ -42,7 +42,7 @@ def avatar_size
     end
   end
 
-  def fix_urls
+  def format_urls
     self.url1 = url1.gsub(%r{(^https?://twitter.com/(#!/)?|@)}, '') unless url1.blank?
     self.url2 = "http://#{url2}" if !url2.blank? && !url2.match(%r{^https?://})
     self.url3 = "http://#{url3}" if !url3.blank? && !url3.match(%r{^https?://})