Excellent work in this library! And I love the documentation!
I recently had to refactor an old form that was using COPY TO...TYPE XLS and decided to replace with this. For the most part things have been easy to understand and implementing is a snap. I added an instance to a form and went from there. I have a couple of thoughts to share. First, I think that integrating this into an EXE requires some refactoring on the error handling. I'd rather have my application level handler kick in instead of throwing up messageboxes with Microsoft Visual Foxpro captions. Second, whilst trying to troubleshoot some issues I was having with repeated calls to create a file, I thought that it was possible some properties needed to be reset to default values after each invocation of SaveTableAsWorkbook. In that process I put a call to deleteallworkbooks at the beginning of my method. I'm not sure if you would classify this as a bug but the destroyallworkbooks method closes the c_strings cursor. According to the comments I saw in the code, the creation of this particular cursor was moved out of the createcursors method into the codepage_assign method. This meant that the first time it worked fine and subsequent passes returned alias not found errors. Once I realized what was happening it was easy enough to handle but I thought I would mention it.
Again, really good work in this class and I tip my hat to you for making it so easy to use.
↧