Improve insitution edit page

Arthur Esposte
1 parent 1d85e3df
Showing 2 changed files with 20 additions and 17 deletions Show diff stats
controllers/gov_user_plugin_myprofile_controller.rb
views/gov_user_plugin_myprofile/edit_institution.html.erb
@@ -6,9 +6,11 @@ class GovUserPluginMyprofileController &lt; MyProfileController
   def edit_institution
     @show_sisp_field = environment.admins.include?(current_user.person)
-    @state_list = NationalRegion.find(:all, :conditions =>
-    { :national_region_type_id =>  2 },
-    :order => 'name')
+    @state_list = NationalRegion.find(
+                    :all,
+                    :conditions => { :national_region_type_id => 2 },
+                    :order => 'name'
+                  )
     @institution = @profile.institution
     update_institution if request.post?
   end
@@ -26,7 +28,11 @@ class GovUserPluginMyprofileController &lt; MyProfileController
         _("Could not find Governmental Power or Governmental Sphere"))
       end
     end
-    flash[:errors] = @institution.errors.full_messages unless @institution.valid?
+    if @institution.valid?
+      redirect_to :controller => 'profile_editor', :action => 'index', :profile => profile.identifier
+    else
+      flash[:errors] = @institution.errors.full_messages
+    end
   end
   def governmental_updates
@@ -6,23 +6,20 @@
   </div>
 <%end %>
-<% unless flash[:errors].nil? %>
-<div class="errorExplanation" id="errorExplanation">
-  <h2> <%= _("Can`t create new Institution: #{flash[:errors].length} errors") %> </h2>
-  <ul>
-  <% flash[:errors].each do |error| %>
-    <li> <%= error %> </li>
-  <% end %>
-  </ul>
-</div>
+<% unless flash[:errors].blank? %>
+  <div class="errorExplanation" id="errorExplanation">
+    <h2> <%= _("Can`t create new Institution: #{flash[:errors].length} errors") %> </h2>
+    <ul>
+    <% flash[:errors].each do |error| %>
+      <li> <%= error %> </li>
+    <% end %>
+    </ul>
+  </div>
 <% end %>
 <div id = 'create_institution_errors' class='errorExplanation hide-field'></div>
 <div>
-  <div class="fields-required">
-    <span class="errorExplanation"><%= _("All fields with (*) are mandatory") %></span>
-  </div>
   <%= labelled_form_for :community,:html => { :multipart => true, :id=>"institution_form" } do |f| %>
     <%= hidden_field_tag "edit_institution_page", true %>
     <%= fields_for :institutions do |inst| %>
@@ -61,7 +58,7 @@
       <%= required f.text_field(:city, :value => @institution.community.city) %>
-      <span class='required-field'>
+      <span class='optional-field'>
         <div class="formfield type-text">
           <%= inst.label("cnpj" ,_("CNPJ"), :class=>"formlabel") %>
           <%= required inst.text_field(:cnpj, :placeholder=>"99.999.999/9999-99", :class=>"intitution_cnpj_field", :value => @institution.cnpj) %>