[Fix-15036] [API] Fix task definition edit doesn't work

[sdhzwc]

fix #15036

Problem description: All versions have the problem that task definition changes do not take effect in the workflow and vice versa. There are several points as follows:

1. Modify the task definition without a predecessor task, but it is not available in the workflow.
   2、Modify the task definition with a predecessor task, but it is not available in the workflow.
   3、Switch the workflow version and modify the task definition, which is not available in the workflow.

Dynamic demonstration of before and after repair comparison:
Before Repair

二、After Repair

[EricGao888]

@sdhzwc Could u plz use English for PR descriptions?

[sdhzwc]

@sdhzwc Could u plz use English for PR descriptions?

Sorry, my English is not very good

[ruanwenjun]

@sdhzwc Could u plz use English for PR descriptions?

Sorry, my English is not very good Chinese is my native language, so describing it more smoothly

Thanks for your contribution, I changed the content to English.

[sdhzwc]

I have completed the modifications

[qingwli]

Please fix ut

Error:  Failures: 
Error:    TaskDefinitionServiceImplTest.updateTaskDefinition:201 Expected org.apache.dolphinscheduler.api.exceptions.ServiceException to be thrown, but nothing was thrown.

[codecov-commenter]

Codecov Report

Merging #14801 (a9c3b1a) into dev (2350775) will decrease coverage by 0.02%.
Report is 1 commits behind head on dev.
The diff coverage is 41.57%.

❗ Current head a9c3b1a differs from pull request most recent head e83e6c0. Consider uploading reports for the commit e83e6c0 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14801      +/-   ##
============================================
- Coverage     39.02%   39.00%   -0.02%     
- Complexity     4600     4603       +3     
============================================
  Files          1231     1231              
  Lines         43300    43362      +62     
  Branches       4792     4794       +2     
============================================
+ Hits          16896    16914      +18     
- Misses        24526    24569      +43     
- Partials       1878     1879       +1     

Files	Coverage Δ
...scheduler/plugin/task/api/model/DependentItem.java	0.00% <ø> (ø)
...er/api/service/impl/TaskDefinitionServiceImpl.java	47.44% <80.43%> (+1.42%)	⬆️
...cheduler/plugin/task/api/utils/DependentUtils.java	76.92% <0.00%> (-11.32%)	⬇️
...k/dependent/DependentAsyncTaskExecuteFunction.java	0.00% <0.00%> (ø)
...cheduler/server/master/utils/DependentExecute.java	0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[sdhzwc]

#15036

[SbloodyS]

LGTM

[davidzollo]

+1
LGTM

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information