Skip to content

Commit 2551eeb

Browse files
committed
fix: add duplicate check
1 parent 37302f1 commit 2551eeb

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

src/main/java/org/casbin/jcasbin/main/InternalEnforcer.java

+7-1
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,9 @@ class InternalEnforcer extends CoreEnforcer {
2626
* addPolicy adds a rule to the current policy.
2727
*/
2828
boolean addPolicy(String sec, String ptype, List<String> rule) {
29+
if (model.hasPolicy(sec, ptype, rule)) {
30+
return false;
31+
}
2932

3033
if (adapter != null && autoSave) {
3134
try {
@@ -47,6 +50,9 @@ boolean addPolicy(String sec, String ptype, List<String> rule) {
4750
* removePolicy removes a rule from the current policy.
4851
*/
4952
boolean removePolicy(String sec, String ptype, List<String> rule) {
53+
if (!model.hasPolicy(sec, ptype, rule)) {
54+
return false;
55+
}
5056

5157
if (adapter != null && autoSave) {
5258
try {
@@ -61,7 +67,7 @@ boolean removePolicy(String sec, String ptype, List<String> rule) {
6167
}
6268
}
6369

64-
return model.removePolicy(sec, ptype, rule);
70+
return model.removePolicy(sec, ptype, rule);
6571
}
6672

6773
/**

0 commit comments

Comments
 (0)