refactor: replace defaultProps with JavaScript default parameters (#9464)
This change in specific to React components and React@18 version. Refs #9456
This commit is contained in:
@@ -20,54 +20,33 @@ const getModelName = (uri) => {
|
||||
return null
|
||||
}
|
||||
|
||||
const Model = forwardRef(({ schema, getComponent, onToggle }, ref) => {
|
||||
const JSONSchema202012 = getComponent("JSONSchema202012")
|
||||
const name = getModelName(schema.get("$$ref"))
|
||||
const Model = forwardRef(
|
||||
({ schema, getComponent, onToggle = () => {} }, ref) => {
|
||||
const JSONSchema202012 = getComponent("JSONSchema202012")
|
||||
const name = getModelName(schema.get("$$ref"))
|
||||
|
||||
const handleExpand = useCallback(
|
||||
(e, expanded) => {
|
||||
onToggle(name, expanded)
|
||||
},
|
||||
[name, onToggle]
|
||||
)
|
||||
const handleExpand = useCallback(
|
||||
(e, expanded) => {
|
||||
onToggle(name, expanded)
|
||||
},
|
||||
[name, onToggle]
|
||||
)
|
||||
|
||||
return (
|
||||
<JSONSchema202012
|
||||
name={name}
|
||||
schema={schema.toJS()}
|
||||
ref={ref}
|
||||
onExpand={handleExpand}
|
||||
/>
|
||||
)
|
||||
})
|
||||
return (
|
||||
<JSONSchema202012
|
||||
name={name}
|
||||
schema={schema.toJS()}
|
||||
ref={ref}
|
||||
onExpand={handleExpand}
|
||||
/>
|
||||
)
|
||||
}
|
||||
)
|
||||
|
||||
Model.propTypes = {
|
||||
schema: ImPropTypes.map.isRequired,
|
||||
getComponent: PropTypes.func.isRequired,
|
||||
getConfigs: PropTypes.func.isRequired,
|
||||
specSelectors: PropTypes.object.isRequired,
|
||||
specPath: ImPropTypes.list.isRequired,
|
||||
name: PropTypes.string,
|
||||
displayName: PropTypes.string,
|
||||
isRef: PropTypes.bool,
|
||||
required: PropTypes.bool,
|
||||
expandDepth: PropTypes.number,
|
||||
depth: PropTypes.number,
|
||||
includeReadOnly: PropTypes.bool,
|
||||
includeWriteOnly: PropTypes.bool,
|
||||
onToggle: PropTypes.func,
|
||||
}
|
||||
|
||||
Model.defaultProps = {
|
||||
name: "",
|
||||
displayName: "",
|
||||
isRef: false,
|
||||
required: false,
|
||||
expandDepth: 0,
|
||||
depth: 1,
|
||||
includeReadOnly: false,
|
||||
includeWriteOnly: false,
|
||||
onToggle: () => {},
|
||||
}
|
||||
|
||||
export default Model
|
||||
|
||||
@@ -5,6 +5,25 @@ import { componentDidCatch } from "../fn"
|
||||
import Fallback from "./fallback"
|
||||
|
||||
export class ErrorBoundary extends Component {
|
||||
static propTypes = {
|
||||
targetName: PropTypes.string,
|
||||
getComponent: PropTypes.func,
|
||||
fn: PropTypes.object,
|
||||
children: PropTypes.oneOfType([
|
||||
PropTypes.arrayOf(PropTypes.node),
|
||||
PropTypes.node,
|
||||
])
|
||||
}
|
||||
|
||||
static defaultProps = {
|
||||
targetName: "this component",
|
||||
getComponent: () => Fallback,
|
||||
fn: {
|
||||
componentDidCatch,
|
||||
},
|
||||
children: null,
|
||||
}
|
||||
|
||||
static getDerivedStateFromError(error) {
|
||||
return { hasError: true, error }
|
||||
}
|
||||
@@ -29,22 +48,5 @@ export class ErrorBoundary extends Component {
|
||||
return children
|
||||
}
|
||||
}
|
||||
ErrorBoundary.propTypes = {
|
||||
targetName: PropTypes.string,
|
||||
getComponent: PropTypes.func,
|
||||
fn: PropTypes.object,
|
||||
children: PropTypes.oneOfType([
|
||||
PropTypes.arrayOf(PropTypes.node),
|
||||
PropTypes.node,
|
||||
])
|
||||
}
|
||||
ErrorBoundary.defaultProps = {
|
||||
targetName: "this component",
|
||||
getComponent: () => Fallback,
|
||||
fn: {
|
||||
componentDidCatch,
|
||||
},
|
||||
children: null,
|
||||
}
|
||||
|
||||
export default ErrorBoundary
|
||||
|
||||
Reference in New Issue
Block a user