Skip to content

Commit 6730f87

Browse files
committed
address code review
1 parent 7940123 commit 6730f87

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

apps/sponsors/manage/views.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -836,9 +836,13 @@ def form_valid(self, form):
836836

837837
if clone_benefits:
838838
for benefit in SponsorshipBenefit.objects.filter(year=source_year):
839-
_, created = benefit.clone(target_year)
839+
new_benefit, created = benefit.clone(target_year)
840840
if created:
841841
cloned_benefits += 1
842+
if not clone_packages:
843+
# benefit.clone() creates packages as a side effect;
844+
# clear them when packages weren't requested
845+
new_benefit.packages.clear()
842846

843847
messages.success(
844848
self.request,

0 commit comments

Comments
 (0)