Browse Source

fix: post-processing for user field

pull/7373/head
mertmit 11 months ago
parent
commit
e19efc0bba
  1. 28
      packages/nocodb/src/db/BaseModelSqlv2.ts

28
packages/nocodb/src/db/BaseModelSqlv2.ts

@ -4719,7 +4719,7 @@ class BaseModelSqlv2 {
await this.model.getColumns(); await this.model.getColumns();
} }
const userColumns = []; let userColumns = [];
const columns = childTable ? childTable.columns : this.model.columns; const columns = childTable ? childTable.columns : this.model.columns;
@ -4743,6 +4743,19 @@ class BaseModelSqlv2 {
} }
} }
// filter user columns that are not present in data
if (userColumns.length) {
if (Array.isArray(data)) {
const row = data[0];
if (row) {
userColumns = userColumns.filter((col) => col.id in row);
}
} else {
userColumns = userColumns.filter((col) => col.id in data);
}
}
// process user columns that are present in data
if (userColumns.length) { if (userColumns.length) {
const baseUsers = await BaseUser.getUsersList({ const baseUsers = await BaseUser.getUsersList({
base_id: childTable ? childTable.base_id : this.model.base_id, base_id: childTable ? childTable.base_id : this.model.base_id,
@ -4767,14 +4780,12 @@ class BaseModelSqlv2 {
) { ) {
try { try {
if (d) { if (d) {
for (const col of userColumns) { const availableUserColumns = userColumns.filter(
if (d[col.id] && d[col.id].length) { (col) => d[col.id] && d[col.id].length,
);
for (const col of availableUserColumns) {
d[col.id] = d[col.id].split(','); d[col.id] = d[col.id].split(',');
} else {
d[col.id] = null;
}
if (d[col.id]?.length) {
d[col.id] = d[col.id].map((fid) => { d[col.id] = d[col.id].map((fid) => {
const { id, email, display_name } = baseUsers.find( const { id, email, display_name } = baseUsers.find(
(u) => u.id === fid, (u) => u.id === fid,
@ -4785,11 +4796,10 @@ class BaseModelSqlv2 {
display_name: display_name?.length ? display_name : null, display_name: display_name?.length ? display_name : null,
}; };
}); });
}
// CreatedBy and LastModifiedBy are always singular // CreatedBy and LastModifiedBy are always singular
if ([UITypes.CreatedBy, UITypes.LastModifiedBy].includes(col.uidt)) { if ([UITypes.CreatedBy, UITypes.LastModifiedBy].includes(col.uidt)) {
d[col.id] = d[col.id]?.[0] ?? null; d[col.id] = d[col.id][0];
} }
} }
} }

Loading…
Cancel
Save