From 1367f96dcd4456e6f8770f050653495a680f79b4 Mon Sep 17 00:00:00 2001 From: dannyrb Date: Sun, 16 Jun 2019 23:44:01 -0400 Subject: [PATCH] Remove code that depends on global "buttons" in Vtk Viewport --- .../src/ConnectedVTKViewport.js | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/extensions/ohif-vtk-extension/src/ConnectedVTKViewport.js b/extensions/ohif-vtk-extension/src/ConnectedVTKViewport.js index 28689a591..322100222 100644 --- a/extensions/ohif-vtk-extension/src/ConnectedVTKViewport.js +++ b/extensions/ohif-vtk-extension/src/ConnectedVTKViewport.js @@ -1,15 +1,10 @@ -import { connect } from 'react-redux'; -import { View2D } from 'react-vtkjs-viewport'; import OHIF from 'ohif-core'; +import { View2D } from 'react-vtkjs-viewport'; +import { connect } from 'react-redux'; -const { - setViewportActive, - setViewportSpecificData, - clearViewportSpecificData -} = OHIF.redux.actions; +const { setViewportActive, setViewportSpecificData } = OHIF.redux.actions; const mapStateToProps = (state, ownProps) => { - const activeButton = state.tools.buttons.find(tool => tool.active === true); let dataFromStore; if (state.extensions && state.extensions.vtk) { @@ -19,9 +14,6 @@ const mapStateToProps = (state, ownProps) => { // If this is the active viewport, enable prefetching. const { viewportIndex } = ownProps; const isActive = viewportIndex === state.viewports.activeViewportIndex; - const viewportSpecificData = - state.viewports.viewportSpecificData[viewportIndex] || {}; - const viewportLayout = state.viewports.layout.viewports[viewportIndex]; const pluginDetails = viewportLayout.vtk || {}; @@ -29,7 +21,8 @@ const mapStateToProps = (state, ownProps) => { layout: state.viewports.layout, isActive, ...pluginDetails, - activeTool: activeButton && activeButton.command, + // Hopefully this doesn't break anything under the hood for this one + // activeTool: activeButton && activeButton.command, ...dataFromStore, enableStackPrefetch: isActive }; @@ -51,7 +44,6 @@ const mapDispatchToProps = (dispatch, ownProps) => { const mergeProps = (propsFromState, propsFromDispatch, ownProps) => { const { afterCreation } = propsFromState; - const { setViewportSpecificData } = propsFromDispatch; const props = { ...propsFromState,