[PEPr] +1 for Tools and Utilities::Address_Book

From: Date: Fri, 03 Dec 2004 12:31:40 +0000
Subject: [PEPr] +1 for Tools and Utilities::Address_Book
References: 1  Groups: php.pear.dev 
Request: Send a blank email to pear-dev+get-34758@lists.php.net to get a copy of this message
Helgi Þormar (http://pear.php.net/user/dufuz) has voted +1 on the proposal for Tools and Utilities::Address_Book. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=156 Vote information: http://pear.php.net/pepr/pepr-vote-show.php?id=156&handle=dufuz This vote is conditional. The condition is: Question: if (eregi('^(http|ftp)://', substr($filename, 0, 10))) { why only http and ftp ? https would be smart also ... Line 146 Convertor.php $map = @array_flip($map); why are you silencing array_flip when you already check if $map is a array or not ? Are you referring to Netscape as Mozilla Mailer/Thunderbird ?? If not then I'd like to see a driver for them :) Can this package only output in .csv via headers or so ? If that's the case I'd also like to see it return only a string with the data, having it a option that is. Probably the same with import :) createComponent should be using similar to loadFile in LiveUser, that way you don't have to deal with "finding the hidden bug because of the silencing of include", anyway that's what I recommend for you :) Maybe you could use File to do the file reading, though I'm not sure if it would benefit you much, though just a idea Anyway looks really cool, something I'll 100% use in the coming months -- Sent by PEPr, the automatic proposal system at http://pear.php.net

« previous php.pear.dev (#34758) next »