[PEPr] +1 for Tools and Utilities::Address_Book
| From: | Helgi �ormar | 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