Conversation
Riki4321
left a comment
Riki4321
left a comment
There was a problem hiding this comment.
@localdotcom нужно внимательно просмотреть дифф. я оставил комменты - там вырезано то, что делалось специально для warecorp когда-то.
сс @hronix
| projects = Project.order('name ASC') | ||
| unless User.current.admin? | ||
| projects = projects.where(Project.visible_condition(User.current)) | ||
| if User.current.admin? |
There was a problem hiding this comment.
А для чего было менять эту конструкцию?
она делает ровно то же самое, что и раньше, только до этого она была написала в Ruby-стиле, т.е. более корректно.
There was a problem hiding this comment.
а, понял, это видимо было взято из оригинального плагина, т.к. изменение конструкции было сделано нами тут: a4ed005
| query = CGI.unescape({ :timesheet => timesheet.to_param }.to_query) | ||
| form_params = query.split("&").map {|pair| pair.split("=")} | ||
| button_to('CSV', | ||
| link_to('CSV', |
There was a problem hiding this comment.
Это изменение было сделано специально для Warecorp когда-то, полагаю, мы не хотим от него отказываться.
59043d1
| include ActiveModel::Conversion | ||
| attr_accessor :date_from, :date_to, :projects, :activities, :users, :groups, :trackers, | ||
| :allowed_projects, :period, :period_type, :include_archived | ||
| :allowed_projects, :period, :period_type |
There was a problem hiding this comment.
Это изменение было сделано специально для Warecorp когда-то, полагаю, мы не хотим от него отказываться: см. коммит
a4ed005
2 participants
No description provided.