Skip to content

Commit

Permalink
Merge branch '2.3' into 2.7
Browse files Browse the repository at this point in the history
* 2.3:
  On line 360 the  header not working
  The `$link` argument must be passed as a reference
  Fix snippet
  • Loading branch information
xabbuh committed Apr 30, 2016
2 parents 373ef72 + 1e1ab96 commit 84a1afd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions book/from_flat_php_to_symfony2.rst
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ of the application are isolated in a new file called ``model.php``::
return $link;
}

function close_database_connection($link)
function close_database_connection(&$link)
{
$link = null;
}
Expand Down Expand Up @@ -357,7 +357,7 @@ on the requested URI::
} elseif ('/index.php/show' === $uri && isset($_GET['id'])) {
show_action($_GET['id']);
} else {
header('Status: 404 Not Found');
header('HTTP/1.1 404 Not Found');
echo '<html><body><h1>Page Not Found</h1></body></html>';
}

Expand Down
2 changes: 1 addition & 1 deletion components/dependency_injection/parentservices.rst
Original file line number Diff line number Diff line change
Expand Up @@ -385,7 +385,7 @@ to the ``NewsletterManager`` class, the config would look like this:
$mailManager = new Definition();
$mailManager
->setAbstract(true);
->setAbstract(true)
->addMethodCall('setMailer', array(
new Reference('my_mailer'),
))
Expand Down

0 comments on commit 84a1afd

Please sign in to comment.