Merge pull request #974 from Koooooo-7/develop

[fix]#972 when 'logo' is set, 'name' is not set correctly inside the 'alt' as same as without 'logo' set.

src/core/render/tpl.js

@@ -1,4 +1,5 @@
 import {isMobile} from '../util/env'
+import {escapeString} from '../util/core'
 /**
  * Render github corner
  * @param  {Object} data
@@ -30,6 +31,9 @@ export function corner(data, cornerExternalLinkTarge) {
  * Render main content
  */
 export function main(config) {
+
+  const name = config.name? escapeString(config.name):''
+
   const aside =
     '<button class="sidebar-toggle" aria-label="Menu">' +
     '<div class="sidebar-toggle-button">' +
@@ -39,9 +43,9 @@ export function main(config) {
     '<aside class="sidebar">' +
     (config.name ?
       `<h1 class="app-name"><a class="app-name-link" data-nosearch>${
-        config.logo ?
+        config.logo ? 
-          `<img alt=${config.name} src=${config.logo}>` :
+           `<img alt="${name}" src=${config.logo}>` :
-          config.name
+          name
       }</a></h1>` :
       '') +
     '<div class="sidebar-nav"><!--sidebar--></div>' +

src/core/util/core.js

@@ -56,3 +56,19 @@ export function noop() {}
 export function isFn(obj) {
   return typeof obj === 'function'
 }
+
+/**
+ * escape String
+ */
+export function escapeString(string) {
+  const entityMap = {
+    '&': '&',
+    '<': '&lt;',
+    '>': '&gt;',
+    '"': '&quot;',
+    '\'': '&#39;',
+    '/': '&#x2F;'
+  }
+
+  return String(string).replace(/[&<>"'/]/g, s => entityMap[s])
+}