Skip to content

Commit

Permalink
fix(database): createWithPopulations looking for wrong properties (#245)
Browse files Browse the repository at this point in the history
  • Loading branch information
kkopanidis committed Jul 15, 2022
1 parent c852a5a commit b8c8c8c
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 12 deletions.
11 changes: 5 additions & 6 deletions modules/database/src/adapters/mongoose-adapter/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,7 @@ async function _createWithPopulations(

if (!fields.hasOwnProperty(key)) continue;
if (!isObject(fields[key])) continue;
if (fields[key].type?.schemaName === 'Mixed') continue;
if (fields[key].schemaName === 'Mixed') continue;
if (fields[key] === 'JSON' || fields[key].type === 'JSON') continue;

if (isArray(document[key])) {
for (let i = 0; i < document[key].length; i++) {
Expand All @@ -49,8 +48,8 @@ async function _createWithPopulations(
if (!isArray(field) && field.type && isArray(field.type)) {
field = field.type;
}
if (field[0].hasOwnProperty('ref')) {
const { model } = adapter.getSchemaModel(field[0].ref);
if (field[0].hasOwnProperty('model')) {
const { model } = adapter.getSchemaModel(field[0].model);
if (validate) {
await model.model.validate(val);
} else {
Expand All @@ -61,8 +60,8 @@ async function _createWithPopulations(
}
}
} else if (isObject(document[key])) {
if (fields[key].hasOwnProperty('ref')) {
const { model } = adapter.getSchemaModel(fields[key].ref);
if (fields[key].hasOwnProperty('model')) {
const { model } = adapter.getSchemaModel(fields[key].model);
if (validate) {
await model.model.validate(document[key]);
} else {
Expand Down
11 changes: 5 additions & 6 deletions modules/database/src/adapters/sequelize-adapter/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -121,17 +121,16 @@ async function _createWithPopulations(

if (!fields.hasOwnProperty(key)) continue;
if (!isObject(fields[key])) continue;
if (fields[key].type?.schemaName === 'Mixed') continue;
if (fields[key].schemaName === 'Mixed') continue;
if (fields[key] === 'JSON' || fields[key].type === 'JSON') continue;

if (isArray(document[key])) {
for (let i = 0; i < document[key].length; i++) {
const val = document[key][i];
if (!isObject(val)) {
continue;
}
if (fields[key][0].hasOwnProperty('ref')) {
const { model } = adapter.getSchemaModel(fields[key][0].ref);
if (fields[key][0].hasOwnProperty('model')) {
const { model } = adapter.getSchemaModel(fields[key][0].model);
if (validate) {
// await model.model.validate(val);
return;
Expand All @@ -143,8 +142,8 @@ async function _createWithPopulations(
}
}
} else if (isObject(document[key])) {
if (fields[key].hasOwnProperty('ref')) {
const { model } = adapter.getSchemaModel(fields[key].ref);
if (fields[key].hasOwnProperty('model')) {
const { model } = adapter.getSchemaModel(fields[key].model);
if (validate) {
// await model.model.validate(document[key]);
return;
Expand Down

0 comments on commit b8c8c8c

Please sign in to comment.