From 8eb6076f1c64e18f0ac58364877eb09d69b801e2 Mon Sep 17 00:00:00 2001
From: Cameron Tod <cameron.tod@gmail.com>
Date: Wed, 22 Aug 2012 14:14:05 +0100
Subject: [PATCH] Added check if role exists before creating.

---
 secure_permissions.module |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/secure_permissions.module b/secure_permissions.module
index 322fea8..bd992f4 100644
--- a/secure_permissions.module
+++ b/secure_permissions.module
@@ -187,9 +187,11 @@ function secure_permissions_build_roles() {
   $remove_roles = array_diff($roles, $secure_roles);
   // Add new roles.
   foreach ($new_roles as $rid => $name) {
-    $role = new stdClass();
-    $role->name = $name;
-    user_role_save($role);
+    if (user_role_load_by_name($name) == FALSE) {
+      $role = new stdClass();
+      $role->name = $name;
+      user_role_save($role);
+    }
   }
   // Delete old roles.
   $omit = array(DRUPAL_ANONYMOUS_RID, DRUPAL_AUTHENTICATED_RID);
-- 
1.7.8.3

