Skip to content

SOLR-16732: misc sonar fixes#1525

Merged
risdenk merged 7 commits intoapache:mainfrom
risdenk:SOLR-16732-misc-sonar-fixes
Apr 3, 2023
Merged

SOLR-16732: misc sonar fixes#1525
risdenk merged 7 commits intoapache:mainfrom
risdenk:SOLR-16732-misc-sonar-fixes

Conversation

@risdenk
Copy link
Contributor

@risdenk risdenk commented Apr 1, 2023

https://issues.apache.org/jira/browse/SOLR-16732

  • String non regex replaceAll with replace
  • indexOf -> contains
  • NBSP in log
  • unindented if statements missing {}
  • Use && instead of & for boolean comparisons
  • Replace "".equals with isEmpty

@risdenk risdenk self-assigned this Apr 1, 2023
@risdenk risdenk force-pushed the SOLR-16732-misc-sonar-fixes branch from 0bb3656 to 56e3de6 Compare April 1, 2023 15:17
@risdenk risdenk force-pushed the SOLR-16732-misc-sonar-fixes branch 2 times, most recently from d1014a4 to 0437060 Compare April 1, 2023 18:19
@risdenk risdenk force-pushed the SOLR-16732-misc-sonar-fixes branch from 0437060 to ca6a2b7 Compare April 1, 2023 19:19
@risdenk risdenk marked this pull request as ready for review April 2, 2023 14:00
@risdenk risdenk merged commit 26e81ab into apache:main Apr 3, 2023
@risdenk risdenk deleted the SOLR-16732-misc-sonar-fixes branch April 3, 2023 13:55
risdenk added a commit that referenced this pull request Apr 3, 2023
* Replace String non regex replaceAll with replace
* indexOf to contains or single character indexOf
* Remove NBSP in log message
* unindented if statements missing {}
* Use && instead of & for boolean comparisons
* Replace .equals with isEmpty
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.

2 participants