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

Firman Wandayandi a écrit :

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.

Hi all, To reply to Helgi: using "\r\n" rather than '\r\n' which defines linebreak for windows environement, did not solved the problem : 1. with File-1.2.0 package 2. with latest version of CVS repository I think Michael has right "This sounds like a bug, I'll have a look." Laurent Laville