[3.1][3.2][beta] Attached image rotator

[Sheer]

Не проверял.

[Pazh]

Sheer, поставил расширение, выставил максимальные размеры - получил ошибку:

Код: Выделить всё

SQL ERROR [ mysqli ]
Duplicate entry 'rotate_img_max_width' for key 'PRIMARY' [1062]
SQL
INSERT INTO phpbb_config (config_name, config_value, is_dynamic) VALUES ('rotate_img_max_width', '1024', 0)

ручная очистка кеша помогает, но осадочек

[Sheer]

Никогда с таким не сталкивался, к тому же непонятно, как этому может привести

Код: Выделить всё

set_config('rotate_img_max_width', $limit_width);

[Yukh]

Pazh, я бы отключил расширение, удалил данные, после этого проверил SQL запрос:

SELECT count(*) FROM phpbb_config WHERE config_name='rotate_img_max_width';

Если результат >0, то удалить вручную эту строку и включить расширение заново.

[Pazh]

Yukh, спасибо за совет, но это излишне, т.к. поле config_name является уникальным primary key, что означает что Ваш запрос вернет 1 или 0. А для проверки есть поле или нет - достаточно обновить страницу управления расширением после очистки кеша из админки.

Отправлено спустя 1 минуту 31 секунду:
Sheer, за что купил, за то и продал. Иногда я тоже удивляюсь таким ошибкам в set_config

[Yukh]

Pazh, смысл не в том, есть ли поле, смысл в том - есть ли конкретная запись в конфиге и если она по какой-то причине не удалилась (именно по этой причине же primary key ругается на Duplicate entry) я предложил удалить ее вручную.

Код: Выделить всё

mysql> SELECT count(*) FROM phpbb_config WHERE config_name='rotate_img_max_width';
+----------+
| count(*) |
+----------+
|        1 |
+----------+
1 row in set (0,00 sec)

mysql> SELECT * FROM phpbb_config WHERE config_name='rotate_img_max_width';
+----------------------+--------------+------------+
| config_name          | config_value | is_dynamic |
+----------------------+--------------+------------+
| rotate_img_max_width | 0            |          0 |
+----------------------+--------------+------------+
1 row in set (0,00 sec)

Насколько я понимаю, при выключенном расширении и удаленных его данных этого быть в таблице не должно.

[Sheer]

при выключенном расширении и удаленных его данных этого быть в таблице не должно.

Не должно.

[southklad]

Не помню уже писал тут или нет, но при повороте изображения, его размер увеличивается.

[Siava]

для PHP 8.0:
event/listener.php

найти

Код: Выделить всё

if (isset($exif['THUMBNAIL']) && $exif['THUMBNAIL']['Orientation'] == 1)

заменить на

Код: Выделить всё

if (isset($exif['THUMBNAIL']) && isset($exif['THUMBNAIL']['Orientation']) == 1)

[Sheer]

Siava, с этой правкой на php7 работает корректно?

[Siava]

Sheer, да, в PHP 7.4.26 нормально.

[Sheer]

Ок, исправлю в репо.

[ronim]

Подтверждаю ,в PHP 7.4.24 тоже все работает нормально

[Sheer]

Исправил
https://github.com/AlexSheer/phpbb3.1-R ... 2b0d6b04c3

[rxu]

Исправил

isset($exif['THUMBNAIL']['Orientation']) == 1 - так не работает. isset() возвращает логическое значение, оно в этом выражении будет просто приведено к единице если переменная определена. Тогда уж

Код: Выделить всё

if (isset($exif['THUMBNAIL']['Orientation']) && $exif['THUMBNAIL']['Orientation'] == 1)

Отправлено спустя 27 минут 8 секунд:
Или то же самое, но еще интереснее

Код: Выделить всё

if (($exif['THUMBNAIL']['Orientation'] ?? false) == 1)