fix: information collect too early pushed to map

DimensionDev · Oct 25, 2019 · 18ddb8d · 18ddb8d
1 parent 214d412
commit 18ddb8d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/social-network-provider/twitter.com/ui/fetch.ts b/src/social-network-provider/twitter.com/ui/fetch.ts
@@ -57,8 +57,6 @@ const registerPostCollector = (self: SocialNetworkUI) => {
     new MutationObserverWatcher(postsSelectors())
         .useForeach((node, _, proxy) => {
             const info = getEmptyPostInfo(postsRootSelector())
-            // push to map
-            self.posts.set(proxy, info)
             const collectPostInfo = () => {
                 const r = postParser(node)
                 if (!r) return
@@ -75,6 +73,8 @@ const registerPostCollector = (self: SocialNetworkUI) => {
             info.postContent.addListener(newValue => {
                 info.postPayload.value = deconstructPayload(newValue, self.payloadDecoder)
             })
+            // push to map
+            self.posts.set(proxy, info)
             return {
                 onNodeMutation: collectPostInfo,
                 onTargetChanged: collectPostInfo,

diff --git a/src/social-network-provider/twitter.com/worker/fetch.ts b/src/social-network-provider/twitter.com/worker/fetch.ts
@@ -11,6 +11,7 @@ export const fetchPostContent = async (post: PostIdentifier<PersonIdentifier>) =
     //  it will be a false-positive and it is dangerous.
     //  There is a build-in parser.
     //  Checkout http://mdn.io/DOMParser and we're already using it.
+    //  Legacy js-free mobile twitter may help.
     const content = twitterWorkerSelf.publicKeyDecoder(d.innerText)
     if (content && content[0].length) return content[0]
     return tasks(toPostUrl(post), {}).getPostContent(post)