Re: pear: Issues about Net_IDNA- / Net_IDNA2-releases

[email protected] ("Daniel O'Connor")
Newsgroups php.pear.qa
Message-ID <[email protected]>
>
> First thing was to install Net_IDNA-0.8.0 because it was the latest.
> That complains because in the constructor "new idna_convert" is used,
> but no class idna_convert is present. I also didn't find how this is
> meant to be used (if I did somehting wrong).
>

idna_convert is a mystery to me: the PHP4 version is undoubtedly completely
broken and has been for some time.

IE:
Revision: 255772
Author: neufeind
Date: 11:09:38 PM, Saturday, March 22, 2008
Message:
first checkin of 0.7.1-version of the package, as as a basis
----
Added : /pear/packages/Net_IDNA/trunk/IDNA
Added : /pear/packages/Net_IDNA/trunk/IDNA/php4.php
Added : /pear/packages/Net_IDNA/trunk/IDNA/php5.php
Added : /pear/packages/Net_IDNA/trunk/IDNA.php
Added : /pear/packages/Net_IDNA/trunk/test
Added : /pear/packages/Net_IDNA/trunk/test/test_php4.php
Added : /pear/packages/Net_IDNA/trunk/test/test_php5.php

= no idna_convert.

The best guess I have is http://www.phpclasses.org/browse/file/5845.html is
also in the include path.

Sigh :(

Looking at the construction even confuses the hell out of me:
    /**
    * Constructor
    *
    * @param  array  $options
    * @access public
    * @see    setParams()
    */
    function Net_IDNA($options = false)
    {
        $this->IC = new idna_convert($options);
        return $this->IC;
    }


No where in the base class does $this->IC get used.

I've removed it in r300837 ; which will probably Destroy The Universe.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.