BugsSite/ChangeLog

   1 2008-10-27  Darin Adler  <darin@apple.com>
   2
   3         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
   4         comment if it's untouched.
   5
   6 2008-10-23  Adam Roben  <aroben@apple.com>
   7
   8         Fix Bug 21401: Comments field on "Review Patch" page should be
   9         initially filled with quoted patch
  10
  11         <https://bugs.webkit.org/show_bug.cgi?id=21401>
  12
  13         Reviewed by Dave Hyatt.
  14
  15         * attachment.cgi:
  16         (edit): Retrieve the attachment data from the database instead of just
  17         its length. When the attachment is a patch, create a quotedpatch
  18         template variable that contains the patch data with each line
  19         prepended with "> ".
  20         * template/en/custom/attachment/reviewform.html.tmpl:
  21           - Changed the comments field to have a monospace font
  22           - Added an "Enter comments below:" caption above the comments field
  23             to make it clearer that this is where comments should go, now that
  24             the comments field is not initially empty
  25           - Fill the comments field with the quoted patch
  26
  27 2008-10-14  Adam Roben  <aroben@apple.com>
  28
  29         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
  30         attachment 24345
  31
  32         https://bugs.webkit.org/show_bug.cgi?id=21602
  33
  34         Reviewed by Dave Kilzer.
  35
  36         * PrettyPatch/PrettyPatch.rb:
  37         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
  38         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
  39         are a binary file marker, mark this FileDiff as binary and stop trying
  40         to process the lines.
  41         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
  42         saying so.
  43
  44 2008-10-02  Adam Roben  <aroben@apple.com>
  45
  46         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
  47
  48 2008-10-02  Adam Roben  <aroben@apple.com>
  49
  50         * template/en/custom/attachment/review.html.tmpl: Fix URLs
  51
  52 2008-10-02  Adam Roben  <aroben@apple.com>
  53
  54         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
  55         interface
  56
  57         https://bugs.webkit.org/show_bug.cgi?id=21315
  58
  59         Reviewed by Tim Hatcher.
  60
  61         * attachment.cgi:
  62         (top level): Added support for the "review" and "reviewform" actions.
  63         (sub edit): Accept the template name as a parameter. If no name is
  64         give, use "edit" as before.
  65         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
  66         link for all patch attachments.
  67         * template/en/custom/attachment/review.html.tmpl: Added. Simple
  68         <frameset> page to show the patch on the top and a comment form on the
  69         bottom.
  70         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
  71         comment form for reviewing patches.
  72
  73 2008-10-02  Adam Roben  <aroben@apple.com>
  74
  75         Remove references to some backup files I created when implementing
  76         PrettyPatch in Bugzilla
  77
  78         Reviewed by Tim Hatcher.
  79
  80         * attachment-aroben.cgi: Removed.
  81         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
  82
  83         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
  84         reference to attachment-aroben.cgi. attachment.cgi will work just
  85         fine.
  86
  87 2008-06-25  David Kilzer  <ddkilzer@apple.com>
  88
  89         Make PrettyPatch handle diffs with no Index or diff headers
  90
  91         Part of Bug 19290: More patches not handled by PrettyPatch.rb
  92         <https://bugs.webkit.org/show_bug.cgi?id=19290>
  93
  94         Reviewed by Adam.
  95
  96         * PrettyPatch/PrettyPatch.rb:
  97         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
  98         match on "+++ " lines for patches with no "Index" or "diff" header.
  99         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
 100         when the first line of a patch has no "Index" or "diff" header.
 101         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
 102         variable to determine when no "Index" or "diff" header has been
 103         found, but a new patch has started with a "--- " line.
 104
 105 2008-06-08  Dan Bernstein  <mitz@apple.com>
 106
 107         Reviewed by Adam Roben.
 108
 109         - update trac URL in PrettyPatch
 110
 111         * PrettyPatch/PrettyPatch.rb:
 112
 113 2008-05-29  Adam Roben  <aroben@apple.com>
 114
 115         Update PrettyPatch directory list
 116
 117         * PrettyPatch/PrettyPatch.rb:
 118
 119 2008-05-28  Adam Roben  <aroben@apple.com>
 120
 121         Make PrettyPatch able to handle diffs taken from outside the WebKit
 122         source tree
 123
 124         Part of Bug 19290: More patches not handled by PrettyPatch.rb
 125         <https://bugs.webkit.org/show_bug.cgi?id=19290>
 126
 127         Reviewed by David Kilzer.
 128
 129         * PrettyPatch/PrettyPatch.rb:
 130         (PrettyPatch.find_url_and_path): Added. Searches the file path from
 131         the bottom up looking for a directory that exists in the source tree.
 132         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
 133
 134 2008-05-28  Adam Roben  <aroben@apple.com>
 135
 136         Print exceptions from PrettyPatch to stdout
 137
 138         This will make it much easier to debug problems with PrettyPatch,
 139         since we'll be able to see the exceptions in the browser.
 140
 141         Reviewed by Sam Weinig.
 142
 143         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
 144         will print exceptions to stdout as HTML.
 145         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
 146
 147 2008-04-26  David Kilzer  <ddkilzer@apple.com>
 148
 149         Removed temp files and added appropriate svn:ignore properties.
 150
 151         Rubber-stamped by Mark Rowe.
 152
 153         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
 154
 155         * data/template: Added svn:ignore property for "template" subdirectory.
 156         * data/template/template: Removed precompiled template subdirectory.
 157
 158         * data/versioncache: Removed.
 159         * data/versioncache.*: Removed backup versioncache files.
 160
 161         * data/webdot: Added svn:ignore property for "*.dot" files.
 162         * data/webdot/*.dot: Removed cached webdot files.
 163
 164 2008-04-08  Adam Roben  <aroben@apple.com>
 165
 166         Combine :equal operations with the following operation if they are
 167         fewer than 3 characters long
 168
 169         This keeps us from showing lots of small changes on long lines, just
 170         because some letters happened to be the same.
 171
 172         Inspired by
 173         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
 174
 175         * PrettyPatch/PrettyPatch.rb:
 176         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
 177         than 3 characters long. The characters from the removed operations
 178         become part of the subsequent operation.
 179         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
 180         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
 181         @toLineNumber.nil? checks as they are no longer reliable now that
 182         we're removing operations.
 183
 184 2008-04-08  Adam Roben  <aroben@apple.com>
 185
 186         Change PrettyPatch to use DiffBuilder for intra-line diffs
 187
 188         This gives us much prettier intra-line diffs, largely because it can
 189         distinguish multiple changes on the same line. e.g., if a line changes
 190         from:
 191
 192         const int myConstant = 0;
 193
 194         to
 195
 196         static const unsigned myConstant;
 197
 198         You will see that "static " was inserted, "int" changed to "unsigned",
 199         and " = 0" was deleted.
 200
 201         This seems to have also gotten rid of some spurious instances of
 202         "<ins></ins>" and "<del></del>" at the end of a line.
 203
 204         * PrettyPatch/PrettyPatch.rb:
 205         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
 206         setChangeExtentFromLine (which had been copied from Trac's diffing
 207         code).
 208         (PrettyPatch::CodeLine):
 209          - Removed the changeExtent attribute
 210          - Added the operations attribute
 211          - Removed the setChangeExtentFromLine method
 212         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
 213         what text to wrap in <ins> and <del> tags.
 214         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
 215         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
 216         method, but left out the calls to perform_operation, since we build
 217         the diff HTML outside of this class.
 218         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
 219         convert_html_to_list_of_words, since we're not diffing HTML.
 220
 221 2008-04-08  Adam Roben  <aroben@apple.com>
 222
 223         Check in diff.rb
 224
 225         This came from
 226         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
 227         revision 28521
 228
 229         * PrettyPatch/diff.rb: Added.
 230
 231 2008-03-04  Adam Roben  <aroben@apple.com>
 232
 233         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
 234         be omitted from the hunk header.
 235