Skip to content

Commit

Permalink
Fixed Logout issue reported in issue mrvautin#154 (mrvautin#155)
Browse files Browse the repository at this point in the history
Co-authored-by: Mark Moffat <hi@markmoffat.com>
  • Loading branch information
acidrs03 and mrvautin authored Oct 21, 2020
1 parent c2cee9f commit fc8d63d
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 23 deletions.
18 changes: 1 addition & 17 deletions lib/cart.js
Original file line number Diff line number Diff line change
Expand Up @@ -109,24 +109,8 @@ const emptyCart = async (req, res, type, customMessage) => {
res.redirect('/');
};

const clearCustomer = (req) => {
// Clear our session
req.session.customerPresent = null;
req.session.customerEmail = null;
req.session.customerFirstname = null;
req.session.customerLastname = null;
req.session.customerAddress1 = null;
req.session.customerAddress2 = null;
req.session.customerCountry = null;
req.session.customerState = null;
req.session.customerPostcode = null;
req.session.customerPhone = null;
req.session.orderComment = null;
};

module.exports = {
updateTotalCart,
updateSubscriptionCheck,
emptyCart,
clearCustomer
emptyCart
};
20 changes: 18 additions & 2 deletions lib/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -268,6 +268,21 @@ const cleanAmount = (amount) => {
return amount;
};

const clearCustomer = (req) => {
// Clear our session
req.session.customerPresent = null;
req.session.customerEmail = null;
req.session.customerFirstname = null;
req.session.customerLastname = null;
req.session.customerAddress1 = null;
req.session.customerAddress2 = null;
req.session.customerCountry = null;
req.session.customerState = null;
req.session.customerPostcode = null;
req.session.customerPhone = null;
req.session.orderComment = null;
};

module.exports = {
allowedMimeType,
fileSizeLimit,
Expand All @@ -287,6 +302,7 @@ module.exports = {
newId,
hooker,
getCountryList,
getCountryNameToCode,
cleanAmount
cleanAmount,
clearCustomer,
getCountryNameToCode
};
8 changes: 4 additions & 4 deletions routes/order.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,14 @@ const {
getCountryList,
getId,
sendEmail,
getEmailTemplate
getEmailTemplate,
clearCustomer
} = require('../lib/common');
const {
paginateData
paginateData,
} = require('../lib/paginate');
const {
emptyCart,
clearCustomer
emptyCart
} = require('../lib/cart');
const { restrict, checkAccess } = require('../lib/auth');
const { indexOrders } = require('../lib/indexing');
Expand Down

0 comments on commit fc8d63d

Please sign in to comment.