Skip to content

Commit 2faa831

Browse files
author
Romain Julien
committed
feat: dont register productClick event if no onProductClick received
1 parent e704edc commit 2faa831

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

packages/display-react/src/index.js

+5-3
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ function unRegisterEvent(controller, name, callbackFromRegister) {
2222
}
2323

2424
const EVENTS = {
25-
PRODUCT_CLICK: 'poductClick',
25+
PRODUCT_CLICK: 'productClick',
2626
CHANGE_SCENE: 'changeScene',
2727
CHANGE_PRODUCT_LOCATION: 'changeSelectedProductLocation',
2828
LOAD_SCENE: 'loadScene',
@@ -110,14 +110,16 @@ function Display(
110110
onChangeSelectedProductLocationStable,
111111
)
112112
const callbackOnError = registerEvent(ctrl, EVENTS.ERROR, onErrorStable)
113-
const callbackOnProductClick = registerEvent(ctrl, EVENTS.PRODUCT_CLICK, onProductClickStable)
113+
const callbackOnProductClick = onProductClickStable.current ? registerEvent(ctrl, EVENTS.PRODUCT_CLICK, onProductClickStable) : undefined
114114
const callbackOnFilter = registerEvent(ctrl, EVENTS.FILTER, onFilterStable)
115115
return () => {
116116
unRegisterEvent(ctrl, EVENTS.LOAD_SCENE, callbackOnLoadScene)
117117
unRegisterEvent(ctrl, EVENTS.CHANGE_SCENE, callbackOnChangeScene)
118118
unRegisterEvent(ctrl, EVENTS.CHANGE_PRODUCT, callbackOnChangeProduct)
119119
unRegisterEvent(ctrl, EVENTS.CHANGE_PRODUCT_LOCATION, callbackOnChangeSelectedProductLocationStable)
120-
unRegisterEvent(ctrl, EVENTS.PRODUCT_CLICK, callbackOnProductClick)
120+
if (callbackOnProductClick) {
121+
unRegisterEvent(ctrl, EVENTS.PRODUCT_CLICK, callbackOnProductClick)
122+
}
121123
unRegisterEvent(ctrl, EVENTS.FILTER, callbackOnFilter)
122124
unRegisterEvent(ctrl, EVENTS.ERROR, callbackOnError)
123125
ctrl.current.destroy()

0 commit comments

Comments
 (0)