BugsSite/ChangeLog

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