Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved wishlist based on @dfalk's work #105

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Improved wishlist based on @dfalk's work #105

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
b888094
store wishlist in database
dfalk Sep 2, 2012
9d389e0
pep8 fixes
dfalk Sep 3, 2012
1f927d8
Extract middleware wishlist loading into WishlistManager and improve …
uranusjr Apr 27, 2013
7b4e30e
Extract wishlist adding and removing
uranusjr Apr 27, 2013
fadb96f
Add hook to transfer cookie-stored wishlist to database when user log…
uranusjr Apr 27, 2013
0f62937
Use absolute import
uranusjr Apr 27, 2013
dbc17b4
Bug fixes and migration rename
uranusjr Apr 27, 2013
2f99de5
Remove duplicate migration file
uranusjr Apr 28, 2013
c6f0abc
Fix AttributeError in login hook
uranusjr Apr 28, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix AttributeError in login hook
  • Loading branch information
@uranusjr
uranusjr committed Apr 28, 2013
commit c6f0abc7ade3c9ed7fe64604811d892e67c326c3
4 changes: 3 additions & 1 deletion cartridge/shop/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -887,7 +887,9 @@ class Meta:

@receiver(user_logged_in, sender=User)
def transfer_wishlist_data(sender, request, user, *args, **kwargs):
skus = request.wishlist
skus = getattr(request, "wishlist", [])
if not skus:
return
existed = (Wishlist.objects.filter(user=user, sku__in=skus)
.values_list("sku", flat=True))
for sku in skus:
Expand Down