Архив | Об архиве | FAQ | New BAN List | Полезные ссылки | Друзья архива | Архив новостей |
|
Регистрация | Справка | Правила форума | Поиск | Сообщения за день | Пользователи | Календарь |
Приёмная ArjLover-a Послания и пожелания по работе Архива |
|
Опции темы | Опции просмотра |
|
|||
Цитата:
2 lamer0: я чисто случайно заглянул на огонек и очень удивился, до сих пор не увидев работающей версии... Счел должным потратить пару часов ради любимого всеми нами сайта |
|
|||
Слегка поправил код и убрал многие ошибки в arj1.php
Из известных неудобств остались ввод даты в виде "1981-12-25 17:15:25" и исчезающий знак "\" в полях ввода. Надо бы еще окончательно почистить код и исправить хотя бы эти недостатки, но сейчас у меня нет времени. ЗЫ. Обязательно нужно переименовать поле "long" базы данных во что-либо более другое. Например "duration". |
|
|||
Посмотрел на изменения. По-большому счету, похвалы заслуживает только одно - вставка htmlspecialchars в выводе полей на экран. Все остальные изменения - мелочи, не сильно относящиеся к делу. Ну, убрали Вы параметры $msg_code и $persistenсy из функций оригинального кода Uswer'а - а зачем, собственно ?
Кстати, заодно объясните мне, чем 2 нижеприведенные строки (первый вариант - мой, второй - ваш) функционально отличаются друг от друга ? Может я чего-то не понимаю... Код:
<input type='hidden' name='f' value='<?=$filename?>'> <input type='hidden' name='f' value='<?php echo $filename; ?>'> |
|
|||
Затем, что код стал меньше и проще. Зачем тут атавизмы phpbb? И я не утверждал, что написал что-то своё, я просто поправил по-мелочи код. И вы вообще запускали этот код, тестировали? Или у вас error_reporting выключен чтобы не видеть ошибок? Та же CRITICAL_ERROR связанная с $msg_code не определена вообще, например, на что у меня PHP сразу выдавал ошибку. И во всех остальных местах исправления сделаны потому что у меня вылазили сообщения PHP.
Код отличается тем, что первая строчка не работает на серверах, где запрещен короткий открывающий тег (в конфиге php.ini стоит short_open_tag=off), а вторая работает везде. |
|
|||
Добавил несколько перепроверок, исправил оба недостатка. Вроде в тексте ничего не пропадает, и от посторонних символов ошибок не вылезает. Даты вводятся тремя списками (день/месяц/год), хотя зачем, я не понимаю, ведь pubdate как я понимаю должна заполняться автоматически?
Отформатировал код под себя, извиняюсь если кому не понравится. В общем проделал бесполезную работу. |
|
||||
Цитата:
Цитата:
Мне проще воткнуть туда пару define, нежели переписывать половину кода. Цитата:
Посыпаю голову пеплом... Цитата:
|
|
|||
Цитата:
Завтра посмотрю насчет объединения последних вариантов во что-то более лучшее. Также хотелось бы уже услышать мнение самого ArjLover'а, в какую сторону далее развивать писанину. |
|
|||
Цитата:
Цитата:
|
|
|||
Если koan не отзовется (вроде как он последнюю версию доделывал) - мой e-mail чуть выше. Если я правильно помню, там выводились все поля, присутствующие в таблице, форма ввода определялась их типом. Нужно точное описание того, что поменялось, не работает или работает не так.
|
|
||||
Так что-то видать первоначальное ТЗ где-то затерялось, все технические поля не редактируются из этой формы.
Даже не знаю отображать их или нет. Коллеги, ведущие высказывайтесь. Я вижу это так что часть надо отображать, часть из них даже сделать редактируемыми, типа картинки или видео - там штук пять avinfo не может распарсить, проще руками их поправить раз в жизни. Остальные техполя по-моему только мешаются в этой форме, хочется чтобы она умещалась на экран и была удобной в работе. |
|
|||
Просто поля надо свернуть сокращённо в одну строчку, и сделать раскрываемыми для редактирования.
Заметил глюк — окна заточены под какой-то другой шрифт, что ли? MD4 и MD5, например, сейчас не показываются целиком. |
|
|||
Ещё надо добавить, и надо подумать, какую, структуру о совместимом / несовместимом железном плеере — будем создавать народую базу знаний непосредственно на основе скачанного, так гораздо проще — "у меня, на таком-то ящике, данное кино видно / не видно".
|
|
|||
Например, выкидное меню для выбора из названий моделей плееров, подтверждённых модератором, с дальнейшим однозначным "да / нет". Тут же, на волне свежих эмоций, можно ещё дать вход в ветку БД по плеерам оставить комментарий про свой, особый случай...
|
|
||||
В поле "year" заранее прописать 19
В поле "country" должен быть сразу чекбокс выбора, стран немного. Вообще должно быть полно чекбоксов - для всех нетехнических полей, кроме сюжета. Причем везде д/б можно как выбрать, так и дописать новое слово, которое попадает сразу и в список выбора. В поле режиссера, актеров, жанра - выбор должен быть множественным - прямо по списку - первый, восемнадцатый, 24-й. То, что не редактируюются технические поля, волнует не в первую очередь. Если будет корректно работать - вообще не волнует. Необходимость прокрутки считаю несущественным недостатком, важнее, чтобы было куда писать все нужное. С расположением меня не спрашивайте, у меня шрифт 22. |