Kiu/xenadmin
1
0
Fork 0
mirror of https://github.com/xcp-ng/xenadmin.git synced 2024-11-25 06:16:37 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity
1225e98225
xenadmin/XenModel/WLB
History
Danilo Del Busso 16d7f74158
CP-36392: Misc fixes to flagged Bugs (#2907) 
* CP-36392: Add several `null` checks

All were flagged by SonarQube

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Specify `namespace` for `SettingsUpdate`

Added to `XenAdmin`

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Collapse useless condition check

Also use pattern matching for type cast

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Specify `namespace` for `ProduceConsumerQueue`

Add to `XenAdmin.Actions`

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Rename field in `BugToolPageSelectCapabilties` to avoid name clash

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Add explicit `null` check

While the `null` check in `GraphDetailsDialog.cs` is not strictly necessary, it has been added to fix a SonarQube issue

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Remove unused field in `WlbPoolConfiguration.cs`

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Replace useless self-assignment with ad-hoc method

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Simplify dictionary key assignment in `WlbScheduledTask.cs`

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Fix minor formatting issues

Add newline at end of file, and reformat content of if statement

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Tidy up latest flagged bugs changes

- Fix whitespace/braces in a few affected files
- Add missing null check in `ValueMaps.cs`
- Check for args length in `RbacCollectorProxy.cs`
- Fix logic in `MainWindow.cs` to revert to previous behaviour

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>

* CP-36392: Tidy up setter in `ChangeableList.cs`

Also update whitespace inconsitencies in `PerfmonAlertEditPage.cs` and `VMHAEditPage.cs`

Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>
2021-12-21 15:53:46 +00:00
..
RetrieveWlbConfigurationAction.cs Fixed workflow when loading the WorkloadReports dialog (includes CA-339666). 2021-05-27 15:39:52 +01:00
WlbConfigurationBase.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
WlbHostConfiguration.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
WlbHostEvacuationRecommendation.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
WlbOptimizationRecommendation.cs CP-36392: Misc fixes to flagged Bugs (#2907) 2021-12-21 15:53:46 +00:00
WlbPoolConfiguration.cs CP-36392: Misc fixes to flagged Bugs (#2907) 2021-12-21 15:53:46 +00:00
WlbReportAction.cs CP-34231: Remove language with negative connotations (#2860) 2021-08-31 11:31:16 +01:00
WlbReportInfo.cs Tidied up WLB report files: 2021-05-26 17:54:53 +01:00
WlbReportSubscription.cs CP-36392: Rename key constants in WlbReportSubscription.cs to avoid field clash 2021-10-05 11:56:15 +01:00
WlbScheduledTask.cs CP-36392: Misc fixes to flagged Bugs (#2907) 2021-12-21 15:53:46 +00:00
WlbServerState.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00