save share response

bitwarden · Oct 23, 2018 · 43c0cbc · 43c0cbc
1 parent 2f510a7
commit 43c0cbc
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/src/abstractions/api.service.ts b/src/abstractions/api.service.ts
@@ -141,7 +141,7 @@ export abstract class ApiService {
     deleteCipherAdmin: (id: string) => Promise<any>;
     deleteManyCiphers: (request: CipherBulkDeleteRequest) => Promise<any>;
     putMoveCiphers: (request: CipherBulkMoveRequest) => Promise<any>;
-    putShareCipher: (id: string, request: CipherShareRequest) => Promise<any>;
+    putShareCipher: (id: string, request: CipherShareRequest) => Promise<CipherResponse>;
     putShareCiphers: (request: CipherBulkShareRequest) => Promise<any>;
     putCipherCollections: (id: string, request: CipherCollectionsRequest) => Promise<any>;
     putCipherCollectionsAdmin: (id: string, request: CipherCollectionsRequest) => Promise<any>;

diff --git a/src/services/api.service.ts b/src/services/api.service.ts
@@ -374,8 +374,9 @@ export class ApiService implements ApiServiceAbstraction {
         return this.send('PUT', '/ciphers/move', request, true, false);
     }
 
-    putShareCipher(id: string, request: CipherShareRequest): Promise<any> {
-        return this.send('PUT', '/ciphers/' + id + '/share', request, true, false);
+    async putShareCipher(id: string, request: CipherShareRequest): Promise<CipherResponse> {
+        const r = await this.send('PUT', '/ciphers/' + id + '/share', request, true, true);
+        return new CipherResponse(r);
     }
 
     putShareCiphers(request: CipherBulkShareRequest): Promise<any> {

diff --git a/src/services/cipher.service.ts b/src/services/cipher.service.ts
@@ -465,9 +465,10 @@ export class CipherService implements CipherServiceAbstraction {
         cipher.collectionIds = collectionIds;
         const encCipher = await this.encrypt(cipher);
         const request = new CipherShareRequest(encCipher);
-        await this.apiService.putShareCipher(cipher.id, request);
+        const response = await this.apiService.putShareCipher(cipher.id, request);
         const userId = await this.userService.getUserId();
-        await this.upsert(encCipher.toCipherData(userId));
+        const data = new CipherData(response, userId, collectionIds);
+        await this.upsert(data);
     }
 
     async shareManyWithServer(ciphers: CipherView[], organizationId: string, collectionIds: string[]): Promise<any> {