diff --git a/src/Orchard.Web/Modules/Orchard.ContentPermissions/Drivers/ContentPermissionsPartDriver.cs b/src/Orchard.Web/Modules/Orchard.ContentPermissions/Drivers/ContentPermissionsPartDriver.cs
index e3d342e36..444d72ac3 100644
--- a/src/Orchard.Web/Modules/Orchard.ContentPermissions/Drivers/ContentPermissionsPartDriver.cs
+++ b/src/Orchard.Web/Modules/Orchard.ContentPermissions/Drivers/ContentPermissionsPartDriver.cs
@@ -119,9 +119,6 @@ namespace Orchard.ContentPermissions.Drivers {
         }
 
         protected override DriverResult Editor(ContentPermissionsPart part, IUpdateModel updater, dynamic shapeHelper) {
-
-            var allRoles = _roleService.GetRoles().Select(x => x.Name).OrderBy(x => x).ToList();
-
             var model = new ContentPermissionsPartViewModel();
 
             if (!updater.TryUpdateModel(model, Prefix, null, null)) {
@@ -142,6 +139,8 @@ namespace Orchard.ContentPermissions.Drivers {
 
                 var settings = part.Settings.TryGetModel<ContentPermissionsPartSettings>();
 
+                var allRoles = _roleService.GetRoles().Select(x => x.Name).OrderBy(x => x).ToList();
+
                 OverrideDefaultPermissions(part, allRoles, settings);
             }