lest (Just Lest)

Name
Just Lest
Website/Blog
http://justlest.info
Company
Twinslash
Location
Belarus/Minsk
Member Since
Nov 22, 2008

Following 2 coders and watching 26 repositories view all →

Public Repositories (20)

Public Activity feed

lest commented on pinifloyd/Playtime Tue Aug 10 05:24:42 -0700 2010
Comment on app/views/products/_comment.html.erb L7 in 1075e5e:

При подобном выводе комментариев мы получаем большое количество SQL-запросов к базе данных (можно увидеть в консоли).

Необходимо сделать так, чтобы количество запросов к базе было фиксированным и не зависело от количества комментариев и уровня их вложенности.

В случае необходимости можно использовать верстку без вложенных списков, например:

<ul>
  <li style="padding-left: 0px;">Комментарий уровня 1</li>
  <li style="padding-left: 20px;">Комментарий уровня 2</li>
  <li style="padding-left: 40px;">Комментарий уровня 3</li>
  <li style="padding-left: 0px;">Комментарий уровня 1</li>
</ul>
lest closed issue 8 on lest/redmine_thumbnails Mon Aug 09 00:46:01 -0700 2010
in redmine 1.0.0 this plugin renders inside <pre> tags
lest pushed to master at lest/redmine_thumbnails Mon Aug 09 00:34:25 -0700 2010
  • a92c3b4
    fix and move thumb size logic into controller
lest commented on Dreamfa11/depot Fri Aug 06 21:50:06 -0700 2010
Comment on app/controllers/products_controller.rb L55 in bf54bb4:

Создание корневого коммента при добавлении продукта необходимо перенести в модель.
В контроллере не должно быть подобной логики, потому как впоследствии, к примеру, продукт может создавать не только в этом контроллере, а еще где-нибудь.
Также хорошим правилом считается в одном action контроллера вызывать один метод модели (или 2 метода отвечающие за разные действия над одной и той же моделью, например, new и save, find и update_attributes, find и destroy).

Еще вместо метода create лучше использовать create! в похожих на этот случай.
Просто create при ошибках валидации только заполнит свойство errors у comment, а раз мы никак ошибки не обрабатываем, но в случае их наличия ничего не узнаем.
А метод create! при ошибке валидации бросит exception, поэтому если что-то написано не так, мы сразу увидим.

lest commented on Dreamfa11/depot Thu Aug 05 08:40:11 -0700 2010
Comment in 4530e52:

Если долго не будет получаться, то не зацикливайся, следующий раз, когда придешь на офис, расскажу и покажу ;-)

lest pushed to master at lest/emacs-config Thu Aug 05 07:13:11 -0700 2010
lest pushed to master at lest/emacs-config Thu Aug 05 06:50:12 -0700 2010
  • 4d27473
    add recentf and organize defuns
  • f56a8d7
    espresso-mode on js.erb files
lest pushed to master at lest/emacs-config Tue Aug 03 22:43:20 -0700 2010
lest pushed to master at lest/emacs-config Tue Aug 03 12:39:54 -0700 2010
  • c148e6e
    ruby-mode for .irbrc and .caprc
lest commented on pinifloyd/Playtime Tue Aug 03 04:24:04 -0700 2010
Comment in b9f84f7:

у продукта может быть много комментов, поэтому product has_many :comments
коммент принадлежит продукту, поэтому comment belongs_to :product

у тебя наоборот сделано

в ближайшее время Витя напишет в группу письмо с рекомендациями по выполнению этой задачи, думаю, что это прояснит остальные вопросы

lest pushed to master at lest/emacs-config Mon Aug 02 12:54:46 -0700 2010
  • e9c401c
    modify ido ignore buffers
  • a2bf722
    remove window switching with arrows
lest pushed to master at lest/emacs-config Mon Aug 02 12:53:43 -0700 2010
  • a7cd08f
    properly turn off scrollbar and toolbar
lest pushed to master at lest/emacs-config Mon Aug 02 12:53:15 -0700 2010
lest pushed to master at lest/emacs-config Fri Jul 30 02:58:06 -0700 2010
  • b4f1426
    ignore and delete ede-projects.el file
  • 46abfe6
    update config and vendored modes
lest commented on pinifloyd/Playtime Thu Jul 29 07:30:43 -0700 2010
Comment on app/views/layouts/store.html.erb L20 in a33ed52:

к тому же условие можно записать так:

if @count && @count > 5
lest commented on pinifloyd/Playtime Thu Jul 29 07:24:08 -0700 2010
Comment on app/controllers/store_controller.rb L11 in a33ed52:

можно заменить на:

session[:counter] ||= 0

если не совсем понятно, как это работает, то попробуй в консоли поиграться с || и ||=

lest commented on pinifloyd/Playtime Thu Jul 29 04:15:52 -0700 2010
Comment on app/models/product.rb L4 in 5a474e9:

проверять наличие загруженной картинки нужно, только это делается немного другим способом

см. документацию по paperclip (методы, начинающиеся с validates_)

lest commented on Dreamfa11/depot Thu Jul 29 00:36:15 -0700 2010
Comment on app/views/products/new.html.erb L8 in 044d861:

f.buttons не должен быть внутри f.inputs, иначе там неправильный html код получается

т.е. должно быть примерно так:

<% f.inputs do %>
....
<% end %>
<%= f.buttons %>
lest pushed to master at lest/depot Tue Jul 27 07:18:39 -0700 2010
lest pushed to master at lest/depot Tue Jul 27 07:14:43 -0700 2010
lest pushed to master at lest/depot Tue Jul 27 07:05:43 -0700 2010
lest commented on lest/depot Tue Jul 27 07:02:33 -0700 2010
Comment in 6fc0706:

test comment

lest created branch master at lest/depot Tue Jul 27 07:02:02 -0700 2010
New branch is at lest/depot/tree/master
lest created repository depot Tue Jul 27 06:53:31 -0700 2010
lest commented on Dreamfa11/depot Fri Jul 23 07:45:32 -0700 2010
Comment on app/controllers/store_controller.rb L6 in ca0e713:

вполне нормальная практика, когда метод изменения значения, возвращает измененное значение

так что вполне ок, за исключением названия метода (лучше уже назвать его increment_visits_counter)

методы контроллера, не являющиеся экшенами нужно делать private (или protected)

lest forked edavis10/redmine_hoptoad_server Thu Jul 22 23:35:55 -0700 2010
Forked repository is at lest/redmine_hoptoad_server
lest commented on yas375/depot Thu Jul 22 01:54:20 -0700 2010
Comment in 06fec8f:

test comment

lest pushed to twinslash at lest/redmine Mon Jul 12 00:42:35 -0700 2010
lest pushed to master at lest/redmine Mon Jul 12 00:42:31 -0700 2010
lest pushed to twinslash at lest/redmine Mon Jul 12 00:40:10 -0700 2010
  • 1a1c9e5
    add current issue tracker on edit issue form
lest pushed to twinslash at lest/redmine Thu Jul 08 12:42:52 -0700 2010
lest pushed to master at lest/redmine Thu Jul 08 12:42:46 -0700 2010
  • 1aacfec
    Added documentation about the session_path. #3968
  • 3dd90ed
    Hide the role forms when editing or adding Project members. #5452
  • c81751e
    Accept email from anonymous users with an empty from address. ...
  • 40 more commits »
lest pushed to master at yas375/redmine_selected_issues_summary Tue Jun 29 01:09:43 -0700 2010
  • c03ad2a
    skip event handling on tag A
lest pushed to master at lest/emacs-config Mon Jun 14 01:46:18 -0700 2010
  • 416670e
    turn on indicate empty lines
  • af73442
    use ruby-mode on .rjs files
lest pushed to master at lest/emacs-config Fri Jun 11 04:14:57 -0700 2010

Your current locale selection: English. Choose another?