c6fe66b1f9fbcc3a6a4391bd5bdab05dc4dcfea1
[WebKit-https.git] / 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