Re: [PEAR-DEV] Text_Password static or non-static?
[email protected] (Martin Jansen) Sun, 31 Jan 2016 10:36:58 +0100
| Newsgroups | php.pear.dev |
|---|---|
| Message-ID | <[email protected]> |
On 30.01.16 23:28, Michael Gauthier wrote: > I'm making a PR to update the Text_Password package to explicitly state > static methods so it works properly in PHP 7. While updating, I noticed > the documentation and internal code seems to be written assuming the API > is entirely static but the unit tests are written assuming the API uses > class instances. > > For example, the docs say to use: > > Text_Password::create(); > > but the unit tests use: > > $subject = new Text_Password(); > $subject->create(); > > My question to the list is does anyone besides the unit tests use the > non-static API for Text_Password? Back when I wrote that code my intention was for the methods to be called statically at all times. I'm not sure why the unit tests differ. but I'd say that this is a bug. That being said I'm extremely happy that you are working on Text_Password. - Martin