Merge pull request #132 from j-joker/master

Armour · web-flow · commit 72b2274e8ddb · 2020-05-07T14:31:16.000+02:00
Fix(directive):remove the dom operation in v-permission which will ca…
diff --git a/src/directives/permission/index.ts b/src/directives/permission/index.ts
@@ -11,7 +11,7 @@ export const permission: DirectiveOptions = {
         return permissionRoles.includes(role)
       })
       if (!hasPermission) {
-        el.parentNode && el.parentNode.removeChild(el)
+        el.style.display = 'none'
       }
     } else {
       throw new Error('need roles! Like v-permission="[\'admin\',\'editor\']"')

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ export const permission: DirectiveOptions = {`
`11`	`11`	`return permissionRoles.includes(role)`
`12`	`12`	`})`
`13`	`13`	`if (!hasPermission) {`
`14`		`- el.parentNode && el.parentNode.removeChild(el)`
	`14`	`+ el.style.display = 'none'`
`15`	`15`	`}`
`16`	`16`	`} else {`
`17`	`17`	`throw new Error('need roles! Like v-permission="[\'admin\',\'editor\']"')`