Merge pull request #25 from FezVrasta/fix/24

fix: don’t attempt to call prop.get() when prop is nullish

Author: jourdain

src/core/View.js

@@ -511,7 +511,7 @@ export default class View extends Component {
         const representationIds = [];
         this.selections.forEach((v) => {
           const { prop } = v.getProperties();
-          const { representationId } = prop.get('representationId');
+          const representationId = prop?.get('representationId').representationId;
           if (representationId) {
             representationIds.push(representationId);
           }
@@ -529,6 +529,9 @@ export default class View extends Component {
       return this.selections.map((v) => {
         const { prop, compositeID, displayPosition } = v.getProperties();
 
+        // Return false to mark this item for removal
+        if (prop == null) return false;
+
         return {
           worldPosition: Array.from(
             this.openglRenderWindow.displayToWorld(
@@ -543,7 +546,7 @@ export default class View extends Component {
           ...prop.get('representationId'),
           ray,
         };
-      });
+      }).filter(Boolean);
     }
     return [];
   }