Skip to content

Store group and project full name and full path in routes table

username-removed-444 requested to merge dz-refactor-full-path into master

What does this MR do?

  • adds name field to routes table
  • moves full_name and full_path implementation from Namespace/Project to Routable concern
  • uses route.name and route.path for full name/path to group or project

Are there points in the code the reviewer needs to double check?

no

Why was this MR needed?

  • less sql queries (no need to collect parents to build full name/path)
  • use routes table as single point of truth for resource location
  • cleaner code

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/2772

Merge request reports