feat: remove id virtual getters

bluehalo · Nov 4, 2024 · 5669d44 · 5669d44
1 parent acbca3c
commit 5669d44
Show file tree

Hide file tree

Showing 8 changed files with 22 additions and 14 deletions.
diff --git a/src/app/core/audit/audit.model.ts b/src/app/core/audit/audit.model.ts
@@ -61,6 +61,7 @@ const AuditSchema = new Schema<
 		}
 	},
 	{
+		id: false,
 		timestamps: {
 			createdAt: 'created',
 			updatedAt: false

diff --git a/src/app/core/audit/audit.service.spec.ts b/src/app/core/audit/audit.service.spec.ts
@@ -47,7 +47,7 @@ describe('Audit Service:', () => {
 			assert(Array.isArray(results), 'results should be an Array');
 			assert.equal(results.length, 1);
 
-			const { created, _id, ...result } = results[0].toObject({
+			const { created, ...result } = results[0].toObject({
 				versionKey: false
 			});
 			/*
@@ -61,7 +61,7 @@ describe('Audit Service:', () => {
 			);
 
 			assert.deepStrictEqual(result, {
-				id: _id.toString(),
+				_id: result._id,
 				message: 'some message',
 				audit: {
 					auditType: 'eventType',

diff --git a/src/app/core/feedback/feedback.model.ts b/src/app/core/feedback/feedback.model.ts
@@ -69,6 +69,7 @@ const FeedbackSchema = new Schema<
 		assignee: { type: String }
 	},
 	{
+		id: false,
 		timestamps: {
 			createdAt: 'created',
 			updatedAt: 'updated'

diff --git a/src/app/core/teams/team-role.model.ts b/src/app/core/teams/team-role.model.ts
@@ -47,18 +47,21 @@ export interface ITeamRole {
  *         role:
  *           type: string
  */
-export const TeamRoleSchema = new Schema<ITeamRole>({
-	_id: {
-		type: Schema.Types.ObjectId,
-		ref: 'Team'
+export const TeamRoleSchema = new Schema<ITeamRole>(
+	{
+		_id: {
+			type: Schema.Types.ObjectId,
+			ref: 'Team'
+		},
+		role: {
+			type: String,
+			trim: true,
+			default: TeamRoles.Member,
+			enum: TeamRoles
+		}
 	},
-	role: {
-		type: String,
-		trim: true,
-		default: TeamRoles.Member,
-		enum: TeamRoles
-	}
-});
+	{ id: false }
+);
 
 TeamRoleSchema.plugin(getterPlugin);
 

diff --git a/src/app/core/teams/team.model.ts b/src/app/core/teams/team.model.ts
@@ -97,6 +97,7 @@ const TeamSchema = new Schema<
 		}
 	},
 	{
+		id: false,
 		timestamps: {
 			createdAt: 'created',
 			updatedAt: 'updated'

diff --git a/src/app/core/teams/teams.service.ts b/src/app/core/teams/teams.service.ts
@@ -187,7 +187,7 @@ class TeamsService {
 				// append isMember field to elements if user is part of the team
 				return {
 					...res.toJSON(),
-					isMember: teamIdStrings.includes(res.id)
+					isMember: teamIdStrings.includes(res._id.toString())
 				} as unknown as TeamDocument;
 			})
 		};

diff --git a/src/app/core/user/eua/eua.model.ts b/src/app/core/user/eua/eua.model.ts
@@ -64,6 +64,7 @@ const UserAgreementSchema = new Schema<
 		}
 	},
 	{
+		id: false,
 		timestamps: {
 			createdAt: 'created',
 			updatedAt: 'updated'

diff --git a/src/app/core/user/user.model.ts b/src/app/core/user/user.model.ts
@@ -247,6 +247,7 @@ const UserSchema = new Schema<
 		teams: [TeamRoleSchema]
 	},
 	{
+		id: false,
 		timestamps: {
 			createdAt: 'created',
 			updatedAt: 'updated'