Please check attached screenshot for reference.

It is not throwing an error for already subscribed with the same email address. Instead, every time accepting the same email address.

Note: But I can't see duplicate entry on Sender.net.

CommentFileSizeAuthor
#3 accepting-duplicate-email-Id-3419155-2.patch2.55 KBvinaygawade
Screenshot from 2024-02-05 10-58-59.png39.5 KBrevathi.b

Issue fork sender_net-3419155

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Revathi.B created an issue. See original summary.

vinaygawade’s picture

vinaygawade
Primary language Marathi
Location Sawantwadi, Maharashtra
commented
Status: Active » Needs review
StatusFileSize
new accepting-duplicate-email-Id-3419155-2.patch2.55 KB

Thank you! @revathib, for reporting this bug. With your detailed description, I was able to identify and resolve the issue. Attached is the patch file for your review.

revathi.b’s picture

revathi.b commented

@vinaygawade, thank you, it is working fine.
Need one change on line 127, instead addStatus() please change as addError().
Thank you!

revathi.b’s picture

revathi.b commented

Can I create an MR for this change?

vinaygawade’s picture

vinaygawade
Primary language Marathi
Location Sawantwadi, Maharashtra
commented

Sure, please go ahead and do this `instead addStatus() please change as addError()` mentioned changes.

Revathi.B opened merge request !2

revathi.b’s picture

revathi.b commented

@vinaygawade, please move this ticket to "fixed".

vinaygawade’s picture

vinaygawade
Primary language Marathi
Location Sawantwadi, Maharashtra
commented
Status: Needs review » Fixed
vinaygawade’s picture

vinaygawade
Primary language Marathi
Location Sawantwadi, Maharashtra
commented
Status: Fixed » Closed (fixed)