Что-то странное творится (хелпер форм)

Нашли ошибку в CI? — пишем сюда!

Что-то странное творится (хелпер форм)

Сообщение taral » 16 мар 2009, 01:32

Очень забавную штуку я заметил. Правда не в CI а в Kohana. Но решил все равно написать, поскольку фраемворки очень похожи.
Есть у коханы в хелпере функция которая создает формочку для загрузки файла.
Вот вырезка из мануала.
Example

Код: Выделить всё
$attributes = array('name' => 'file_1', 'class' => 'your-class');
echo form::upload($attributes, 'path/to/local/file')


Result in HTML:

Код: Выделить всё
<input type="file" name="file_1" value="path/to/local/file" class="your-class" />


Меня смущает второй параметр у функции. Он типа значение вбивает в форму. Но это же абсурд. У формы загрузки файлов запрещено это поле.
Я бы не обратил внимание если бы это поле передавалось в масиве. Тогда бы ладно. Но они сочли это поле настолько важным что оно передается во втором параметре=) Или от того что я жутко спать хочу голова моя не варит. Или программеры коханы напортачили (понятно что ошибка несущественная, но очень забавная)
Аватара пользователя
taral
 
Сообщения: 166
Зарегистрирован: 03 ноя 2008, 21:01

Re: Что-то странное творится (хелпер форм)

Сообщение Big_Shark » 16 мар 2009, 02:13

хм возможно какие то браузеры поддерживают данное свойство или планирую его ввести.
Отвечу на ваши вопросы по CodeIgniter и Kohana3
Благодарности принимаются тут: 41001216991355 (ЯД)
Big_Shark
Модератор
 
Сообщения: 1307
Зарегистрирован: 03 фев 2009, 02:12
Откуда: Владивосток

Re: Что-то странное творится (хелпер форм)

Сообщение taral » 17 мар 2009, 00:59

хм возможно какие то браузеры поддерживают данное свойство или планирую его ввести.

Очень очень сомневаюсь. Это свойство закрыто неспроста. Причина безопасность. Как известно js может отправлять форму. А теперь представляем сайт на который заходит пользователь, в форме отправки файла вбито C:\WINDOWS\system32\system.exe или что то другое. И скрипт отправляет форму сразу после загрузки страницы=) Ой как тырить файлы можно=) Тут и трояны ненужны. Собсно потому я и создал тему. Логика тех программеров мне не ясна=)
Аватара пользователя
taral
 
Сообщения: 166
Зарегистрирован: 03 ноя 2008, 21:01

Re: Что-то странное творится (хелпер форм)

Сообщение Big_Shark » 17 мар 2009, 01:40

А ведь верно)
Попробуйте задать этот вопрос программерам которые писали этот код может они знают)
Отвечу на ваши вопросы по CodeIgniter и Kohana3
Благодарности принимаются тут: 41001216991355 (ЯД)
Big_Shark
Модератор
 
Сообщения: 1307
Зарегистрирован: 03 фев 2009, 02:12
Откуда: Владивосток

Re: Что-то странное творится (хелпер форм)

Сообщение Master » 17 мар 2009, 02:20

taral писал(а):Правда не в CI а в Kohana. Но решил все равно написать, поскольку фраемворки очень похожи.

Весьма странная логика. Давайте я буду писать сюда о багах Zend Framework только потому что это_тоже_фреймворк и написан_он_тоже_на_похапэ?

В CI нет кода, который вы написали.
Master
 
Сообщения: 230
Зарегистрирован: 13 мар 2009, 13:22

Re: Что-то странное творится (хелпер форм)

Сообщение taral » 17 мар 2009, 17:03

Весьма странная логика. Давайте я буду писать сюда о багах Zend Framework только потому что это_тоже_фреймворк и написан_он_тоже_на_похапэ?
В CI нет кода, который вы написали.

С ваших слов сразу ясно что с кохана вы дело не имели. Когда я говорил что фраемворки очень похожи то говорил не просто так. Кохана это вообще ветка развития CI. О они очень похожи.
И вот доказательство. Я специально проверил присутствует ли этот баг в CI. И был прав. Смотрим хелпер form функцию form_upload
Она аналогичная.
В CI нет кода, который вы написали.

Спасибо за информацию.
Аватара пользователя
taral
 
Сообщения: 166
Зарегистрирован: 03 ноя 2008, 21:01

Re: Что-то странное творится (хелпер форм)

Сообщение Master » 17 мар 2009, 18:57

taral писал(а):Я специально проверил присутствует ли этот баг в CI. И был прав. Смотрим хелпер form функцию form_upload
Она аналогичная.

Я тоже проверил, и этого кода там не нашёл :)
Master
 
Сообщения: 230
Зарегистрирован: 13 мар 2009, 13:22

Re: Что-то странное творится (хелпер форм)

Сообщение taral » 17 мар 2009, 23:10

Я тоже проверил, и этого кода там не нашёл :)

Ну что же поздравляю, вы плохо искали. Сейчас я вам его покажу.
system -> helpers -> form_helper.php -> 186 строка (в сборке CI 1.7)
Вот собсно и она.
Код: Выделить всё
function form_upload($data = '', $value = '', $extra = '')
{
   if ( ! is_array($data))
   {
      $data = array('name' => $data);
   }
   $data['type'] = 'file';
   return form_input($data, $value, $extra);
}

Как видим второй параметр $value. Что и требовалось доказать.
Аватара пользователя
taral
 
Сообщения: 166
Зарегистрирован: 03 ноя 2008, 21:01

Re: Что-то странное творится (хелпер форм)

Сообщение Master » 18 мар 2009, 02:43

Да, действительно. Правда, это глобальная функция, а не метод класса, и параметров в ней три, а не два (последний - $extra).
Я думаю, сабж был сделан ради единообразия, подобия функции form_input(), как это и написано в документации.
Master
 
Сообщения: 230
Зарегистрирован: 13 мар 2009, 13:22

Re: Что-то странное творится (хелпер форм)

Сообщение Sam Dark » 18 мар 2009, 13:38

Переносим в баги? Репортим?
Аватара пользователя
Sam Dark
 
Сообщения: 824
Зарегистрирован: 16 апр 2008, 19:23
Откуда: Воронеж

Re: Что-то странное творится (хелпер форм)

Сообщение taral » 18 мар 2009, 18:47

Переносим в баги? Репортим?

Да, не против.
Правда, это глобальная функция, а не метод класса, и параметров в ней три, а не два

А я что говорил что это метод класса? И что говорил что параметров 3?
Я говорил у хелпера (у коханы это метод, у CI функция). И говорил "второй" параметр, а не что их 3.
Аватара пользователя
taral
 
Сообщения: 166
Зарегистрирован: 03 ноя 2008, 21:01


Вернуться в Баг-репорты

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 0

cron