refactor(core): check if element is object
This commit is contained in:
@@ -69,7 +69,7 @@ export function getHostForElement(element: HTMLElement): Document {
|
|||||||
|
|
||||||
// todo: refactor generic to use MaybeElement
|
// todo: refactor generic to use MaybeElement
|
||||||
export function isEdge<Data = ElementData>(element: MaybeElement): element is Edge<Data> {
|
export function isEdge<Data = ElementData>(element: MaybeElement): element is Edge<Data> {
|
||||||
return element && 'id' in element && 'source' in element && 'target' in element
|
return typeof element === 'object' && 'id' in element && 'source' in element && 'target' in element
|
||||||
}
|
}
|
||||||
|
|
||||||
// todo: refactor generic to use MaybeElement
|
// todo: refactor generic to use MaybeElement
|
||||||
@@ -79,7 +79,7 @@ export function isGraphEdge<Data = ElementData>(element: MaybeElement): element
|
|||||||
|
|
||||||
// todo: refactor generic to use MaybeElement
|
// todo: refactor generic to use MaybeElement
|
||||||
export function isNode<Data = ElementData>(element: MaybeElement): element is Node<Data> {
|
export function isNode<Data = ElementData>(element: MaybeElement): element is Node<Data> {
|
||||||
return element && 'id' in element && !isEdge(element)
|
return typeof element === 'object' && 'id' in element && !isEdge(element)
|
||||||
}
|
}
|
||||||
|
|
||||||
// todo: refactor generic to use MaybeElement
|
// todo: refactor generic to use MaybeElement
|
||||||
|
|||||||
Reference in New Issue
Block a user