diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-03-20 15:00:09 +0000 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2017-03-20 15:00:09 +0000 |
commit | cd3e410110a5c6f33c5e873f8fb54883a8e11754 (patch) | |
tree | abd4161440349232ddeebe144d4ffa095cba13c4 | |
parent | b075d38c9b98d4b39441bd29d2c36be4637b6f72 (diff) | |
parent | 666c20cb705a9e65e2700b1fb0c6bac8f73e827e (diff) | |
download | gitlab-ce-cd3e410110a5c6f33c5e873f8fb54883a8e11754.tar.gz |
Merge branch 'routes-lower-case' into 'master'
Remove repeated routes.path check for postgresql database
See merge request !10063
-rw-r--r-- | app/models/concerns/routable.rb | 10 | ||||
-rw-r--r-- | changelogs/unreleased/routes-lower-case.yml | 4 |
2 files changed, 10 insertions, 4 deletions
diff --git a/app/models/concerns/routable.rb b/app/models/concerns/routable.rb index 9f6d215ceb3..529fb5ce988 100644 --- a/app/models/concerns/routable.rb +++ b/app/models/concerns/routable.rb @@ -51,11 +51,13 @@ module Routable paths.each do |path| path = connection.quote(path) - where = "(routes.path = #{path})" - if cast_lower - where = "(#{where} OR (LOWER(routes.path) = LOWER(#{path})))" - end + where = + if cast_lower + "(LOWER(routes.path) = LOWER(#{path}))" + else + "(routes.path = #{path})" + end wheres << where end diff --git a/changelogs/unreleased/routes-lower-case.yml b/changelogs/unreleased/routes-lower-case.yml new file mode 100644 index 00000000000..2110956680c --- /dev/null +++ b/changelogs/unreleased/routes-lower-case.yml @@ -0,0 +1,4 @@ +--- +title: Remove repeated routes.path check for postgresql database +merge_request: +author: mhasbini |