Skip to content

Commit

Permalink
TODO for strategy removal
Browse files Browse the repository at this point in the history
  • Loading branch information
st8st8 committed Mar 15, 2024
1 parent 2516a2d commit 7a5fa8f
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/oscar/apps/basket/abstract_models.py
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,10 @@ def merge_line(self, line, add_quantities=True, strategy=None):
This is used with the "Saved" basket functionality.
"""
# TODO: If the login causes the strategy to be None, the line won't save.
# IntegrityError at /checkout/
# (1048, "Column 'stockrecord_id' cannot be null")
# We'll need to display a message if that's the case.
try:
existing_line = self.lines.get(line_reference=line.line_reference)
except ObjectDoesNotExist:
Expand Down

0 comments on commit 7a5fa8f

Please sign in to comment.