postgres arrays are stringified wrong

[ArdentZeal]

Hello,

I am using knex with a postgres database.

To build a subquery I am using knex to build the subquery. I am then using toString() method to put the query part into a strucd.knex.raw() which again is in a knex chain for the outer query. See full example below.

...
knex.where(columnName, '&&', [1,2,3])
...
.toString()

knex turns the array into a valid postgresarray '{1,2,3}', but as soon as i call toString() it will be '{"1","2","3"}'.

So at the moment I have to put a .replace(/"/g, '') after each toString() just to circumvent that problem.

I would prefer knex handling that situation correctly instead of me hacking around it.

Kind Regards

Full example

 _strucd.knex().debug()
                        .select(_strucd.knex.raw('date, ' + UserListDataController._buildGraphSelectStringFromElementsPostgresql(list, elementsWithEC, elementcategories, graphParameters)))
                        .from(_strucd.knex.raw(utils.__buildGenerateSeriesString(columnName, interval, generateIntervalMin, generateIntervalMax)))
                        .leftOuterJoin(
                            _strucd.knex.raw(
                                '(' +
                                _strucd.knex(tableName)
                                    .select(_strucd.knex.raw(utils.__buildGraphSubquerySelectDateString(columnName, interval) + ', ' + UserListDataController._buildGraphSubquerySelectStringFromElementsPostgresql(list, elementsWithEC, elementcategories, graphParameters)))
                                    .whereNull('deprecated_at')
                                    .where(function() {
                                        UserListDataController.__buildFilterQuery(this, findParams.filter, list, elementsWithEC, elementcategories);
                                        UserListDataController.__buildInstantSearchQuery(this, findParams.instantSearch, list, elementsWithEC, elementcategories);
                                    })
                                    .groupBy('interval').toString().replace(/\\"/g, '') +
                                    ') results'
                            ),
                            'date', '=', 'results.interval'
                        )
                        .then(function(results) {
                            console.log(results);
                            return callback(null, results);
                        })
                        .catch(function(err) {
                            return callback(err);
                        });

[wubzz]

Seems fixed according to commits and tests on knexjs.org