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
7144528f6b
xenadmin/XenModel/ServerDBs
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
..
FakeAPI CP-36392: Misc fixes to flagged Bugs (#2907) 2021-12-21 15:53:46 +00:00
Db.cs CA-339526: Ignore classes that are present in the database but not exposed in the SDK. 2020-05-29 14:53:54 +01:00
Db.Dictionaries.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
DbProxy.cs CP-34231: Remove language with negative connotations (#2860) 2021-08-31 11:31:16 +01:00
DbProxyInvokingEventArgs.cs CA-176169: Changed copyright statements to include the comma in Citrix Systems, 2017-01-16 19:59:50 +00:00
HighLoadEventGenerator.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
Parser.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
ProxyMethodNameParser.cs CA-176169: Changed copyright statements to include the comma in Citrix Systems, 2017-01-16 19:59:50 +00:00
SimulatorWebProxy.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
SimulatorWebStream.cs CA-176169: Changed copyright statements to include the comma in Citrix Systems, 2017-01-16 19:59:50 +00:00
StatusReportXmlDocReader.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
TypeCache.cs CA-339526: Ignore classes that are present in the database but not exposed in the SDK. 2020-05-29 14:53:54 +01:00