Modify ↓

#25027 new task

[Patch] Fix for Sonar issues: False-positive report (java:S2259) in TagChecker

Reported by: DanProgs <Abenteurer.2901@…> Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: Cc:

Description

Utils.isEmpty(harmonizedValue) has been replaced by an explicit check for null and emptiness, allowing static code analysis to verify that harmonizedValue is not null. Furthermore, control structures have been updated to include curly braces, and fields and variables have been declared as final.
See: https://josm.openstreetmap.de/sonar/project/issues?s=TYPE_SEVERITY&issueStatuses=CONFIRMED%2COPEN&types=BUG&id=josm&open=AXvWAEXUyP4JTgY8QMT7
See: https://josm.openstreetmap.de/sonar/project/issues?s=TYPE_SEVERITY&issueStatuses=CONFIRMED%2COPEN&types=BUG&id=josm&open=AXvWAEXUyP4JTgY8QMT8

Attachments (1)

#25027-Fix-Sonar-issues-in-TagChecker.patch​ (22.9 KB ) - added by DanProgs <Abenteurer.2901@…> 23 minutes ago.

Download all attachments as: .zip

Change History (1)

by DanProgs <Abenteurer.2901@…>, 23 minutes ago

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to DanProgs <Abenteurer.2901@…>.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.