This project is read-only.

Use TraceSource

please use a TraceSource instead of logging to Debug (ie Debug.WriteLine()). this makes it possible to configure logging for your module via the app.config file

Id #7867 | Release: None | Updated: Aug 6, 2014 at 9:09 AM by uTILLIty | Created: Aug 6, 2014 at 9:09 AM by uTILLIty

Events should not have Event suffix

the name of your events end in "Event". By convention, this is not nessesary Examples: WebDavManager.DownloadProgressEvent should be called WebDavManager.DownloadProgress

Id #7866 | Release: None | Updated: Aug 6, 2014 at 9:08 AM by uTILLIty | Created: Aug 6, 2014 at 9:08 AM by uTILLIty

don't use local filenames, use stream

send and receive files to System.IO.Stream, not to files in the file-system. This makes it easier for in-memory work and flexible for unit-testing etc. if you must access the filesystem, make sure...

Id #7865 | Release: None | Updated: Aug 6, 2014 at 9:06 AM by uTILLIty | Created: Aug 6, 2014 at 9:06 AM by uTILLIty

Don't swallow exceptions

your code swallows server exceptions. If it can't handle them, it should rethrow or wrap them, or don't catch them in the first place. Otherwise it's impossible to find out the reason for non-worki...

Id #7864 | Release: None | Updated: Aug 6, 2014 at 9:04 AM by uTILLIty | Created: Aug 6, 2014 at 9:04 AM by uTILLIty

Set ContentLength on requests

OpenERP requires the webRequest.ContentLength to be set to something other than -1. for List() it should be 0, for example. For Uploads it should be the size of the file

Id #7863 | Release: None | Updated: Aug 6, 2014 at 9:02 AM by uTILLIty | Created: Aug 6, 2014 at 9:02 AM by uTILLIty

  • 1-5 of 5 Work Items
    • Previous
    • 1
    • Next
    • Showing
    • All
    • Work Items