https://bugzilla.wikimedia.org/show_bug.cgi?id=37225

--- Comment #36 from Aaron Schulz <aschulz4...@gmail.com> 2012-06-22 16:45:34 
UTC ---
Logging confirms a second bug that can cause this. In doEdit(), "$changed = (
strcmp( $text, $oldtext ) != 0 );" doesn't account for the rtrim() call on the
text in the Revision constructor. If the pre-trimmed text is used for
comparison but the trimmed text is inserted, you could have duplicate edits.
However, none of the logged instances resulted in duplicated edits so far
(since the new text had changes other than whitespace in the edits):
2012-06-22 12:39:18 mw19 trwiki: Rev text trimmed for page 1469915             
                                                            2012-06-22 13:06:08
mw35 trwiki: Rev text trimmed for page 970567                                  
                                        2012-06-22 13:45:02 srv288 enwiki: Rev
text trimmed for page 26204397                                                 
                       2012-06-22 15:11:55 mw24 enwiki: Rev text trimmed for
page 26204397                                                                  
      2012-06-22 15:40:52 mw24 enwiki: Rev text trimmed for page 26204397

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to