aleksb86 / news Goto Github PK

FIXME:

• результаты поиска, при клике на заголовок найденных постов (переход в show) - template missing.
• рендеринг пагинации после апдейта, удаления - неправильно рендерятся ссылки страниц.
• отрисовка страницы show (post) после редактирования в show.
• под учеткой редактора не видно кнопки "создать статью".
• порядок вывода статей - стэк - последняя вверху списка (так должно быть, сейчас посты выводятся в обратном порядке).
• после создания статьи - нужно переходить в show созданной статьи (сейчас происходит редирект к общему списку).

Its better to put default image file to '/public'.

1. Удалить бестолковые (незначащие) комменты везде, кроме файлов, сгенерированных автоматически и не редактированных вручную.

2. Удалить объекты, функциональность которых не реализована до конца - например, методы контроллеров, недоступные из вью; неиспользуемая конфигурация; любой лишний недоступный код.

3. posts_controller:
   3.1. убрать лишние вызовы respond_to - не должно быть пустых вызовов, оставить только там, где происходит отправка через конкретный формат: js, html, json.
   3.2. устранить дублирование создания Attachment для поста - вынести в отдельный метод из create и update.
   3.3. разобраться, почему происходит лишний GET запрос при создании поста (проверить также, не происходит ли этого при create).
   3.4. убрать проверку отправки формы (if request.xhr? || remotipart_submitted?) - пока это бессмысленно.
   3.5. исправить экшны create, update - происходит(missing template) http 500.
   3.6. вычитстить код контроллера - комменты, посторонние фраменты и прочий мусор.

4. attachments_controller:
   4.1. убрать проверку last_modified для пустого аттачмента - ибо не нужно.
   4.2. убрать (переписать) безобразное условие - проверку :noimage.
   4.3. убрать @post из дестроя - он не нужен, т.к надо работать с attachments напрямую.
   4.4. убрать из дестроя лишние вызовы respond_to.
   4.5. провести в порядок код контроллера - устранить лишний мусор.

5. Локализация! при помощи стандартной локали фреймворка.