fix(AbstractUser): make isOwner return false when this.provider is not defined

Hans Kristian Flaatten · Hans Kristian Flaatten · commit 912f6c756975 · 2016-07-05T13:04:37.000+02:00
diff --git a/lib/AuthUser.js b/lib/AuthUser.js
@@ -20,7 +20,7 @@ class AuthUser extends AbstractUser {
   }
 
   isOwner(doc) {
-    return !!doc.tilbyder && this.provider === doc.tilbyder;
+    return !!doc.tilbyder && !!this.provider && this.provider === doc.tilbyder;
   }
 
   can() {

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ class AuthUser extends AbstractUser {`
`20`	`20`	`}`
`21`	`21`
`22`	`22`	`isOwner(doc) {`
`23`		`- return !!doc.tilbyder && this.provider === doc.tilbyder;`
	`23`	`+ return !!doc.tilbyder && !!this.provider && this.provider === doc.tilbyder;`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`can() {`