Possible Bug

The text entered was not recognized as an acceptable email address, i.e. Name email@domain or email@domain or email@domain or any combination of these separated by a comma. Please re-enter, the unrecognized values have been removed.

Send Copy of Order Confirmation Emails To

Send Copy of Order Confirmation Emails To
Send COPIES of order confirmation emails to the following email addresses, in this format: Name 1 email@address1, Name 2 email@address2

Input to error
Word word [email protected], Word word [email protected]

In fact if you use the provided example
Name 1 email@address1, Name 2 email@address2
also errors

The text entered was not recognized as an acceptable email address, i.e. Name email@domain or email@domain or email@domain or any combination of these separated by a comma. Please re-enter, the unrecognized values have been removed.

Zen Cart 2.2.2 PHP 8.5

Are you suggesting that the messages (and configuration-setting descriptions) be updated to fully spell out that email@domain is [email protected]

Noting that your "Word word" example was missing the required brackets surrounding the email address.

Send COPIES of order confirmation emails to the following email addresses, in this format: Name 1 <email@address1>, Name 2 <email@address2>

Sorry accidentally dropped them but NO it doesn't work =)

Name 1 <[email protected]>, Name 2 <[email protected]>

i have confirmed this bug.

by the time, we get to here:

https://github.com/zencart/zencart/blob/5f150dca7b1ca539234419ad8aec3a72e64b9797/admin/includes/classes/configurationValidation.php#L33

the < and the > have been encoded to < and >

converting them back prior to that line validates the entry correctly.

they are then stored in the database using the < and > but displayed on the configuration screen using the htmlentities.

Corrected in the 2.3 branch on GitHub: https://github.com/zencart/zencart/commit/5b29fdf8c1a2abe50926cc39b2fa8d8ce8163e8b

lat9:

Corrected in the 2.3 branch on GitHub: https://github.com/zencart/zencart/commit/5b29fdf8c1a2abe50926cc39b2fa8d8ce8163e8b

in my testing using the master branch with the latest commits, this bug still exists.

in my FURTHER testing on the 2.3 branch, this bug also still exists.

i will say that on the master branch, there are 2 error messages that get added to the messageStack; on branch 2.3, there is only 1 error message that gets added to the stack.

i have not dug deeper, as to their differences.

specifically because (thread drift), i'm a little aghast at the number of branches that exist on this repo. the management of which, to me, looks like an incredible amount of work. 16 active branches?

but this bug (or a slight variation) still exists on both branches. and i would think if the bug was corrected in 2.3, it should also be corrected in the master branch. but here, we have the master branch throwing 2 messages, while 2.3 only has 1.

best.

For both the 2.3 and master branches, I've changed Send Low Stock Emails To to Cindy <[email protected]>; that's how it displays on the screen and also how it's stored in the configuration table (no < >).

@carlwhat what messages are getting added to the messageStack?

hmm.... this seems like my mistake. the 2.3 and master branch both seem fine. apologies about that... i have far too many dev domains on my dev server.

i was getting that 2nd error due to an issue with a configuration value database setting. that also seems to have been a problem on my end.

best.