Download as
Browse Code »

Commit 77861d20827f064e837054347732fb8ac1af4999

Authored by Antonio Terceiro
1 parent 1f3a4c22
Exists in master and in 28 other branches 1.2+SPB2, 2_split_by_role, add_fields_in_api, api_fixies, api_pagination, api_private_token, api_tasks, article_hit_count, blog_page_bug, checkbox_to_user_can_edit_page, communities_ratings, communities_ratings_moderation, community_ratings_rebase, fix_cloned_article_parent, fix_gallery_image_url, fix_javascript_categories, fix_optional_fields, fix_suggest_article, fix_user_email_validation, highlight_block_fix, notification_plugin, notification_plugin_rebase, script_to_update, split_by_role, stable, temp_ratings, tests, tests_fixies

ActionItem1167: hide not informed city and state

Showing 2 changed files with 5 additions and 3 deletions   Show diff stats
app/controllers/public/contact_controller.rb
  Diff comments   View file @77861d2
... ... @@ -10,8 +10,8 @@ class ContactController < PublicController
10 10 if request.post? && params[self.icaptcha_field].blank?
11 11 @contact = Contact.new(params[:contact])
12 12 @contact.dest = profile
13   - @contact.city = (!params[:city].blank? && City.exists?(params[:city])) ? City.find(params[:city]).name : _('Missing')
14   - @contact.state = (!params[:state].blank? && State.exists?(params[:state])) ? State.find(params[:state]).name : _('Missing')
  13 + @contact.city = (!params[:city].blank? && City.exists?(params[:city])) ? City.find(params[:city]).name : nil
  14 + @contact.state = (!params[:state].blank? && State.exists?(params[:state])) ? State.find(params[:state]).name : nil
15 15 if @contact.deliver
16 16 flash[:notice] = _('Contact successfully sent')
17 17 redirect_to :action => 'new'
... ...
app/views/contact/sender/mail.rhtml
  Diff comments   View file @77861d2
1 1 <%= _('Name: %s') % @name %>
2 2 <%= _('e-Mail: %s') % @email %>
3   -<%= _('City and state: %s-%s') % [@city, @state] %>
  3 +<% if @city || @state %>
  4 + <%= _('City and state: %s/%s') % [@city || '?', @state || '?'] %>
  5 +<% end %>
4 6  
5 7 <%= _('Message:') %>
6 8 --
... ...