From 82fa2dcc844a019e71284ddae4d7cf67a2804e37 Mon Sep 17 00:00:00 2001
From: Kajetan Kazimierczak <kajetan-kazimierczak@users.noreply.github.com>
Date: Fri, 30 Apr 2021 10:37:44 +0200
Subject: [PATCH] =?UTF-8?q?fix:=20=F0=9F=90=9B=20F=C3=B6rb=C3=A4ttrad=20pa?=
 =?UTF-8?q?rsning=20av=20nyhetsbrev=20(#125)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* fix: 🐛 Förbättrad parsning av nyhetsbrev

* refactor: 💡 unfolded loop to get rid of linting error. :(

* Clean up rearrangeWhitespace

* undo

Co-authored-by: Viktor Sarström <viktorsarstrom@gmail.com>
---
 lib/parse/__tests__/news.test.ts |  2 +-
 lib/parseHtml.test.ts            | 11 ++++-----
 lib/parseHtml.ts                 | 39 +++++++++++++++++++++++++++++---
 3 files changed, 42 insertions(+), 10 deletions(-)

diff --git a/lib/parse/__tests__/news.test.ts b/lib/parse/__tests__/news.test.ts
index a74cf56d..29648fc0 100644
--- a/lib/parse/__tests__/news.test.ts
+++ b/lib/parse/__tests__/news.test.ts
@@ -168,7 +168,7 @@ describe('newsItem', () => {
     const item = newsItemDetails(response)
 
     const expected =
-      '[1177 hemsida](https://www.1177.se/sjukdomar--besvar/hud-har-och-naglar/harbotten-och-harsackar/huvudloss/)​​​​​​​'
+      '[1177 hemsida](https://www.1177.se/sjukdomar--besvar/hud-har-och-naglar/harbotten-och-harsackar/huvudloss/)'
     expect(item.body).toContain(expected)
     expect(item.body).toContain(' **tillfället** ')
   })
diff --git a/lib/parseHtml.test.ts b/lib/parseHtml.test.ts
index 7a4681aa..dad4ce91 100644
--- a/lib/parseHtml.test.ts
+++ b/lib/parseHtml.test.ts
@@ -65,7 +65,7 @@ describe('parseHtml', () => {
           </tbody>
         </table>
       </div>`
-      const expected = `# Hello #
+      const expected = `# Hello
 **World**
 - Foo
 
@@ -92,8 +92,7 @@ Alla knep är tillåtna.
 Kolla in Reddit: [https://reddit.com/water-balloons/where-to-buy/](https://reddit.com/water-balloons/where-to-buy/)
 
 ...och här: [https://reddit.com/splash-wars/](https://reddit.com/splash-wars/)
-
-## Om att vara hemma vid symtom ##
+## Om att vara hemma vid symtom
 
 Även HackerNews är bra.
 
@@ -101,10 +100,10 @@ Kolla in Reddit: [https://reddit.com/water-balloons/where-to-buy/](https://reddi
 
 Vi fortsätter också att: 
 
-- hålla avstånd. 
+- hålla avstånd.
 - ha flera digitala möten.
-- tvätta händerna. 
-- undvika kollektivtrafik om det är möjligt. 
+- tvätta händerna.
+- undvika kollektivtrafik om det är möjligt.
 
 - stanna hemma även när man bara känner sig lite sjuk.
 - vädra ofta
diff --git a/lib/parseHtml.ts b/lib/parseHtml.ts
index a2a76d30..a7847ed3 100644
--- a/lib/parseHtml.ts
+++ b/lib/parseHtml.ts
@@ -41,13 +41,40 @@ const deepClean = (node: HTMLElement): HTMLElement => {
 }
 
 const rearrangeWhitespace = (html: string = ''): string => {
-  let content = html.split('&#160;').join('&amp;nbsp;')
+  let content = html
+  .replace(/<span[^>]*>/gm, '')
+  .split('</span>').join('')
+  .replace(/<div[^>]*>/gm, '')
+  .split('</div>').join('')
+  .split('&#160;').join('&amp;nbsp;')
+  
+  // FIXME: Make a loop that doesn't break linting
   trimNodes.forEach((trimNode) => {
     content = content.split(`<${trimNode}> `).join(` <${trimNode}>`)
     content = content.split(` </${trimNode}>`).join(`</${trimNode}> `)
-    content = content.split(`<${trimNode}>&amp;nbsp;`).join(`&amp;nbsp;<${trimNode}>`)
-    content = content.split(`&amp;nbsp;</${trimNode}>`).join(`</${trimNode}>&amp;nbsp;`)
+    content = content.split(`<${trimNode}>&amp;nbsp;`).join(` <${trimNode}>`)
+    content = content.split(`&amp;nbsp;</${trimNode}>`).join(`</${trimNode}> `)    
   })
+    
+  trimNodes.forEach((trimNode) => {
+    content = content.split(`<${trimNode}> `).join(` <${trimNode}>`)
+    content = content.split(` </${trimNode}>`).join(`</${trimNode}> `)
+    content = content.split(`<${trimNode}>&amp;nbsp;`).join(` <${trimNode}>`)
+    content = content.split(`&amp;nbsp;</${trimNode}>`).join(`</${trimNode}> `)    
+  })
+  trimNodes.forEach((trimNode) => {
+    content = content.split(`<${trimNode}> `).join(` <${trimNode}>`)
+    content = content.split(` </${trimNode}>`).join(`</${trimNode}> `)
+    content = content.split(`<${trimNode}>&amp;nbsp;`).join(` <${trimNode}>`)
+    content = content.split(`&amp;nbsp;</${trimNode}>`).join(`</${trimNode}> `)    
+  })
+  trimNodes.forEach((trimNode) => {
+    content = content.split(`<${trimNode}> `).join(` <${trimNode}>`)
+    content = content.split(` </${trimNode}>`).join(`</${trimNode}> `)
+    content = content.split(`<${trimNode}>&amp;nbsp;`).join(` <${trimNode}>`)
+    content = content.split(`&amp;nbsp;</${trimNode}>`).join(`</${trimNode}> `)    
+  })
+
   return content
 }
 
@@ -66,6 +93,12 @@ const overides = {
   img: (node: Node) => `![${node.attrs.title || ''}](${node.attrs.src})`,
   i: (node: Node) => `*${node.md}*`,
   b: (node: Node) => `**${node.md}**`,
+  'h1': (node: Node) => `# ${node.md}\n`,
+  'h2': (node: Node) => `## ${node.md}\n`,
+  'h3': (node: Node) => `### ${node.md}\n`,
+  'h4': (node: Node) => `#### ${node.md}\n`,
+  'h5': (node: Node) => `##### ${node.md}\n`,
+  'h6': (node: Node) => `###### ${node.md}\n`,
 }
 
 export const toMarkdown = (html: string): string => {