[JBIDE-24025] returning null when there's no application (was: assert…

[adietish]

…ion error)

Pull Request Checklist

General

• Is this a blocking issue or new feature? If yes, QE needs to +1 this PR

Code

• Are method-/class-/variable-names meaningful?
• Are methods concise, not too long?
• Are catch blocks catching precise Exceptions only (no catch all)?

Testing

• Are there unit-tests?
• Are there integration tests (or at least a jira to tackle these)?
• Is the non-happy path working, too?
• Are other parts that use the same component still working fine?

Function

• Does it work?

[rhdevelopers-ci]

SonarQube analysis reported 5 issues

1. ExpressServerUtils.java#L557: Close this "Repository".
2. ExpressServerUtils.java#L234: NullPointerException will be thrown when invoking method getUsername().
3. ExpressServerUtils.java#L239: NullPointerException might be thrown as 'attributes' is nullable here
4. ExpressServerUtils.java#L241: NullPointerException might be thrown as 'attributes' is nullable here
5. ExpressServerUtils.java#L579: Extract this nested ternary operation into an independent statement.