save & destroyAll missing

[janober]

After testing changing some of my datasources to couchbase, loopback did always stop and exit with the message "stop here". After looking through the code in "lib/cb.js" I found the reason. It seems like that "save" (which I did use, Line 217) and "destroyAll" (Line 339) are not implemented yet.

[nolandubeau]

Hi yes this is correct. We have yet to complete those features. I will
check with our team tomorrow on timing for that, but feel free to submit a
PR should you want to contribute those methods.

On Tue, Jan 27, 2015 at 7:06 PM, Jan [email protected] wrote:

After testing changing some of my datasources to couchbase, loopback did
always stop and exit with the message "stop here". After looking through
the code in "lib/cb.js" I found the reason. It seems like that "save"
(which I did use, Line 217) and "destroyAll" (Line 339) are not implemented
yet.

—
Reply to this email directly or view it on GitHub
#6.

_Nolan Dubeau_VP, Engineering, Guardly Corp.
Online: https://www.guardly.com
LinkedIn: http://www.linkedin.com/in/nolandubeau
20 Maud Street, Suite 307, Toronto, ON M5V 2M5
CA (416) 434-8029 | US (650) 262-3599 | Skype: nolandubeau
Please consider the environment before printing this email.

[janober]

I added at least the "save" it looked simple enough. Is in my fork. Sorry should probably have created a different branch because do not see a way to create a pull request as long as the other one is still open. Do sadly have not to much experience with github... Maybe next time directly a branch...

With the destroyAll. I was not sure how to most efficiently get all the instances so I left it before I code total crap which you can then not use in the end anyway. Hope at least the save is usefull.

[nolandubeau]

Great. If you want to post the code for the save() I can get it merged
in. Will look into destroyAll() tomorrow. Thanks.

On Tue, Jan 27, 2015 at 8:24 PM, Jan [email protected] wrote:

I added at least the "save" it looked simple enough. Is in my fork. Sorry
should probably have created a different branch because do not see a way to
create a pull request as long as the other one is still open. Do sadly have
not to much experience with github... Maybe next time directly a branch...

With the destroyAll. I was not sure how to most efficiently get all the
instances so I left it before I code total crap which you can then not use
in the end anyway. Hope at least the save is usefull.

—
Reply to this email directly or view it on GitHub
#6 (comment)
.

_Nolan Dubeau_VP, Engineering, Guardly Corp.
Online: https://www.guardly.com
LinkedIn: http://www.linkedin.com/in/nolandubeau
20 Maud Street, Suite 307, Toronto, ON M5V 2M5
CA (416) 434-8029 | US (650) 262-3599 | Skype: nolandubeau
Please consider the environment before printing this email.

[janober]

The save code is here:
janober@69ba1cc

So "CouchbaseDB.prototype.save" from Line 217 and "CouchbaseDB.prototype.updateAttributes" Line 348. The other stuff is just WebStorm removing not needed spaces from the end of the lines.