github
/
nocodb
mirror of https://github.com/nocodb/nocodb
1
0
Fork 0
Code Issues Releases Wiki Activity
Browse Source

fix: use asId prop to override as selector instead of modifying column obj (#9493)

pull/9519/head
Mert E. 2 months ago committed by GitHub
parent
c194d55bec
commit
b64809dbb7
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 107 additions and 96 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Unified View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 199
      packages/nocodb/src/db/BaseModelSqlv2.ts
  2. 4
      packages/nocodb/src/models/Column.ts

199
packages/nocodb/src/db/BaseModelSqlv2.ts
Unescape View File

@ -165,6 +165,10 @@ export async function getColumnName(
} }
} }
export function getAs(column: Column) {
return column.asId || column.id;
}
export function replaceDynamicFieldWithValue( export function replaceDynamicFieldWithValue(
_row: any, _row: any,
_rowId, _rowId,
@ -814,12 +818,11 @@ class BaseModelSqlv2 {
...(await column ...(await column
.getColOptions<BarcodeColumn | QrCodeColumn>(this.context) .getColOptions<BarcodeColumn | QrCodeColumn>(this.context)
.then((col) => col.getValueColumn(this.context))), .then((col) => col.getValueColumn(this.context))),
title: column.title, asId: column.id,
id: column.id,
}); });
} }
groupByColumns[column.id] = column; groupByColumns[getAs(column)] = column;
switch (column.uidt) { switch (column.uidt) {
case UITypes.Attachment: case UITypes.Attachment:
@ -844,9 +847,9 @@ class BaseModelSqlv2 {
this.context, this.context,
)) as RollupColumn, )) as RollupColumn,
}) })
).builder.as(column.id), ).builder.as(getAs(column)),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
case UITypes.Formula: { case UITypes.Formula: {
let selectQb; let selectQb;
@ -856,14 +859,14 @@ class BaseModelSqlv2 {
); );
selectQb = this.dbDriver.raw(`?? as ??`, [ selectQb = this.dbDriver.raw(`?? as ??`, [
_selectQb.builder, _selectQb.builder,
column.id, getAs(column),
]); ]);
} catch (e) { } catch (e) {
console.log(e); console.log(e);
selectQb = this.dbDriver.raw(`'ERR' as ??`, [column.id]); selectQb = this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]);
} }
colSelectors.push(selectQb); colSelectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
@ -878,10 +881,10 @@ class BaseModelSqlv2 {
}); });
const selectQb = this.dbDriver.raw(`?? as ??`, [ const selectQb = this.dbDriver.raw(`?? as ??`, [
this.dbDriver.raw(_selectQb.builder).wrap('(', ')'), this.dbDriver.raw(_selectQb.builder).wrap('(', ')'),
column.id, getAs(column),
]); ]);
colSelectors.push(selectQb); colSelectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
case UITypes.DateTime: case UITypes.DateTime:
@ -898,7 +901,7 @@ class BaseModelSqlv2 {
colSelectors.push( colSelectors.push(
this.dbDriver.raw( this.dbDriver.raw(
"date_trunc('minute', ??) + interval '0 seconds' as ??", "date_trunc('minute', ??) + interval '0 seconds' as ??",
[columnName, column.id], [columnName, getAs(column)],
), ),
); );
} else if ( } else if (
@ -906,10 +909,10 @@ class BaseModelSqlv2 {
this.dbDriver.clientType() === 'mysql2' this.dbDriver.clientType() === 'mysql2'
) { ) {
colSelectors.push( colSelectors.push(
// this.dbDriver.raw('??::date as ??', [columnName, column.id]), // this.dbDriver.raw('??::date as ??', [columnName, getAs(column)]),
this.dbDriver.raw( this.dbDriver.raw(
"DATE_SUB(CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00'), INTERVAL SECOND(??) SECOND) as ??", "DATE_SUB(CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00'), INTERVAL SECOND(??) SECOND) as ??",
[columnName, columnName, column.id], [columnName, columnName, getAs(column)],
), ),
); );
} else if (this.dbDriver.clientType() === 'sqlite3') { } else if (this.dbDriver.clientType() === 'sqlite3') {
@ -930,7 +933,7 @@ class BaseModelSqlv2 {
END) AS :id:`, END) AS :id:`,
{ {
column: columnName, column: columnName,
id: column.id, id: getAs(column),
}, },
), ),
); );
@ -938,11 +941,11 @@ class BaseModelSqlv2 {
colSelectors.push( colSelectors.push(
this.dbDriver.raw('DATE(??) as ??', [ this.dbDriver.raw('DATE(??) as ??', [
columnName, columnName,
column.id, getAs(column),
]), ]),
); );
} }
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
default: { default: {
@ -952,9 +955,9 @@ class BaseModelSqlv2 {
columns, columns,
); );
colSelectors.push( colSelectors.push(
this.dbDriver.raw('?? as ??', [columnName, column.id]), this.dbDriver.raw('?? as ??', [columnName, getAs(column)]),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
} }
@ -1133,12 +1136,11 @@ class BaseModelSqlv2 {
...(await column ...(await column
.getColOptions<BarcodeColumn | QrCodeColumn>(this.context) .getColOptions<BarcodeColumn | QrCodeColumn>(this.context)
.then((col) => col.getValueColumn(this.context))), .then((col) => col.getValueColumn(this.context))),
title: column.title, asId: column.id,
id: column.id,
}); });
} }
groupByColumns[column.id] = column; groupByColumns[getAs(column)] = column;
switch (column.uidt) { switch (column.uidt) {
case UITypes.Attachment: case UITypes.Attachment:
@ -1163,9 +1165,9 @@ class BaseModelSqlv2 {
this.context, this.context,
)) as RollupColumn, )) as RollupColumn,
}) })
).builder.as(column.id), ).builder.as(getAs(column)),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
case UITypes.Formula: { case UITypes.Formula: {
let selectQb; let selectQb;
@ -1175,14 +1177,14 @@ class BaseModelSqlv2 {
); );
selectQb = this.dbDriver.raw(`?? as ??`, [ selectQb = this.dbDriver.raw(`?? as ??`, [
_selectQb.builder, _selectQb.builder,
column.id, getAs(column),
]); ]);
} catch (e) { } catch (e) {
console.log(e); console.log(e);
selectQb = this.dbDriver.raw(`'ERR' as ??`, [column.id]); selectQb = this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]);
} }
colSelectors.push(selectQb); colSelectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
@ -1197,10 +1199,10 @@ class BaseModelSqlv2 {
}); });
const selectQb = this.dbDriver.raw(`?? as ??`, [ const selectQb = this.dbDriver.raw(`?? as ??`, [
this.dbDriver.raw(_selectQb.builder).wrap('(', ')'), this.dbDriver.raw(_selectQb.builder).wrap('(', ')'),
column.id, getAs(column),
]); ]);
colSelectors.push(selectQb); colSelectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
case UITypes.DateTime: case UITypes.DateTime:
@ -1217,7 +1219,7 @@ class BaseModelSqlv2 {
colSelectors.push( colSelectors.push(
this.dbDriver.raw( this.dbDriver.raw(
"date_trunc('minute', ??) + interval '0 seconds' as ??", "date_trunc('minute', ??) + interval '0 seconds' as ??",
[columnName, column.id], [columnName, getAs(column)],
), ),
); );
} else if ( } else if (
@ -1225,10 +1227,10 @@ class BaseModelSqlv2 {
this.dbDriver.clientType() === 'mysql2' this.dbDriver.clientType() === 'mysql2'
) { ) {
colSelectors.push( colSelectors.push(
// this.dbDriver.raw('??::date as ??', [columnName, column.id]), // this.dbDriver.raw('??::date as ??', [columnName, getAs(column)]),
this.dbDriver.raw( this.dbDriver.raw(
"DATE_SUB(CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00'), INTERVAL SECOND(??) SECOND) as ??", "DATE_SUB(CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00'), INTERVAL SECOND(??) SECOND) as ??",
[columnName, columnName, column.id], [columnName, columnName, getAs(column)],
), ),
); );
} else if (this.dbDriver.clientType() === 'sqlite3') { } else if (this.dbDriver.clientType() === 'sqlite3') {
@ -1249,7 +1251,7 @@ class BaseModelSqlv2 {
END) AS :id:`, END) AS :id:`,
{ {
column: columnName, column: columnName,
id: column.id, id: getAs(column),
}, },
), ),
); );
@ -1257,11 +1259,11 @@ class BaseModelSqlv2 {
colSelectors.push( colSelectors.push(
this.dbDriver.raw('DATE(??) as ??', [ this.dbDriver.raw('DATE(??) as ??', [
columnName, columnName,
column.id, getAs(column),
]), ]),
); );
} }
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
default: { default: {
@ -1271,9 +1273,9 @@ class BaseModelSqlv2 {
columns, columns,
); );
colSelectors.push( colSelectors.push(
this.dbDriver.raw('?? as ??', [columnName, column.id]), this.dbDriver.raw('?? as ??', [columnName, getAs(column)]),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
} }
@ -1383,7 +1385,7 @@ class BaseModelSqlv2 {
); );
} else { } else {
tQb.orderBy( tQb.orderBy(
column.id, getAs(column),
sort.direction, sort.direction,
sort.direction === 'desc' ? 'LAST' : 'FIRST', sort.direction === 'desc' ? 'LAST' : 'FIRST',
); );
@ -1797,11 +1799,10 @@ class BaseModelSqlv2 {
...(await column ...(await column
.getColOptions<BarcodeColumn | QrCodeColumn>(this.context) .getColOptions<BarcodeColumn | QrCodeColumn>(this.context)
.then((col) => col.getValueColumn(this.context))), .then((col) => col.getValueColumn(this.context))),
title: column.title, asId: column.id,
id: column.id,
}); });
groupByColumns[column.id] = column; groupByColumns[getAs(column)] = column;
switch (column.uidt) { switch (column.uidt) {
case UITypes.Attachment: case UITypes.Attachment:
@ -1827,9 +1828,9 @@ class BaseModelSqlv2 {
this.context, this.context,
)) as RollupColumn, )) as RollupColumn,
}) })
).builder.as(column.id), ).builder.as(getAs(column)),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
case UITypes.Formula: case UITypes.Formula:
{ {
@ -1841,16 +1842,16 @@ class BaseModelSqlv2 {
selectQb = this.dbDriver.raw(`?? as ??`, [ selectQb = this.dbDriver.raw(`?? as ??`, [
_selectQb.builder, _selectQb.builder,
column.id, getAs(column),
]); ]);
} catch (e) { } catch (e) {
logger.log(e); logger.log(e);
// return dummy select // return dummy select
selectQb = this.dbDriver.raw(`'ERR' as ??`, [column.id]); selectQb = this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]);
} }
selectors.push(selectQb); selectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
case UITypes.Lookup: case UITypes.Lookup:
@ -1866,11 +1867,11 @@ class BaseModelSqlv2 {
const selectQb = this.dbDriver.raw(`?? as ??`, [ const selectQb = this.dbDriver.raw(`?? as ??`, [
this.dbDriver.raw(_selectQb.builder).wrap('(', ')'), this.dbDriver.raw(_selectQb.builder).wrap('(', ')'),
column.id, getAs(column),
]); ]);
selectors.push(selectQb); selectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
case UITypes.CreatedTime: case UITypes.CreatedTime:
@ -1887,7 +1888,7 @@ class BaseModelSqlv2 {
selectors.push( selectors.push(
this.dbDriver.raw( this.dbDriver.raw(
"date_trunc('minute', ??) + interval '0 seconds' as ??", "date_trunc('minute', ??) + interval '0 seconds' as ??",
[columnName, column.id], [columnName, getAs(column)],
), ),
); );
} else if ( } else if (
@ -1895,10 +1896,10 @@ class BaseModelSqlv2 {
this.dbDriver.clientType() === 'mysql2' this.dbDriver.clientType() === 'mysql2'
) { ) {
selectors.push( selectors.push(
// this.dbDriver.raw('??::date as ??', [columnName, column.id]), // this.dbDriver.raw('??::date as ??', [columnName, getAs(column)]),
this.dbDriver.raw( this.dbDriver.raw(
"DATE_SUB(CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00'), INTERVAL SECOND(??) SECOND) as ??", "DATE_SUB(CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00'), INTERVAL SECOND(??) SECOND) as ??",
[columnName, columnName, column.id], [columnName, columnName, getAs(column)],
), ),
); );
} else if (this.dbDriver.clientType() === 'sqlite3') { } else if (this.dbDriver.clientType() === 'sqlite3') {
@ -1919,16 +1920,19 @@ class BaseModelSqlv2 {
END) AS :id:`, END) AS :id:`,
{ {
column: columnName, column: columnName,
id: column.id, id: getAs(column),
}, },
), ),
); );
} else { } else {
selectors.push( selectors.push(
this.dbDriver.raw('DATE(??) as ??', [columnName, column.id]), this.dbDriver.raw('DATE(??) as ??', [
columnName,
getAs(column),
]),
); );
} }
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
default: default:
@ -1939,9 +1943,9 @@ class BaseModelSqlv2 {
columns, columns,
); );
selectors.push( selectors.push(
this.dbDriver.raw('?? as ??', [columnName, column.id]), this.dbDriver.raw('?? as ??', [columnName, getAs(column)]),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
} }
@ -2055,7 +2059,7 @@ class BaseModelSqlv2 {
); );
} else { } else {
qb.orderBy( qb.orderBy(
column.id, getAs(column),
sort.direction, sort.direction,
sort.direction === 'desc' ? 'LAST' : 'FIRST', sort.direction === 'desc' ? 'LAST' : 'FIRST',
); );
@ -2104,8 +2108,7 @@ class BaseModelSqlv2 {
...(await column ...(await column
.getColOptions<BarcodeColumn | QrCodeColumn>(this.context) .getColOptions<BarcodeColumn | QrCodeColumn>(this.context)
.then((col) => col.getValueColumn(this.context))), .then((col) => col.getValueColumn(this.context))),
title: column.title, asId: column.id,
id: column.id,
}); });
switch (column.uidt) { switch (column.uidt) {
@ -2132,9 +2135,9 @@ class BaseModelSqlv2 {
this.context, this.context,
)) as RollupColumn, )) as RollupColumn,
}) })
).builder.as(column.id), ).builder.as(getAs(column)),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
case UITypes.Formula: { case UITypes.Formula: {
let selectQb; let selectQb;
@ -2145,16 +2148,16 @@ class BaseModelSqlv2 {
selectQb = this.dbDriver.raw(`?? as ??`, [ selectQb = this.dbDriver.raw(`?? as ??`, [
_selectQb.builder, _selectQb.builder,
column.id, getAs(column),
]); ]);
} catch (e) { } catch (e) {
logger.log(e); logger.log(e);
// return dummy select // return dummy select
selectQb = this.dbDriver.raw(`'ERR' as ??`, [column.id]); selectQb = this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]);
} }
selectors.push(selectQb); selectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
break; break;
} }
case UITypes.Lookup: case UITypes.Lookup:
@ -2170,11 +2173,11 @@ class BaseModelSqlv2 {
const selectQb = this.dbDriver.raw(`?? as ??`, [ const selectQb = this.dbDriver.raw(`?? as ??`, [
this.dbDriver.raw(_selectQb.builder).wrap('(', ')'), this.dbDriver.raw(_selectQb.builder).wrap('(', ')'),
column.id, getAs(column),
]); ]);
selectors.push(selectQb); selectors.push(selectQb);
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
case UITypes.CreatedTime: case UITypes.CreatedTime:
@ -2191,7 +2194,7 @@ class BaseModelSqlv2 {
selectors.push( selectors.push(
this.dbDriver.raw( this.dbDriver.raw(
"date_trunc('minute', ??) + interval '0 seconds' as ??", "date_trunc('minute', ??) + interval '0 seconds' as ??",
[columnName, column.id], [columnName, getAs(column)],
), ),
); );
} else if ( } else if (
@ -2201,7 +2204,7 @@ class BaseModelSqlv2 {
selectors.push( selectors.push(
this.dbDriver.raw( this.dbDriver.raw(
"CONVERT_TZ(DATE_SUB(??, INTERVAL SECOND(??) SECOND), @@GLOBAL.time_zone, '+00:00') as ??", "CONVERT_TZ(DATE_SUB(??, INTERVAL SECOND(??) SECOND), @@GLOBAL.time_zone, '+00:00') as ??",
[columnName, columnName, column.id], [columnName, columnName, getAs(column)],
), ),
); );
} else if (this.dbDriver.clientType() === 'sqlite3') { } else if (this.dbDriver.clientType() === 'sqlite3') {
@ -2222,16 +2225,19 @@ class BaseModelSqlv2 {
END) as :id:`, END) as :id:`,
{ {
column: columnName, column: columnName,
id: column.id, id: getAs(column),
}, },
), ),
); );
} else { } else {
selectors.push( selectors.push(
this.dbDriver.raw('DATE(??) as ??', [columnName, column.id]), this.dbDriver.raw('DATE(??) as ??', [
columnName,
getAs(column),
]),
); );
} }
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
default: default:
@ -2242,9 +2248,9 @@ class BaseModelSqlv2 {
columns, columns,
); );
selectors.push( selectors.push(
this.dbDriver.raw('?? as ??', [columnName, column.id]), this.dbDriver.raw('?? as ??', [columnName, getAs(column)]),
); );
groupBySelectors.push(column.id); groupBySelectors.push(getAs(column));
} }
break; break;
} }
@ -4154,7 +4160,7 @@ class BaseModelSqlv2 {
// the value 2023-01-01 10:00:00 (UTC) would display as 2023-01-01 18:00:00 (UTC+8) // the value 2023-01-01 10:00:00 (UTC) would display as 2023-01-01 18:00:00 (UTC+8)
// our existing logic is based on UTC, during the query, we need to take the UTC value // our existing logic is based on UTC, during the query, we need to take the UTC value
// hence, we use CONVERT_TZ to convert back to UTC value // hence, we use CONVERT_TZ to convert back to UTC value
res[sanitize(column.id || columnName)] = this.dbDriver.raw( res[sanitize(getAs(column) || columnName)] = this.dbDriver.raw(
`CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00')`, `CONVERT_TZ(??, @@GLOBAL.time_zone, '+00:00')`,
[`${sanitize(alias || this.tnPath)}.${columnName}`], [`${sanitize(alias || this.tnPath)}.${columnName}`],
); );
@ -4167,7 +4173,7 @@ class BaseModelSqlv2 {
column.dt !== 'timestamp with time zone' && column.dt !== 'timestamp with time zone' &&
column.dt !== 'timestamptz' column.dt !== 'timestamptz'
) { ) {
res[sanitize(column.id || columnName)] = this.dbDriver res[sanitize(getAs(column) || columnName)] = this.dbDriver
.raw( .raw(
`?? AT TIME ZONE CURRENT_SETTING('timezone') AT TIME ZONE 'UTC'`, `?? AT TIME ZONE CURRENT_SETTING('timezone') AT TIME ZONE 'UTC'`,
[`${sanitize(alias || this.tnPath)}.${columnName}`], [`${sanitize(alias || this.tnPath)}.${columnName}`],
@ -4180,14 +4186,14 @@ class BaseModelSqlv2 {
// convert to database timezone, // convert to database timezone,
// then convert to UTC // then convert to UTC
if (column.dt !== 'datetimeoffset') { if (column.dt !== 'datetimeoffset') {
res[sanitize(column.id || columnName)] = this.dbDriver.raw( res[sanitize(getAs(column) || columnName)] = this.dbDriver.raw(
`CONVERT(DATETIMEOFFSET, ?? AT TIME ZONE 'UTC')`, `CONVERT(DATETIMEOFFSET, ?? AT TIME ZONE 'UTC')`,
[`${sanitize(alias || this.tnPath)}.${columnName}`], [`${sanitize(alias || this.tnPath)}.${columnName}`],
); );
break; break;
} }
} }
res[sanitize(column.id || columnName)] = sanitize( res[sanitize(getAs(column) || columnName)] = sanitize(
`${alias || this.tnPath}.${columnName}`, `${alias || this.tnPath}.${columnName}`,
); );
} }
@ -4201,7 +4207,7 @@ class BaseModelSqlv2 {
); );
if (!qrCodeColumn.fk_qr_value_column_id) { if (!qrCodeColumn.fk_qr_value_column_id) {
qb.select(this.dbDriver.raw(`? as ??`, ['ERR!', column.id])); qb.select(this.dbDriver.raw(`? as ??`, ['ERR!', getAs(column)]));
break; break;
} }
@ -4244,7 +4250,7 @@ class BaseModelSqlv2 {
); );
if (!barcodeColumn.fk_barcode_value_column_id) { if (!barcodeColumn.fk_barcode_value_column_id) {
qb.select(this.dbDriver.raw(`? as ??`, ['ERR!', column.id])); qb.select(this.dbDriver.raw(`? as ??`, ['ERR!', getAs(column)]));
break; break;
} }
@ -4267,7 +4273,7 @@ class BaseModelSqlv2 {
aliasToColumnBuilder, aliasToColumnBuilder,
); );
qb.select({ qb.select({
[column.id]: selectQb.builder, [getAs(column)]: selectQb.builder,
}); });
} catch { } catch {
continue; continue;
@ -4275,7 +4281,7 @@ class BaseModelSqlv2 {
break; break;
default: { default: {
qb.select({ qb.select({
[column.id]: barcodeValueColumn.column_name, [getAs(column)]: barcodeValueColumn.column_name,
}); });
break; break;
} }
@ -4293,12 +4299,15 @@ class BaseModelSqlv2 {
aliasToColumnBuilder, aliasToColumnBuilder,
); );
qb.select( qb.select(
this.dbDriver.raw(`?? as ??`, [selectQb.builder, column.id]), this.dbDriver.raw(`?? as ??`, [
selectQb.builder,
getAs(column),
]),
); );
} catch (e) { } catch (e) {
logger.log(e); logger.log(e);
// return dummy select // return dummy select
qb.select(this.dbDriver.raw(`'ERR' as ??`, [column.id])); qb.select(this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]));
} }
} }
break; break;
@ -4321,7 +4330,7 @@ class BaseModelSqlv2 {
colOption.type, colOption.type,
`${colOption.label}`, `${colOption.label}`,
selectQb.builder, selectQb.builder,
column.id, getAs(column),
], ],
), ),
); );
@ -4334,7 +4343,7 @@ class BaseModelSqlv2 {
colOption.type, colOption.type,
`${colOption.label}`, `${colOption.label}`,
selectQb.builder, selectQb.builder,
column.id, getAs(column),
], ],
), ),
); );
@ -4347,13 +4356,13 @@ class BaseModelSqlv2 {
colOption.type, colOption.type,
`${colOption.label}`, `${colOption.label}`,
selectQb.builder, selectQb.builder,
column.id, getAs(column),
], ],
), ),
); );
break; break;
default: default:
qb.select(this.dbDriver.raw(`'ERR' as ??`, [column.id])); qb.select(this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]));
} }
} else if (colOption.type === 'webhook') { } else if (colOption.type === 'webhook') {
switch (this.dbDriver.client.config.client) { switch (this.dbDriver.client.config.client) {
@ -4365,7 +4374,7 @@ class BaseModelSqlv2 {
colOption.type, colOption.type,
`${colOption.label}`, `${colOption.label}`,
colOption.fk_webhook_id, colOption.fk_webhook_id,
column.id, getAs(column),
], ],
), ),
); );
@ -4378,7 +4387,7 @@ class BaseModelSqlv2 {
colOption.type, colOption.type,
`${colOption.label}`, `${colOption.label}`,
colOption.fk_webhook_id, colOption.fk_webhook_id,
column.id, getAs(column),
], ],
), ),
); );
@ -4391,19 +4400,19 @@ class BaseModelSqlv2 {
colOption.type, colOption.type,
`${colOption.label}`, `${colOption.label}`,
colOption.fk_webhook_id, colOption.fk_webhook_id,
column.id, getAs(column),
], ],
), ),
); );
break; break;
default: default:
qb.select(this.dbDriver.raw(`'ERR' as ??`, [column.id])); qb.select(this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]));
} }
} }
} catch (e) { } catch (e) {
logger.log(e); logger.log(e);
// return dummy select // return dummy select
qb.select(this.dbDriver.raw(`'ERR' as ??`, [column.id])); qb.select(this.dbDriver.raw(`'ERR' as ??`, [getAs(column)]));
} }
break; break;
} }
@ -4421,7 +4430,7 @@ class BaseModelSqlv2 {
this.context, this.context,
)) as RollupColumn, )) as RollupColumn,
}) })
).builder.as(column.id), ).builder.as(getAs(column)),
); );
break; break;
case UITypes.CreatedBy: case UITypes.CreatedBy:
@ -4432,7 +4441,7 @@ class BaseModelSqlv2 {
_columns || (await this.model.getColumns(this.context)), _columns || (await this.model.getColumns(this.context)),
); );
res[sanitize(column.id || columnName)] = sanitize( res[sanitize(getAs(column) || columnName)] = sanitize(
`${alias || this.tnPath}.${columnName}`, `${alias || this.tnPath}.${columnName}`,
); );
break; break;
@ -4440,7 +4449,7 @@ class BaseModelSqlv2 {
default: default:
if (this.isPg) { if (this.isPg) {
if (column.dt === 'bytea') { if (column.dt === 'bytea') {
res[sanitize(column.id || column.column_name)] = res[sanitize(getAs(column) || column.column_name)] =
this.dbDriver.raw( this.dbDriver.raw(
`encode(??.??, '${ `encode(??.??, '${
column.meta?.format === 'hex' ? 'hex' : 'escape' column.meta?.format === 'hex' ? 'hex' : 'escape'
@ -4451,7 +4460,7 @@ class BaseModelSqlv2 {
} }
} }
res[sanitize(column.id || column.column_name)] = sanitize( res[sanitize(getAs(column) || column.column_name)] = sanitize(
`${alias || this.tnPath}.${column.column_name}`, `${alias || this.tnPath}.${column.column_name}`,
); );
break; break;

4
packages/nocodb/src/models/Column.ts
Unescape View File

@ -97,7 +97,9 @@ export default class Column<T = any> implements ColumnType {
public validate: any; public validate: any;
public meta: any; public meta: any;
constructor(data: Partial<ColumnType | Column>) { public asId?: string;
constructor(data: Partial<(ColumnType & { asId?: string }) | Column>) {
Object.assign(this, data); Object.assign(this, data);
} }

Write Preview
Loading…
Cancel
Save