changeset 65:69ee2e406f71 luisf

Correctly outputs an error message if the user's description field is not set.
author luisf
date Mon, 06 Dec 2010 12:54:56 +0000
parents 9d42bcda8cea
children 29d087fde142
files app/models/user.rb app/views/account/register.rhtml
diffstat 2 files changed, 3 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/app/models/user.rb	Mon Dec 06 11:15:03 2010 +0000
+++ b/app/models/user.rb	Mon Dec 06 12:54:56 2010 +0000
@@ -52,6 +52,7 @@
   belongs_to :auth_source
   
   has_one :ssamr_user_detail, :dependent => :destroy, :class_name => 'SsamrUserDetail'
+  accepts_nested_attributes_for :ssamr_user_detail
 
   # Active non-anonymous users scope
   named_scope :active, :conditions => "#{User.table_name}.status = #{STATUS_ACTIVE}"
--- a/app/views/account/register.rhtml	Mon Dec 06 11:15:03 2010 +0000
+++ b/app/views/account/register.rhtml	Mon Dec 06 12:54:56 2010 +0000
@@ -2,12 +2,7 @@
 
 <% form_tag({:action => 'register'}, :class => "tabular") do %>
 <%= error_messages_for 'user' %>
-
-<ul>
-  <% for message in @ssamr_user_details.errors.full_messages %>
-  <li><%= message %></li>
-<% end %>
-</ul>
+<%= error_messages_for 'ssamr_user_details' %>
 
 <div class="box">
 <!--[form:user]-->
@@ -37,7 +32,7 @@
 
 
 <h3><%=l(:label_ssamr_details)%></h3>
-	<%= error_messages_for :ssamr_user_details %>
+	
 		<% fields_for :ssamr_user_details, :builder => TabularFormBuilder, :lang => current_language do |ssamr_user_detail| %>	
 			<p><%= ssamr_user_detail.text_area :description, :rows => 3, :cols => 40, :required => true  %></p>
 		<% end %>