Skip to content

Commit a5d7c24

Browse files
authored
Merge pull request hs-web#14 from hs-web/2.2
修复动态脚本更新引发类不存在的bug
2 parents 522fbeb + e04e35c commit a5d7c24

File tree

4 files changed

+6
-10
lines changed

4 files changed

+6
-10
lines changed

hsweb-web-concurrent/hsweb-web-concurrent-lock/src/main/java/org/hsweb/concurrent/lock/support/AnnotationLockAopAdvice.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public class AnnotationLockAopAdvice {
3232

3333
@Autowired
3434
private LockFactory lockFactory;
35-
private ConcurrentMap<String, Lock> lockMap = new ConcurrentHashMap<>();
35+
private ConcurrentMap<String, Lock> lockMap = new ConcurrentHashMap<>();
3636
private ConcurrentMap<String, ReadWriteLock> readWriteLockMap = new ConcurrentHashMap<>();
3737

3838
@Autowired(required = false)
@@ -175,8 +175,7 @@ public String getLockName(ProceedingJoinPoint pjp) throws Throwable {
175175
}
176176
if (expressionScopeBeanMap != null) var.putAll(expressionScopeBeanMap);
177177
ExecuteResult result = engine.execute(expressionId, var);
178-
if (result.getException() != null) throw result.getException();
179-
lockNameStr = result.getResult().toString();
178+
lockNameStr = (String) result.getIfSuccess();
180179
} else {
181180
lockNameStr = lockName.value();
182181
}

hsweb-web-controller/src/main/java/org/hsweb/web/controller/script/DynamicScriptController.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,10 +107,7 @@ public ResponseMessage runtime(@PathVariable("type") String type,
107107
String id = "script.runtime";
108108
engine.compile(id, script);
109109
ExecuteResult result = engine.execute(id);
110-
if (!result.isSuccess()) {
111-
if (result.getException() != null) throw result.getException();
112-
}
113-
return ResponseMessage.ok(result.getResult());
110+
return ResponseMessage.ok(result.getIfSuccess());
114111
}
115112

116113

hsweb-web-core/src/main/java/org/hsweb/web/core/authorize/validator/SimpleAuthorizeValidator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,14 @@ public boolean validate(User user, Map<String, Object> param, AuthorizeValidator
5555
DynamicScriptEngine engine = DynamicScriptEngineFactory.getEngine(expression.getLanguage());
5656
Map<String, Object> var = getExpressionRoot(user);
5757
var.putAll(param);
58-
return StringUtils.isTrue(engine.execute(expression.getId(), var).getResult());
58+
return StringUtils.isTrue(engine.execute(expression.getId(), var).get());
5959
});
6060
else
6161
access = expressions.stream().anyMatch(expression -> {
6262
DynamicScriptEngine engine = DynamicScriptEngineFactory.getEngine(expression.getLanguage());
6363
Map<String, Object> var = getExpressionRoot(user);
6464
var.putAll(param);
65-
return StringUtils.isTrue(engine.execute(expression.getId(), var).getResult());
65+
return StringUtils.isTrue(engine.execute(expression.getId(), var).get());
6666
});
6767
}
6868

hsweb-web-service/hsweb-web-service-simple/src/main/java/org/hsweb/web/service/impl/form/validator/GroovyDycBeanValidator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public GroovyDycBeanValidator(String className, javax.validation.Validator hiber
3535
public boolean validateMap(Map<Object, Object> data, Operation operation) {
3636
ValidateResults results = new ValidateResults();
3737
try {
38-
Class validatorTargetClass = (Class) engine.execute(className, new HashMap<>()).getResult();
38+
Class validatorTargetClass = (Class) engine.execute(className, new HashMap<>()).getIfSuccess();
3939
Object validatorTarget = validatorTargetClass.newInstance();
4040
Set<ConstraintViolation<Object>> result = new LinkedHashSet<>();
4141
if (operation == Operation.INSERT) {

0 commit comments

Comments
 (0)