Покинул форум
Сообщений всего: 7
Дата рег-ции: Июнь 2013
Помог: 0 раз(а)
Оцените сайт, сайт написан на PHP, с нуля, уже много чего было переписано/переработано, вот теперь хотелось бы услышать как вам сайт, что еще нужно сделать/переделать/доработать. Зарание спасибо
ссылка на GitHub: https://github[dot]com/Seriyyy95/pcpostОтредактировано администратором: EuGen, 03 Июня, 2013 - 15:09:27 Ссылки не допускаются к размещению на конференции, желающие могут уточнить адрес через сервис личных сообщений
EuGen
Отправлено: 03 Июня, 2013 - 15:08:43
Профессионал
Покинул форум
Сообщений всего: 9095
Дата рег-ции: Июнь 2007 Откуда: Berlin
/sections/show/page/-1
MySQL error: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '-20, 10' at line 1
- как обычно, проверки на int не везде достаточна;
- выдаёт 400 bad request (правда, похитрее оформленный). Введена была тарабарщина, но я ожидал увидеть, где ошибся.
Это всё бегло просмотренные ошибки. Вероятно, есть и более серьёзные недочёты.
Ну и клон habrahabr, думаю, не будет особенно популярным на фоне оригинала.
----- Есть в мире две бесконечные вещи - это Вселенная и человеческая глупость. Но насчет первой .. я не уверен.
LinuxUser
Отправлено: 03 Июня, 2013 - 15:17:16
Новичок
Покинул форум
Сообщений всего: 7
Дата рег-ции: Июнь 2013
Помог: 0 раз(а)
EuGen пишет:
Стандартно
Спойлер (Отобразить)
- как обычно, проверки на int не везде достаточна;
Спойлер (Отобразить)
- мало того, что удаётся вызвать листинг всех материалов (привет, DoS на страницу), так ещё и 400 от хостинга выдаётся при использовании пагинатора.
Спасибо, буду исправлять
esterio
Отправлено: 03 Июня, 2013 - 15:43:38
Активный участник
Покинул форум
Сообщений всего: 5025
Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов
Помог: 127 раз(а)
ну скиньте уже и мне в личку урл. уж больно интересно стало после поста EuGen. И если несекрет Z_LoadClassException я так понял ето Zend Framework?
Crate
Отправлено: 03 Июня, 2013 - 15:45:04
Посетитель
Покинул форум
Сообщений всего: 312
Дата рег-ции: Янв. 2012
Покинул форум
Сообщений всего: 5025
Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов
Помог: 127 раз(а)
почему мне дизайн тянет так на твердую 3+. Может просто я не дизайнер и чаще всего оцениваю код. А так кода нет, то и оценивать нечего. А багы искать щс как-то не охота
LinuxUser
Отправлено: 03 Июня, 2013 - 16:20:21
Новичок
Покинул форум
Сообщений всего: 7
Дата рег-ции: Июнь 2013
Помог: 0 раз(а)
Crate пишет:
Ну чтож, по коду вам уже все сказали, а дизайн остался в районе веб 0.1
- значит еще поработаю и над дизайном, спасибо
DeepVarvar
Отправлено: 03 Июня, 2013 - 16:30:26
Активный участник
Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008 Откуда: Альфа Центавра
Помог: 353 раз(а)
Код выложи - люди просят.
На гитхаб например.
Видно же что ты зарегался только ради этого раздела ))
Покинул форум
Сообщений всего: 7
Дата рег-ции: Июнь 2013
Помог: 0 раз(а)
DeepVarvar пишет:
Код выложи
Окей, сделаю
LinuxUser
Отправлено: 03 Июня, 2013 - 20:08:38
Новичок
Покинул форум
Сообщений всего: 7
Дата рег-ции: Июнь 2013
Помог: 0 раз(а)
esterio пишет:
почему мне дизайн тянет так на твердую 3+. Может просто я не дизайнер и чаще всего оцениваю код. А так кода нет, то и оценивать нечего. А багы искать щс как-то не охота
DeepVarvar пишет:
Код выложи - люди просят.
На гитхаб например.
Ссылку на Github скинул по емейл, потому что не уверен можно ли ее размещать здесь)
DeepVarvar
Отправлено: 03 Июня, 2013 - 21:29:17
Активный участник
Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008 Откуда: Альфа Центавра
Помог: 353 раз(а)
1) sitemap.xml - лишние указатели на шемы, почитай за формат.
2) /Settings/site.ini - считай что в базу тебе уже зашли, т.к. конфиг доступен по прямой ссылке.
3) Все пароли пользаков в БД хранятся в открытом виде - считай что их ВСЕХ у тебя тоже слили.
По коду все нормально, обычно, не сложно, понятно.
Если и искать косяки - то надо вникать.
Ссылку с гитхаба сюда положить конечно можно.
Может кто-то еще найдет что-то.
Покинул форум
Сообщений всего: 7
Дата рег-ции: Июнь 2013
Помог: 0 раз(а)
DeepVarvar пишет:
1) sitemap.xml - лишние указатели на шемы, почитай за формат.
2) /Settings/site.ini - считай что в базу тебе уже зашли, т.к. конфиг доступен по прямой ссылке.
3) Все пароли пользаков в БД хранятся в открытом виде - считай что их ВСЕХ у тебя тоже слили.
По коду все нормально, обычно, не сложно, понятно.
Если и искать косяки - то надо вникать.
Ссылку с гитхаба сюда положить конечно можно.
Может кто-то еще найдет что-то.
Все гости форума могут просматривать этот раздел. Только зарегистрированные пользователи могут создавать новые темы в этом разделе. Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.