Friday, January 20, 2012

[Rails] Re: Any help with refactory this ugly code?

Thanks a lot.

I maked some improvments =>

#verifications/index.haml
- if Ad.unverified_ads.size == 0
- else
= link_to "Fast Verify",
fast_verify_info_verification_path(Ad.unverified_ads.last), :method =>
:put
= link_to "Normal Verify", verify_info_verification_path(ad), :method =>
:put

#verifications/verify_info.haml
= link_to "Verify", verify_verification_path(@ad)

#verifications/fast_verify_info.haml
= link_to "Verify",
fast_verify_verification_path(Ad.unverified_ads.last), :method => :put

#verification_controller.rb
def verify_info
@ad = Ad.find(params[:id])
end

def fast_verify_info
@ad = Ad.find(params[:id])
end

def fast_verify
@ad = Ad.find(params[:id])
@ad.verify!
if Ad.unverified_ads.size == 0
redirect_to verifications_path
else
redirect_to
fast_verify_info_verification_path(Ad.unverified_ads.last)
end
end

def verify
@ad = Ad.find(params[:id])
@ad.verify!
if params[:fast] == true
redirect_to Ad.unverified_ads.last
else
redirect_to verifications_path
end
end

--
Posted via http://www.ruby-forum.com/.

--
You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group.
To post to this group, send email to rubyonrails-talk@googlegroups.com.
To unsubscribe from this group, send email to rubyonrails-talk+unsubscribe@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en.

0 Comments:

Post a Comment

Subscribe to Post Comments [Atom]

<< Home


Real Estate