Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

Node 4.4.5 (METEOR) - Chyba při definici: let {pkp, bkp} #15

Closed
BenjaminMerlynS opened this issue Feb 28, 2017 · 7 comments
Closed

Node 4.4.5 (METEOR) - Chyba při definici: let {pkp, bkp} #15

BenjaminMerlynS opened this issue Feb 28, 2017 · 7 comments

Comments

@BenjaminMerlynS
Copy link
Contributor

Ahoj ! Zaprví moc děkuji za modul, nevim co bych bez něho dělal...

Pracuji s frameworkem Meteor.js který běží na Node 4.4.5. Když spouštím funkci eet(options, items), tak se mi zastaví server a zobrazí se mi chyba při definici
let {pkp, bkp} = getFooterItems(options.privateKey, items)

Chyba:
SyntaxError: Unexpected token {

Jestli správně chápu tak Node/Meteor nedokažou přečíst definici proměné v tomto formátu

Máte někdo nápad co spůsobuje tuto chybu? Čili jestli máte řešení?

Díky moc !

@BenjaminMerlynS BenjaminMerlynS changed the title Node 4.4.5 (METEOR) - Chyba při definice: let {pkp, bkp} Node 4.4.5 (METEOR) - Chyba při definici: let {pkp, bkp} Feb 28, 2017
@JakubMrozek
Copy link
Owner

JakubMrozek commented Feb 28, 2017

Problém je v tom, že Node.js 4.x neumí destruktury.

Existuji tři možnosti, jak to řešit:
a) upgrade na Node.js 6+
b) spouštět node.js s harmony přiznakem jako node --harmony_destructuring server.js
c) forknout repo a upravit tuto část např. takto:

let code = getFooterItems(options.privateKey, items)
let bkp = code.bkp
let pkp = code.pkp

@BenjaminMerlynS
Copy link
Contributor Author

Děkuji za opdověď a rešení!
Je to co jsem si myslel !

Bohužel nejde Node v Meteoru updatnout... To musejí oni s pokrokem verzi. Stejně tak neni možné zvlášť spouštět Node. Takže forknu repo !

Jestli je zájem tak rád udělám Pull Request se změnou pro kompatibilitu s Node v4 !

@mfeuermann
Copy link

Ahoj to by bylo fajn, urcite bych taky vyuzil...

@omasopust
Copy link

+1, taky bych to využil.

@omasopust
Copy link

@BenjaminMerlynS ahoj, jak to vypadá s PR? Bude, nebo se do toho mám pustit?

@BenjaminMerlynS
Copy link
Contributor Author

BenjaminMerlynS commented Mar 4, 2017

@omasopust Omlouvám se za spoždění... Tento týden jsem nestíhal !

#16 Adaptace Node v4

@JakubMrozek
Copy link
Owner

Doplněno ve v0.7, díky.

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

No branches or pull requests

4 participants