-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[DomCrawler] Abstract URI logic and crawl images #13620
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
201 changes: 201 additions & 0 deletions
201
src/Symfony/Component/DomCrawler/AbstractUriElement.php
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,201 @@ | ||
<?php | ||
|
||
/* | ||
* This file is part of the Symfony package. | ||
* | ||
* (c) Fabien Potencier <fabien@symfony.com> | ||
* | ||
* For the full copyright and license information, please view the LICENSE | ||
* file that was distributed with this source code. | ||
*/ | ||
|
||
namespace Symfony\Component\DomCrawler; | ||
|
||
/** | ||
* Any HTML element that can link to an URI. | ||
* | ||
* @author Fabien Potencier <fabien@symfony.com> | ||
* | ||
* @api | ||
*/ | ||
abstract class AbstractUriElement implements UriElementInterface | ||
{ | ||
/** | ||
* @var \DOMElement | ||
*/ | ||
protected $node; | ||
|
||
/** | ||
* @var string The method to use for the element URI | ||
*/ | ||
protected $method; | ||
|
||
/** | ||
* @var string The URI of the page where the element is embedded (or the base href) | ||
*/ | ||
protected $currentUri; | ||
|
||
/** | ||
* Constructor. | ||
* | ||
* @param \DOMElement $node A \DOMElement instance | ||
* @param string $currentUri The URI of the page where the element is embedded (or the base href) | ||
* @param string $method The method to use for the element URI (get by default) | ||
* | ||
* @throws \InvalidArgumentException if the node is not a link | ||
* | ||
* @api | ||
*/ | ||
public function __construct(\DOMElement $node, $currentUri, $method = 'GET') | ||
{ | ||
if (!in_array(strtolower(substr($currentUri, 0, 4)), array('http', 'file'))) { | ||
throw new \InvalidArgumentException(sprintf('Current URI must be an absolute URL ("%s").', $currentUri)); | ||
} | ||
|
||
$this->setNode($node); | ||
$this->method = $method ? strtoupper($method) : null; | ||
$this->currentUri = $currentUri; | ||
} | ||
|
||
public function getNode() | ||
{ | ||
return $this->node; | ||
} | ||
|
||
public function getMethod() | ||
{ | ||
return $this->method; | ||
} | ||
|
||
public function getUri() | ||
{ | ||
$uri = trim($this->getRawUri()); | ||
|
||
// absolute URL? | ||
if (null !== parse_url($uri, PHP_URL_SCHEME)) { | ||
return $uri; | ||
} | ||
|
||
// empty URI | ||
if (!$uri) { | ||
return $this->currentUri; | ||
} | ||
|
||
// an anchor | ||
if ('#' === $uri[0]) { | ||
return $this->cleanupAnchor($this->currentUri).$uri; | ||
} | ||
|
||
$baseUri = $this->cleanupUri($this->currentUri); | ||
|
||
if ('?' === $uri[0]) { | ||
return $baseUri.$uri; | ||
} | ||
|
||
// absolute URL with relative schema | ||
if (0 === strpos($uri, '//')) { | ||
return preg_replace('#^([^/]*)//.*$#', '$1', $baseUri).$uri; | ||
} | ||
|
||
$baseUri = preg_replace('#^(.*?//[^/]*)(?:\/.*)?$#', '$1', $baseUri); | ||
|
||
// absolute path | ||
if ('/' === $uri[0]) { | ||
return $baseUri.$uri; | ||
} | ||
|
||
// relative path | ||
$path = parse_url(substr($this->currentUri, strlen($baseUri)), PHP_URL_PATH); | ||
$path = $this->canonicalizePath(substr($path, 0, strrpos($path, '/')).'/'.$uri); | ||
|
||
return $baseUri.('' === $path || '/' !== $path[0] ? '/' : '').$path; | ||
} | ||
|
||
/** | ||
* Returns raw URI data. | ||
* | ||
* @return string | ||
*/ | ||
abstract protected function getRawUri(); | ||
|
||
/** | ||
* Returns the canonicalized URI path (see RFC 3986, section 5.2.4). | ||
* | ||
* @param string $path URI path | ||
* | ||
* @return string | ||
*/ | ||
protected function canonicalizePath($path) | ||
{ | ||
if ('' === $path || '/' === $path) { | ||
return $path; | ||
} | ||
|
||
if ('.' === substr($path, -1)) { | ||
$path = $path.'/'; | ||
} | ||
|
||
$output = array(); | ||
|
||
foreach (explode('/', $path) as $segment) { | ||
if ('..' === $segment) { | ||
array_pop($output); | ||
} elseif ('.' !== $segment) { | ||
array_push($output, $segment); | ||
} | ||
} | ||
|
||
return implode('/', $output); | ||
} | ||
|
||
/** | ||
* Sets current \DOMElement instance. | ||
* | ||
* @param \DOMElement $node A \DOMElement instance | ||
*/ | ||
abstract protected function setNode(\DOMElement $node); | ||
|
||
/** | ||
* Removes the query string and the anchor from the given uri. | ||
* | ||
* @param string $uri The uri to clean | ||
* | ||
* @return string | ||
*/ | ||
private function cleanupUri($uri) | ||
{ | ||
return $this->cleanupQuery($this->cleanupAnchor($uri)); | ||
} | ||
|
||
/** | ||
* Remove the query string from the uri. | ||
* | ||
* @param string $uri | ||
* | ||
* @return string | ||
*/ | ||
private function cleanupQuery($uri) | ||
{ | ||
if (false !== $pos = strpos($uri, '?')) { | ||
return substr($uri, 0, $pos); | ||
} | ||
|
||
return $uri; | ||
} | ||
|
||
/** | ||
* Remove the anchor from the uri. | ||
* | ||
* @param string $uri | ||
* | ||
* @return string | ||
*/ | ||
private function cleanupAnchor($uri) | ||
{ | ||
if (false !== $pos = strpos($uri, '#')) { | ||
return substr($uri, 0, $pos); | ||
} | ||
|
||
return $uri; | ||
} | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
<?php | ||
|
||
/* | ||
* This file is part of the Symfony package. | ||
* | ||
* (c) Fabien Potencier <fabien@symfony.com> | ||
* | ||
* For the full copyright and license information, please view the LICENSE | ||
* file that was distributed with this source code. | ||
*/ | ||
|
||
namespace Symfony\Component\DomCrawler; | ||
|
||
/** | ||
* Image represents an HTML image (an HTML img tag). | ||
* | ||
* @api | ||
*/ | ||
class Image extends AbstractUriElement | ||
{ | ||
protected function getRawUri() | ||
{ | ||
return $this->node->getAttribute('src'); | ||
} | ||
|
||
protected function setNode(\DOMElement $node) | ||
{ | ||
if ('img' !== $node->nodeName) { | ||
throw new \LogicException(sprintf('Unable to visualize a "%s" tag.', $node->nodeName)); | ||
} | ||
|
||
$this->node = $node; | ||
} | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please make these private (we have public getters for them anyway)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, no. These properties are currently protected in Link, so we cannot make them private yet