search

coresmart / persistencejs

persistence.js is an asynchronous Javascript database mapper library. You can use it in the browser, as well on the server (and you can share data models between them).
http://persistencejs.org
1.73k stars 240 forks source link

Untitled #18

Closed bjouhier closed 13 years ago

bjouhier commented 13 years ago

Hi Zef,

I have modified the count and destroyAll methods to handle mixins.

The destroyAll method does not seem to work with many to many relations. The generated SQL is invalid and I don't think that this is related to the code I added for mixins. I have added a unit test for destroyAll in test.mixin.js but I have commented it out because it did not pass.

I did not have any time to investigate this destroyAll problem so far and I wanted to check with you first. Is this something you want to investigate, or should I go ahead and try to fix it?

Bruno

zefhemel commented 13 years ago

Hi, sorry for the late reply. Had a paper deadline. I integrated your patch, thanks! If you want to go ahead and fix the destroyAll method that would be great, otherwise I can also look into it.