On Sunday 13 May 2012 06:30 Frank Karlitschek wrote: > On 13.05.2012, at 00:31, Thomas Tanghus <[email protected]> wrote: > > On Saturday 12 May 2012 22:07 Frank Karlitschek wrote: > >> Hi, > >> > >> this is not good. > >> Is there something we can do to fix this? Or do we have to revert to the > >> old MDB driver? > >> > >> I just released RC anyways but we have to fix this for the final release. > >> > >> I recommend all developers to run ownCloud with notices switched on. > >> Because it easier for debugging and motivates us all to do cleaner > >> coding. ;-)> > > I actually just pushed a commit that suppresses notices when importing > > events or contacts from the files app. To be honest I think that hook > > should be disabled for this release because it has inter-app dependencies > > that aren't properly handled, it introduces two extra writable > > directories, and there's hardly any error checking. > > Your´re the expert here. :-) > Let´s disable it if it´s not working properly.
Not really because I have no idea how the filesystem hooks work - yet ;-) I may have found what caused the errors, and in the process also found some errors in my js code. If I add more error checking and reporting, would it then be OK to push those changes? I guess I was a bit grumpy when I wrote the last mail, because I really think we should have the import functionality. -- Med venlig hilsen / Best Regards Thomas Tanghus _______________________________________________ Owncloud mailing list [email protected] https://mail.kde.org/mailman/listinfo/owncloud
