From 636659681ea5272168ec8312b2d4dec0fc2319a0 Mon Sep 17 00:00:00 2001
From: Joris-K <joris.kosacki@vates.tech>
Date: Wed, 26 Feb 2025 17:44:23 +0100
Subject: [PATCH] review feedback

---
 .../components/host/network/HostPifsTable.vue | 31 +++++++++++--------
 .../lite/src/libs/xen-api/xen-api.types.ts    |  1 +
 2 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/@xen-orchestra/lite/src/components/host/network/HostPifsTable.vue b/@xen-orchestra/lite/src/components/host/network/HostPifsTable.vue
index 200f73fde32..b47e3500b3f 100644
--- a/@xen-orchestra/lite/src/components/host/network/HostPifsTable.vue
+++ b/@xen-orchestra/lite/src/components/host/network/HostPifsTable.vue
@@ -109,7 +109,7 @@
                 />
               </div>
               <div v-else-if="column.id === 'IP'" class="ip-addresses">
-                <span class="value text-ellipsis">{{ column.value[0] }}</span>
+                <span v-tooltip class="value text-ellipsis">{{ column.value[0] }}</span>
                 <span v-if="column.value.length > 1" class="typo-body-regular-small more-ips">
                   {{ `+${column.value.length - 1}` }}
                 </span>
@@ -174,19 +174,8 @@ const { getByOpaqueRef } = useNetworkStore().subscribe()
 const { getPifStatus } = usePifStore().subscribe()
 
 const { t } = useI18n()
-const selectedPifId = useRouteQuery('id')
-const searchQuery = ref('')
-
-// TODO change to match with network name
-const filteredPifs = computed(() => {
-  const searchTerm = searchQuery.value.trim().toLocaleLowerCase()
 
-  if (!searchTerm) {
-    return pifs
-  }
-
-  return pifs.filter(pif => Object.values(pif).some(value => String(value).toLocaleLowerCase().includes(searchTerm)))
-})
+const selectedPifId = useRouteQuery('id')
 
 const pifsUuids = computed(() => pifs.map(pif => pif.uuid))
 
@@ -209,6 +198,22 @@ const getIpConfigurationMode = (ipMode: string) => {
   }
 }
 
+const searchQuery = ref('')
+
+const filteredPifs = computed(() => {
+  const searchTerm = searchQuery.value.trim().toLocaleLowerCase()
+
+  if (!searchTerm) {
+    return pifs
+  }
+
+  return pifs.filter(pif =>
+    [...Object.values(pif), getNetworkName(pif.network)].some(value =>
+      String(value).toLocaleLowerCase().includes(searchTerm)
+    )
+  )
+})
+
 const { visibleColumns, rows } = useTable('pifs', filteredPifs, {
   rowId: record => record.uuid,
   columns: define => [
diff --git a/@xen-orchestra/lite/src/libs/xen-api/xen-api.types.ts b/@xen-orchestra/lite/src/libs/xen-api/xen-api.types.ts
index 771f124195c..c7f0dbdec89 100644
--- a/@xen-orchestra/lite/src/libs/xen-api/xen-api.types.ts
+++ b/@xen-orchestra/lite/src/libs/xen-api/xen-api.types.ts
@@ -53,6 +53,7 @@ export type RawTypeToType<RawType extends RawObjectType> = Lowercase<RawType>
 export type TypeToRawType<Type extends ObjectType> = TypeMapping[Type]
 
 type ObjectTypeToRecordMapping = {
+  bond: XenApiBond
   console: XenApiConsole
   host: XenApiHost
   host_metrics: XenApiHostMetrics