keyword_sprint type_defect | by rjsparks@nostrum.com
There are more than a dozen places in the code where querysets are constructed to select the currently active ads. There are variants of that selection that include Person objects with pre-ad roles. The logic defining is also captured in the active_ad_positions method.
This should be refactored to capture the logic in one place and allow it to be conveniently used in those others.
Issue migrated from trac:1576 at 2022-03-04 03:58:53 +0000
keyword_sprinttype_defect| by rjsparks@nostrum.comThere are more than a dozen places in the code where querysets are constructed to select the currently active ads. There are variants of that selection that include Person objects with pre-ad roles. The logic defining is also captured in the active_ad_positions method.
This should be refactored to capture the logic in one place and allow it to be conveniently used in those others.
Issue migrated from trac:1576 at 2022-03-04 03:58:53 +0000