fix(xo-web/pool/advanced): show backup/migration network even when deleted

CHANGELOG.unreleased.md

@@ -24,6 +24,7 @@
 - [Jobs] Reset parameters when editing method to avoid invalid parameters on execution [Forum#69299](https://xcp-ng.org/forum/post/69299)
 - [Metadata Backup] Fix `ENOENT` error when restoring an _XO Config_ backup [Forum#68999](https://xcp-ng.org/forum/post/68999)
 - [REST API] Fix `/backup/log/<id>` which was broken by the `/backups` to `/backup` renaming [Forum#69426](https://xcp-ng.org/forum/post/69426)
+- [Pool/Advanced] Show pool backup/migration network even if they no longer exist (PR [#7303](https://github.com/vatesfr/xen-orchestra/pull/7303))
 
 ### Packages to release
 

packages/xo-web/src/common/intl/messages.js

@@ -900,6 +900,7 @@ const messages = {
   // ----- Pool advanced tab -----
   backupNetwork: 'Backup network',
   crashDumpSr: 'Crash dump SR',
+  networkIdMissing: '{networkID} no longer exists, please select a new one',
   poolEditAll: 'Edit all',
   poolHaStatus: 'High Availability',
   poolHaEnabled: 'Enabled',

packages/xo-web/src/xo-app/pool/tab-advanced.js

@@ -4,6 +4,7 @@ import _, { messages } from 'intl'
 import ActionButton from 'action-button'
 import ActionRowButton from 'action-row-button'
 import Component from 'base-component'
+import Copiable from 'copiable'
 import Icon from 'icon'
 import renderXoItem, { Network, Sr } from 'render-xo-item'
 import SelectFiles from 'select-files'
@@ -440,7 +441,17 @@ export default class TabAdvanced extends Component {
                         value={migrationNetwork}
                         xoType='network'
                       >
-                        {migrationNetwork !== undefined ? <Network id={migrationNetwork.id} /> : _('noValue')}
+                        {pool.otherConfig['xo:migrationNetwork'] === undefined ? (
+                          _('noValue')
+                        ) : migrationNetwork !== undefined ? (
+                          <Network id={migrationNetwork.id} />
+                        ) : (
+                          <span className='text-danger'>
+                            {_('networkIdMissing', {
+                              networkID: <Copiable>{pool.otherConfig['xo:migrationNetwork']}</Copiable>,
+                            })}
+                          </span>
+                        )}
                       </XoSelect>{' '}
                       {migrationNetwork !== undefined && (
                         <a role='button' onClick={this._removeMigrationNetwork}>
@@ -458,7 +469,17 @@ export default class TabAdvanced extends Component {
                         value={backupNetwork}
                         xoType='network'
                       >
-                        {backupNetwork !== undefined ? <Network id={backupNetwork.id} /> : _('noValue')}
+                        {pool.otherConfig['xo:backupNetwork'] === undefined ? (
+                          _('noValue')
+                        ) : backupNetwork !== undefined ? (
+                          <Network id={backupNetwork.id} />
+                        ) : (
+                          <span className='text-danger'>
+                            {_('networkIdMissing', {
+                              networkID: <Copiable>{pool.otherConfig['xo:backupNetwork']}</Copiable>,
+                            })}
+                          </span>
+                        )}
                       </XoSelect>{' '}
                       {backupNetwork !== undefined && (
                         <a role='button' onClick={this._removeBackupNetwork}>