Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Bugfix for Zend\Log\Writer\Syslog #137

Closed
wants to merge 1 commit into from

3 participants

@subsven

_facility got overwritten by the current object which lead to errors in subsequent openlog() calls

@weierophinney

Sven --

I need you to sign and return a CLA before I can merge your pull request:

(You can scan and email it, if desired). Make sure you also create an account in our issue tracker to which to link the CLA:

Once you have, update the ticket, and I'll merge your pull request.

Thanks!
Matthew

@b-durand

This bug has already a resolved issue ZF-10769. There's just no sync between the SVN repository and Git.

I work on synchronization on my branch "svn/log". The commit for this issue is Intiilapa/zf2@ba18a9a.
I send a pull request soon.

@weierophinney

intiilapa: when you issue your pull request, reference this one so I can close it.

@weierophinney

These changes are fixed with intiilapa's pull request.

@laupiFrpar laupiFrpar referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 1, 2011
  1. @subsven

    bugfix: _facility got overwritten by the current object which lead

    subsven authored
    to errors in subsequent openlog() calls
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  library/Zend/Log/Writer/Syslog.php
View
2  library/Zend/Log/Writer/Syslog.php
@@ -105,7 +105,7 @@ public function __construct(array $params = array())
$runInitializeSyslog = true;
if (isset($params['facility'])) {
- $this->_facility = $this->setFacility($params['facility']);
+ $this->setFacility($params['facility']);
$runInitializeSyslog = false;
}
Something went wrong with that request. Please try again.