Oops, not an update

guardian · Feb 19, 2018 · 5bd6b6f · 5bd6b6f
1 parent e9176f4
commit 5bd6b6f
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/main/scala/com/gu/editorial/permissions/client/PermissionsStore.scala b/src/main/scala/com/gu/editorial/permissions/client/PermissionsStore.scala
@@ -42,10 +42,10 @@ class PermissionsStore(config: PermissionsConfig, provider: Option[PermissionsSt
 
   def list(implicit user: PermissionsUser): Future[PermissionsMap] = {
     if (config.enablePermissionsStore)
-      storeProvider.store.modify {
-        case PermissionsStoreModel.empty => Failure(PermissionsStoreEmptyException())
-        case s: PermissionsStoreModel => Success(s)
-      }.map(s => s.defaultsMap ++ s.userOverrides.getOrElse(user.userId.toLowerCase, Map.empty))
+      Future.successful(storeProvider.store.get).flatMap {
+        case PermissionsStoreModel.empty => Future.failed(PermissionsStoreEmptyException())
+        case s: PermissionsStoreModel => Future.successful(s.defaultsMap ++ s.userOverrides.getOrElse(user.userId.toLowerCase, Map.empty))
+      }
 
     else Future.failed(PermissionsStoreDisabledException())
   }