69793 - check in new files for QuickSearch

Status: VERIFIED FIXED

(Bugzilla :: Bugzilla-General, enhancement)

Description

[Andreas Franke (gone)]

This bug is a request to add the QuickSearch tool to the bugzilla cvs
repository. It is complementary to bug 61561 which is a request to integrate
this tool into the bugzilla front page on bugzilla.mozilla.org .

Description of the tool:
========================
The QuickSearch tool is an alternative query tool for bugzilla. The UI is as
simple as possible: a single text box. And it should be simple to use for
newbies: you can just enter some words and press return, and it will give you a
list of bugs.
Currently, this is a JS-based client-side tool, but it could be ported to perl
in the future.

Reasons for including it into CVS:
==================================
The tool can be useful for all bugzilla installations. All mozilla-specific
values are defined in a separate file (currently called "localconfig.js"), so it
should be easy to use it in other installations. It works fine for me so far,
and I already prefer it over query.cgi more than 80 percent of the time. I
consider it production level quality. Checking it in will hopefully increase the
chance that someone finds it useful and starts hacking on it.

Files involved:
===============
quicksearch.html     (query form with first part of the documentation)
quicksearchhack.html (extended query form with second part of the documentation)
quicksearch.js       (main part of the JS sources)
localconfig.js       (installation-specific values)
optional: index.html (for integration into the front page)

I'm going to attach the files soon.

Comment 1

[Andreas Franke (gone)]

Attachment: localconfig.js

Comment 2

[Andreas Franke (gone)]

Attachment: quicksearch.js

Comment 3

[Andreas Franke (gone)]

Attachment: quicksearch.html

Comment 4

[Andreas Franke (gone)]

Attachment: quicksearchhack.html

Comment 5

[Andreas Franke (gone)]

Attachment: cvs diff index.html

Comment 6

[Andreas Franke (gone)]

Attachment: cvs diff -u index.html

Comment 7

[Andreas Franke (gone)]

Attachment: new index.html

Comment 8

[Andreas Franke (gone)]

Ok, patches complete. Decorating bug with keywords and status whiteboard stuff.
Please review, approve, checkin asap. ;)

Comment 9

[Andreas Franke (gone)]

Ok, maybe it was a little too much... :)

Comment 10

[Andreas Franke (gone)]

Attachment: small fix for quicksearch.js (product_exceptions)

Comment 11

[Andreas Franke (gone)]

Attachment: quicksearch.js *final* :)

Comment 12

[Andreas Franke (gone)]

Attachment: quicksearch.js *final* (without searching for DUPLICATES)

Comment 13

[Andreas Franke (gone)]

So I'm sorry that the first version was the wrong one. Attachment 26162 [details] [diff] from
02/25/01 4:12 is the correct one. The difference is that the new one only
searches for OPEN bugs:

319c319
<         add("DUPLICATE",resolution);
---
>         // add("DUPLICATE",resolution);

Having localconfig.js and quicksearch.js in the bugzilla cvs would have the
advantage that one could truly operate with diffs instead of whole files...

Comment 14

[Chris Yeh]

re-assigning to me to evaluate

Comment 15

[Chris Yeh]

I've applied all your patches (quicksearch.html, quicksearchhack.html, 
index.html, localconfig.js and quicksearch.js) and the searching part of it 
doesn't appear to work at all.

I have a bug with a summary of "google foo bar".

If I do a quick search for any of these summary words, I get zero bugs returned 
from the query.

Comment 16

[Andreas Franke (gone)]

It would probably help if I had the following info:
- the url of your bugzilla test installation
- the url generated by your query
- the browser name and version

Comment 17

[Chris Yeh]

The bugzilla test installation is behind a firewall, so you will not be able to 
reach it.

Communicator 4.7

http://crucible.bluemartini.com/mozilla/webtools/bugzilla/buglist.cgi?bug_status
=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&field0-0-0=p
roduct&type0-0-0=substring&value0-0-0=foo&field0-0-1=component&type0-0-1=substri
ng&value0-0-1=foo&field0-0-2=short_desc&type0-0-2=substring&value0-0-2=foo&field
0-0-3=status_whiteboard&type0-0-3=substring&value0-0-3=foo

Comment 18

[Andreas Franke (gone)]

This is strange. I'm using Communicator 4.61, and the url is exactly what it 
should be. If I replace the bugzilla location part
"http://crucible.bluemartini.com/mozilla/webtools/bugzilla/" with
"http://bugzilla.mozilla.org/", it works correctly.

Are you sure you have an _OPEN_ bug with foo in the summary? Can you find it
using your query.cgi form (select UNCONFIRMED,NEW,ASSIGNED,REOPENED, enter foo
in the summary field, and submit)?

Comment 19

[Dave Miller [:justdave]]

moving to real milestones...

Comment 20

[Andreas Franke (gone)]

Attachment: diff quicksearch.html (small correctness fix for the docs)

Comment 21

[Andreas Franke (gone)]

Attachment: (same as above, but as diff -u)

Comment 22

[Andreas Franke (gone)]

Attachment: diff quicksearchhack.html (small documentation fix, part 2)

Comment 23

[Andreas Franke (gone)]

Attachment: (same as above, but as diff -u)

Comment 24

[Chris Yeh]

new feature and patches checked into trunk

Comment 25

[Andreas Franke (gone)]

Verified fixed. All files have been checked in. Any bugs related to this feature
should be filed separately.

Comment 26

[Andreas Franke (gone)]

.

Comment 27

[Dave Miller [:justdave]]

Moving closed bugs to Bugzilla product

Comment 28

[Dave Miller [:justdave]]

*** Bug 33067 has been marked as a duplicate of this bug. ***