Esri/arcgis-rest-js
`isItemSharedWithGroup` returns `undefined` instead of false.
Open
#665 opened on Mar 3, 2020
good first issuehelp wanted
Repository metrics
- Stars
- (389 stars)
- PR merge metrics
- (PR metrics pending)
Description
The return statement in isItemSharedWithGroup is incorrectly nested in an if block. This one flies under the radar because undefined is falsy, but it's a bug nonetheless.
function isItemSharedWithGroup(requestOptions) {
var searchOpts = {
q: "id: " + requestOptions.id + " AND group: " + requestOptions.groupId,
start: 1,
num: 10,
sortField: "title",
authentication: requestOptions.authentication,
httpMethod: "POST"
};
return Object(_items_search__WEBPACK_IMPORTED_MODULE_7__["searchItems"])(searchOpts).then(function (searchResponse) {
var result = false;
if (searchResponse.total > 0) {
result = searchResponse.results.some(function (itm) {
return itm.id === requestOptions.id;
});
return result; // SHOULD BE OUTSIDE THE IF BLOCK
}
});
}