Fix #6766: Added checks for dragdropScope being defined before conver…

…sion (#6767)
primefaces · Jun 18, 2024 · 825a52d · 825a52d
1 parent a698936
commit 825a52d
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/components/lib/tree/UITreeNode.js b/components/lib/tree/UITreeNode.js
@@ -620,20 +620,20 @@ export const UITreeNode = React.memo((props) => {
     };
 
     const onDropPointDragOver = (event) => {
-        if (event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase()) {
+        if (props.dragdropScope && event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase()) {
             event.dataTransfer.dropEffect = 'move';
             event.preventDefault();
         }
     };
 
     const onDropPointDragEnter = (event) => {
-        if (event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase()) {
+        if (props.dragdropScope && event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase()) {
             DomHandler.addClass(event.target, 'p-treenode-droppoint-active');
         }
     };
 
     const onDropPointDragLeave = (event) => {
-        if (event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase()) {
+        if (props.dragdropScope && event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase()) {
             DomHandler.removeClass(event.target, 'p-treenode-droppoint-active');
         }
     };
@@ -655,21 +655,21 @@ export const UITreeNode = React.memo((props) => {
     };
 
     const onDragOver = (event) => {
-        if (event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase() && props.node.droppable !== false) {
+        if (props.dragdropScope && event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase() && props.node.droppable !== false) {
             event.dataTransfer.dropEffect = 'move';
             event.preventDefault();
             event.stopPropagation();
         }
     };
 
     const onDragEnter = (event) => {
-        if (event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase() && props.node.droppable !== false) {
+        if (props.dragdropScope && event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase() && props.node.droppable !== false) {
             DomHandler.addClass(contentRef.current, 'p-treenode-dragover');
         }
     };
 
     const onDragLeave = (event) => {
-        if (event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase() && props.node.droppable !== false) {
+        if (props.dragdropScope && event.dataTransfer.types[1] === props.dragdropScope.toLocaleLowerCase() && props.node.droppable !== false) {
             let rect = event.currentTarget.getBoundingClientRect();
 
             if (event.nativeEvent.x > rect.left + rect.width || event.nativeEvent.x < rect.left || event.nativeEvent.y >= Math.floor(rect.top + rect.height) || event.nativeEvent.y < rect.top) {