Commit graph

16 commits

Author SHA1 Message Date
Yorick Peterse
5819ca1a24
Added Cop to blacklist polymorphic associations
One should really use a separate table instead of using polymorphic
associations.

See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11168 for
more information.
2017-06-07 17:36:55 +02:00
Dmitriy Zaporozhets
bbd1be002b
Change project list cache key to use route.cache_key instead of namespace
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2017-05-12 18:27:59 +03:00
Michael Kozono
b0ee22609a Reduce risk of deadlocks
We’ve seen a deadlock in CI here https://gitlab.com/mkozono/gitlab-ce/builds/15644492#down-build-trace. This commit should not fix that particular failure, but perhaps it will avoid others.

* Don’t call delete_conflicting_redirects after update if the path wasn’t changed
* Rename descendants without using recursion again, so we can run delete_conflicting_redirects exactly once.
2017-05-05 14:31:33 -07:00
Michael Kozono
e1c245af51 Resolve discussions 2017-05-05 12:12:51 -07:00
Michael Kozono
0c866f4a57 Resolve discussions 2017-05-05 12:12:49 -07:00
Michael Kozono
ca5c762cf5 Refactor 2017-05-05 12:11:58 -07:00
Michael Kozono
72872ee213 Delete conflicting redirects 2017-05-05 12:11:58 -07:00
Michael Kozono
a0368e9131 Create redirect routes on path change 2017-05-05 12:11:58 -07:00
Jarka Kadlecova
3750766f42 Escape route path for LIKE queries 2017-03-21 17:04:12 +01:00
Robert Speicher
9bf9e6eb38 Handle Route#name being nil after an update
It was possible for the `routes.name` field to be `NULL`, causing
`name_was` to be `nil` after a rename, resulting a bad first argument to
`sub` when attempting to rename descendants. This change adds a
condition to make sure `name_was` is present before attempting the
descendant update.
2017-03-20 19:17:14 -04:00
Dmitriy Zaporozhets
4d40c3c30a
Fix route rename descendants if route.name is blank
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2017-02-08 23:36:23 +02:00
Dmitriy Zaporozhets
2989192d1a
Store group and project full name and full path in routes table
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2017-02-08 19:14:29 +02:00
Dmitriy Zaporozhets
52c5f9c97f
Add User#nested_groups and User#nested_projects methods
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2017-01-25 13:43:15 +02:00
Dmitriy Zaporozhets
51c4b20c48 Refactor Namespace code related to nested groups
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2017-01-13 10:05:02 -05:00
Dmitriy Zaporozhets
95e0fac59a
Fix Route#rename_children behavior
Given group `gitlab` and `gitlab-org` exists. When rename `gitlab` it
will rename `gitlab-org` group route too. This commit fixes it

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2016-12-19 22:36:47 +02:00
Dmitriy Zaporozhets
83232be0e1
Add nested groups support on data level
* add parent_id field to namespaces table to store relation with nested groups
* create routes table to keep information about full path of every group and project
* project/group lookup by full path from routes table

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
2016-12-08 11:47:16 +02:00