Skip to content

Commit a39818f

Browse files
committed
Let FastImageSize see if image file exists
Signed-off-by: Matt Friedman <maf675@gmail.com>
1 parent 70140d5 commit a39818f

File tree

4 files changed

+10
-19
lines changed

4 files changed

+10
-19
lines changed

event/listener.php

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -181,15 +181,8 @@ public function validate_pwa_options($event)
181181
return;
182182
}
183183

184-
// Check if image file exists
184+
// Check if image is valid
185185
$image = $this->root_path . $this->config['icons_path'] . '/' . $value;
186-
if (!file_exists($image))
187-
{
188-
$this->add_error($event, 'PWA_IMAGE_NOT_FOUND', $value);
189-
return;
190-
}
191-
192-
// Check if image is the correct dimensions and type
193186
$image_info = $this->imagesize->getImageSize($image);
194187
if ($image_info !== false)
195188
{

language/en/webpushnotifications_common_acp.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,5 @@
4848
'PWA_ICON_SIZE_INVALID' => '“%s” does not have the correct image dimensions.',
4949
'PWA_ICON_MIME_INVALID' => '“%s” must be a PNG image file.',
5050
'PWA_IMAGE_INVALID' => '“%s” does not appear to be a valid image file.',
51-
'PWA_IMAGE_NOT_FOUND' => '“%s” could not be found.',
52-
'PWA_IMAGE_NOT_PROVIDED' => '%s must not be empty. All icon fields must contain and image.',
51+
'PWA_IMAGE_NOT_PROVIDED' => '%s field must not be empty. All icon fields must contain and image.',
5352
]);

language/ru/webpushnotifications_common_acp.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,8 @@
4545
'PWA_ICON_SMALL_EXPLAIN' => 'Имя файла изображения формата PNG размером 192 x 192 пикселя. Файл изображения должен быть загружен на сервер в папку <samp>images/icons</samp>.',
4646
'PWA_ICON_LARGE' => 'Большой значок для мобильного устройства',
4747
'PWA_ICON_LARGE_EXPLAIN' => 'Имя файла изображения формата PNG размером 512 x 512 пикселей. Файл изображения должен быть загружен на сервер в папку <samp>images/icons</samp>.',
48-
'PWA_ICON_SIZE_INVALID' => 'Изображение “%s” имеет некорректные размеры.',
49-
'PWA_ICON_MIME_INVALID' => 'Файл изображения “%s” должен иметь формат PNG.',
50-
'PWA_IMAGE_INVALID' => 'Файл “%s” не яввляется файлом изображения.',
51-
'PWA_IMAGE_NOT_FOUND' => 'Файл “%s” не найден.',
52-
'PWA_IMAGE_NOT_PROVIDED' => '“%s” must not be empty. All icon fields must contain and image.',
48+
'PWA_ICON_SIZE_INVALID' => 'Изображение «%s» имеет некорректные размеры.',
49+
'PWA_ICON_MIME_INVALID' => 'Файл изображения «%s» должен иметь формат PNG.',
50+
'PWA_IMAGE_INVALID' => 'Файл «%s» не яввляется файлом изображения.',
51+
'PWA_IMAGE_NOT_PROVIDED' => 'Настройка «%s» не может быть пустой. Необходимо задать все пути к значкам для мобильного устройства.',
5352
]);

tests/event/listener_test.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -310,19 +310,19 @@ public function validate_pwa_options_data()
310310
return [
311311
[
312312
['pwa_icon_small' => '192.png', 'pwa_icon_large' => '512.png'],
313-
['PWA_IMAGE_NOT_FOUND'],
313+
[],
314314
],
315315
[
316316
['pwa_icon_small' => '1.png', 'pwa_icon_large' => '512.png'],
317-
['PWA_IMAGE_NOT_FOUND', 'PWA_ICON_SIZE_INVALID'],
317+
['PWA_ICON_SIZE_INVALID'],
318318
],
319319
[
320320
['pwa_icon_small' => '1.png', 'pwa_icon_large' => '12.png'],
321-
['PWA_IMAGE_NOT_FOUND', 'PWA_ICON_SIZE_INVALID'],
321+
['PWA_ICON_SIZE_INVALID'],
322322
],
323323
[
324324
['pwa_icon_small' => '192.jpg', 'pwa_icon_large' => '512.gif'],
325-
['PWA_IMAGE_NOT_FOUND', 'PWA_ICON_MIME_INVALID'],
325+
['PWA_ICON_MIME_INVALID'],
326326
],
327327
[
328328
['pwa_icon_small' => '', 'pwa_icon_large' => ''],

0 commit comments

Comments
 (0)