Describe your bug or feature request.

Make store email optional, if blank defaults to site mail.

On simple sites, it's common for the store email address to be the same as the site mail. It's a shame to have to configure the same value in two places, and when the customer ask to change the site mail it's easy to forget to change the store also.

CommentFileSizeAuthor
#3 3347292-3.patch1.86 KBjsacksick

Comments

AdamPS created an issue. See original summary.

jsacksick’s picture

jsacksick commented

hm... Why not... So I think we should have Store::getEmail() defaults to the site mail if the email field is empty, and then the email field should be made optional.

jsacksick’s picture

jsacksick commented
Status: Active » Needs review
StatusFileSize
new 3347292-3.patch1.86 KB
adamps’s picture

adamps commented
Status: Needs review » Reviewed & tested by the community

Great many thanks, it works

  • jsacksick committed 02edd544 on 8.x-2.x 
    Issue #3347292 by jsacksick, AdamPS: Make store email optional.
jsacksick’s picture

jsacksick commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks for the review!

  • jsacksick committed b64bb73b on 3.0.x 
    Issue #3347292 by jsacksick, AdamPS: Make store email optional.
adamps’s picture

adamps commented

Great many thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.