Loopback
|
Posted: Fri Apr 06, 2012 22:16 Post subject: |
|
|
Фикс - 2 строчки кода. Стоило из-за этого спорить?
Что касается стандартов. Спецификация торрентов допускает любые нестандартные сегменты. При этом парсер, разумеется, не должен падать, если встретит такой сегмент. Обрабатывать его или нет - это дело разработчика парсера, но если не обрабатывать - значит нужно корректно пропускать.
TCTorrent - плагин для получения информации о торрент-файле, в этом случае информация лишней не бывает. Хуже от того, что пользователь будет знать о наличии этих ссылок в файле точно не будет. Так или иначе, было принято решение показывать содержимое этого сегмента. То, что он нестандартный, я вообще узнал только сейчас.
Итак, фича уже добавлена, но неправильно, в результате плагин падает. После обнаружения ошибки есть два пути: добавить 2 строки кода, исправляющие проблему, или убрать поддержку "url-list" совсем, причем для этого надо убрать несколько кусков кода в разных местах: код парсера, список для URL, вывод поля, локализацию поля. По-моему, выбор пути очевиден. |
|