Page MenuHomePhabricator

Automatic edit summaries should not trigger edit summary pings
Open, LowestPublic


Edit summary notification (i.e. T32750) can be triggered by automatic summaries generated for instance on page creation.
In practice, once the feature is deployed, this will unexpectedly notify users from time to time when archiving talk pages, for instance
It would be good to prevent this.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

I can still reproduce this on Do you propose to close the task because you think this should not be fixed? Or is it fixed by something not deployed yet on the beta cluster?

In case my example was not clear: in the diff, the autogenerated edit summary is "←Created page with '== Old discussion == Short message. User:Orlodrim2'". This pings Orlodrim2, although no edit summary was explicitly provided, so this is unexpected.

@Orlodrim We deployed patches which should have fixed this issue. The code ought to be on the beta cluster.
Pinging @MaxSem to look into this.

@Orlodrim The diff you point to is from 6 March. Did you happen to try this again recently?

Confirm that autosummary triggers a ping. Is it really a behavior we don't want?

We will leave this ticket open for someday, but this is not a release blocker for our 2017 Wishlist project of T32750: [Epic] Ping/notify user when username used in an edit summary

I believe there is a valid use case for users to receive this notification so we will leave this behavior as-is for now. (If someone is being talked about, they arguably have the right to know about it...)

If you want to create a page with a username, you can prepend it with a colon (e.g. [[:User:Foobar]]) or provide a manual summary, which prevents the automatic edit summary.