BugsSite/ChangeLog

   1 2009-12-07  Shinichiro Hamaji  <hamaji@chromium.org>
   2
   3         Reviewed by David Kilzer.
   4
   5         Bugzilla should show images in git patches
   6         https://bugs.webkit.org/show_bug.cgi?id=31395
   7
   8         Attempt to go r51748 again using --directory option of git-apply.
   9
  10         * PrettyPatch/PrettyPatch.rb:
  11
  12 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
  13
  14         Unreviewed. Revert r51748.
  15
  16         Bugzilla should show images in git patches
  17         https://bugs.webkit.org/show_bug.cgi?id=31395
  18
  19         * PrettyPatch/PrettyPatch.rb:
  20
  21 2009-12-06  Shinichiro Hamaji  <hamaji@chromium.org>
  22
  23         Reviewed by David Kilzer.
  24
  25         Bugzilla should show images in git patches
  26         https://bugs.webkit.org/show_bug.cgi?id=31395
  27
  28         Show images in git patches using git-apply.
  29
  30         * PrettyPatch/PrettyPatch.rb:
  31
  32 2009-10-23  Eric Seidel  <eric@webkit.org>
  33
  34         Reviewed by Adam Roben.
  35
  36         PrettyPatch should show images even when they have the wrong mime type
  37         https://bugs.webkit.org/show_bug.cgi?id=29506
  38
  39         * PrettyPatch/PrettyPatch.rb:
  40
  41 2009-10-17  Alexey Proskuryakov  <ap@apple.com>
  42
  43         Reviewed by Eric Carlson.
  44
  45         https://bugs.webkit.org/show_bug.cgi?id=30470
  46         Make marking a bug as duplicate easier
  47
  48         * js/field.js: (setResolutionToDuplicate): Focus the edit field, too.
  49
  50 2009-10-16  Alexey Proskuryakov  <ap@apple.com>
  51
  52         Reviewed by Mark Rowe.
  53
  54         https://bugs.webkit.org/show_bug.cgi?id=30470
  55         Make marking a bug as duplicate easier
  56
  57         * js/field.js: (setResolutionToDuplicate): Show edit field right away, don't make the user
  58         click again.
  59
  60 2009-09-09  David Kilzer  <ddkilzer@apple.com>
  61
  62         Update data/params parameters
  63
  64         * data/params: Updated 'attachment_base' parameter for secure
  65         attachments.  Updated 'mybugstemplate' parameter to search for
  66         bugs in UNCONFIRMED, ASSIGNED and REOPENED status in addition to
  67         NEW status.
  68
  69 2009-08-11  David Kilzer  <ddkilzer@apple.com>
  70
  71         Updated data/params parameter
  72
  73         * data/params: Updated 'loginnetmask' parameter to allow users
  74         to choose whether their cookies are restricted to a single IP
  75         address or not.
  76
  77 2009-07-07  David Kilzer  <ddkilzer@apple.com>
  78
  79         Updated BugsSite with local changes on server
  80
  81         The new files were created by checksetup.pl during the upgrade
  82         to Bugzilla 3.2.3.  The rest are self-explanatory.
  83
  84         * data: Added "bugzilla-update.xml" to svn:ignore.
  85         * data/attachments: Added "groups.*" to svn:ignore.
  86         * data/params: Updated for a few changed parameters.
  87
  88         * lib/.htaccess: Added.
  89         * skins/contrib/Dusk/IE-fixes.css: Added.
  90         * skins/contrib/Dusk/admin.css: Added.
  91         * skins/contrib/Dusk/create_attachment.css: Added.
  92         * skins/contrib/Dusk/dependency-tree.css: Added.
  93         * skins/contrib/Dusk/duplicates.css: Added.
  94         * skins/contrib/Dusk/editusers.css: Added.
  95         * skins/contrib/Dusk/help.css: Added.
  96         * skins/contrib/Dusk/index.css: Added.
  97         * skins/contrib/Dusk/panel.css: Added.
  98         * skins/contrib/Dusk/params.css: Added.
  99         * skins/contrib/Dusk/release-notes.css: Added.
 100         * skins/contrib/Dusk/show_bug.css: Added.
 101         * skins/contrib/Dusk/show_multiple.css: Added.
 102         * skins/contrib/Dusk/summarize-time.css: Added.
 103         * skins/contrib/Dusk/voting.css: Added.
 104         * skins/contrib/Dusk/yui: Added.
 105         * skins/contrib/Dusk/yui/calendar.css: Added.
 106         * skins/custom/IE-fixes.css: Added.
 107         * skins/custom/admin.css: Added.
 108         * skins/custom/buglist.css: Added.
 109         * skins/custom/create_attachment.css: Added.
 110         * skins/custom/dependency-tree.css: Added.
 111         * skins/custom/duplicates.css: Added.
 112         * skins/custom/editusers.css: Added.
 113         * skins/custom/help.css: Added.
 114         * skins/custom/panel.css: Added.
 115         * skins/custom/params.css: Added.
 116         * skins/custom/release-notes.css: Added.
 117         * skins/custom/show_bug.css: Added.
 118         * skins/custom/show_multiple.css: Added.
 119         * skins/custom/summarize-time.css: Added.
 120         * skins/custom/voting.css: Added.
 121         * skins/custom/yui: Added.
 122         * skins/custom/yui/calendar.css: Added.
 123
 124 2009-07-03  David Kilzer  <ddkilzer@apple.com>
 125
 126         Bug 26958: Change edit link to review link in request messages
 127
 128         <https://bugs.webkit.org/show_bug.cgi?id=26958>
 129
 130         Reviewed by Dan Bernstein.
 131
 132         * template/en/custom/request/email.txt.tmpl: Copied from BugsSite/template/en/default/request/email.txt.tmpl.
 133         Changed link from "action=edit" to "action=review".
 134
 135 2009-07-03  David Kilzer  <ddkilzer@apple.com>
 136
 137         Bug 26950: Make the summary and alias fields support click-to-edit
 138         <https://bugs.webkit.org/show_bug.cgi?id=26950>
 139
 140         Reviewed by Maciej Stachowiak.
 141
 142         Original patch by Maciej Stachowiak.
 143
 144         * js/field.js:
 145         (hideEditableField): Updated to add click event listeners to the
 146         alias and short description elements to make them easier to
 147         edit.  Renamed field_id parameter to field2_id (short
 148         description id) and added a field1_id parameter (alias id).
 149         (showEditableField): If a third parameter is passed in the
 150         ContainerInputArray parameter, use it to find the element to
 151         focus.  Otherwise, fall back to the original behavior of
 152         focusing the first input field.
 153         (hideAliasAndSummary): Changed to pass the id for the alias
 154         element to hideEditableField().
 155
 156 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 157
 158         Config file for contrib/recode.pl when upgrading bugs.webkit.org
 159
 160         Current as of midday on Wednesday, July 1, 2009.
 161
 162         * contrib/recode-overrides.txt: Added.
 163
 164 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 165
 166         Workaround for WebKit Bug 9630 when running contrib/recode.pl
 167
 168         A number of WebKit nightly builds included Bug 9630 which caused
 169         non-breaking space characters (0xA0) to be submitted within the
 170         content of textarea elements.
 171
 172         * contrib/recode.pl: To work around these 0xA0 characters, try
 173         an encoding of cp1252 and use it if it works.
 174
 175 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 176
 177         Don't print "Failed to guess" warning if an override is available
 178
 179         * contrib/recode.pl: Check that %overrides does NOT contain a
 180         key matching the current digest before printing out the warning
 181         about a failed charset guess.
 182
 183 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 184
 185         Added --[no-]truncate switch to contrib/recode.pl
 186
 187         * contrib/recode.pl: Added --[no-]truncate switch to make
 188         debugging of failed charset guessing easier.  Often times the
 189         illegal character was truncated in the output.
 190
 191 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 192
 193         Changed Perl scripts to use #!/usr/bin/env perl
 194
 195         perl -pi -e 's@#\!/usr/bin/perl@#\!/usr/bin/env perl@' *.cgi *.pl */*.pl
 196
 197 2009-07-02  David D. Kilzer  <ddkilzer@apple.com>
 198
 199         Replace Bugzilla favicon with webkit.org favicon
 200
 201         * favicon.ico: Removed.
 202         * images/favicon.ico: Replaced.  Copied from favicon.ico.
 203
 204 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
 205
 206         Merged BugsSite to Bugzilla-3.2.3
 207
 208         Updated to the latest-and-greatest stable version.
 209
 210 2009-07-02  David Kilzer  <ddkilzer@webkit.org>
 211
 212         Merged BugsSite to Bugzilla-3.0.3
 213
 214         Nothing to see here.  Move along.
 215
 216 2009-06-09  Eric Seidel  <eric@webkit.org>
 217
 218         Reviewed by Adam Roben.
 219
 220         Add support for displaying added png files in PrettyPatch diffs
 221         https://bugs.webkit.org/show_bug.cgi?id=26210
 222
 223         Currently this is SVN only (git-send-bugzilla patches exclude binary data)
 224         and only works for PNG files but could easily be made to work for other images as needed.
 225
 226         * PrettyPatch/PrettyPatch.rb:
 227
 228 2009-05-19  Eric Seidel  <eric@webkit.org>
 229
 230         Reviewed by Adam Roben.
 231
 232         Make PrettyPatch understand quoted filenames in git diffs.
 233         https://bugs.webkit.org/show_bug.cgi?id=25881
 234
 235         * PrettyPatch/PrettyPatch.rb:
 236
 237 2009-05-15  Simon Fraser  <simon.fraser@apple.com>
 238
 239         Carrying forwards Darin Adler's rubber stamp
 240
 241         Make the comment field wider too.
 242
 243         * skins/custom/global.css:
 244
 245 2009-05-14  Simon Fraser  <simon.fraser@apple.com>
 246
 247         Reviewed by Darin Adler
 248
 249         Make the URL, Summary and Keyword fields wider to mitigate the
 250         effects of https://bugs.webkit.org/show_bug.cgi?id=25566 and make
 251         things generally better.
 252
 253         * skins/custom/global.css:
 254         * template/en/custom/bug/edit.html.tmpl:
 255
 256 2008-11-10  Darin Adler  <darin@apple.com>
 257
 258         - fix more of https://bugs.webkit.org/show_bug.cgi?id=21400
 259           "Edit" links for patches in comments, review queue, review emails
 260           should be replaced by "Review Patch" links
 261
 262         * globals.pl: Fix cases that are automatically generated, such as links
 263         in comments in bugs.
 264
 265         * template/en/custom/attachment/created.html.tmpl: Fix the link on the
 266         "I just created an attachment" page.
 267
 268         * template/en/custom/request/queue.html.tmpl: Fix the link in the queue
 269         page.
 270
 271 2008-11-10  Darin Adler  <darin@apple.com>
 272
 273         * template/en/default/request/queue.html.tmpl: Use review links instead
 274         of edit links in the review queue.
 275
 276 2008-11-10  Darin Adler  <darin@apple.com>
 277
 278         * template/en/custom/request/email.txt.tmpl: Send review links instead
 279         of edit links when flags are set on a bug.
 280
 281 2008-10-27  Darin Adler  <darin@apple.com>
 282
 283         * template/en/custom/attachment/reviewform.html.tmpl: Leave out the
 284         comment if it's untouched.
 285
 286 2008-10-23  Adam Roben  <aroben@apple.com>
 287
 288         Fix Bug 21401: Comments field on "Review Patch" page should be
 289         initially filled with quoted patch
 290
 291         <https://bugs.webkit.org/show_bug.cgi?id=21401>
 292
 293         Reviewed by Dave Hyatt.
 294
 295         * attachment.cgi:
 296         (edit): Retrieve the attachment data from the database instead of just
 297         its length. When the attachment is a patch, create a quotedpatch
 298         template variable that contains the patch data with each line
 299         prepended with "> ".
 300         * template/en/custom/attachment/reviewform.html.tmpl:
 301           - Changed the comments field to have a monospace font
 302           - Added an "Enter comments below:" caption above the comments field
 303             to make it clearer that this is where comments should go, now that
 304             the comments field is not initially empty
 305           - Fill the comments field with the quoted patch
 306
 307 2008-10-14  Adam Roben  <aroben@apple.com>
 308
 309         Fix Bug 21602: Bugzilla times out trying to display formatted diff for
 310         attachment 24345
 311
 312         https://bugs.webkit.org/show_bug.cgi?id=21602
 313
 314         Reviewed by Dave Kilzer.
 315
 316         * PrettyPatch/PrettyPatch.rb:
 317         (PrettyPatch.BINARY_FILE_MARKER_FORMAT): Added.
 318         (PrettyPatch.FileDiff.initialize): If any of the lines in the diff
 319         are a binary file marker, mark this FileDiff as binary and stop trying
 320         to process the lines.
 321         (PrettyDiff.FileDiff.to_html): If we're binary, just print a string
 322         saying so.
 323
 324 2008-10-02  Adam Roben  <aroben@apple.com>
 325
 326         * template/en/custom/attachment/reviewform.html.tmpl: Fix typo.
 327
 328 2008-10-02  Adam Roben  <aroben@apple.com>
 329
 330         * template/en/custom/attachment/review.html.tmpl: Fix URLs
 331
 332 2008-10-02  Adam Roben  <aroben@apple.com>
 333
 334         Fix Bug 21315: bugs.webkit.org should have a nicer patch review
 335         interface
 336
 337         https://bugs.webkit.org/show_bug.cgi?id=21315
 338
 339         Reviewed by Tim Hatcher.
 340
 341         * attachment.cgi:
 342         (top level): Added support for the "review" and "reviewform" actions.
 343         (sub edit): Accept the template name as a parameter. If no name is
 344         give, use "edit" as before.
 345         * template/en/custom/attachment/list.html.tmpl: Added a "Review Patch"
 346         link for all patch attachments.
 347         * template/en/custom/attachment/review.html.tmpl: Added. Simple
 348         <frameset> page to show the patch on the top and a comment form on the
 349         bottom.
 350         * template/en/custom/attachment/reviewform.html.tmpl: Added. Simple
 351         comment form for reviewing patches.
 352
 353 2008-10-02  Adam Roben  <aroben@apple.com>
 354
 355         Remove references to some backup files I created when implementing
 356         PrettyPatch in Bugzilla
 357
 358         Reviewed by Tim Hatcher.
 359
 360         * attachment-aroben.cgi: Removed.
 361         * template/en/default/attachment/edit-aroben.html.tmpl: Removed.
 362
 363         * template/en/custom/attachment/edit.html.tmpl: Removed an erroneous
 364         reference to attachment-aroben.cgi. attachment.cgi will work just
 365         fine.
 366
 367 2008-06-25  David Kilzer  <ddkilzer@apple.com>
 368
 369         Make PrettyPatch handle diffs with no Index or diff headers
 370
 371         Part of Bug 19290: More patches not handled by PrettyPatch.rb
 372         <https://bugs.webkit.org/show_bug.cgi?id=19290>
 373
 374         Reviewed by Adam.
 375
 376         * PrettyPatch/PrettyPatch.rb:
 377         (PrettyPatch.DIFF_HEADER_FORMATS): Added regular expression to
 378         match on "+++ " lines for patches with no "Index" or "diff" header.
 379         (PrettyPatch.FileDiff.initialize): Look for filename on "+++ " line
 380         when the first line of a patch has no "Index" or "diff" header.
 381         (PrettyPatch.FileDiff.parse): Added haveSeenDiffHeader state
 382         variable to determine when no "Index" or "diff" header has been
 383         found, but a new patch has started with a "--- " line.
 384
 385 2008-06-08  Dan Bernstein  <mitz@apple.com>
 386
 387         Reviewed by Adam Roben.
 388
 389         - update trac URL in PrettyPatch
 390
 391         * PrettyPatch/PrettyPatch.rb:
 392
 393 2008-05-29  Adam Roben  <aroben@apple.com>
 394
 395         Update PrettyPatch directory list
 396
 397         * PrettyPatch/PrettyPatch.rb:
 398
 399 2008-05-28  Adam Roben  <aroben@apple.com>
 400
 401         Make PrettyPatch able to handle diffs taken from outside the WebKit
 402         source tree
 403
 404         Part of Bug 19290: More patches not handled by PrettyPatch.rb
 405         <https://bugs.webkit.org/show_bug.cgi?id=19290>
 406
 407         Reviewed by David Kilzer.
 408
 409         * PrettyPatch/PrettyPatch.rb:
 410         (PrettyPatch.find_url_and_path): Added. Searches the file path from
 411         the bottom up looking for a directory that exists in the source tree.
 412         (PrettyPatch.linkifyFilename): Changed to call find_url_and_path.
 413
 414 2008-05-28  Adam Roben  <aroben@apple.com>
 415
 416         Print exceptions from PrettyPatch to stdout
 417
 418         This will make it much easier to debug problems with PrettyPatch,
 419         since we'll be able to see the exceptions in the browser.
 420
 421         Reviewed by Sam Weinig.
 422
 423         * PrettyPatch/prettify.rb: Added a --html-exceptions option, which
 424         will print exceptions to stdout as HTML.
 425         * attachment.cgi: Changed to pass --html-exceptions to prettify.rb.
 426
 427 2008-04-26  David Kilzer  <ddkilzer@apple.com>
 428
 429         Removed temp files and added appropriate svn:ignore properties.
 430
 431         Rubber-stamped by Mark Rowe.
 432
 433         * data: Added svn:ignore property for "versioncache" and "versioncache.*" files.
 434
 435         * data/template: Added svn:ignore property for "template" subdirectory.
 436         * data/template/template: Removed precompiled template subdirectory.
 437
 438         * data/versioncache: Removed.
 439         * data/versioncache.*: Removed backup versioncache files.
 440
 441         * data/webdot: Added svn:ignore property for "*.dot" files.
 442         * data/webdot/*.dot: Removed cached webdot files.
 443
 444 2008-04-08  Adam Roben  <aroben@apple.com>
 445
 446         Combine :equal operations with the following operation if they are
 447         fewer than 3 characters long
 448
 449         This keeps us from showing lots of small changes on long lines, just
 450         because some letters happened to be the same.
 451
 452         Inspired by
 453         http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/diffviewer/diffutils.py?r=1264#147
 454
 455         * PrettyPatch/PrettyPatch.rb:
 456         (PrettyPatch::DiffSection.initialize): Remove :equal operations fewer
 457         than 3 characters long. The characters from the removed operations
 458         become part of the subsequent operation.
 459         (PrettyPatch::CodeLine.text_as_html): Don't wrap 0-length strings in
 460         <ins>/<del> tags. Also removed the @fromLineNumber.nil? and
 461         @toLineNumber.nil? checks as they are no longer reliable now that
 462         we're removing operations.
 463
 464 2008-04-08  Adam Roben  <aroben@apple.com>
 465
 466         Change PrettyPatch to use DiffBuilder for intra-line diffs
 467
 468         This gives us much prettier intra-line diffs, largely because it can
 469         distinguish multiple changes on the same line. e.g., if a line changes
 470         from:
 471
 472         const int myConstant = 0;
 473
 474         to
 475
 476         static const unsigned myConstant;
 477
 478         You will see that "static " was inserted, "int" changed to "unsigned",
 479         and " = 0" was deleted.
 480
 481         This seems to have also gotten rid of some spurious instances of
 482         "<ins></ins>" and "<del></del>" at the end of a line.
 483
 484         * PrettyPatch/PrettyPatch.rb:
 485         (PrettyPatch::DiffSection.initialize): Use DiffBuilder instead of
 486         setChangeExtentFromLine (which had been copied from Trac's diffing
 487         code).
 488         (PrettyPatch::CodeLine):
 489          - Removed the changeExtent attribute
 490          - Added the operations attribute
 491          - Removed the setChangeExtentFromLine method
 492         (PrettyPatch::CodeLine.text_as_html): Uses @operations to determine
 493         what text to wrap in <ins> and <del> tags.
 494         * PrettyPatch/diff.rb: Deleted a bunch of code we don't use.
 495         (HTMLDiff::DiffBuilder.initialize): Moved code here from the old build
 496         method, but left out the calls to perform_operation, since we build
 497         the diff HTML outside of this class.
 498         (HTMLDiff::DiffBuilder.split_inputs_to_words): Removed calls to
 499         convert_html_to_list_of_words, since we're not diffing HTML.
 500
 501 2008-04-08  Adam Roben  <aroben@apple.com>
 502
 503         Check in diff.rb
 504
 505         This came from
 506         http://instiki.rubyforge.org/svn/instiki/trunk/lib/diff.rb
 507         revision 28521
 508
 509         * PrettyPatch/diff.rb: Added.
 510
 511 2008-03-04  Adam Roben  <aroben@apple.com>
 512
 513         * PrettyPatch/PrettyPatch.rb: Allow the extent of the previous file to
 514         be omitted from the hunk header.
 515