Index: branches/RL2/extensions/Gadgets/Gadgets.hooks.php |
— | — | @@ -79,7 +79,9 @@ |
80 | 80 | } |
81 | 81 | |
82 | 82 | $prevTs = wfTimestampNow(); |
| 83 | + $thisTs = wfTimestampNow(); |
83 | 84 | if ( $revision ) { // $revision is null for null edits |
| 85 | + $thisTs = $revision->getTimestamp(); |
84 | 86 | $previousRev = $revision->getPrevious(); |
85 | 87 | if ( $previousRev ) { // $previousRev is null if there is no previous revision |
86 | 88 | $prevTs = $previousRev->getTimestamp(); |
— | — | @@ -90,7 +92,7 @@ |
91 | 93 | $repo = LocalGadgetRepo::singleton(); |
92 | 94 | // TODO: Timestamp in the constructor is ugly |
93 | 95 | $gadget = new Gadget( $id, $repo, $text, $prevTs ); |
94 | | - $repo->modifyGadget( $gadget, $revision ? $revision->getTimestamp() : $revision ); |
| 96 | + $repo->modifyGadget( $gadget, $thisTs ); |
95 | 97 | |
96 | 98 | // modifyGadget() returns a Status object with an error if there was a conflict, |
97 | 99 | // but we don't care. If a conflict occurred, that must be because a newer edit's |