diff options
author | Jeremy Benoist <jeremy.benoist@gmail.com> | 2019-02-27 14:59:50 +0100 |
---|---|---|
committer | Jeremy Benoist <jeremy.benoist@gmail.com> | 2019-02-27 15:01:03 +0100 |
commit | 8c0ba953070dca22e9a06999cfe355ea01847c64 (patch) | |
tree | 994fc7fa9e96ef9f15ef311cbc103fab41746e0f /src/Wallabag/CoreBundle | |
parent | ea925bb112ab99efbb29d8e7113e80357a70bd18 (diff) | |
download | wallabag-8c0ba953070dca22e9a06999cfe355ea01847c64.tar.gz wallabag-8c0ba953070dca22e9a06999cfe355ea01847c64.tar.zst wallabag-8c0ba953070dca22e9a06999cfe355ea01847c64.zip |
Adding more tests
Diffstat (limited to 'src/Wallabag/CoreBundle')
4 files changed, 4 insertions, 4 deletions
diff --git a/src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php b/src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php index 2e57aac8..90e00c62 100644 --- a/src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php +++ b/src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php | |||
@@ -114,7 +114,7 @@ class GrabySiteConfigBuilder implements SiteConfigBuilder | |||
114 | $extraFields = []; | 114 | $extraFields = []; |
115 | foreach ($extraFieldsStrings as $extraField) { | 115 | foreach ($extraFieldsStrings as $extraField) { |
116 | if (false === strpos($extraField, '=')) { | 116 | if (false === strpos($extraField, '=')) { |
117 | break; | 117 | continue; |
118 | } | 118 | } |
119 | 119 | ||
120 | list($fieldName, $fieldValue) = explode('=', $extraField, 2); | 120 | list($fieldName, $fieldValue) = explode('=', $extraField, 2); |
diff --git a/src/Wallabag/CoreBundle/Helper/DownloadImages.php b/src/Wallabag/CoreBundle/Helper/DownloadImages.php index 8c1c208f..cc3dcfce 100644 --- a/src/Wallabag/CoreBundle/Helper/DownloadImages.php +++ b/src/Wallabag/CoreBundle/Helper/DownloadImages.php | |||
@@ -56,7 +56,7 @@ class DownloadImages | |||
56 | $imagePath = $this->processSingleImage($entryId, $image, $url, $relativePath); | 56 | $imagePath = $this->processSingleImage($entryId, $image, $url, $relativePath); |
57 | 57 | ||
58 | if (false === $imagePath) { | 58 | if (false === $imagePath) { |
59 | break; | 59 | continue; |
60 | } | 60 | } |
61 | 61 | ||
62 | // if image contains "&" and we can't find it in the html it might be because it's encoded as & | 62 | // if image contains "&" and we can't find it in the html it might be because it's encoded as & |
diff --git a/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php b/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php index d156df84..fbdf2ac7 100644 --- a/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php +++ b/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php | |||
@@ -37,7 +37,7 @@ class RuleBasedTagger | |||
37 | 37 | ||
38 | foreach ($rules as $rule) { | 38 | foreach ($rules as $rule) { |
39 | if (!$this->rulerz->satisfies($entry, $rule->getRule())) { | 39 | if (!$this->rulerz->satisfies($entry, $rule->getRule())) { |
40 | break; | 40 | continue; |
41 | } | 41 | } |
42 | 42 | ||
43 | $this->logger->info('Matching rule.', [ | 43 | $this->logger->info('Matching rule.', [ |
diff --git a/src/Wallabag/CoreBundle/Helper/TagsAssigner.php b/src/Wallabag/CoreBundle/Helper/TagsAssigner.php index 519150f5..e6b4989f 100644 --- a/src/Wallabag/CoreBundle/Helper/TagsAssigner.php +++ b/src/Wallabag/CoreBundle/Helper/TagsAssigner.php | |||
@@ -49,7 +49,7 @@ class TagsAssigner | |||
49 | 49 | ||
50 | // avoid empty tag | 50 | // avoid empty tag |
51 | if (0 === \strlen($label)) { | 51 | if (0 === \strlen($label)) { |
52 | break; | 52 | continue; |
53 | } | 53 | } |
54 | 54 | ||
55 | if (isset($tagsNotYetFlushed[$label])) { | 55 | if (isset($tagsNotYetFlushed[$label])) { |