@Qgil I just tried to reproduce this - but sadly couldnt. I just get one notification. My only assumption was that the form was submitted twice. Can you check this thing now (after our yesterdays patch - which was unrelated though) ?

I just got two notifications when being added as a publisher.

(And no notification when being removed as publisher, if that's relevant.)

@01tonythomas I just added you as publisher to https://test.wikipedia.org/wiki/Newsletter:The_Technical_Collaboration_Observer

Did you receive one or two notifications?

In T174602#3569472, @Qgil wrote:

@01tonythomas I just added you as publisher to https://test.wikipedia.org/wiki/Newsletter:The_Technical_Collaboration_Observer
Did you receive one or two notifications?

I can see it there. Woah. Strange I cannot reproduce (making me think that someone should look at the Echo jobs logs).

In T174602#3569466, Johan wrote:

(And no notification when being removed as publisher, if that's relevant.)

(Thanks! This is now T174667: Newsletter extension doesn't notify users when they are removed as publishers)

I think I have found the problem. There is something fishy triggering notifications when the user who adds a publisher is an administrator:

• When a publisher who is not an administrator adds a publisher, one notification is sent.
• When a publisher who is an administrator adds a publisher, two notifications are sent.
• When an administrator who is not a publisher adds a publisher, two notifications are sent.

I am not sure. I still think there is something with the Echo job queue here. Might not be a newsletter triggered thing as I tried condition (3) mentioned above just now - and only got *one*. Can someone from the echo team take a look, or someone with Echo jobs take a look at what is happening ?

In any case, I don't think this is a blocker for deployment.

In T173111#3572946, @Qgil wrote:

@Addshore I think this extension is ready to go.
The testing helped finding one major bug (fixed) and one glitch (T174602) that @01tonythomas cannot reproduce outside of test.wikipedia. That glitch only represents a very minor problem to a few and special users (users granted publisher permissions by an admin), and in fact checking whether it can be reproduced in production will be useful to find the cause.

There are 2 EchoEvent::create calls for "EchoEvent::create"

One in NewsletterDataUpdate and one in NewsletterEditPage

Change 375371 had a related patch set uploaded (by Addshore; owner: Addshore):
[mediawiki/extensions/Newsletter@master] Remove duplicate newsletter-newpublisher create event

https://gerrit.wikimedia.org/r/375371

In T174602#3572967, @Addshore wrote:

There are 2 EchoEvent::create calls for "EchoEvent::create"
One in NewsletterDataUpdate and one in NewsletterEditPage

But is the NewsletterDataUpdate called on normal edits ? I cannot find that happening on my local instance. In fact, after applying the ^^ patch - I do not get a notification at all. Probably if someone is free, we can test this thing out at newsletter-test.wmflabs.org and decide for sure.

Else, there is something very related to my machine install which do not produce double notifications.

I just added @Quiddity as publisher of Newsletter², a newsletter in MediaWiki.org, and he only got one notification. Should we close this task as... Resolved? If someone hits this problem again, we can reopen.

(Since then I have added more publishers to newsletters, and I have heard no further complaints.)

Change 375371 abandoned by Addshore:
Remove duplicate newsletter-newpublisher create event

https://gerrit.wikimedia.org/r/375371