Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sequelize delete property of meta object error #234

Open
knoxcard2 opened this issue Feb 18, 2023 · 7 comments
Open

Sequelize delete property of meta object error #234

knoxcard2 opened this issue Feb 18, 2023 · 7 comments

Comments

@knoxcard2
Copy link
Contributor

/etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/dialects/mariadb/query.js:110
delete data.meta;

TypeError: Cannot delete property 'meta' of [object Array]
at Query.formatResults (/etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/dialects/mariadb/query.js:110:7)
at Query.run (/etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/dialects/mariadb/query.js:73:17)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async /etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/sequelize.js:314:16

Node.js v19.6.1

@PeppeL-G
Copy link

This rather looks like a problem with the sequelize package?

@WikiRik
Copy link

WikiRik commented Feb 26, 2023

It is, sequelize does not support v3 of this connector yet. Almost all tests of the sequelize package pass on 3.0.2 but more fail on 3.1.0 due to the non-enumerable meta. The advice of the sequelize team is to not upgrade to v3 of the mariadb connector until v7 of sequelize

@rusher
Copy link
Collaborator

rusher commented Feb 27, 2023

do you have some link @WikiRik / @knoxcard2 ? correction is easy to do, and i can make a PR

@WikiRik
Copy link

WikiRik commented Feb 27, 2023

do you have some link @WikiRik / @knoxcard2 ? correction is easy to do, and i can make a PR

You can check out which tests are failing in this renovate PR; sequelize/sequelize#15311

@rusher
Copy link
Collaborator

rusher commented Jun 21, 2023

from Rik Smale :

Starting with v7.0.0-alpha27 (released soon) sequelize will support 3.x. See sequelize/sequelize#16139
Support for 3.x is not planned for sequelize v6 at this moment

@WikiRik
Copy link

WikiRik commented Jun 21, 2023

Ah yes, I forgot this issue was open as well. Thanks for posting it here!

@dcolley
Copy link

dcolley commented Jun 28, 2023

Until v3 of the connector is supported:

npm install --save mariadb@2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants