fix(repository): remove hidden properties from entities

See #1914
loopbackio · Oct 30, 2018 · 356d579 · 356d579
1 parent 14d9419
commit 356d579
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 2 deletions.
diff --git a/packages/repository/package.json b/packages/repository/package.json
@@ -30,7 +30,7 @@
     "@types/debug": "0.0.30",
     "debug": "^4.0.1",
     "lodash": "^4.17.10",
-    "loopback-datasource-juggler": "^4.0.0"
+    "loopback-datasource-juggler": "^4.1.2"
   },
   "files": [
     "README.md",

diff --git a/packages/repository/src/repositories/legacy-juggler-bridge.ts b/packages/repository/src/repositories/legacy-juggler-bridge.ts
@@ -326,7 +326,7 @@ export class DefaultCrudRepository<T extends Entity, ID>
   }
 
   protected toEntity(model: juggler.PersistedModel): T {
-    return new this.entityClass(model.toObject()) as T;
+    return new this.entityClass(model.toJSON()) as T;
   }
 
   protected toEntities(models: juggler.PersistedModel[]): T[] {

diff --git a/packages/repository/test/unit/repositories/legacy-juggler-bridge.unit.ts b/packages/repository/test/unit/repositories/legacy-juggler-bridge.unit.ts
@@ -49,12 +49,17 @@ describe('DefaultCrudRepository', () => {
       properties: {
         title: 'string',
         content: 'string',
+        secret: 'string',
         id: {name: 'id', type: 'number', id: true},
       },
+      settings: {
+        hiddenProperties: ['secret'],
+      },
     });
 
     title?: string;
     content?: string;
+    secret?: string;
     id: number;
 
     constructor(data: Partial<Note>) {
@@ -137,6 +142,18 @@ describe('DefaultCrudRepository', () => {
     expect(result.toJSON()).to.eql(note.toJSON());
   });
 
+  it('hides hidden properties', async () => {
+    const repo = new DefaultCrudRepository(Note, ds);
+    const note = await repo.create({
+      title: 't3',
+      content: 'c3',
+      secret: 'secret',
+    });
+    expect(note.secret).to.be.undefined();
+    const result = await repo.findById(note.id);
+    expect(result.secret).to.be.undefined();
+  });
+
   it('implements Repository.createAll()', async () => {
     const repo = new DefaultCrudRepository(Note, ds);
     const notes = await repo.createAll([