fix: Crash when filtering unique on a pointer field

src/components/BrowserCell/BrowserCell.react.js

@@ -71,7 +71,7 @@ export default class BrowserCell extends Component {
       );
       let value = this.props.value;
       let dataValue = this.props.value.id || this.props.value.objectId;
-      if (defaultPointerKey !== 'objectId') {
+      if (defaultPointerKey !== 'objectId' && typeof this.props.value.get === 'function') {
         dataValue = this.props.value.get(defaultPointerKey);
         if (dataValue && typeof dataValue === 'object') {
           if (dataValue instanceof Date) {

src/components/BrowserRow/BrowserRow.react.js

@@ -67,13 +67,15 @@ export default class BrowserRow extends Component {
       return acc;
     }, requiredCols);
     // for dynamically changing required field on _User class
-    if (
-      obj.className === '_User' &&
-      (obj.get('username') !== undefined || obj.get('password') !== undefined)
-    ) {
-      requiredCols = ['username', 'password'];
-    } else if (obj.className === '_User' && obj.get('authData') !== undefined) {
-      requiredCols = ['authData'];
+    if (!isUnique) {
+      if (
+        obj.className === '_User' &&
+        (obj.get('username') !== undefined || obj.get('password') !== undefined)
+      ) {
+        requiredCols = ['username', 'password'];
+      } else if (obj.className === '_User' && obj.get('authData') !== undefined) {
+        requiredCols = ['authData'];
+      }
     }
     const highlightColor = '#eef4fb';
     const stickyHighlightColor = '#d6e4f0';

src/lib/tests/BrowserRow.test.js

@@ -104,4 +104,29 @@ describe('BrowserRow', () => {
       expect(checkboxCell.props.style.background).toBe('#ffffff');
     });
   });
+
+  describe('Unique filter', () => {
+    beforeAll(() => {
+      global.localStorage = {
+        getItem: () => null,
+        setItem: () => {},
+        removeItem: () => {},
+      };
+    });
+
+    it('does not crash rendering a unique _User pointer value', () => {
+      const rawUserPointer = { __type: 'Pointer', className: '_User', objectId: 'user1' };
+      expect(() =>
+        renderer.create(
+          <BrowserRow
+            {...defaultProps}
+            isUnique={true}
+            obj={rawUserPointer}
+            columns={{ owner: { type: 'Pointer', targetClass: '_User' } }}
+            order={[{ name: 'owner', width: 150, visible: true }]}
+          />
+        )
+      ).not.toThrow();
+    });
+  });
 });