Closed Bug 410134 Opened 17 years ago Closed 17 years ago

[Oracle] Make Oracle case-insensitive for Bugzilla

Categories

(Bugzilla :: Database, enhancement)

3.1.2
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.2

People

(Reporter: xiaoou.wu, Assigned: xiaoou.wu)

Details

Attachments

(4 files, 3 obsolete files)

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
Attached patch v1Splinter Review
Accent-Insensitive
Attachment #294805 - Flags: review?(mkanat)
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 on attachment 294805 [details] [diff] [review]
v1

Beautiful! Case-insensitive and accent-insensitive! Great! :-)
Attachment #294805 - Flags: review?(mkanat) → review+
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
Closed: 17 years ago
Flags: documentation?
Flags: approval+
Resolution: --- → FIXED
should we put this in "5.5. Searching for Bugs" or "5.8. Hints and Tips"?
Attached patch draft 1 (obsolete) — Splinter Review
is this what you have in mind for documentation?
Attachment #297572 - Flags: review?(documentation)
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-
Attachment #297572 - Attachment is obsolete: true
Attachment #297584 - Flags: review?(documentation)
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+
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.
Attached patch backport to 2.22 (obsolete) — Splinter Review
for 2.22 branch
Attachment #297993 - Flags: review?(documentation)
Attached patch patch for 3.0 (obsolete) — Splinter Review
patch for 3.0
Attachment #297994 - Flags: review?(documentation)
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 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-
Attachment #297993 - Attachment is obsolete: true
Attachment #298023 - Flags: review?(documentation)
Attached patch 3.0 doc patchSplinter Review
Attachment #297994 - Attachment is obsolete: true
Attachment #298025 - Flags: review?(documentation)
Attachment #298023 - Flags: review?(documentation) → review+
Comment on attachment 298023 [details] [diff] [review]
version 2.22 patch

>+      the way PostreSQL handles case and accent sensitivity. 

s/PostreSQL/PostgreSQL/
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+
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.

Attachment

General

Creator:
Created:
Updated:
Size: