Actions
Bug #8442
closed
Eager loading on audits page does not work
Description
ActiveRecord::EagerLoadPolymorphicError Can not eagerly load the polymorphic association :user app/views/audits/_list.html.erb:1:in `_53d8a19f5b9e17a52dbbf2112ae4186e' app/views/audits/index.html.erb:3:in `_cde286859771808c11fbbbdf6d2f4212' app/models/concerns/foreman/thread_session.rb:33:in `clear_thread' lib/middleware/catch_json_parse_errors.rb:9:in `call'
Updated by Ohad Levy over 9 years ago
- Related to Bug #8425: n+1 query on audits index added
Updated by Dominic Cleal over 9 years ago
- Category set to Database
- Priority changed from Normal to High
- translation missing: en.field_release set to 28
Updated by The Foreman Bot over 9 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/1964 added
- Pull request deleted (
)
Updated by Joseph Magen over 9 years ago
- Status changed from Ready For Testing to Closed
merged and changed commit msg to fixes #8442 - add eager loading of users on audit show page for history tab
Updated by Shlomi Zadok over 9 years ago
- Status changed from Closed to New
Weird enough, https://github.com/theforeman/foreman/pull/1953/files and https://github.com/theforeman/foreman/pull/1964 should have cover this issue (which was tested by Ori and me) and now there is a regression.
re-opening (and fixing...)
Updated by Shlomi Zadok over 9 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2040 added
- Pull request deleted (
https://github.com/theforeman/foreman/pull/1964)
Fixed at https://github.com/theforeman/foreman/pull/2040 (typo :/ )
Updated by Shlomi Zadok over 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Updated by Dominic Cleal over 9 years ago
- Assignee changed from Shlomi Zadok to Joseph Magen
Actions