Defect #12349

Watchers auto-complete search on non-latin chars

Added by Alexander Oryol about 5 years ago. Updated about 5 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:Database
Target version:2.1.3
Resolution:Fixed Affected version:

Description

The current code generates SQL like as:


SELECT "users".* FROM "users" WHERE "users"."type" IN ('User', 'AnonymousUser') AND (users.status = 1) AND (LOWER(login) LIKE '%Собо%' OR LOWER(firstname) LIKE '%Собо%' OR LOWER(lastname) LIKE '%Собо%' OR LOWER(mail) LIKE '%Собо%') LIMIT 100

So, searching user "Соболев Денис" on "Собо" as result - not found. There is the case sensitive.

Patch attached.

watcher_search.patch Magnifier (875 Bytes) Alexander Oryol, 2012-11-12 16:02

Associated revisions

Revision 10801
Added by Jean-Philippe Lang about 5 years ago

Fixed that Principal#like scope does not work with cyrillic characters and Postgresql (#12349).

Revision 10819
Added by Jean-Philippe Lang about 5 years ago

Merged r10801 from trunk (#12349).

History

#1 Updated by Jean-Philippe Lang about 5 years ago

  • Status changed from New to Confirmed
  • Target version set to 2.1.3

#2 Updated by Jean-Philippe Lang about 5 years ago

  • Status changed from Confirmed to Resolved
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Was working fine with Mysql but not Postgresql. Fixed in r10801 with a test.

#3 Updated by Alexander Oryol about 5 years ago

Hmmm...
Repo https://github.com/redmine/redmine syncronization broken again. Fix it please!

#4 Updated by Toshi MARUYAMA about 5 years ago

Alexander Oryol wrote:

Hmmm...
Repo https://github.com/redmine/redmine syncronization broken again. Fix it please!

See #11918.

Bitbucket Mercurial mirror is syncronized.
https://bitbucket.org/redmine

#5 Updated by Daniel Felix about 5 years ago

  • Status changed from Resolved to Closed

I close this one too as it seems to work.

#6 Updated by Toshi MARUYAMA about 5 years ago

  • Status changed from Closed to Reopened

Not yet merged 2.1-stable branch.

#7 Updated by Jean-Philippe Lang about 5 years ago

  • Status changed from Reopened to Closed

Merged.

Also available in: Atom PDF