View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0004463 | OXID eShop (all versions) | 1.05. Users | public | 2012-08-28 15:59 | 2012-11-22 10:50 |
Reporter | WBL_BjoernLange | Assigned To | |||
Priority | high | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | Past development | ||||
Fixed in Version | 4.6.6 revision 54646 | ||||
Summary | 0004463: oxnewssubscribed should be fixated for the mail adress and not for the user | ||||
Description | oxUser::getNewsSubscription: if ( !$this->_oNewsSubscription->loadFromUserId( $this->getId() ) ) { if ( !$this->_oNewsSubscription->loadFromEmail( $this->oxuser__oxusername->value ) ) { The Double Opt is used to validate if the mail address really exists and if the user wants to get the newsletter. To fixate the state for the user instead for the mail address does not fulfill these requirements imho. | ||||
Tags | No tags attached. | ||||
Theme | Both | ||||
Browser | All | ||||
PHP Version | any | ||||
Database Version | any | ||||
|
When request for subscription is sent by user (i.e. during the registration), who created an account: this user can change his email at some moment later. But once he confirmed the subscription via double opt in - I think it's not needed to ask him again on each email change. Thus - binding to user account has more sense than binding to email. but @Developers: please check that from the source code if that works correctly, also in cases, when user without account subscribes for newsletter (then record to oxuser table is still created with supplied data). |
|
I think Bjoern may have thought of legal requirements, if the mail address has not been verified it may not be legal to send a newsletter to that address. The new address may belong to someone else. |
|
Yes leofonic, you got me right. |
|
Hi, subscriptions is still fixated to user account, but we are now sending subscription confirmation on email change. If you have any further questions regarding emails, please reopen this ticket. |