Re: [Fwd: extend PEAR::OLE to get Spreadsheet_Excel_Reader into PEAR]

Hi Andreas, Ask Vadim to propose his package to PEPr, even tho the ole read things are missing in PEAR::OLE then it is okey he can just implement those in his package until PEAR::OLE gets them (If ever), the most important part is that he made his code PEAR CS and structured the right way. Could you pass this on to him since there probably much interested for this kind of package, I know also that Alan K. once made a package like this for him self, think it was even made with reference to Vadmins package so Alan should be able (if he has time) to give somewhat of a nice review on the code. Regards Helgi On Wed, 21 Sep 2005 22:07:35 +0200, Arnaud Limbourg wrote: > vadim, propose your code via PePr, if it works and conforms to CS then I > see no reason to delay it further :) > > Arnaud. > > Helgi Þormar wrote: >> I'm not sure why you emailed me in particular since I don't have >> anything to do with the OLE, mearly committed some CS fixes. >> >> Anyway I'm forwarding this to pear-dev >> >> >> -------- Forwarded Message -------- >> >>>From: Andreas Korthaus <akorthaus@web.de> To: helgi@trance.is Subject: >>>extend PEAR::OLE to get Spreadsheet_Excel_Reader into PEAR Date: 21 Sep >>>2005 12:25:45 -0000 >>>[This message has been brought to you via pear.php.net.] >>> >>>Hi Helgi ! >>> >>>I've talked to Vadim Tkachenko, author of phpexcelreader, a package for >>>reading Excel files some time ago. He proposed his package for PEAR >>>(Spreadsheet_Excel_Reader) last year: >>> >>>http://marc.theaimsgroup.com/?t=107409182100002&r=1&w=2 >>> >>>PEAR devs told him to change his code to be conform to PEAR coding >>>standards, to change the license and to use PEAR::OLE. He completed all >>>the tasks, except using PEAR::OLE, because there are missing some >>>methodes he needs for reading Excel. >>> >>>He told me that he has contacted the author of OLE, but he seems to be >>>very busy, so there has not been changed anything which prevents >>>Spreadsheet_Excel_Reader to be included in PEAR for 1.5 years now. >>> >>>I've seen that you committed some code to PEAR::OLE too (which has not >>>been released by the way), so perhaps you have an idea what to do to >>>make it possible to include Spreadsheet_Excel_Reader into PEAR soon. >>> >>>That's the needed functionality: >>>http://cvs.sourceforge.net/viewcvs.py/phpexcelreader/base/Excel/oleread.inc?rev=1.1.1.1&view=auto >>> >>>phpexcelreader project page: >>>http://sourceforge.net/projects/phpexcelreader/ >>> >>>best regards >>>Andreas >> >>