Hello,

With administrator user or authentificated user, when you register a newsleter from the block all is OK, but when you do it from the subscription node, it tells you that the e-mail is invalid "newsletter/subscriptions/".

If you do that with anonymous user in the subscription node where it lets you enter the e-mail all is fine.

It seems that in the node of subscription the e-mail " data is not arriving.

Anyone has this problem?

Best Regards.

CommentFileSizeAuthor
#4 simplenews.module.112191.patch1.65 KBsutharsan
subscriptionok.JPG84.59 KBrubensans

Comments

sutharsan’s picture

sutharsan commented

I can not reproduce your problem.

Do you use the same e-mail address for anonymous and authenticated?
Which e-mail address are you using?
Is the e-mail address really (in)valid?
Do you have the same problems if you are running the English interface?
Can you reproduce the problem on a clean drupal installation (only core modules)?

rubensans’s picture

rubensans commented

Do you use the same e-mail address for anonymous and authenticated?
--> It makes no change using the same or another valid

Which e-mail address are you using?
--> 2 of my addresses; one is gmail.

Is the e-mail address really (in)valid?
--> No.

Do you have the same problems if you are running the English interface?
--> Yes

Can you reproduce the problem on a clean drupal installation (only core module)
--> Yes

sutharsan’s picture

sutharsan commented
Assigned: Unassigned » sutharsan
sutharsan’s picture

sutharsan commented
StatusFileSize
new simplenews.module.112191.patch1.65 KB

First subscription of a user via url newsletter/subscriptions is now handled correctly.

Testing the functionality of the newsletter subscription form I decided to remove the 'Delete' button and allow the user to unsubscribe to (all) newsletter(s) using the 'Update' button. This seems more logical to me.

rubensans’s picture

rubensans commented
Status: Active » Closed (fixed)

Thanks you Sutharsan.

I now works great.

Best Regards.

sutharsan’s picture

sutharsan commented
Status: Closed (fixed) » Reviewed & tested by the community

This issue was unintentionally closed, the patch still needs to be committed.
The fix must be applied to HEAD and and 4-7-2 too.

sutharsan’s picture

sutharsan commented
Assigned: sutharsan » Unassigned
sutharsan’s picture

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

Patch committed to 5-1-dev.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)