[Feature-11195][UI] Add re-upload feature for resource files

[sketchmind]

Purpose of the pull request

Add re-upload feature for resource files
close #11195

Brief change log

[Feature-11195][UI] Add re-upload feature for resource files and udf files

• fix resource file incorrect button link
• add re-upload feature for resource files
• add re-upload feature for udf files
• modify udf edit button to rename button
• fix file resource page button-link constraint
• encapsulate file and udf resource code
• place the breadcrumbs in the upper left corner
• optimized some code
• modify relative e2e test
• fix the problcem of UDF page breadcrumbs when using the browser to go back or forward
• the create folder button on the file page uses the primary type to make the button more user-friendly

Verify this pull request

Existing tests already cover part of this pull request, such as (please describe tests).

• org.apache.dolphinscheduler.e2e.pages.resource
  • FileManagePage.java
  • UdfManagePage.java

but reuploading resource feature should add more tests.

[codecov-commenter]

Codecov Report

Merging #11203 (9cde6da) into dev (4f73fe1) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11203      +/-   ##
============================================
- Coverage     39.46%   39.44%   -0.02%     
+ Complexity     4313     4308       -5     
============================================
  Files          1083     1083              
  Lines         40756    40756              
  Branches       4670     4673       +3     
============================================
- Hits          16083    16075       -8     
- Misses        22886    22895       +9     
+ Partials       1787     1786       -1     

Impacted Files	Coverage Δ
...er/master/dispatch/host/assign/RandomSelector.java	77.77% <0.00%> (-5.56%)	⬇️
...erver/master/processor/queue/TaskEventService.java	75.00% <0.00%> (-5.36%)	⬇️
...org/apache/dolphinscheduler/remote/utils/Host.java	42.55% <0.00%> (-2.13%)	⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java	51.38% <0.00%> (-1.39%)	⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java	51.19% <0.00%> (-1.20%)	⬇️

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

[sketchmind]

The code smell reported by SonarCloud is prevalent in the front-end code of the resource, do I need to resolve this?

[ruanwenjun]

This PR only include front-end code, can it close #11195? I will remove the close action.

[sketchmind]

This PR only include front-end code, can it close #11195? I will remove the close action.

Yes, it can, the backend has been implemented

[Amy0104]

It’s better to reuse the upload logic instead of creating a new reupload. Because if the upload logic has changed, the reupload need to be changed too. There will be a bug in the feature.BTW, you can format your code by eslint.

[sketchmind]

It’s better to reuse the upload logic instead of creating a new reupload. Because if the upload logic has changed, the reupload need to be changed too. There will be a bug in the feature.BTW, you can format your code by eslint.

I thought about it again, let's imagine a very possible change, if uploading files later supports batch uploading or uploading folders, it may lead to bugs in re-uploading, and if processed separately, the function of re-uploading files uses different front-end code and different interface, it will not affect.
Of course, this can also use only one logic, and then patch the logic according to the parameters. If do this, does UDF resource upload also need to use this logic?

[Amy0104]

It’s better to reuse the upload logic instead of creating a new reupload. Because if the upload logic has changed, the reupload need to be changed too. There will be a bug in the feature.BTW, you can format your code by eslint.

I thought about it again, let's imagine a very possible change, if uploading files later supports batch uploading or uploading folders, it may lead to bugs in re-uploading, and if processed separately, the function of re-uploading files uses different front-end code and different interface, it will not affect. Of course, this can also use only one logic, and then patch the logic according to the parameters. If do this, does UDF resource upload also need to use this logic?

The similarity reaches nearly eighty percent. I think it's better to reuse and handle the differences by parameters.

[sketchmind]

It’s better to reuse the upload logic instead of creating a new reupload. Because if the upload logic has changed, the reupload need to be changed too. There will be a bug in the feature.BTW, you can format your code by eslint.

I thought about it again, let's imagine a very possible change, if uploading files later supports batch uploading or uploading folders, it may lead to bugs in re-uploading, and if processed separately, the function of re-uploading files uses different front-end code and different interface, it will not affect. Of course, this can also use only one logic, and then patch the logic according to the parameters. If do this, does UDF resource upload also need to use this logic?

The similarity reaches nearly eighty percent. I think it's better to reuse and handle the differences by parameters.

ok~

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
1 Security Hotspot
3 Code Smells

No Coverage information
0.0% Duplication

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
1 Security Hotspot
17 Code Smells

No Coverage information
0.0% Duplication

[LiJie20190102]

@sketchmind @SbloodyS
Excuse me, when will this PR be closed? We found that many scenes need resources to be uploaded again.

[Amy0104]

Excuse me, when will this PR be closed? We found that many scenes need resources to be uploaded again.

Pls resolve the conflicts.

[sketchmind]

I'd like to know too, I've resolved conflicts so many times

Sorry for that, I've already checked the front end side. it can be merged from the front end side. @SbloodyS pls check the back end side.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
1 Security Hotspot
10 Code Smells

No Coverage information
0.0% Duplication

[SbloodyS]

+1

[710961349]

When will the version with the added feature of a 're-upload' button be updated? This feature is very practical

[linxd23496]

This is a very creative feature request。When will the version with the added feature of a 're-upload' button be updated?