third_codes/PowerJob
1
0
Fork 1
mirror of https://github.com/PowerJob/PowerJob.git synced 2025-07-17 00:00:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: can creating a job with the wrong parameters #312

Browse Source
This commit is contained in:
tjq 2021-11-03 21:26:25 +08:00
parent e094c22952
commit 4c0e5f348b
2 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
powerjob-common/src/main/java/tech/powerjob/common/utils/CommonUtils.java
View File

@ -173,4 +173,11 @@ public class CommonUtils {
} catch (InterruptedException ignore) {
}
}
/**
* ignore compile warning of 'Result of 'Long.valueOf()' is ignored '
* @param obj anything
*/
public static void ignoreIgnoredResultWarning(Object obj) {
}
}

12
powerjob-server/powerjob-server-core/src/main/java/tech/powerjob/server/core/service/JobService.java
View File

@ -6,6 +6,7 @@ import tech.powerjob.common.PowerQuery;
import tech.powerjob.common.enums.TimeExpressionType;
import tech.powerjob.common.request.http.SaveJobInfoRequest;
import tech.powerjob.common.response.JobInfoDTO;
import tech.powerjob.common.utils.CommonUtils;
import tech.powerjob.server.common.SJ;
import tech.powerjob.server.common.constants.SwitchableStatus;
import tech.powerjob.server.common.utils.CronExpression;
@ -61,7 +62,7 @@ public class JobService {
*/
public Long saveJob(SaveJobInfoRequest request) throws ParseException {
request.valid();
checkValid(request);
JobInfoDO jobInfoDO;
if (request.getId() != null) {
@ -278,4 +279,13 @@ public class JobService {
return jobInfoDTO;
}
private static void checkValid(SaveJobInfoRequest request) {
request.valid();
if (TimeExpressionType.frequentTypes.contains(request.getTimeExpressionType().getV())) {
CommonUtils.ignoreIgnoredResultWarning(Long.valueOf(request.getTimeExpression()));
}
}
}