components: replace defaultProps with default params

* Replaces `defaultProps` declarations with ES6 default parameter
  syntax across all function components. React 18 deprecates
  defaultProps on function components, producing console errors.

* Fixes currentResultsState propType from array to object.

* Fixes index propType from string to number.

Author: mirekys

src/demos/cern-videos-namespaced/Results.js

@@ -70,9 +70,7 @@ export class Results extends Component {
 }
 
 Results.propTypes = {
-  currentResultsState: PropTypes.array.isRequired,
+  currentResultsState: PropTypes.object.isRequired,
   sortValues: PropTypes.array.isRequired,
   resultsPerPageValues: PropTypes.array.isRequired,
 };
-
-Results.defaultProps = {};

src/demos/cern-videos/App.js

@@ -98,7 +98,7 @@ const CERNVideosResultsListItem = ({ result, index }) => {
 
 CERNVideosResultsListItem.propTypes = {
   result: PropTypes.object.isRequired,
-  index: PropTypes.string.isRequired,
+  index: PropTypes.number,
 };
 
 const CERNVideosResultsGridItem = ({ result, index }) => {
@@ -119,7 +119,7 @@ const CERNVideosResultsGridItem = ({ result, index }) => {
 
 CERNVideosResultsGridItem.propTypes = {
   result: PropTypes.object.isRequired,
-  index: PropTypes.string.isRequired,
+  index: PropTypes.number,
 };
 
 const overriddenComponents = {

src/demos/cern-videos/Results.js

@@ -70,9 +70,7 @@ export class Results extends Component {
 }
 
 Results.propTypes = {
-  currentResultsState: PropTypes.array.isRequired,
+  currentResultsState: PropTypes.object.isRequired,
   sortValues: PropTypes.array.isRequired,
   resultsPerPageValues: PropTypes.array.isRequired,
 };
-
-Results.defaultProps = {};

src/demos/opensearch/App.js

@@ -9,6 +9,7 @@
 import _truncate from "lodash/truncate";
 import PropTypes from "prop-types";
 import { Component } from "react";
+import { connect } from "react-redux";
 import { OverridableContext } from "react-overridable";
 import {
   Button,
@@ -21,6 +22,7 @@ import {
   Menu,
 } from "semantic-ui-react";
 import { OSSearchApi } from "../../lib/api/contrib/opensearch";
+import { updateQueryFilters } from "../../lib/state/actions";
 import {
   BucketAggregation,
   RangeFacet,
@@ -79,9 +81,8 @@ const customAggValueCmp = (
 
 class Tags extends Component {
   onClick = (event, value) => {
-    window.history.push({
-      search: `${window.location.search}&f=tags_agg:${value}`,
-    });
+    const { updateQueryFilters } = this.props;
+    updateQueryFilters(["tags_agg", value]);
     event.preventDefault();
   };
 
@@ -102,8 +103,13 @@ class Tags extends Component {
 
 Tags.propTypes = {
   tags: PropTypes.array.isRequired,
+  updateQueryFilters: PropTypes.func.isRequired,
 };
 
+const TagsComponent = connect(null, (dispatch) => ({
+  updateQueryFilters: (filter) => dispatch(updateQueryFilters(filter)),
+}))(Tags);
+
 const OpenSearchResultsListItem = ({ result, index }) => {
   return (
     <Item key={index} href="#">
@@ -119,7 +125,7 @@ const OpenSearchResultsListItem = ({ result, index }) => {
           {_truncate(result.about, { length: 200 })}
         </Item.Description>
         <Item.Extra>
-          <Tags tags={result.tags} />
+          <TagsComponent tags={result.tags} />
         </Item.Extra>
       </Item.Content>
     </Item>
@@ -128,7 +134,7 @@ const OpenSearchResultsListItem = ({ result, index }) => {
 
 OpenSearchResultsListItem.propTypes = {
   result: PropTypes.object.isRequired,
-  index: PropTypes.string.isRequired,
+  index: PropTypes.number,
 };
 
 const OpenSearchResultsGridItem = ({ result, index }) => {
@@ -144,15 +150,15 @@ const OpenSearchResultsGridItem = ({ result, index }) => {
         </Card.Description>
       </Card.Content>
       <Card.Content extra>
-        <Tags tags={result.tags} />
+        <TagsComponent tags={result.tags} />
       </Card.Content>
     </Card>
   );
 };
 
 OpenSearchResultsGridItem.propTypes = {
   result: PropTypes.object.isRequired,
-  index: PropTypes.string.isRequired,
+  index: PropTypes.number,
 };
 
 const overriddenComponents = {

src/demos/opensearch/Results.js

@@ -46,7 +46,5 @@ export function Results(props) {
 }
 
 Results.propTypes = {
-  currentResultsState: PropTypes.array.isRequired,
+  currentResultsState: PropTypes.object.isRequired,
 };
-
-Results.defaultProps = {};

src/demos/zenodo/App.js

@@ -112,7 +112,7 @@ export const ZenodoResultsListItem = ({ result, index }) => {
 
 ZenodoResultsListItem.propTypes = {
   result: PropTypes.object.isRequired,
-  index: PropTypes.string.isRequired,
+  index: PropTypes.number,
 };
 
 export const ZenodoResultsGridItem = ({ result, index }) => {
@@ -132,7 +132,7 @@ export const ZenodoResultsGridItem = ({ result, index }) => {
 
 ZenodoResultsGridItem.propTypes = {
   result: PropTypes.object.isRequired,
-  index: PropTypes.string.isRequired,
+  index: PropTypes.number,
 };
 
 const overriddenComponents = {

src/demos/zenodo/Results.js

@@ -70,9 +70,7 @@ export class Results extends Component {
 }
 
 Results.propTypes = {
-  currentResultsState: PropTypes.array.isRequired,
+  currentResultsState: PropTypes.object.isRequired,
   sortValues: PropTypes.array.isRequired,
   resultsPerPageValues: PropTypes.array.isRequired,
 };
-
-Results.defaultProps = {};

src/lib/components/ActiveFilters/ActiveFilters.js

@@ -60,7 +60,12 @@ ActiveFilters.defaultProps = {
   overridableId: "",
 };
 
-const Element = ({ overridableId, filters, removeActiveFilter, getLabel }) => {
+const Element = ({
+  overridableId = "",
+  filters,
+  removeActiveFilter,
+  getLabel,
+}) => {
   const { buildUID } = useContext(AppContext);
 
   return (
@@ -97,8 +102,4 @@ Element.propTypes = {
   overridableId: PropTypes.string,
 };
 
-Element.defaultProps = {
-  overridableId: "",
-};
-
 export default Overridable.component("ActiveFilters", ActiveFilters);

src/lib/components/BucketAggregation/BucketAggregation.js

@@ -106,10 +106,10 @@ BucketAggregation.defaultProps = {
 };
 
 const Element = ({
-  overridableId,
+  overridableId = "",
   agg,
   title,
-  containerCmp,
+  containerCmp = null,
   updateQueryFilters,
 }) => {
   const { buildUID } = useContext(AppContext);
@@ -142,9 +142,4 @@ Element.propTypes = {
   updateQueryFilters: PropTypes.func.isRequired,
 };
 
-Element.defaultProps = {
-  containerCmp: null,
-  overridableId: "",
-};
-
 export default Overridable.component("BucketAggregation", BucketAggregation);

src/lib/components/BucketAggregation/BucketAggregationValues.js

@@ -113,7 +113,7 @@ BucketAggregationValues.defaultProps = {
   overridableId: "",
 };
 
-const ContainerElement = ({ valuesCmp, overridableId }) => {
+const ContainerElement = ({ valuesCmp, overridableId = "" }) => {
   const { buildUID } = useContext(AppContext);
 
   return (
@@ -131,16 +131,12 @@ ContainerElement.propTypes = {
   overridableId: PropTypes.string,
 };
 
-ContainerElement.defaultProps = {
-  overridableId: "",
-};
-
 const ValueElement = ({
   bucket,
   isSelected,
   onFilterClicked,
   getChildAggCmps,
-  overridableId,
+  overridableId = "",
   keyField,
 }) => {
   const { buildUID, nextComponentIndex } = useContext(AppContext);
@@ -183,10 +179,6 @@ ValueElement.propTypes = {
   overridableId: PropTypes.string,
 };
 
-ValueElement.defaultProps = {
-  overridableId: "",
-};
-
 export default Overridable.component(
   "BucketAggregationValues",
   BucketAggregationValues