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
acc5ed0210
xenadmin/XenAdmin/Actions/GUIActions
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
..
Wlb CP-36392: Change condition that always evaluates to false in WlbOptimizePoolAction 2021-10-01 11:56:34 +01:00
DeleteAllAlertsAction.cs CP-34231: Remove language with negative connotations (#2860) 2021-08-31 11:31:16 +01:00
ExportResourceReportAction.cs CP-34231: Remove language with negative connotations (#2860) 2021-08-31 11:31:16 +01:00
ExternalPluginAction.cs CP-36392: Misc fixes to flagged Bugs (#2907) 2021-12-21 15:53:46 +00:00
GeneralEditPageAction.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
MeddlingAction.cs CP-34231: Remove language with negative connotations (#2860) 2021-08-31 11:31:16 +01:00
MeddlingActionManager.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
RestoreDismissedUpdatesAction.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
SaveCustomFieldsAction.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00
SaveDataSourceStateAction.cs CA-176169: Changed copyright statements to include the comma in Citrix Systems, 2017-01-16 19:59:50 +00:00
SearchAction.cs CP-36392 pt.1: Removed unused using directives. 2021-04-07 10:04:47 +01:00