fix bug of reporting 0 total claims #251

Merged
jessopb merged 1 commit from fix-zeroClaimsBug into master 2019-12-13 22:25:20 +01:00
jessopb commented 2019-12-13 02:39:18 +01:00 (Migrated from github.com)
No description provided.
jessopb (Migrated from github.com) reviewed 2019-12-13 02:42:16 +01:00
@ -2044,3 +2044,3 @@
const makeSelectOmittedCountForChannel = uri => reselect.createSelector(makeSelectTotalItemsForChannel(uri), makeSelectTotalClaimsInChannelSearch(uri), (claimsInChannel, claimsInSearch) => {
if (claimsInChannel && claimsInSearch) {
if (claimsInChannel && typeof claimsInSearch === 'number' && claimsInSearch >= 0) {
return claimsInChannel - claimsInSearch;
jessopb (Migrated from github.com) commented 2019-12-13 02:42:16 +01:00

apparently null >= 0. so the typeof is probably a good defensive move

apparently null >= 0. so the typeof is probably a good defensive move
kauffj (Migrated from github.com) approved these changes 2019-12-13 16:38:25 +01:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-redux#251
No description provided.