fixes

Francisco Marcelo de Araújo Lima Júnior
1 parent a24518ab
Showing 3 changed files with 9 additions and 23 deletions Show diff stats
app/helpers/profile_helper.rb
app/views/account/_signup_form.html.erb
app/views/profile_editor/_person_form.html.erb
@@ -45,22 +45,16 @@ module ProfileHelper
       return
     end
     value = profile.custom_fields[field][:value]
-    if !value.blank?
-      if block_given?
-        value = yield(value)
-      end
-      content_tag('tr', content_tag('td', title, :class => 'field-name') + content_tag('td', value))
-    else
-      ''
-    end
+    !value.blank? ? content_tag('tr', content_tag('td', title) + content_tag('td', value)) : ''
   end
   def display_custom_fields(profile)
     fields = []
-    profile.custom_fields.each { |key,value|
-      fields << display_custom_field(value[:label], profile, key)
+    profile.custom_fields.each { |custom_field_key,custom_field_data|
+      fields << display_custom_field(custom_field_data[:title], profile, custom_field_key)
     }
-    content_tag('tr', content_tag('th', _('Custom Fields'), { :colspan => 2 })) + fields.join.html_safe
+    fields.reject!(&:blank?)
+    fields.size >= 1 ? content_tag('tr', content_tag('th', _('Custom Fields'), { :colspan => 2 })) + fields.join.html_safe : ''
   end
 end
@@ -117,8 +117,7 @@
             data.custom_fields.each(function(field) {
               html = '<div class="formfieldline">' +
                            '<label class="formlabel" for="profile_data_custom_fields_{#CUSTOM_FIELD_ID#}">{#CUSTOM_FIELD_NAME#}</label>' +
-                           '<input type="hidden" name="profile_data[custom_fields][{#CUSTOM_FIELD_ID#}[required]" id="profile_data_custom_fields_{#CUSTOM_FIELD_ID#}_required" value="on" />' +
-                           '<input type="hidden" name="profile_data[custom_fields][{#CUSTOM_FIELD_ID#}][label]" id="profile_data_custom_fields_{#CUSTOM_FIELD_ID#}_label" value="{#CUSTOM_FIELD_NAME#}" />' +
+                           '<input type="hidden" name="profile_data[custom_fields][{#CUSTOM_FIELD_ID#}][title]" id="profile_data_custom_fields_{#CUSTOM_FIELD_ID#}_title" value="{#CUSTOM_FIELD_NAME#}" />' +
                            '<div class="formfield type-text">' +
                               '<input type="text" name="profile_data[custom_fields][{#CUSTOM_FIELD_ID#}][value]" id="profile_data_custom_fields_{#CUSTOM_FIELD_ID#}_value" />' +
                            '</div>' +
@@ -77,8 +77,7 @@
   <table border="0" style="display: none;">
     <tr>
-      <th><%= _('Field name') %></th>
-      <th><%= _('Required') %></th>
+      <th align="left"><%= _('Field name') %></th>
       <th><%= _('Display on signup?') %></th>
       <th>&nbsp;</th>
     </tr>
@@ -89,9 +88,6 @@
             <%= text_field_tag( "profile_data[custom_fields][#{key}][label]", value[:label], :style => "display:block") %>
           </td>
           <td align="center">
-            <%= check_box_tag "profile_data[custom_fields][#{key}][required]", value['required'], value['required'], :onclick => "required_action('profile_data[custom_fields][#{key}][active]','profile_data[custom_fields][#{key}][required]', 'profile_data[custom_fields][#{key}][signup]')" %>
-          </td>
-          <td align="center">
             <%= check_box_tag "profile_data[custom_fields][#{key}][signup]", value['signup'], value['signup'], :onclick => "signup_action('profile_data[custom_fields][#{key}][active]','profile_data[custom_fields][#{key}][required]', 'profile_data[custom_fields][#{key}][signup]')" %>
           </td>
           <td align="center">
@@ -122,14 +118,11 @@
         <div class="formfieldline">
           <span style="display: block;">
-            <% if value[:required] %>
-              <%= label_tag value[:label] + ' (*)', nil, class: 'required' %>
-            <% else %>
-              <%= label_tag value[:label] %>
-            <% end %>
+            <%= label_tag value[:label] %>
           </span>
           <div class="formfield type-text" style="display: inline-block;">
+            <%= hidden_field_tag "profile_data[custom_fields][#{key}[title]", @person.custom_fields[key][:title] %>
             <% valor = @person.custom_fields[key] ? @person.custom_fields[key][:value] : "" %>
             <%= text_field_tag( "profile_data[custom_fields][#{key}][value]", valor, :size => 30 ) %>
           </div>