Re: Re: search CVS file (perharps a bug)

On 5/6/05, Helgi Þormar <helgi@trance.is> wrote: > On Mon, 2005-05-02 at 22:32 +0200, Laurent Laville wrote: > > Michael Wallner a écrit : > > > I wrote: > > > > > > > > >>This sounds like a bug, I'll have a look. > > > > > > > > > In case you didn't notice, I comitted a possible fix to File. > > > I'd appreciate if you could check if that was the cause. > > > > > > Thanks, > > > -- > > > Michael - < mike(@)php.net > > > > > Hi Michael > > > > Everybody have noticed that i means CSV and not CVS > > > > I've tested the version 1.30 of File.php in CVS repository and it won't > > fix the problem. I've even a warning message now ! > > > > Warning: Illegal offset type in [...]File.php on line 414 > > > > Laurent > > Like Mike said earlier, you have to do "\r\n" not '\r\n' for the CSV > stuff to work :) > > - Helgi > Seems, config 'crlf' option need to renamed to linebreak and a value is a constant that is one of FILE_CSV_CRLF_MAC, FILE_CSV_CRLF_WINDOWS and FILE_CSV_CRLF_UNIX. If it's not one of these just throw the error, and this solution could be eliminated a mistake like what laurent do. -- Firman Wandayandi : Never Dreamt Before