changing upsertData and deleteData to use an autoreleasing error parameter and return the dataObject.

All error returning methods should probably use this structure. It’s
cleaner.
Feedback reporting now uses the feedback database object for its api call.

Haven’t pulled out all the old cruft yet, though I will real soon.