Sign upLogin

GeekPark/gpk_account

View on GitHub
Star
app/controllers/api/v1/direct_messages_controller.rb

Summary

Maintainability
A
0 mins
Test Coverage

Avoid using update_all because it skips validations. (http://guides.rubyonrails.org/active_record_validations.html#skipping-validations)
Open

    current_user.unread_dm_between(params[:user_id]).update_all(unread: false)
Severity: Minor
Found in app/controllers/api/v1/direct_messages_controller.rb by rubocop

This cop checks for the use of methods which skip validations which are listed in http://guides.rubyonrails.org/active_record_validations.html#skipping-validations

Example:

# bad
Article.first.decrement!(:view_count)
DiscussionBoard.decrement_counter(:post_count, 5)
Article.first.increment!(:view_count)
DiscussionBoard.increment_counter(:post_count, 5)
person.toggle :active
product.touch
Billing.update_all("category = 'authorized', author = 'David'")
user.update_attribute(website: 'example.com')
user.update_columns(last_request_at: Time.current)
Post.update_counters 5, comment_count: -1, action_count: 1

# good
user.update_attributes(website: 'example.com')
FileUtils.touch('file')

There are no issues that match your filters.

Category
Status