diff options
author | Jeremy Benoist <jeremy.benoist@gmail.com> | 2018-12-18 13:14:42 +0100 |
---|---|---|
committer | Jeremy Benoist <jeremy.benoist@gmail.com> | 2019-01-03 09:42:06 +0100 |
commit | 9f8f188d928b47503d39348c5990379a572b570a (patch) | |
tree | 0e2b7e488312b7bcb51efee8979ff35544997716 /src/Wallabag/ImportBundle/Import/InstapaperImport.php | |
parent | 4d0c632c70ea50d459c3c55ddda2e0f394dd51cb (diff) | |
download | wallabag-9f8f188d928b47503d39348c5990379a572b570a.tar.gz wallabag-9f8f188d928b47503d39348c5990379a572b570a.tar.zst wallabag-9f8f188d928b47503d39348c5990379a572b570a.zip |
Validate imported entry to avoid error on import
We got some imports with a missing `url` field generating some errors while trying to retrieve an existing entry with that url.
Introducing the `validateEntry` allow us to dismiss a message when it doesn't have an url (or other missing stuff in the future)
Diffstat (limited to 'src/Wallabag/ImportBundle/Import/InstapaperImport.php')
-rw-r--r-- | src/Wallabag/ImportBundle/Import/InstapaperImport.php | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/src/Wallabag/ImportBundle/Import/InstapaperImport.php b/src/Wallabag/ImportBundle/Import/InstapaperImport.php index e4f0970c..5a18c7c0 100644 --- a/src/Wallabag/ImportBundle/Import/InstapaperImport.php +++ b/src/Wallabag/ImportBundle/Import/InstapaperImport.php | |||
@@ -108,6 +108,18 @@ class InstapaperImport extends AbstractImport | |||
108 | /** | 108 | /** |
109 | * {@inheritdoc} | 109 | * {@inheritdoc} |
110 | */ | 110 | */ |
111 | public function validateEntry(array $importedEntry) | ||
112 | { | ||
113 | if (empty($importedEntry['url'])) { | ||
114 | return false; | ||
115 | } | ||
116 | |||
117 | return true; | ||
118 | } | ||
119 | |||
120 | /** | ||
121 | * {@inheritdoc} | ||
122 | */ | ||
111 | public function parseEntry(array $importedEntry) | 123 | public function parseEntry(array $importedEntry) |
112 | { | 124 | { |
113 | $existingEntry = $this->em | 125 | $existingEntry = $this->em |