From f6b1ae4031ded2c68c991be8635129a05f01f9fd Mon Sep 17 00:00:00 2001
From: Kevin LaBranche <klabran@hotmail.com>
Date: Tue, 7 Dec 2010 22:20:38 -0700
Subject: [PATCH] Basic fix for Work Item 16558 - Adding too long role name
 causes page error.  Added Data annotation to limit role name to 255 chars to
 avoid YSOD.

---
 .../Modules/Orchard.Roles/ViewModels/RoleCreateViewModel.cs     | 2 +-
 .../Modules/Orchard.Roles/ViewModels/RoleEditViewModel.cs       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleCreateViewModel.cs b/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleCreateViewModel.cs
index b0fbc44e0..72ea8eecf 100644
--- a/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleCreateViewModel.cs
+++ b/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleCreateViewModel.cs
@@ -4,7 +4,7 @@ using Orchard.Security.Permissions;
 
 namespace Orchard.Roles.ViewModels {
     public class RoleCreateViewModel  {
-        [Required]
+        [Required, StringLength(255)]
         public string Name { get; set; }
         public IDictionary<string, IEnumerable<Permission>> FeaturePermissions { get; set; }
     }
diff --git a/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleEditViewModel.cs b/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleEditViewModel.cs
index 49b7b2647..2046327ca 100644
--- a/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleEditViewModel.cs
+++ b/src/Orchard.Web/Modules/Orchard.Roles/ViewModels/RoleEditViewModel.cs
@@ -5,7 +5,7 @@ using Orchard.Security.Permissions;
 namespace Orchard.Roles.ViewModels {
     public class RoleEditViewModel  {
         public int Id { get; set; }
-        [Required]
+        [Required, StringLength(255)]
         public string Name { get; set; }
         public IDictionary<string, IEnumerable<Permission>> RoleCategoryPermissions { get; set; }
         public IEnumerable<string> CurrentPermissions { get; set; }