Skip to content

Commit c24db28

Browse files
Merge pull request #556 from Kitware/get-fix
added safer get(taskflow, '...cluster._id) to inline conditionals
2 parents e110b8b + 42de0e1 commit c24db28

4 files changed

Lines changed: 4 additions & 4 deletions

File tree

  • src/workflows
    • nwchem/common/steps/Visualization/View
    • pyfr/components/steps
    • visualizer/components/steps/Visualization/View

src/workflows/nwchem/common/steps/Visualization/View/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ const VisualizationView = React.createClass({
108108
return (
109109
<div>
110110
<JobMonitor taskflowId={ taskflowId }
111-
clusterId={taskflow.flow.meta ? taskflow.flow.meta.cluster._id : null}
111+
clusterId={get(taskflow, 'flow.meta.cluster._id') ? taskflow.flow.meta.cluster._id : null}
112112
/>
113113
<FileListing title="Input Files" folderId={simulation.metadata.inputFolder._id}
114114
actionsDisbled={fileActionsDisabled} />

src/workflows/pyfr/components/steps/Simulation/View/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ const SimualtionView = React.createClass({
9595
return (
9696
<div>
9797
<JobMonitor taskflowId={ taskflowId }
98-
clusterId={taskflow.flow.meta ? taskflow.flow.meta.cluster._id : null}
98+
clusterId={get(taskflow, 'flow.meta.cluster._id') ? taskflow.flow.meta.cluster._id : null}
9999
/>
100100
<FileListing title="Input Files" folderId={simulation.metadata.inputFolder._id} actionsDisabled={fileActionsDisabled} />
101101
<FileListing title="Output Files" folderId={simulation.metadata.outputFolder._id} actionsDisabled={fileActionsDisabled} />

src/workflows/pyfr/components/steps/Visualization/View/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ const VisualizationView = React.createClass({
106106
return (
107107
<div>
108108
<JobMonitor taskflowId={ taskflowId }
109-
clusterId={taskflow.flow.meta ? taskflow.flow.meta.cluster._id : null}
109+
clusterId={get(taskflow, 'flow.meta.cluster._id') ? taskflow.flow.meta.cluster._id : null}
110110
/>
111111
<FileListing title="Input Files" folderId={simulation.metadata.outputFolder._id}
112112
actionsDisbled={fileActionsDisabled} />

src/workflows/visualizer/components/steps/Visualization/View/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ const visualizationView = React.createClass({
100100
return (
101101
<div>
102102
<JobMonitor taskflowId={ taskflowId }
103-
clusterId={taskflow.flow.meta ? taskflow.flow.meta.cluster._id : null}
103+
clusterId={get(taskflow, 'flow.meta.cluster._id') ? taskflow.flow.meta.cluster._id : null}
104104
/>
105105
<FileListing title="Input Files" folderId={simulation.metadata.inputFolder._id} actionsDisabled={fileActionsDisabled} />
106106
<FileListing title="Output Files" folderId={simulation.steps.Visualization.folderId} actionsDisabled={fileActionsDisabled} />

0 commit comments

Comments
 (0)