1.0.9.8 Questions / Comments / Concerns / Feedback

[heatherbell]

Screenshot 2024-11-10 090852.png

Input Comment and proceed to checkout_confirmation.

Screenshot 2024-11-10 090920.png

Customer decides to delete comment, click Edit goes back to checkout_payment as expected.
Delete Comment and proceed but Comment persists through checkout.
Seems impossible to delete Comment after initial input.

[burt]

I think this will take an architecture change;

- remove these comment input boxes from the checkout_payment and checkout_shipping.
- add comment input box on checkout_confirmation
- amend processing logic to catch _POST

Maybe Matt @ecartz has a less intrusive way.

Removing those input boxes from the two pages is probably a positive thing anyway.

[heatherbell]

- remove these comment input boxes from the checkout_payment and checkout_shipping.
- add comment input box on checkout_confirmation

Issue aside, think this is better than how it is handled now anyway. It also seems to be the most common way it is handled in most carts. The feature does not suit all store-owners so maybe good if it can be enabled/disabled.

[frankl]

Excellent burt!

One small thing, could this line be changed in catalog/product_info.php

from:

Code: Select all

$db->query("UPDATE products_description SET products_viewed = products_viewed+1 WHERE products_id = " . (int)$_GET['products_id'] . " and language_id = " . (int)$_SESSION['languages_id']);

to

Code: Select all

if (SESSION_BLOCK_SPIDERS == 'True') {
  $db->query("UPDATE products_description SET products_viewed = products_viewed+1 WHERE products_id = " . (int)$_GET['products_id'] . " and language_id = " . (int)$_SESSION['languages_id']);
}

which would stop counting views from spiders and only count views from real users.

If you don't want to change it, no biggie!

[ecartz]

which would stop counting views from spiders and only count views from real users.

That's not what that would do. That would turn off view counting for both spiders and real users whenever it's set not to block spider sessions. It would do nothing when it is set to block spider sessions.

Code: Select all

if ($session_started) {

would make it so that only real users trigger it if spider sessions are blocked.

[burt]

TY
https://github.com/CE-PhoenixCart/Phoen ... 32f9d56e71

[14Steve14]

I think this will take an architecture change;

- remove these comment input boxes from the checkout_payment and checkout_shipping.
- add comment input box on checkout_confirmation
- amend processing logic to catch _POST

Maybe Matt @ecartz has a less intrusive way.

Removing those input boxes from the two pages is probably a positive thing anyway.

We have a lot of customers leave things like 'Safe Place' delivery comments and other things such as leave with Neighbours and this seems the most logical place to leave that type of comments.

We don't get that many comments relating to payments. Would it be easier to remove just one of the comments boxes.

After the order has been placed may be the wrong place and some customers would not expect to see such a box on that page. Most may never scroll down the page expecting to find it. I would never look for a box there. If I have a shipping comment it would always be left on the shipping page.

[burt]

We have a lot of customers leave things like 'Safe Place' delivery comments and other things such as leave with Neighbours and this seems the most logical place to leave that type of comments.

We don't get that many comments relating to payments. Would it be easier to remove just one of the comments boxes.

After the order has been placed may be the wrong place and some customers would not expect to see such a box on that page. Most may never scroll down the page expecting to find it. I would never look for a box there. If I have a shipping comment it would always be left on the shipping page.

TY for feedback. It's been moved to the checkout_confirmation page to help mitigate a session problem.