Don't support images in account descriptions
authorChocobozzz <me@florianbigard.com>
Fri, 6 Dec 2019 13:01:28 +0000 (14:01 +0100)
committerChocobozzz <me@florianbigard.com>
Fri, 6 Dec 2019 13:01:28 +0000 (14:01 +0100)
client/src/app/+accounts/account-about/account-about.component.ts
client/src/app/shared/renderer/markdown.service.ts

index c65bad3f2935cfc91de6133b423a2b79c415edd3..d1616490ff33a85b1392874780ad2b86a0684b85 100644 (file)
@@ -27,7 +27,7 @@ export class AccountAboutComponent implements OnInit, OnDestroy {
     this.accountSub = this.accountService.accountLoaded
       .subscribe(async account => {
         this.account = account
-        this.descriptionHTML = await this.markdownService.enhancedMarkdownToHTML(this.account.description, true)
+        this.descriptionHTML = await this.markdownService.textMarkdownToHTML(this.account.description, true)
       })
   }
 
index 09bbbfb7ef863f1752a954f0c0482cbe36abc32b..fc1f18c2ba3829ac15069a4705c0d0fd508c5fe2 100644 (file)
@@ -1,12 +1,14 @@
 import { Injectable } from '@angular/core'
 import { MarkdownIt } from 'markdown-it'
 import { HtmlRendererService } from '@app/shared/renderer/html-renderer.service'
-import { mark } from '@angular/compiler-cli/src/ngtsc/perf/src/clock'
 
 type MarkdownParsers = {
   textMarkdownIt: MarkdownIt
+  textWithHTMLMarkdownIt: MarkdownIt
+
   enhancedMarkdownIt: MarkdownIt
-  enhancedMarkdownWithHTMLIt: MarkdownIt
+  enhancedWithHTMLMarkdownIt: MarkdownIt
+
   completeMarkdownIt: MarkdownIt
 }
 
@@ -30,36 +32,45 @@ export class MarkdownService {
     'newline',
     'list'
   ]
+  static TEXT_WITH_HTML_RULES = MarkdownService.TEXT_RULES.concat([ 'html_inline', 'html_block' ])
+
   static ENHANCED_RULES = MarkdownService.TEXT_RULES.concat([ 'image' ])
-  static ENHANCED_WITH_HTML_RULES = MarkdownService.ENHANCED_RULES.concat([ 'html_inline', 'html_block' ])
+  static ENHANCED_WITH_HTML_RULES = MarkdownService.TEXT_WITH_HTML_RULES.concat([ 'image' ])
+
   static COMPLETE_RULES = MarkdownService.ENHANCED_WITH_HTML_RULES.concat([ 'block', 'inline', 'heading', 'paragraph' ])
 
   private markdownParsers: MarkdownParsers = {
     textMarkdownIt: null,
+    textWithHTMLMarkdownIt: null,
     enhancedMarkdownIt: null,
-    enhancedMarkdownWithHTMLIt: null,
+    enhancedWithHTMLMarkdownIt: null,
     completeMarkdownIt: null
   }
   private parsersConfig: MarkdownParserConfigs = {
     textMarkdownIt: { rules: MarkdownService.TEXT_RULES, html: false },
+    textWithHTMLMarkdownIt: { rules: MarkdownService.TEXT_WITH_HTML_RULES, html: true, escape: true },
+
     enhancedMarkdownIt: { rules: MarkdownService.ENHANCED_RULES, html: false },
-    enhancedMarkdownWithHTMLIt: { rules: MarkdownService.ENHANCED_WITH_HTML_RULES, html: true, escape: true },
+    enhancedWithHTMLMarkdownIt: { rules: MarkdownService.ENHANCED_WITH_HTML_RULES, html: true, escape: true },
+
     completeMarkdownIt: { rules: MarkdownService.COMPLETE_RULES, html: true }
   }
 
   constructor (private htmlRenderer: HtmlRendererService) {}
 
-  textMarkdownToHTML (markdown: string) {
+  textMarkdownToHTML (markdown: string, withHtml = false) {
+    if (withHtml) return this.render('textWithHTMLMarkdownIt', markdown)
+
     return this.render('textMarkdownIt', markdown)
   }
 
-  async enhancedMarkdownToHTML (markdown: string, withHtml = false) {
-    if (withHtml) return this.render('enhancedMarkdownWithHTMLIt', markdown)
+  enhancedMarkdownToHTML (markdown: string, withHtml = false) {
+    if (withHtml) return this.render('enhancedWithHTMLMarkdownIt', markdown)
 
     return this.render('enhancedMarkdownIt', markdown)
   }
 
-  async completeMarkdownToHTML (markdown: string) {
+  completeMarkdownToHTML (markdown: string) {
     return this.render('completeMarkdownIt', markdown)
   }