Apply a discount code and delete it from the Django session, if it exists [on hold]
up vote
0
down vote
favorite
This piece of code is placed at the end of my checkout process. Once the user paid successfully, I want to check at the end, if there a discount code was used. If so, I want to remove the discount code session. When the user goes back to the event/product page, he has to type in the discount again, in case he wants to use it.
Is there a better way to first check for the session, and then delete it?
# If set, delete discount cookie
discount_code_session_name = 'discount_code_' + str(
self.order_items_dict['event'].pk
)
discount_code_session = request.session.get(discount_code_session_name, None)
if discount_code_session:
del request.session[discount_code_session_name]
python django session
put on hold as off-topic by Mathias Ettinger, Graipher, alecxe, Malachi♦ yesterday
This question appears to be off-topic. The users who voted to close gave this specific reason:
- "Lacks concrete context: Code Review requires concrete code from a project, with sufficient context for reviewers to understand how that code is used. Pseudocode, stub code, hypothetical code, obfuscated code, and generic best practices are outside the scope of this site." – Mathias Ettinger, Graipher, alecxe, Malachi
If this question can be reworded to fit the rules in the help center, please edit the question.
add a comment |
up vote
0
down vote
favorite
This piece of code is placed at the end of my checkout process. Once the user paid successfully, I want to check at the end, if there a discount code was used. If so, I want to remove the discount code session. When the user goes back to the event/product page, he has to type in the discount again, in case he wants to use it.
Is there a better way to first check for the session, and then delete it?
# If set, delete discount cookie
discount_code_session_name = 'discount_code_' + str(
self.order_items_dict['event'].pk
)
discount_code_session = request.session.get(discount_code_session_name, None)
if discount_code_session:
del request.session[discount_code_session_name]
python django session
put on hold as off-topic by Mathias Ettinger, Graipher, alecxe, Malachi♦ yesterday
This question appears to be off-topic. The users who voted to close gave this specific reason:
- "Lacks concrete context: Code Review requires concrete code from a project, with sufficient context for reviewers to understand how that code is used. Pseudocode, stub code, hypothetical code, obfuscated code, and generic best practices are outside the scope of this site." – Mathias Ettinger, Graipher, alecxe, Malachi
If this question can be reworded to fit the rules in the help center, please edit the question.
with suppress(KeyError): del request.session[discount_code_session_name]
. But this question is so stripped of context that I’m voting to close it as off-topic.
– Mathias Ettinger
2 days ago
You mean I should have posted more of my code, what's around it?
– Jon Programmer
2 days ago
4
Yes. It helps getting the bigger picture. Right now you're retrieving the session information name from a dict we know nothing about. Maybe changing this bit could avoid suppressing the session altogether… Who knows?
– Mathias Ettinger
2 days ago
Please see the related meta posts here and there.
– Mathias Ettinger
2 days ago
I tried to explain it better in what context I use it (see edit). I will also check the links you posted, thank you.
– Jon Programmer
2 days ago
add a comment |
up vote
0
down vote
favorite
up vote
0
down vote
favorite
This piece of code is placed at the end of my checkout process. Once the user paid successfully, I want to check at the end, if there a discount code was used. If so, I want to remove the discount code session. When the user goes back to the event/product page, he has to type in the discount again, in case he wants to use it.
Is there a better way to first check for the session, and then delete it?
# If set, delete discount cookie
discount_code_session_name = 'discount_code_' + str(
self.order_items_dict['event'].pk
)
discount_code_session = request.session.get(discount_code_session_name, None)
if discount_code_session:
del request.session[discount_code_session_name]
python django session
This piece of code is placed at the end of my checkout process. Once the user paid successfully, I want to check at the end, if there a discount code was used. If so, I want to remove the discount code session. When the user goes back to the event/product page, he has to type in the discount again, in case he wants to use it.
Is there a better way to first check for the session, and then delete it?
# If set, delete discount cookie
discount_code_session_name = 'discount_code_' + str(
self.order_items_dict['event'].pk
)
discount_code_session = request.session.get(discount_code_session_name, None)
if discount_code_session:
del request.session[discount_code_session_name]
python django session
python django session
edited 2 days ago
200_success
127k15148411
127k15148411
asked 2 days ago
Jon Programmer
1073
1073
put on hold as off-topic by Mathias Ettinger, Graipher, alecxe, Malachi♦ yesterday
This question appears to be off-topic. The users who voted to close gave this specific reason:
- "Lacks concrete context: Code Review requires concrete code from a project, with sufficient context for reviewers to understand how that code is used. Pseudocode, stub code, hypothetical code, obfuscated code, and generic best practices are outside the scope of this site." – Mathias Ettinger, Graipher, alecxe, Malachi
If this question can be reworded to fit the rules in the help center, please edit the question.
put on hold as off-topic by Mathias Ettinger, Graipher, alecxe, Malachi♦ yesterday
This question appears to be off-topic. The users who voted to close gave this specific reason:
- "Lacks concrete context: Code Review requires concrete code from a project, with sufficient context for reviewers to understand how that code is used. Pseudocode, stub code, hypothetical code, obfuscated code, and generic best practices are outside the scope of this site." – Mathias Ettinger, Graipher, alecxe, Malachi
If this question can be reworded to fit the rules in the help center, please edit the question.
with suppress(KeyError): del request.session[discount_code_session_name]
. But this question is so stripped of context that I’m voting to close it as off-topic.
– Mathias Ettinger
2 days ago
You mean I should have posted more of my code, what's around it?
– Jon Programmer
2 days ago
4
Yes. It helps getting the bigger picture. Right now you're retrieving the session information name from a dict we know nothing about. Maybe changing this bit could avoid suppressing the session altogether… Who knows?
– Mathias Ettinger
2 days ago
Please see the related meta posts here and there.
– Mathias Ettinger
2 days ago
I tried to explain it better in what context I use it (see edit). I will also check the links you posted, thank you.
– Jon Programmer
2 days ago
add a comment |
with suppress(KeyError): del request.session[discount_code_session_name]
. But this question is so stripped of context that I’m voting to close it as off-topic.
– Mathias Ettinger
2 days ago
You mean I should have posted more of my code, what's around it?
– Jon Programmer
2 days ago
4
Yes. It helps getting the bigger picture. Right now you're retrieving the session information name from a dict we know nothing about. Maybe changing this bit could avoid suppressing the session altogether… Who knows?
– Mathias Ettinger
2 days ago
Please see the related meta posts here and there.
– Mathias Ettinger
2 days ago
I tried to explain it better in what context I use it (see edit). I will also check the links you posted, thank you.
– Jon Programmer
2 days ago
with suppress(KeyError): del request.session[discount_code_session_name]
. But this question is so stripped of context that I’m voting to close it as off-topic.– Mathias Ettinger
2 days ago
with suppress(KeyError): del request.session[discount_code_session_name]
. But this question is so stripped of context that I’m voting to close it as off-topic.– Mathias Ettinger
2 days ago
You mean I should have posted more of my code, what's around it?
– Jon Programmer
2 days ago
You mean I should have posted more of my code, what's around it?
– Jon Programmer
2 days ago
4
4
Yes. It helps getting the bigger picture. Right now you're retrieving the session information name from a dict we know nothing about. Maybe changing this bit could avoid suppressing the session altogether… Who knows?
– Mathias Ettinger
2 days ago
Yes. It helps getting the bigger picture. Right now you're retrieving the session information name from a dict we know nothing about. Maybe changing this bit could avoid suppressing the session altogether… Who knows?
– Mathias Ettinger
2 days ago
Please see the related meta posts here and there.
– Mathias Ettinger
2 days ago
Please see the related meta posts here and there.
– Mathias Ettinger
2 days ago
I tried to explain it better in what context I use it (see edit). I will also check the links you posted, thank you.
– Jon Programmer
2 days ago
I tried to explain it better in what context I use it (see edit). I will also check the links you posted, thank you.
– Jon Programmer
2 days ago
add a comment |
active
oldest
votes
active
oldest
votes
active
oldest
votes
active
oldest
votes
active
oldest
votes
with suppress(KeyError): del request.session[discount_code_session_name]
. But this question is so stripped of context that I’m voting to close it as off-topic.– Mathias Ettinger
2 days ago
You mean I should have posted more of my code, what's around it?
– Jon Programmer
2 days ago
4
Yes. It helps getting the bigger picture. Right now you're retrieving the session information name from a dict we know nothing about. Maybe changing this bit could avoid suppressing the session altogether… Who knows?
– Mathias Ettinger
2 days ago
Please see the related meta posts here and there.
– Mathias Ettinger
2 days ago
I tried to explain it better in what context I use it (see edit). I will also check the links you posted, thank you.
– Jon Programmer
2 days ago