Przeglądaj źródła

add null check for onSwitch

kaoritokashiki 5 lat temu
rodzic
commit
d0edaeb3d4

+ 5 - 0
src/client/js/components/TopOfTableContents.jsx

@@ -15,6 +15,7 @@ import PageAccessoriesModal from './PageAccessoriesModal';
 import { withUnstatedContainers } from './UnstatedUtils';
 import { withUnstatedContainers } from './UnstatedUtils';
 
 
 const TopOfTableContents = (props) => {
 const TopOfTableContents = (props) => {
+  const { onSwitch } = props;
 
 
   const [isPageAccessoriesModalShown, setIsPageAccessoriesModalShown] = useState(false);
   const [isPageAccessoriesModalShown, setIsPageAccessoriesModalShown] = useState(false);
   const [activeTab, setActiveTab] = useState('');
   const [activeTab, setActiveTab] = useState('');
@@ -27,6 +28,9 @@ const TopOfTableContents = (props) => {
   }
   }
 
 
   function switchActiveTab(clickedTab) {
   function switchActiveTab(clickedTab) {
+    if (onSwitch == null) {
+      return;
+    }
     activeComponents.add(clickedTab);
     activeComponents.add(clickedTab);
     setActiveComponents(activeComponents);
     setActiveComponents(activeComponents);
     setActiveTab(clickedTab);
     setActiveTab(clickedTab);
@@ -86,6 +90,7 @@ const TopOfTableContentsWrapper = withUnstatedContainers(TopOfTableContents, [Pa
 
 
 TopOfTableContents.propTypes = {
 TopOfTableContents.propTypes = {
   pageContainer: PropTypes.instanceOf(PageContainer).isRequired,
   pageContainer: PropTypes.instanceOf(PageContainer).isRequired,
+  onSwitch: PropTypes.func,
 };
 };
 
 
 export default withTranslation()(TopOfTableContentsWrapper);
 export default withTranslation()(TopOfTableContentsWrapper);