Skip to content

[Fix-3058][ui]Method catches with variables#3464

Merged
xingchun-chen merged 3 commits intoapache:1.3.2-releasefrom
break60:1.3.2-release
Aug 11, 2020
Merged

[Fix-3058][ui]Method catches with variables#3464
xingchun-chen merged 3 commits intoapache:1.3.2-releasefrom
break60:1.3.2-release

Conversation

@break60
Copy link
Copy Markdown
Contributor

@break60 break60 commented Aug 11, 2020

fix issue #3058

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (1.8.0_265) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@break60 break60 changed the title [ui]Code optimization [ui]Method catches with variables Aug 11, 2020
@xingchun-chen xingchun-chen changed the title [ui]Method catches with variables [Fix-3058][ui]Method catches with variables Aug 11, 2020
@xingchun-chen xingchun-chen merged commit e2553e3 into apache:1.3.2-release Aug 11, 2020
Comment on lines +694 to +695
let rtTargetarr = rtTargetarrArr(targetId)
let rtSoucearr = rtTargetarrArr(sourceId)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest:
rtTargetarr -> rtTargetArr
rtSoucearr -> rtSourceArr

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest:
rtTargetarr -> rtTargetArr
rtSoucearr -> rtSourceArr

ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants