aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorThomas Citharel <tcit@tcit.fr>2019-01-12 13:09:36 +0100
committerThomas Citharel <tcit@tcit.fr>2019-01-12 13:09:36 +0100
commitceddccb736d7c82ffae068df4d9f334a48aff2f4 (patch)
treef079612ca9b3bde8189c8152da0742e63da9bfdc
parent5c331bf0f9ef679aaf91ef29b13120272fcccbf5 (diff)
downloadwallabag-php73.tar.gz
wallabag-php73.tar.zst
wallabag-php73.zip
Replace continue; with break; to avoid PHP 7.3 warningsphp73
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
-rw-r--r--src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php2
-rw-r--r--src/Wallabag/CoreBundle/Helper/DownloadImages.php2
-rw-r--r--src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php2
-rw-r--r--src/Wallabag/CoreBundle/Helper/TagsAssigner.php2
-rw-r--r--src/Wallabag/ImportBundle/Import/AbstractImport.php2
-rw-r--r--src/Wallabag/ImportBundle/Import/BrowserImport.php6
-rw-r--r--src/Wallabag/ImportBundle/Import/InstapaperImport.php2
7 files changed, 9 insertions, 9 deletions
diff --git a/src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php b/src/Wallabag/CoreBundle/GuzzleSiteAuthenticator/GrabySiteConfigBuilder.php
index 90e00c62..2e57aac8 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 continue; 117 break;
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 cc3dcfce..8c1c208f 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 continue; 59 break;
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 &amp; 62 // if image contains "&" and we can't find it in the html it might be because it's encoded as &amp;
diff --git a/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php b/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php
index 63f65067..da02fbee 100644
--- a/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php
+++ b/src/Wallabag/CoreBundle/Helper/RuleBasedTagger.php
@@ -36,7 +36,7 @@ class RuleBasedTagger
36 36
37 foreach ($rules as $rule) { 37 foreach ($rules as $rule) {
38 if (!$this->rulerz->satisfies($entry, $rule->getRule())) { 38 if (!$this->rulerz->satisfies($entry, $rule->getRule())) {
39 continue; 39 break;
40 } 40 }
41 41
42 $this->logger->info('Matching rule.', [ 42 $this->logger->info('Matching rule.', [
diff --git a/src/Wallabag/CoreBundle/Helper/TagsAssigner.php b/src/Wallabag/CoreBundle/Helper/TagsAssigner.php
index e6b4989f..519150f5 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 continue; 52 break;
53 } 53 }
54 54
55 if (isset($tagsNotYetFlushed[$label])) { 55 if (isset($tagsNotYetFlushed[$label])) {
diff --git a/src/Wallabag/ImportBundle/Import/AbstractImport.php b/src/Wallabag/ImportBundle/Import/AbstractImport.php
index 58a234f4..ea86c528 100644
--- a/src/Wallabag/ImportBundle/Import/AbstractImport.php
+++ b/src/Wallabag/ImportBundle/Import/AbstractImport.php
@@ -156,7 +156,7 @@ abstract class AbstractImport implements ImportInterface
156 $entry = $this->parseEntry($importedEntry); 156 $entry = $this->parseEntry($importedEntry);
157 157
158 if (null === $entry) { 158 if (null === $entry) {
159 continue; 159 break;
160 } 160 }
161 161
162 // store each entry to be flushed so we can trigger the entry.saved event for each of them 162 // store each entry to be flushed so we can trigger the entry.saved event for each of them
diff --git a/src/Wallabag/ImportBundle/Import/BrowserImport.php b/src/Wallabag/ImportBundle/Import/BrowserImport.php
index 614386cb..b55525e5 100644
--- a/src/Wallabag/ImportBundle/Import/BrowserImport.php
+++ b/src/Wallabag/ImportBundle/Import/BrowserImport.php
@@ -158,13 +158,13 @@ abstract class BrowserImport extends AbstractImport
158 158
159 foreach ($entries as $importedEntry) { 159 foreach ($entries as $importedEntry) {
160 if ((array) $importedEntry !== $importedEntry) { 160 if ((array) $importedEntry !== $importedEntry) {
161 continue; 161 break;
162 } 162 }
163 163
164 $entry = $this->parseEntry($importedEntry); 164 $entry = $this->parseEntry($importedEntry);
165 165
166 if (null === $entry) { 166 if (null === $entry) {
167 continue; 167 break;
168 } 168 }
169 169
170 // @see AbstractImport 170 // @see AbstractImport
@@ -206,7 +206,7 @@ abstract class BrowserImport extends AbstractImport
206 { 206 {
207 foreach ($entries as $importedEntry) { 207 foreach ($entries as $importedEntry) {
208 if ((array) $importedEntry !== $importedEntry) { 208 if ((array) $importedEntry !== $importedEntry) {
209 continue; 209 break;
210 } 210 }
211 211
212 // set userId for the producer (it won't know which user is connected) 212 // set userId for the producer (it won't know which user is connected)
diff --git a/src/Wallabag/ImportBundle/Import/InstapaperImport.php b/src/Wallabag/ImportBundle/Import/InstapaperImport.php
index e113ba00..60da7751 100644
--- a/src/Wallabag/ImportBundle/Import/InstapaperImport.php
+++ b/src/Wallabag/ImportBundle/Import/InstapaperImport.php
@@ -65,7 +65,7 @@ class InstapaperImport extends AbstractImport
65 $handle = fopen($this->filepath, 'rb'); 65 $handle = fopen($this->filepath, 'rb');
66 while (false !== ($data = fgetcsv($handle, 10240))) { 66 while (false !== ($data = fgetcsv($handle, 10240))) {
67 if ('URL' === $data[0]) { 67 if ('URL' === $data[0]) {
68 continue; 68 break;
69 } 69 }
70 70
71 // last element in the csv is the folder where the content belong 71 // last element in the csv is the folder where the content belong