Project

General

Profile

Actions

Defect #12349

closed

Watchers auto-complete search on non-latin chars

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

Status:
Closed
Priority:
Normal
Category:
Database
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
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.


Files

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

Updated by Jean-Philippe Lang about 12 years ago

  • Status changed from New to Confirmed
  • Target version set to 2.1.3
Actions #2

Updated by Jean-Philippe Lang about 12 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.

Actions #3

Updated by Alexander Oryol about 12 years ago

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

Actions #4

Updated by Toshi MARUYAMA about 12 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

Actions #5

Updated by Daniel Felix about 12 years ago

  • Status changed from Resolved to Closed

I close this one too as it seems to work.

Actions #6

Updated by Toshi MARUYAMA about 12 years ago

  • Status changed from Closed to Reopened

Not yet merged 2.1-stable branch.

Actions #7

Updated by Jean-Philippe Lang about 12 years ago

  • Status changed from Reopened to Closed

Merged.

Actions

Also available in: Atom PDF