Reviewed by Mark Rowe.
[WebKit-https.git] / 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