Merge pull request #866 from swarmcity/bug_fixing

Bug fixing

src/redux/app-reducers.html

@@ -40,6 +40,7 @@
             },
             balance: {},
             hashtagList: [],
+            visitedHashtags: {},
             fetching: {},
             hashtags: {},
             current: {},
@@ -313,6 +314,13 @@
                         }
                     }
                 }
+                case 'ADD_VISITED_HASHTAG': {
+                    return deepmerge(state, {
+                        visitedHashtags: {
+                            [action.hashtag.hashtagAddress]: action.hashtag
+                        }
+                    })
+                }
                 default: {
                     return state
                 }

src/redux/app-selectors.html

@@ -55,6 +55,7 @@
             },
             confirmedTx: (state) => state.confirmedTx,
             hashtagList: (state) => state.hashtagList,
+            visitedHashtags: (state) => state.visitedHashtags,
             address: (state) => state.publicKey,
             username: (state) => state.username,
             language: (state) => state.language,

src/redux/sagas/handleReplyItem.html

@@ -30,7 +30,8 @@
                 // Step 1. Store the reply info
                 console.log('Handling ReplyEvent', event)
                 const { itemHash, replyMetadataHash, provider: providerAddress } = event.returnValues
-                const { address: hashtagAddress } = event
+                // Protection against calling toLowerCase() on undefined variables
+                const hashtagAddress = event.address ? event.address.toLowerCase() : event.address
                 const reply = {
                     blockNumber: event.blockNumber,
                     providerAddress,

src/redux/sagas/hashtag-saga.html

@@ -78,7 +78,8 @@
                 hashtagMetadataHash: await hashtagContract.methods.hashtagMetadataHash().call(),
                 hashtagFee: parseInt(await hashtagContract.methods.hashtagFee().call()),
                 deployBlock: deployBlock || 8149489,
-                address: hashtagContract._address
+                address: hashtagContract._address,
+                hashtagAddress: hashtagContract._address
             }
         }
 
@@ -157,6 +158,7 @@
                 //   - deployBlock() -> deployBlock
                 const hashtagContractData = yield call(getHashtagContractData, hashtagContract)
                 yield put({ type: 'UPDATE_NEWHASHTAG', hashtagAddress, data: hashtagContractData });
+                yield put({ type: 'ADD_VISITED_HASHTAG', hashtag: hashtagContractData })
 
                 // Step 2A. Resolve hashtagMetadata with IPFS
                 //   - ipfs.cat(hashtagMetadata) -> description

src/redux/sagas/hashtagList-saga.html

@@ -56,7 +56,18 @@
                 if (!hashtagList.length) {
                     hashtagList = yield call(getHashtagList)
                 }
-                const reputationsObj = yield call(getReputationFromHashtags, hashtagList, address)
+
+                // Append the visited hashtags and remove duplicates
+                // visitedHashtags is an object
+                const visitedHashtags = yield select(selectors.visitedHashtags)
+                const uniqueHashtags = visitedHashtags
+                hashtagList.forEach(hashtag => {
+                    uniqueHashtags[hashtag.hashtagAddress] = hashtag
+                })
+                const uniqueHashtagList = Object.values(uniqueHashtags)
+
+                // Get reputation for all hashtags in the list
+                const reputationsObj = yield call(getReputationFromHashtags, uniqueHashtagList, address)
                 yield put({ type: 'UPDATE_REPUTATION', address, reputationsObj });
             } catch (e) {
                 console.error(`Error fetching user's reputation`, e)

src/redux/sagas/router-saga.html

@@ -41,7 +41,7 @@
             const path = window.location.pathname
             //     [0 /  [1]  /    [2]   /   [3]
             // path = /hashtag/0xCeb...fE/0x2342tds
-            const [, pageName, hashtag, item] = path.split('/')
+            let [, pageName, hashtag, item] = path.split('/')
 
             const params = parseSearch()
 
@@ -71,6 +71,9 @@
                 || pageName === 'new-request'
             ) {
                 if (hashtag && hashtag !== _hashtag) {
+                    // Prevent inconsistencies with address with/without checksum case
+                    // This parameter is entered by the user 
+                    hashtag = hashtag.toLowerCase()
                     yield put({ type: 'GET_HASHTAG', hashtag })
                     yield put({ type: 'CURRENT_HASHTAG', hashtag })
                     _hashtag = hashtag;