Skip to content

🎨 UI/UX: Missing null check before accessing MainWindow.unityInstallationsSource #222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
halinhtvn3a wants to merge 1 commit into from
+8 −1

fix(ui): missing null check before accessing mainwindow.unityinstalla…

444978b
Select commit
Loading
Failed to load commit list.
Closed

🎨 UI/UX: Missing null check before accessing MainWindow.unityInstallationsSource #222

fix(ui): missing null check before accessing mainwindow.unityinstalla…
444978b
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 27, 2026 in 11s

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 57 in UnityLauncherPro/UpgradeWindow.xaml.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'MainWindow.unityInstallationsSource' is null on at least one execution path. 

See more on https://sonarcloud.io/project/issues?id=unitycoder_UnityLauncherPro&issues=AZ0rRahiH2jBfq5nbcnk&open=AZ0rRahiH2jBfq5nbcnk&pullRequest=222

Check failure on line 16 in UnityLauncherPro/UpgradeWindow.xaml.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this constructor to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=unitycoder_UnityLauncherPro&issues=AZ0rRahiH2jBfq5nbcnl&open=AZ0rRahiH2jBfq5nbcnl&pullRequest=222
View more details on SonarQubeCloud