fix(accounts): ignore non-hoodie account docs in accounts.findAll()

hoodiehq · Mar 7, 2017 · 3f40085 · 3f40085
1 parent b59366b
commit 3f40085
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/lib/accounts/find-all.js b/lib/accounts/find-all.js
@@ -1,5 +1,6 @@
 module.exports = findAllAccount
 
+var isHoodieAccountDoc = require('../utils/is-hoodie-account-doc')
 var toAccount = require('../utils/doc-to-account')
 
 function findAllAccount (state, options) {
@@ -12,10 +13,14 @@ function findAllAccount (state, options) {
 
   .then(function (response) {
     options = options || {}
-    return response.rows.map(function (row) {
-      return toAccount(row.doc, {
-        includeProfile: options.include === 'profile'
+    return response.rows
+      .filter(function (row) {
+        return isHoodieAccountDoc(row.doc)
+      })
+      .map(function (row) {
+        return toAccount(row.doc, {
+          includeProfile: options.include === 'profile'
+        })
       })
-    })
   })
 }
diff --git a/lib/utils/is-hoodie-account-doc.js b/lib/utils/is-hoodie-account-doc.js
@@ -0,0 +1,7 @@
+module.exports = isHoodieAccountDoc
+
+var findIdInRoles = require('./find-id-in-roles')
+
+function isHoodieAccountDoc (doc) {
+  return !!findIdInRoles(doc.roles)
+}