[API] [bug] dependent task need regenerate up-stream code when import

[xiaolailong]

Purpose of the pull request

When import a dependent task from json file, the up-stream code of process and task is not re-generated, which cause the import error. (The import operation return success, but when you open the detail of dependend task, it will show many error message like: "can not found process definition")

fix #15493

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

[SbloodyS]

Please link this pr to your issue. @xiaolailong

[xiaolailong]

Please link this pr to your issue. @xiaolailong

Thank you for reminding！There is no relevant issue, should I create one?

[xiaolailong]

Please link this pr to your issue. @xiaolailong

issue is #15493

[fuchanghai]

@xiaolailong plz run 'mvn spotless:apply' to fix these violations.

[fuchanghai]

hi @SbloodyS plz check #14956

Importing subprocess, condition, switch, and dependcy is actually a type of problem. do we need to split it into multiple PRs to do it? . cc @EricGao888 @chenshuai1995 @ruanwenjun

[SbloodyS]

Importing subprocess, condition, switch, and dependcy is actually a type of problem.

Thanks. I overlooked some potential issues. I've replyed in #14956 (comment).

[xiaolailong]

@xiaolailong plz run 'mvn spotless:apply' to fix these violations.

Thank you for guideline! already done.

[codecov-commenter]

Codecov Report

Attention: 40 lines in your changes are missing coverage. Please review.

Comparison is base (308e4fb) 38.01% compared to head (279007d) 37.99%.
Report is 1 commits behind head on dev.

❗ Current head 279007d differs from pull request most recent head 2bf3b23. Consider uploading reports for the commit 2bf3b23 to get more accurate results

Files	Patch %	Lines
...api/service/impl/ProcessDefinitionServiceImpl.java	0.00%	40 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                dev   #15492      +/-   ##
============================================
- Coverage     38.01%   37.99%   -0.03%     
  Complexity     4692     4692              
============================================
  Files          1304     1304              
  Lines         44814    44843      +29     
  Branches       4804     4812       +8     
============================================
  Hits          17038    17038              
- Misses        25924    25953      +29     
  Partials       1852     1852              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rickchengx]

@xiaolailong Thanks for your contributions. Maybe we should also regenerate the task code when import other task types, such as SUB_PROCESS and CONDITIONS, what do u think? Can this replacement be done in a uniform way?

[sonarqubecloud]

Quality Gate failed

Failed conditions

0.0% Coverage on New Code (required ≥ 60%)

See analysis details on SonarCloud

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

[github-actions]

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.