Пару вопросов по lmbHttpRequest

[hidrarg]

Пару вопросов по lmbHttpRequest.
Зачем при инициализации выбрасывается исключение? Мне кажется будет достаточно пропускать недопустимые параметры

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


     foreach($this->__request as $k => $v)
     {
       if(in_array($k, $this->__reserved_attrs))
-         throw new lmbException("Attribute with name '$k' is reserved for internal usage");
+        continue;
       $this->set($k, $v);
     }


Сейчас можно на любом сайте на лимб вызвать ошибку, подставив всего один параметр в запрос. Это нормально?

Какой смысл имеет _ensureMultipartFormData?

[korchasa]

Пару вопросов по lmbHttpRequest.
Зачем при инициализации выбрасывается исключение? Мне кажется будет достаточно пропускать недопустимые параметры

Согласен. Поправишь?

Какой смысл имеет _ensureMultipartFormData?

Насколько я понимаю это проверка на то, что файлы могут приходить только из формы с multipart/form-data. Но там тогда ошибка:

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

- if(!$this->hasPost() || $this->__files)
+ if(!$this->hasPost() || !$this->__files)


[hidrarg]

первое - поправлю.
Второе - уберу. если кто вспомнит зачем это надо - вернуть не долго.

[hidrarg]

закомител

[pachanga]

Пару вопросов по lmbHttpRequest.
Зачем при инициализации выбрасывается исключение? Мне кажется будет достаточно пропускать недопустимые параметры

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


     foreach($this->__request as $k => $v)
     {
       if(in_array($k, $this->__reserved_attrs))
-         throw new lmbException("Attribute with name '$k' is reserved for internal usage");
+        continue;
       $this->set($k, $v);
     }


С другой стороны, во время разработки "тихий игнор" этой ошибки можеть иметь большой "WTF?" фактор....

[korchasa]

Второе - уберу. если кто вспомнит зачем это надо - вернуть не долго.

Это была защита от дурака. Вроде бы особого смысла в ней нет.

[korchasa]

С другой стороны, во время разработки "тихий игнор" этой ошибки можеть иметь большой "WTF?" фактор....

Крайне маловероятно, но возможно.

[wIliaM]

Второе - уберу. если кто вспомнит зачем это надо - вернуть не долго.

Это была защита от дурака. Вроде бы особого смысла в ней нет.

Это была действительно защита от дурака. и я, и pachanga в свое время при разработке сайтов натыкались на эту проблему - в форму надо добавить аплоад файла, ты забываешь изменить атрибут формы и тупо втыкаешь почему это не работает. в зависимости от фазы луны , энергетических потоков и эмоционального состояния это стоит от 30 секунд до 30 минут жизни.
имхо можно оставить, т.к. вызов не дает большого оверхеда.

[korchasa]

Второе - уберу. если кто вспомнит зачем это надо - вернуть не долго.

Это была защита от дурака. Вроде бы особого смысла в ней нет.

Это была действительно защита от дурака. и я, и pachanga в свое время при разработке сайтов натыкались на эту проблему - в форму надо добавить аплоад файла, ты забываешь изменить атрибут формы и тупо втыкаешь почему это не работает. в зависимости от фазы луны , энергетических потоков и эмоционального состояния это стоит от 30 секунд до 30 минут жизни.
имхо можно оставить, т.к. вызов не дает большого оверхеда.

Хм, согласен.

[hidrarg]

Второе - уберу. если кто вспомнит зачем это надо - вернуть не долго.

Это была защита от дурака. Вроде бы особого смысла в ней нет.

Это была действительно защита от дурака. и я, и pachanga в свое время при разработке сайтов натыкались на эту проблему - в форму надо добавить аплоад файла, ты забываешь изменить атрибут формы и тупо втыкаешь почему это не работает. в зависимости от фазы луны , энергетических потоков и эмоционального состояния это стоит от 30 секунд до 30 минут жизни.
имхо можно оставить, т.к. вызов не дает большого оверхеда.

А я не согласен. Ситуация такая же, как и в первом случае - внешние данные вызывают не обрабатываемое исключение.
Для защиты от ошибок в форме можно использовать проверку hasFiles - ведь файлы в любом случае должны прийти, если форма была с полями выбора файла, хоть и могут быть пустыми (с кодом 4).

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