BugsSite/ChangeLog

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