includes create bad queries. don't use them for content lists
Change-Id: I4afafae06d4a799aaa626f40f37f49dae4af7c13 Reviewed-on: https://gerrit.instructure.com/24609 Reviewed-by: Brian Palmer <brianp@instructure.com> Reviewed-by: James Williams <jamesw@instructure.com> Tested-by: Jenkins <jenkins@instructure.com> Product-Review: Cody Cutrer <cody@instructure.com> QA-Review: Cody Cutrer <cody@instructure.com>
This commit is contained in:
parent
c4914183bc
commit
803ca91821
|
@ -216,7 +216,7 @@ module Canvas::Migration::Helpers
|
|||
elsif type == 'discussion_topics'
|
||||
count = source.discussion_topics.active.only_discussion_topics.count
|
||||
elsif source.respond_to?(type) && source.send(type).respond_to?(:count)
|
||||
scope = source.send(type)
|
||||
scope = source.send(type).except(:includes)
|
||||
if scope.respond_to?(:not_deleted)
|
||||
scope = scope.not_deleted
|
||||
elsif scope.respond_to?(:active)
|
||||
|
|
Loading…
Reference in New Issue