diff Agendas/trunk/src/Agendas.Web/Controllers/PropuestaController.cs @ 53:d149bfea3892

Tratamos las validaciones como resultados y no como excepciones, dejamos las excepciones solo para lo que no esperamos que suceda.
author nelopauselli
date Sat, 14 May 2011 12:45:50 -0300
parents 475be11edf56
children 3059a5f8930f
line wrap: on
line diff
--- a/Agendas/trunk/src/Agendas.Web/Controllers/PropuestaController.cs	Sat May 14 12:17:35 2011 -0300
+++ b/Agendas/trunk/src/Agendas.Web/Controllers/PropuestaController.cs	Sat May 14 12:45:50 2011 -0300
@@ -36,16 +36,10 @@
 			{
 				var agenda = AgendaFactory.GetAgenda();
 
-				try
-				{
-					agenda.Proponer(model.Titulo, model.Ponente);
-
+				var r = agenda.Proponer(model.Titulo, model.Ponente);
+				if (r.Succeful)
 					return RedirectToAction("Index");
-				}
-				catch (ValidationException ex)
-				{
-					ModelState.AddModelError("error", ex.ToString());
-				}
+				ModelState.AddModelError("error", r.ToString());
 			}
 			return View(model);
 		}
@@ -77,16 +71,10 @@
 			{
 				var agenda = AgendaFactory.GetAgenda();
 
-				try
-				{
-					agenda.ModificarPropuesta(new Guid(model.Id), model.Titulo, model.Ponente);
-
+				var r = agenda.ModificarPropuesta(new Guid(model.Id), model.Titulo, model.Ponente);
+				if (r.Succeful)
 					return RedirectToAction("Index");
-				}
-				catch (ValidationException ex)
-				{
-					ModelState.AddModelError("error", ex.ToString());
-				}
+				ModelState.AddModelError("error", r.ToString());
 			}
 			return View(model);
 		}