Изменить расширение загружаемого файла

[Vlad__]

Как? Если загружается png меняем расширение на jpg, чтобы в строке загруженного файла filename.jpg

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

if (function_exists('exif_imagetype') && ($filedata['extension'] == 'png'))

что написать дальше, чтобы переименовать уже загруженный файл после этого условия?

[Sumanai]

Один вопрос- зачем?

[Vlad__]

Пишу небольшое расширение, конвертирующее автоматом загружаемый png в jpg. Все в принципе уже работает, только файл остается с расширением png.

[LavIgor]

Тоже думаю, зачем, я бы обратно переводил в PNG, который лучше масштабируется, хоть и тоже растровый.

[Vlad__]

Господа, неужели непонятно зачем? Когда некоторые пользователи начинают грузить фото и картинки по 1,5 -2 мб в png, когда в jpg их размер 200 кил. Накой мне такая радость?
Я не жду обсуждений зачем, а спрашиваю решение вопроса у опытных товарищей. Как я понимаю нужно изменить для этого файла extension в phpbb_attachments. Напомните как?

$db->sql_query( ..... ? как именно для этого файла?

[Sheer]

http://php.net/manual/ru/function.rename.php

[Vlad__]

Sheer, rename не то. Файл загружается с physical_filename вида 61_09deb2b77a79d3f9b37d89ed38cf9c0b. Его я конвертирую в jpg, это не проблема. Но к нему в бд прописано как я понимаю real_filename и extension. Имхо мне там нужно поменять, чтобы было кошерно.

[Sheer]

Ну и меняй. Ссылку на руководство по MySQL прислать?

[Vlad__]

Ну и меняй. Ссылку на руководство по MySQL прислать?

Читаю, но немного торможу, т.к. опыта во всем этом крайне мало.

[Sheer]

Как-то так

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

UPDATE phpbb_attachments SET real_filename = 'under_construction.jpg', extension = 'jpg', mimetype = 'image/jpg' WHERE attach_id = 1;

[Vlad__]

Не получается, чуствую что просто, но ....

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

if (function_exists('exif_imagetype') && ($filedata['extension'] == 'png'))
{
$sql = 'UPDATE ' . phpbb_attachments . 'SET real_filename =' . $filedata['physical_filename'] . '.jpg' . 'SET extension = 'jpg'' .  'WHERE attach_id = ' . $filedata['attach_id'];
$this->db->sql_query($sql);
}

Этот код не работает. Или ATTACHMENTS_TABLE вместо phpbb_attachments?

[Sheer]

ATTACHMENTS_TABLE вместо phpbb_attachments

Семён Семёныч!
Ну для начала стоит еще проверить, а что там с массивом $filedata

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

print"<pre>";print_r($filedata);print"</pre>";

Второе. А выполняется ли условие if (function_exists('exif_imagetype') && ($filedata['extension'] == 'png'))
И третье, нужно кавычки экранировать.
Вообще это лучше так делать

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

$sql_data = array(
	'real_filename'	=> $filedata['physical_filename'],
	'extension '	=> $filedata['extension'],
);

$sql = 'UPDATE ' . ATTACHMENTS_TABLE . '
		SET ' . $this->db->sql_build_array('UPDATE', $sql_data) . '
		WHERE attach_id = ' . $filedata['attach_id'];

И четвертое. mimetype тоже наужно менять.

Отправлено спустя 54 секунды:
И при разработке всегда включать debug

[Vlad__]

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

PHP Fatal error:  Uncaught Error: Call to a member function sql_build_array() on null in /var/www/mysite$
Stack trace:
#0 [internal function]: vlad\image_convert\event\listener->upload_image_convert(Object(phpbb\event\data), 'core.modify_upl...', Object(phpbb$
#1 /var/www/mysite/vendor/symfony/event-dispatcher/EventDispatcher.php(184): call_user_func(Array, Object(phpbb\event\data), 'core.modify$
#2 /var/www/mysite/vendor/symfony/event-dispatcher/EventDispatcher.php(46): Symfony\Component\EventDispatcher\EventDispatcher->doDispatch$
#3 /var/www/mysite/phpbb/event/dispatcher.php(60): Symfony\Component\EventDispatcher\EventDispatcher->dispatch('core.modify_upl...', Obje$
#4 /var/www/mysite/phpbb/event/dispatcher.php(46): phpbb\event\dispatcher->dispatch('core.modify_upl...', Object(phpbb\event\data))
#5 /var/www/mysite/phpbb/attachment/up in /var/www/mysite/ext/vlad/image_convert/event/listener.php on line 65

[Sheer]

В конструкторе (__construct) среди аргументов функции должна быть строка \phpbb\db\driver\driver_interface $db, и соответственно в services.yml - '@dbal.conn'
Создание страниц в стиле конференции

Отправлено спустя 1 минуту 36 секунд:

Обратите внимание: аргументы класса Sheer\mytest\controller\mytest размещаются строго в том же порядке, в каком они перечислены в списке аргументов функции __construct.

Отправлено спустя 2 минуты 12 секунд:
Да, и не забываем, что в теле функции __construct должна быть при этом стока

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

$this->db					= $db;

[Vlad__]

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

$sql = 'UPDATE ' . ATTACHMENTS_TABLE . '
		SET ' . $this->db->sql_build_array('UPDATE', $sql_data) . '
		WHERE attach_id = ' . $filedata['attach_id'];

А тут UPDATE два раза - так нужно?

В конструкторе (__construct)

Спасибо! Внес изменения, теперь не ругается, но теперь ничего не происходит. Уже всю голову поломал )))

Привожу всю функцию:

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

public function upload_image_convert($event)
        {
                $is_image = $event['is_image'];
                $filedata = $event['filedata'];
                $destination_file = $this->phpbb_root_path . $this->config['upload_path'] . '/' . $filedata['physical_filename'];
                if ($is_image)
                {
                        if (function_exists('exif_imagetype') && ($filedata['extension'] == 'png'))
                        {

// вот тут нужен код замены имени файла, расширения, миметипа этого вложения в дб
// затем будет идти код конвертации png в jpg (он у меня есть и работает), вопрос только в расширении файла
                        }
          }
}

А выполняется ли условие if (function_exists('exif_imagetype') && ($filedata['extension'] == 'png'))

Выполняется.

Отправлено спустя 23 минуты 2 секунды:
Конечно лучше менять расширение файла до загрузки или во время ее, а формат файла определять только по exif_imagetype (думаю это не критично), но моих знаний не хватает это реализовать (((