xueli.xue 8 năm trước cách đây
mục cha
commit
7d31e9838a

+ 11 - 13
xxl-job-admin/src/main/java/com/xxl/job/admin/service/impl/XxlJobServiceImpl.java

@@ -168,6 +168,7 @@ public class XxlJobServiceImpl implements IXxlJobService {
 
 		// stage job info
 		XxlJobInfo exists_jobInfo = xxlJobInfoDao.loadById(jobInfo.getId());
+        String old_cron = exists_jobInfo.getJobCron();
 		if (exists_jobInfo == null) {
 			return new ReturnT<String>(500, "参数异常");
 		}
@@ -181,19 +182,16 @@ public class XxlJobServiceImpl implements IXxlJobService {
 		exists_jobInfo.setExecutorParam(jobInfo.getExecutorParam());
 		exists_jobInfo.setGlueSwitch(jobInfo.getGlueSwitch());
 		exists_jobInfo.setChildJobKey(jobInfo.getChildJobKey());
-		
-		try {
-			// fresh quartz
-			boolean ret = XxlJobDynamicScheduler.rescheduleJob(String.valueOf(exists_jobInfo.getJobGroup()), String.valueOf(exists_jobInfo.getId()), exists_jobInfo.getJobCron());
-			if (ret) {
-				xxlJobInfoDao.update(exists_jobInfo);
-				return ReturnT.SUCCESS;
-			} else {
-				return new ReturnT<String>(500, "更新任务失败");
-			}
-		} catch (SchedulerException e) {
-			logger.error("", e);
-		}
+        xxlJobInfoDao.update(exists_jobInfo);
+
+        try {
+            // fresh quartz
+            boolean ret = XxlJobDynamicScheduler.rescheduleJob(String.valueOf(exists_jobInfo.getJobGroup()), String.valueOf(exists_jobInfo.getId()), exists_jobInfo.getJobCron());
+            return ret?ReturnT.SUCCESS:ReturnT.FAIL;
+        } catch (SchedulerException e) {
+            logger.error("", e);
+        }
+
 		return ReturnT.FAIL;
 	}
 

+ 1 - 1
xxl-job-admin/src/main/webapp/static/js/joblog.index.1.js

@@ -11,7 +11,7 @@ $(function() {
 			dataType : "json",
 			success : function(data){
 				if (data.code == 200) {
-					$("#jobId").html('<option value="" >请选择</option>');
+					$("#jobId").html('<option value="0" >请选择</option>');
 					$.each(data.content, function (n, value) {
                         $("#jobId").append('<option value="' + value.id + '" >' + value.jobDesc + '</option>');
                     });