Skip to content

Commit

Permalink
[FIX] easy_my_coop: Fix name computation crash
Browse files Browse the repository at this point in the history
Don't use value from self when looping over a list

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
  • Loading branch information
carmenbianca committed Jun 15, 2022
1 parent 0c18cae commit ab75fd1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion easy_my_coop/models/subscription_request.py
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ def check_iban(self, iban):
def _compute_name(self):
for sub_request in self:
sub_request.name = " ".join(
part for part in (self.firstname, self.lastname) if part
part for part in (sub_request.firstname, sub_request.lastname) if part
)

@api.multi
Expand Down
1 change: 1 addition & 0 deletions easy_my_coop/readme/newsfragments/330.bugfix.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fix name computation crash

0 comments on commit ab75fd1

Please sign in to comment.