Support server methods about meteor-cfs-autoform HOT 5 CLOSED

I guess the thing to decide here is: what would one expect a form like this to do? Since it's submitting to a method, autoform doesn't have any idea what you're going to do with the doc. Would we expect it to go ahead and upload all the files anyway, and call the method only after all the files are inserted and their IDs are added to doc? Is that what you were expecting?

I think as long as we document what use cases are supported, this could be technically possible. Maybe the method could pass back false or throw an error to tell cfs-autoform to cancel the uploads and delete the files, if the insert/update failed on the server.

from meteor-cfs-autoform.

Thanks for the reply. I have it working in my app and its been a relatively simple change that I will try to contribute back. I'll throw my changes into a branch and you can see if you want to integrate any of them.

Yes. Keep the behavior the same as it currently is. Upload the files and delete if there is a problem, just like it currently does.

As for the error cases, you can throw an Meteor.Error object in the server method and that will be presented later in the autoform "after" event. So knowing whether to rollback shouldn't be any more difficult than the client. At least, in my testing that seems to work fine. That is consistent with the client-only code as well so there are not a bunch of corner cases for testing.

BTW, I had to overcome bug #9 for this to work on my system so I already have the code to fix that for you.

from meteor-cfs-autoform.

Great @dpankros. I'm working on autoform fixes today and was hoping to fix these cfs-autoform issues after getting the next af release out, but I'm not sure whether I'll get to it today. Would like to see your PR, though if you've already solved a couple.

from meteor-cfs-autoform.

Pull request was sent for the updated code. I have not updated the readme.md to reflect the changes, however.

from meteor-cfs-autoform.

Changes were merged. Issue closed.

from meteor-cfs-autoform.