feat(xo-web/backup-reports): hide merge task when no merge (#5263)

CHANGELOG.unreleased.md

@@ -8,6 +8,7 @@
 > Users must be able to say: “Nice enhancement, I'm eager to test it”
 
 - [Host/Advanced] Add the field `IOMMU` if it is defined (PR [#5294](https://github.com/vatesfr/xen-orchestra/pull/5294))
+- [Backup logs/report] Hide merge task when no merge is done (PR [#5263](https://github.com/vatesfr/xen-orchestra/pull/5263))
 
 ### Bug fixes
 
@@ -37,5 +38,6 @@
 >
 > In case of conflict, the highest (lowest in previous list) `$version` wins.
 
+- xo-server-backup-reports patch
 - xo-server minor
 - xo-web minor

packages/xo-server-backup-reports/src/index.js

@@ -513,6 +513,8 @@ class BackupReportsXoPlugin {
             } else {
               globalTransferSize += size
             }
+          } else if (operationLog.status === 'success') {
+            return
           }
 
           const operationText = [

packages/xo-web/src/xo-app/logs/backup-ng/log-alert-body.js

@@ -136,8 +136,8 @@ const Warnings = ({ warnings }) =>
     </div>
   ) : null
 
-const VmTask = ({ children, restartVmJob, task }) => (
+const VmTask = ({ children, className, restartVmJob, task }) => (
-  <div>
+  <li className={className}>
     <Vm id={task.data.id} link newTab /> <TaskStateInfos status={task.status} />{' '}
     {restartVmJob !== undefined && hasTaskFailed(task) && (
       <ButtonGroup>
@@ -190,11 +190,11 @@ const VmTask = ({ children, restartVmJob, task }) => (
     )}
     {task.isFull !== undefined &&
       _.keyValue(_('exportType'), task.isFull ? 'full' : 'delta')}
-  </div>
+  </li>
 )
 
-const PoolTask = ({ children, task }) => (
+const PoolTask = ({ children, className, task }) => (
-  <div>
+  <li className={className}>
     <Pool id={task.data.id} link newTab />{' '}
     <TaskStateInfos status={task.status} />
     <Warnings warnings={task.warnings} />
@@ -203,11 +203,11 @@ const PoolTask = ({ children, task }) => (
     <TaskEnd task={task} />
     <TaskDuration task={task} />
     <TaskError task={task} />
-  </div>
+  </li>
 )
 
-const XoTask = ({ children, task }) => (
+const XoTask = ({ children, className, task }) => (
-  <div>
+  <li className={className}>
     <Icon icon='menu-xoa' /> XO <TaskStateInfos status={task.status} />
     <Warnings warnings={task.warnings} />
     {children}
@@ -215,22 +215,22 @@ const XoTask = ({ children, task }) => (
     <TaskEnd task={task} />
     <TaskDuration task={task} />
     <TaskError task={task} />
-  </div>
+  </li>
 )
 
-const SnapshotTask = ({ task }) => (
+const SnapshotTask = ({ className, task }) => (
-  <div>
+  <li className={className}>
     <Icon icon='task' /> {_('snapshotVmLabel')}{' '}
     <TaskStateInfos status={task.status} />
     <Warnings warnings={task.warnings} />
     <TaskStart task={task} />
     <TaskEnd task={task} />
     <TaskError task={task} />
-  </div>
+  </li>
 )
 
-const RemoteTask = ({ children, task }) => (
+const RemoteTask = ({ children, className, task }) => (
-  <div>
+  <li className={className}>
     <Remote id={task.data.id} link newTab />{' '}
     <TaskStateInfos status={task.status} />
     <Warnings warnings={task.warnings} />
@@ -239,11 +239,11 @@ const RemoteTask = ({ children, task }) => (
     <TaskEnd task={task} />
     <TaskDuration task={task} />
     <TaskError task={task} />
-  </div>
+  </li>
 )
 
-const SrTask = ({ children, task }) => (
+const SrTask = ({ children, className, task }) => (
-  <div>
+  <li className={className}>
     <Sr id={task.data.id} link newTab /> <TaskStateInfos status={task.status} />
     <Warnings warnings={task.warnings} />
     {children}
@@ -251,13 +251,17 @@ const SrTask = ({ children, task }) => (
     <TaskEnd task={task} />
     <TaskDuration task={task} />
     <TaskError task={task} />
-  </div>
+  </li>
 )
 
-const TransferMergeTask = ({ task }) => {
+const TransferMergeTask = ({ className, task }) => {
-  const size = get(() => task.result.size)
+  const size = defined(() => task.result.size, 0)
+  if (task.status === 'success' && size === 0) {
+    return null
+  }
+
   return (
-    <div>
+    <li className={className}>
       <Icon icon='task' /> {task.message}{' '}
       <TaskStateInfos status={task.status} />
       <Warnings warnings={task.warnings} />
@@ -275,7 +279,7 @@ const TransferMergeTask = ({ task }) => {
           )}
         </div>
       )}
-    </div>
+    </li>
   )
 }
 
@@ -293,7 +297,7 @@ const COMPONENT_BY_MESSAGE = {
   transfer: TransferMergeTask,
 }
 
-const TaskLi = ({ className, task, ...props }) => {
+const TaskLi = ({ task, ...props }) => {
   let Component
   if (
     (Component = defined(
@@ -305,11 +309,7 @@ const TaskLi = ({ className, task, ...props }) => {
   ) {
     return null
   }
-  return (
+  return <Component task={task} {...props} />
-    <li className={className}>
-      <Component task={task} {...props} />
-    </li>
-  )
 }
 
 const SEARCH_BAR_FILTERS = { name: 'name:' }