fix(page/title): check if the container exists before displaying its …

…name in the title
vatesfr · Jul 13, 2016 · 539d136 · 539d136
1 parent 4c28b57
commit 539d136
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/xo-app/host/index.js b/src/xo-app/host/index.js
@@ -284,7 +284,7 @@ export default class Host extends Component {
       installPatch: this._installPatch
     }
    )
-    return <Page header={this.header()} title={`${host.name_label} (${pool.name_label})`}>
+    return <Page header={this.header()} title={`${host.name_label}${pool ? ` (${pool.name_label})` : ''}`}>
       {cloneElement(this.props.children, childProps)}
     </Page>
   }

diff --git a/src/xo-app/sr/index.js b/src/xo-app/sr/index.js
@@ -160,7 +160,7 @@ export default class Sr extends Component {
       'vdisToVmIds'
     ])
    )
-    return <Page header={this.header()} title={`${sr.name_label} (${container.name_label})`}>
+    return <Page header={this.header()} title={`${sr.name_label}${container ? ` (${container.name_label})` : ''}`}>
       {cloneElement(this.props.children, childProps)}
     </Page>
   }

diff --git a/src/xo-app/vm/index.js b/src/xo-app/vm/index.js
@@ -262,7 +262,7 @@ export default class Vm extends Component {
     ]), pick(this.state, [
       'statsOverview'
     ]))
-    return <Page header={this.header()} title={`${vm.name_label} (${container.name_label})`}>
+    return <Page header={this.header()} title={`${vm.name_label}${container ? ` (${container.name_label})` : ''}`}>
       {cloneElement(this.props.children, childProps)}
     </Page>
   }