Skip to content

NH-2558 Fixed various bugs from Local Sonarqube scan

Simon Li requested to merge NH-2558 into master

Der er blevet lavet nogle ændringer i koden som skulle fjerne nogle af de bugs som der bliver fundet i Sonarqube. Disse ændringer retter ikke alle bugs og vulnerabilities da dette er kørt lokalt på min PC som ikke finder lige så mange som ERST Sonarqube. Dette skulle dog gerne rette 10/12 bugs som ERST Sonarqube finder.
Jeg har ikke rettet på vulnerabilities da disse ikke bliver fundet lokalt og der kan derfor ikke testes for fejlrettelser inden de kommer ind i master.

Disse kodeændringer burde rette følgende bugs i Sonarqube:

  • Either remove this useless object instantiation of class 'Program' or use it.
  • Change this condition so that it does not always evaluate to 'false'; some subsequent code is never executed.
  • Remove this conditional structure or edit its code blocks so that they're not all the same.
  • This class overrides 'GetHashCode' and should therefore also override 'Equals'.
  • Pair this "BeginInvoke" with an "EndInvoke".
  • 'pServiceHost' is null on at least one execution path.
  • Refactor 'GetHashCode' to not reference mutable fields.
  • 'compiledXslt' is null on at least one execution path.
  • Either remove this useless object instantiation of class 'Program' or use it.

Merge request reports