search

appy-one / acebase

A fast, low memory, transactional, index & query enabled NoSQL database engine and server for node.js and browser with realtime data change notifications
MIT License
491 stars 27 forks source link

Error: Unexpected token in storage-acebase.js #55

Closed holeq closed 2 years ago

holeq commented 2 years ago

Hello. I faced the problem in version 1.12.3. Node v13.10.1 on Windows 10

...\node_modules\acebase\src\storage-acebase.js:979 if (this.settings.transactions?.log !== true) { throw new Error('Transaction logging is not enabled'); } ^

SyntaxError: Unexpected token '.' at wrapSafe (internal/modules/cjs/loader.js:1063:16) at Module._compile (internal/modules/cjs/loader.js:1111:27) at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10) at Module.load (internal/modules/cjs/loader.js:996:32) at Function.Module._load (internal/modules/cjs/loader.js:896:14) at Module.require (internal/modules/cjs/loader.js:1036:19) at require (internal/modules/cjs/helpers.js:72:18) at Object. (D:\PROJECTS\njs-querize\node_modules\acebase\src\api-local.js:3:52) at Module._compile (internal/modules/cjs/loader.js:1147:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)

appy-one commented 2 years ago

The optional chaining operator ?. is supported by Node.js 14+. If you can update, do that. I will change the code to be backward compatible with older Node.js versions again but updating is your quickest solution for now

holeq commented 2 years ago

I hear you. Thank you.

appy-one commented 2 years ago

I published the fix for this in acebase v1.12.4