merged branch matthijsvandenbos/matthijsvandenbos/ticket_7210 (PR #7211)

This PR was submitted for the 2.1 branch but it was merged into the master branch instead (closes #7211).

Commits
-------

41abfd8 [DomCrawler] fixed ticket 7210

Discussion
----------

[DomCrawler] fixed ticket 7210: DomCrawler is not properly extendable

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #7210
| License       | MIT
| Doc PR        |

See https://github.com/symfony/symfony/issues/7210 for a description of the argument
This commit is contained in:
Fabien Potencier 2013-03-06 20:37:18 +01:00
commit af28b0c3fc
1 changed files with 3 additions and 3 deletions

View File

@ -25,7 +25,7 @@ class Crawler extends \SplObjectStorage
/**
* @var string The current URI or the base href value
*/
private $uri;
protected $uri;
/**
* Constructor.
@ -704,7 +704,7 @@ class Crawler extends \SplObjectStorage
return sprintf("concat(%s)", implode($parts, ', '));
}
private function getNode($position)
protected function getNode($position)
{
foreach ($this as $i => $node) {
if ($i == $position) {
@ -717,7 +717,7 @@ class Crawler extends \SplObjectStorage
// @codeCoverageIgnoreEnd
}
private function sibling($node, $siblingDir = 'nextSibling')
protected function sibling($node, $siblingDir = 'nextSibling')
{
$nodes = array();