Issue Type: Improvement Created: 2010-01-27T08:31:32.000+0000 Last Updated: 2010-07-19T16:59:48.000+0000 Status: Closed Fix version(s): - 1.11.0 (02/Nov/10)
Reporter: Ota Mares (ota) Assignee: Ramon Henrique Ornelas (ramon) Tags: - Zend_Log
Related issues: Attachments: - Log.patch
All public methods of Zend_Log should return the current instance (return $this) so a fluent usage is possible.
Sometimes its necessary to add more writers or event items at once and a fluent interface makes this nicer to implement. There should be no downside because in its current state all public methods return nothing.
<pre class="highlight">$logger->addWriter($writer); $logger->setEventItem('foo1', 'bar1'); $logger->setEventItem('foo2', 'bar2');
<pre class="highlight">$logger->addWriter($writer) ->setEventItem('foo1', 'bar1') ->setEventItem('foo2', 'bar2');
Posted by Ramon Henrique Ornelas (ramon) on 2010-01-30T09:25:19.000+0000
Reassigned to @matthew.
Posted by Ramon Henrique Ornelas (ramon) on 2010-02-19T14:59:58.000+0000
Attached the patch.
Posted by Ramon Henrique Ornelas (ramon) on 2010-07-15T20:51:07.000+0000
Fixed with the r22568.
Posted by Dolf Schimmel (Freeaqingme) (freak) on 2010-07-18T04:57:53.000+0000
Doesn't this - strictly spoken - break bc, and therefore may not be committed into the release branch?
Posted by Ramon Henrique Ornelas (ramon) on 2010-07-18T06:43:00.000+0000
Soon I will revert branch but I believe should be implemented in the minor release ;).
Posted by Ramon Henrique Ornelas (ramon) on 2010-07-18T11:33:39.000+0000
Reverted in branch r22632.
Posted by Matthew Weier O'Phinney (matthew) on 2010-07-19T07:02:31.000+0000
I was originally going to re-open the issue, but I will note that it does introduce a slight BC break.
Technically speaking, we usually characterize BC breaks as one of the following:
However, changing the visibility at all actually can introduce some BC breakage. If a method is marked as private, and we change it to protected, we can cause BC breakage in the case where somebody has extended the class and overridden that method, while retaining the private visibility. In this case, they will get an E_FATAL as their method is now more restrictive than the method they're extending.
That said, they also need to override a public or protected method in the first place in order to even call the formerly private method. I would hope they have a good test suite in place. :)
I still think this is a good idea, but it's easier to call this sort of thing out in a minor release than in a maintenance release.
Posted by Ramon Henrique Ornelas (ramon) on 2010-07-19T16:59:47.000+0000
Thanks you Matthew and Freeaqingme by reported. Only applied to branch because I saw a similar issue ZF-9913 be applied in release 1.10.6.
Have you found an issue?
See the Overview section for more details.