Last modified: 2014-02-12 23:35:35 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T6323, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 4323 - Send a diff in email on page changes in addition to URL
Send a diff in email on page changes in addition to URL
Status: NEW
Product: MediaWiki
Classification: Unclassified
Email (Other open bugs)
All All
: Normal enhancement with 9 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-reviewed
Depends on:
Blocks: 1932
  Show dependency treegraph
Reported: 2005-12-20 14:37 UTC by Greg Sabino Mullane
Modified: 2014-02-12 23:35 UTC (History)
10 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Superceded by attachment #1229 (9.08 KB, patch)
2005-12-20 14:40 UTC, Greg Sabino Mullane
Show diffs in notification emails, full file if new (9.07 KB, patch)
2005-12-22 20:00 UTC, Greg Sabino Mullane
Email diff patch v3 (9.12 KB, patch)
2006-01-13 20:49 UTC, Greg Sabino Mullane
Allow text diffs in notification emails (8.67 KB, patch)
2006-04-04 04:48 UTC, Greg Sabino Mullane
diffNotify.patch (foro 1.8.1 but not working) (8.06 KB, patch)
2006-10-21 00:44 UTC, David Grant

Description Greg Sabino Mullane 2005-12-20 14:37:54 UTC
When a user gets a notification that a page has changed, the attached patch
sends a diff of what has changed on the page. This allows a quick viewing of the
changes without having to go to a browser and visit the page.
Comment 1 Greg Sabino Mullane 2005-12-20 14:40:45 UTC
Created attachment 1219 [details]
Superceded by attachment #1229 [details]
Comment 2 Greg Sabino Mullane 2005-12-22 20:00:10 UTC
Created attachment 1229 [details]
Show diffs in notification emails, full file if new

Slight modification: show full file if new.
Comment 3 Greg Sabino Mullane 2006-01-13 20:49:49 UTC
Created attachment 1305 [details]
Email diff patch v3

Minor fixes to previous patch.
Comment 4 Greg Sabino Mullane 2006-04-04 04:48:55 UTC
Created attachment 1495 [details]
Allow text diffs in notification emails

Run against latest cvs; has minor cleanups.
Comment 5 David Grant 2006-10-21 00:20:39 UTC
oh man, this really does not work in 1.8.1. Many rejected hunks and a failed one
since Messages.php is no longer there.
Comment 6 David Grant 2006-10-21 00:44:28 UTC
Created attachment 2527 [details]
diffNotify.patch (foro 1.8.1 but not working)

I have updated this patch to work with 1.8.1. The patch succeeds, but it
doesn't work. I'm not very good at debugging this though...
Comment 7 Sorin Sbarnea 2009-01-04 11:31:31 UTC
What's the current status of this bug/feature?

I consider it quite important and I should add that the diff code it's allready included in RSS feeds. 

RSS feeds are currently doing quite a good job in highlighting the diffs but they cannot replace email.
Comment 8 Greg Sabino Mullane 2009-01-08 02:43:22 UTC
It's not been maintained moving forward, but it does work great, and has been used successfully for years on production wikis. Please vote for it in bugzilla if you would like to see it get more attention. If I get time, I'll try and make an updated version at some point, and post it here.
Comment 9 Danny B. 2009-01-08 02:47:52 UTC
A preference to set if recipient does or does not want to receive the diff too or link only should come along with this feature.
Comment 10 Anne KG Murphy 2009-11-10 15:41:45 UTC
I also think this is very important, but I do not know how to vote for something in bugzilla to make it get more attention.
Comment 11 FT2 2011-05-16 22:12:01 UTC

Now that user talk page edits are being email-notified on major wikis including a diff and not just a bare URL would be very helpful.
Comment 12 John Du Hart 2011-08-31 22:20:35 UTC
Thank you for submitting patches for MediaWiki. I've looked over the patches however I cannot commit them in their current state. I don't think that writing to files and running them through diff is the best method for this. Instead could you possibly use the Diff class? Also, take a look at our coding conventions (especially in relation to spacing)

Thank you for your continued help!

Note You need to log in before you can comment on or make changes to this bug.