[Oracle] Make Oracle case-insensitive for Bugzilla

RESOLVED FIXED in Bugzilla 3.2

Status

()

Bugzilla
Database
--
enhancement
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Xiaoou, Assigned: Xiaoou)

Tracking

3.1.2
Bugzilla 3.2
Bug Flags:
approval +
documentation +
documentation3.0 +
documentation2.22 +

Details

Attachments

(4 attachments, 3 obsolete attachments)

v1
708 bytes, patch
Max Kanat-Alexander
: review+
Details | Diff | Splinter Review
893 bytes, patch
Frédéric Buclin
: review+
Details | Diff | Splinter Review
834 bytes, patch
Frédéric Buclin
: review+
Details | Diff | Splinter Review
909 bytes, patch
Frédéric Buclin
: review+
Details | Diff | Splinter Review
(Assignee)

Description

10 years ago
User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 2.0.50727; .NET CLR 3.0.04506.30)
Build Identifier: 

By default, query in Oracle is case-sensitive, but Bugzilla is 
case-insensitive.

Reproducible: Always
(Assignee)

Comment 1

10 years ago
Created attachment 294805 [details] [diff] [review]
v1

Accent-Insensitive
Attachment #294805 - Flags: review?(mkanat)

Updated

10 years ago
Assignee: database → xiaoou.wu
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: [Oracle] To allow case-insensitive queries. → [Oracle] Make Oracle case-insensitive for Bugzilla
Target Milestone: --- → Bugzilla 3.2
Version: unspecified → 3.1.2

Comment 2

10 years ago
Comment on attachment 294805 [details] [diff] [review]
v1

Beautiful! Case-insensitive and accent-insensitive! Great! :-)
Attachment #294805 - Flags: review?(mkanat) → review+

Comment 3

10 years ago
This is going to need to be documented somewhere, that Oracle databases are both case-insensitive and accent-insensitive for Bugzilla, meaning that, for example, "FrÈdèriC" is the same as "frederic".

Checking in Bugzilla/DB/Oracle.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Oracle.pm,v  <--  Oracle.pm
new revision: 1.4; previous revision: 1.3
done
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Flags: documentation?
Flags: approval+
Resolution: --- → FIXED

Comment 4

10 years ago
should we put this in "5.5. Searching for Bugs" or "5.8. Hints and Tips"?

Comment 5

10 years ago
Created attachment 297572 [details] [diff] [review]
draft 1

is this what you have in mind for documentation?
Attachment #297572 - Flags: review?(documentation)

Comment 6

10 years ago
Comment on attachment 297572 [details] [diff] [review]
draft 1

You don't need to add the bit about Oracle.

You should note that PostgreSQL is case-sensitive for some things, though, and case-insensitive for others.
Attachment #297572 - Flags: review?(documentation) → review-

Comment 7

10 years ago
Created attachment 297584 [details] [diff] [review]
clarified the 3 dbs
Attachment #297572 - Attachment is obsolete: true
Attachment #297584 - Flags: review?(documentation)

Comment 8

10 years ago
Comment on attachment 297584 [details] [diff] [review]
clarified the 3 dbs

Looks good to me and the information is accurate. r=LpSolit
Attachment #297584 - Flags: review?(documentation) → review+

Comment 9

10 years ago
Sam, we need to backport the doc patch to 3.0 and 2.22 as we don't mention the difference between MySQL and PostgreSQL on these branches.

Comment 10

10 years ago
Created attachment 297993 [details] [diff] [review]
backport to 2.22

for 2.22 branch
Attachment #297993 - Flags: review?(documentation)

Comment 11

10 years ago
Created attachment 297994 [details] [diff] [review]
patch for 3.0

patch for 3.0
Attachment #297994 - Flags: review?(documentation)

Comment 12

10 years ago
Comment on attachment 297993 [details] [diff] [review]
backport to 2.22

>+      Bugzilla queries are case-insensitive and accent-insensitive, when
>+      used with either MySQL or Oracle databases.

Doesn't make sense to talk about Oracle on 2.22 as we don't even support it on this branch.
Attachment #297993 - Flags: review?(documentation) → review-

Comment 13

10 years ago
Comment on attachment 297994 [details] [diff] [review]
patch for 3.0

>     <section id="list">
>+
>+    <section id="casesensitivity">

Err... You pasted the text at the wrong place. id="list" should come later.


>+      Bugzilla queries are case-insensitive and accent-insensitive, when
>+      used with either MySQL or Oracle databases.

Don't talk about Oracle as 3.0 doesn't support it.
Attachment #297994 - Flags: review?(documentation) → review-

Comment 14

10 years ago
Created attachment 298023 [details] [diff] [review]
version 2.22 patch
Attachment #297993 - Attachment is obsolete: true
Attachment #298023 - Flags: review?(documentation)

Comment 15

10 years ago
Created attachment 298025 [details] [diff] [review]
3.0 doc patch
Attachment #297994 - Attachment is obsolete: true
Attachment #298025 - Flags: review?(documentation)

Updated

10 years ago
Attachment #298023 - Flags: review?(documentation) → review+

Comment 16

10 years ago
Comment on attachment 298023 [details] [diff] [review]
version 2.22 patch

>+      the way PostreSQL handles case and accent sensitivity. 

s/PostreSQL/PostgreSQL/

Comment 17

10 years ago
Comment on attachment 298025 [details] [diff] [review]
3.0 doc patch

>+      the way PostreSQL handles case and accent sensitivity. 

PostgreSQL


>     <section id="list">
>+
>       <title>Bug Lists</title>

Useless newline. Should be removed on checkin. r=LpSolit
Attachment #298025 - Flags: review?(documentation) → review+

Comment 18

10 years ago
tip:

Checking in docs/xml/using.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/using.xml,v  <--  using.xml
new revision: 1.77; previous revision: 1.76
done

3.0.3:

Checking in docs/xml/using.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/using.xml,v  <--  using.xml
new revision: 1.64.2.10; previous revision: 1.64.2.9
done

2.22.3:

Checking in docs/xml/using.xml;
/cvsroot/mozilla/webtools/bugzilla/docs/xml/using.xml,v  <--  using.xml
new revision: 1.37.2.24; previous revision: 1.37.2.23
done
Flags: documentation?
Flags: documentation3.0+
Flags: documentation2.22+
Flags: documentation+
You need to log in before you can comment on or make changes to this bug.