Skip to content

Commit dfeaab4

Browse files
committed
Merge pull request #109 from ParsePlatform/array-of-null
Fix issues with arrays contiaining null or pointers to users. Fixes #105.
2 parents fb9c19d + 417654d commit dfeaab4

File tree

2 files changed

+6
-14
lines changed

2 files changed

+6
-14
lines changed

src/components/BrowserCell/BrowserCell.react.js

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -39,16 +39,10 @@ let BrowserCell = ({ type, value, hidden, width, current, onSelect, readonly, on
3939
content = dateStringUTC(value);
4040
} else if (type === 'Boolean') {
4141
content = value ? 'True' : 'False';
42-
} else if (type === 'Array' || type === 'Object') {
43-
if (type === 'Array') {
44-
let _value = [];
45-
value.forEach((val) => {
46-
_value.push(val.constructor === Parse.Object ? val.toPointer() : val);
47-
});
48-
content = JSON.stringify(_value);
49-
} else {
50-
content = JSON.stringify(value);
51-
}
42+
} else if (type === 'Array') {
43+
content = JSON.stringify(value.map(val => val instanceof Parse.Object ? val.toPointer() : val))
44+
} else if (type === 'Object') {
45+
content = JSON.stringify(value);
5246
} else if (type === 'File') {
5347
if (value.url()) {
5448
content = <a href={value.url()} target='_blank'><Pill value={getFileName(value)} /></a>;
@@ -85,7 +79,7 @@ let BrowserCell = ({ type, value, hidden, width, current, onSelect, readonly, on
8579
</div>
8680
);
8781
}
88-
82+
8983
if (current) {
9084
classes.push(styles.current);
9185
}

src/dashboard/Data/Browser/BrowserTable.react.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -202,9 +202,7 @@ export default class BrowserTable extends React.Component {
202202
value = '';
203203
} else if (type === 'Array') {
204204
if (value) {
205-
value = value.map((val) => {
206-
return val.constructor === Parse.Object ? val.toPointer() : val;
207-
});
205+
value = value.map(val => val instanceof Parse.Object ? val.toPointer() : val);
208206
}
209207
}
210208
let wrapTop = Math.max(0, this.props.current.row * ROW_HEIGHT);

0 commit comments

Comments
 (0)