Cleanup

searchlib/components/SearchApp/BasicSearchApp.jsx

@@ -11,7 +11,6 @@ import useDeepCompareEffect from 'use-deep-compare-effect';
 import { AppConfigContext, SearchContext } from '@eeacms/search/lib/hocs';
 import { bindOnAutocomplete, bindOnSearch } from '@eeacms/search/lib/request';
 import useSearchApp from './useSearchApp';
-// import useWhyDidYouUpdate from '@eeacms/search/lib/hocs/useWhyDidYouUpdate';
 
 function SearchWrappers(SearchViewComponent) {
   function Wrapper(props) {
@@ -80,7 +79,6 @@ export default function BasicSearchApp(props) {
   });
 
   useDeepCompareEffect(() => {
-    console.log('reset stable context');
     setStableContext({ appConfig, registry });
   }, [appConfig, registry]);
 
@@ -90,17 +88,6 @@ export default function BasicSearchApp(props) {
     );
   }, [mapContextToProps, searchViewComponent, children]);
 
-  // useWhyDidYouUpdate('BasicSearchapp', {
-  //   mapContextToProps,
-  //   searchViewComponent,
-  //   appConfig,
-  //   registry,
-  //   driverInstance,
-  //   elasticConfig,
-  //   facetOptions,
-  //   stableContext,
-  // });
-
   return driverInstance ? (
     <SearchProvider config={elasticConfig} driver={driverInstance}>
       <WrappedSearchView

searchlib/components/SearchApp/FacetApp.jsx

@@ -47,7 +47,6 @@ function BoostrapFacetView(props) {
     const activeFilter = filters?.find((filter) => filter.field === field);
     if (value && !activeFilter) {
       const sortedFilters = [...filters, value].sort(sorter);
-      // console.log('initializing', sortedFilters);
       driver._setState({ filters: sortedFilters });
     }
   }, [value, filters, field, setFilter, driver]); // searchContext
@@ -61,16 +60,10 @@ function BoostrapFacetView(props) {
         const { filters } = driver.state;
         const activeValue = filters.find((f) => f.field === field);
         if (!activeValue) {
-          // console.log('onChange null', field);
           onChange(null);
           return;
         }
         if (!isEqual(activeValue, value)) {
-          // console.log('onChange', {
-          //   activeValue,
-          //   value,
-          //   filters: driver.state.filters,
-          // });
           onChange(activeValue);
         }
       }
@@ -81,7 +74,6 @@ function BoostrapFacetView(props) {
     }
 
     return () => {
-      // console.log('unsubscribe', field);
       driver.events.plugins = driver.events.plugins.filter(
         (plug) => plug.id !== plugId,
       );

searchlib/components/SearchApp/SearchResultsApp.jsx

@@ -6,13 +6,6 @@ import BasicSearchApp from './BasicSearchApp';
 function BootstrapSearchResultsView(props) {
   const { appConfig, registry } = props;
 
-  React.useEffect(
-    () => () => {
-      console.log('unmount BootstrapSearchResultsView');
-    },
-    [],
-  );
-
   const searchContext = useSearchContext();
   const { results = [] } = searchContext;
   const { resultViews } = appConfig;

src/SearchBlock/SearchBlockView.jsx

@@ -4,7 +4,6 @@ import { SearchBlockSchema } from './schema';
 import { applySchemaEnhancer, withBlockExtensions } from '@plone/volto/helpers';
 import { applyBlockSettings } from './utils';
 import { useDebouncedStableData } from './hocs';
-import useWhyDidYouUpdate from '@eeacms/search/lib/hocs/useWhyDidYouUpdate';
 
 import './less/styles.less';
 
@@ -46,14 +45,9 @@ function SearchBlockView(props) {
 
   const Variation = variation.view;
 
-  useWhyDidYouUpdate('Variation', { Variation });
-
-  // TODO: why the double rendering?
-
   return (
     <div>
       {mode !== 'view' && 'EEA Semantic Search block'}
-      {JSON.stringify(data)}
       <Variation
         registry={registry}
         appName={appName}

src/SearchBlock/templates/SearchResultsView.jsx

@@ -12,13 +12,6 @@ const overlayStyle = {
 export default function SearchResultsView(props) {
   const { appName, mode } = props;
 
-  React.useEffect(
-    () => () => {
-      console.log('unmount SRV');
-    },
-    [],
-  );
-
   return (
     <BodyClass className={`${appName}-view searchlib-page`}>
       <div className="searchlib-block">
@@ -72,7 +65,6 @@ const setFacetWidgetProps = (appConfig, registry, appName) => {
     schema.properties.value.appName = appName;
     schema.properties.value.registry = registry;
 
-    // console.log('setfacet', { schema, appConfig, data });
     return schema;
   };
 };