fix: remove group role manager code and fix vector iteration in DeleteRole.

Author: divy9881

casbin/casbin.vcxproj

@@ -198,7 +198,6 @@
     <ClCompile Include="persist\file_adapter\file_adapter.cpp" />
     <ClCompile Include="persist\file_adapter\filtered_adapter.cpp" />
     <ClCompile Include="rbac\default_role_manager.cpp" />
-    <ClCompile Include="rbac\group_role_manager.cpp" />
     <ClCompile Include="rbac_api.cpp" />
     <ClCompile Include="rbac_api_with_domains.cpp" />
     <ClCompile Include="util\array_equals.cpp" />
@@ -281,7 +280,6 @@
     <ClInclude Include="persist\watcher_ex.h" />
     <ClInclude Include="rbac.h" />
     <ClInclude Include="rbac\default_role_manager.h" />
-    <ClInclude Include="rbac\group_role_manager.h" />
     <ClInclude Include="rbac\pch.h" />
     <ClInclude Include="rbac\role_manager.h" />
     <ClInclude Include="util.h" />

casbin/casbin.vcxproj.filters

@@ -216,9 +216,6 @@
     <ClCompile Include="rbac\default_role_manager.cpp">
       <Filter>Source Files\rbac</Filter>
     </ClCompile>
-    <ClCompile Include="rbac\group_role_manager.cpp">
-      <Filter>Source Files\rbac</Filter>
-    </ClCompile>
     <ClCompile Include="persist\file_adapter\batch_file_adapter.cpp">
       <Filter>Source Files\persist\file_adapter</Filter>
     </ClCompile>
@@ -287,9 +284,6 @@
     <ClInclude Include="rbac\default_role_manager.h">
       <Filter>Header Files\rbac</Filter>
     </ClInclude>
-    <ClInclude Include="rbac\group_role_manager.h">
-      <Filter>Header Files\rbac</Filter>
-    </ClInclude>
     <ClInclude Include="rbac\role_manager.h">
       <Filter>Header Files\rbac</Filter>
     </ClInclude>

casbin/rbac.h

@@ -2,7 +2,6 @@
 #define CASBIN_CPP_RBAC
 
 #include "./rbac/default_role_manager.h"
-#include "./rbac/group_role_manager.h"
 #include "./rbac/role_manager.h"
 
 #endif

casbin/rbac/default_role_manager.cpp

@@ -21,9 +21,9 @@ void Role :: AddRole(Role* role) {
 }
 
 void Role :: DeleteRole(Role* role) {
-    for (vector<Role*> :: iterator it = roles.begin() ; it != roles.end() ; it++) {
-        if (!(*it)->name.compare(role->name)) {
-            roles.erase(it);
+    for (int i = 0; i < roles.size();i++) {
+        if (!roles[i]->name.compare(role->name)) {
+            roles.erase(roles.begin()+i);
         }
     }
 }

casbin/rbac/default_role_manager.h

@@ -69,28 +69,28 @@ class DefaultRoleManager : public RoleManager {
         // AddLink adds the inheritance link between role: name1 and role: name2.
         // aka role: name1 inherits role: name2.
         // domain is a prefix to the roles.
-        void AddLink(string name1, string name2, vector<string> domain);
+        void AddLink(string name1, string name2, vector<string> domain = vector<string>{});
 
         /**
          * deleteLink deletes the inheritance link between role: name1 and role: name2.
          * aka role: name1 does not inherit role: name2 any more.
          * domain is a prefix to the roles.
          */
-        void DeleteLink(string name1, string name2, vector<string> domain);
+        void DeleteLink(string name1, string name2, vector<string> domain = vector<string>{});
 
         /**
          * hasLink determines whether role: name1 inherits role: name2.
          * domain is a prefix to the roles.
          */
-        bool HasLink(string name1, string name2, vector<string> domain);
+        bool HasLink(string name1, string name2, vector<string> domain = vector<string>{});
 
         /**
          * getRoles gets the roles that a subject inherits.
          * domain is a prefix to the roles.
          */
-        vector <string> GetRoles(string name, vector<string> domain);
+        vector <string> GetRoles(string name, vector<string> domain = vector<string>{});
 
-        vector<string> GetUsers(string name, vector<string> domain);
+        vector<string> GetUsers(string name, vector<string> domain = vector<string>{});
 
         /**
          * printRoles prints all the roles to log.

casbin/rbac/group_role_manager.cpp

@@ -6,65 +6,28 @@
 
 using namespace std;
 
+// RoleManager provides interface to define the operations for managing roles.
 class RoleManager {
     public:
-        /**
-         * Clear clears all stored data and resets the role manager to the initial state.
-         */
-        virtual void Clear() = 0;
-
-        /**
-         * AddLink adds the inheritance link between two roles. role: name1 and role: name2.
-         * domain is a prefix to the roles.
-         *
-         * @param name1 the first role (or user).
-         * @param name2 the second role.
-         * @param domain the domain the roles belong to.
-         */
-        virtual void AddLink(string name1, string name2, vector<string> domain) = 0;
-
-        /**
-         * DeleteLink deletes the inheritance link between two roles. role: name1 and role: name2.
-         * domain is a prefix to the roles.
-         *
-         * @param name1 the first role (or user).
-         * @param name2 the second role.
-         * @param domain the domain the roles belong to.
-         */
-        virtual void DeleteLink(string name1, string name2, vector<string> domain) = 0;
-
-        /**
-         * hasLink determines whether a link exists between two roles. role: name1 inherits role: name2.
-         * domain is a prefix to the roles.
-         *
-         * @param name1 the first role (or a user).
-         * @param name2 the second role.
-         * @param domain the domain the roles belong to.
-         * @return whether name1 inherits name2 (name1 has role name2).
-         */
-        virtual bool HasLink(string name1, string name2, vector<string> domain) = 0;
-
-        /**
-         * GetRoles gets the roles that a user inherits.
-         * domain is a prefix to the roles.
-         *
-         * @param name the user (or a role).
-         * @param domain the domain the roles belong to.
-         * @return the roles.
-         */
-        virtual vector<string> GetRoles(string name, vector<string> domain) = 0;
-
-        /**
-         * GetUsers gets the users that inherits a role.
-         * @param name the role.
-         * @return the users.
-         */
-        virtual vector<string> GetUsers(string name, vector<string> domain) = 0;
-
-        /**
-         * PrintRoles prints all the roles to log.
-         */
-        virtual void PrintRoles() = 0;
+    // Clear clears all stored data and resets the role manager to the initial state.
+    virtual void Clear() = 0;
+    // AddLink adds the inheritance link between two roles. role: name1 and role: name2.
+    // domain is a prefix to the roles (can be used for other purposes).
+    virtual void AddLink(string name1, string name2, vector<string> domain = vector<string>{}) = 0;
+    // DeleteLink deletes the inheritance link between two roles. role: name1 and role: name2.
+    // domain is a prefix to the roles (can be used for other purposes).
+    virtual void DeleteLink(string name1, string name2, vector<string> domain = vector<string>{}) = 0;
+    // HasLink determines whether a link exists between two roles. role: name1 inherits role: name2.
+    // domain is a prefix to the roles (can be used for other purposes).
+    virtual bool HasLink(string name1, string name2, vector<string> domain = vector<string>{}) = 0;
+    // GetRoles gets the roles that a user inherits.
+    // domain is a prefix to the roles (can be used for other purposes).
+    virtual vector<string> GetRoles(string name, vector<string> domain = vector<string>{}) = 0;
+    // GetUsers gets the users that inherits a role.
+    // domain is a prefix to the users (can be used for other purposes).
+    virtual vector<string> GetUsers(string name, vector<string> domain = vector<string>{}) = 0;
+    // PrintRoles prints all the roles to log.
+    virtual void PrintRoles() = 0;
 };
 
 #endif