open-consul/ui-v2/app
John Cowen c62b974222
ui: Fix using 'ui-like' KVs when using an empty default nspace (#7734)
When using namespaces, the 'default' namespace is a little special in
that we wanted the option for all our URLs to stay the same when using
namespaces if you are using the default namespace, with the option of
also being able to explicitly specify `~default` as a namespace.

In other words both `ui/services/service-name` and
`ui/~default/services/service-name` show the same thing.

This means that if you switch between OSS and Enterprise, all of your
URLs stay the same, but you can still specifically link to the default
namespace itself.

Our routing configuration is duplicated in order to achieve this:

```
- :dc
  - :service
  - :kv
    - :edit
- :nspace
  - :dc
    - :service
    - :kv
      - :edit
```

Secondly, ember routing resolves/matches routes in the order that you specify
them, unless, its seems, when using wildcard routes, like we do in the
KV area.

When not using the wildcard routes the above routing configuration
resolves/matches a `/dc-1/kv/service` to the `dc.kv.edit` route correctly
(dc:dc-1, kv:services), that route having been configured in a higher
priority than the nspace routes.

However when configured with wildcards (required in the KV area), note
the asterisk below:

```
- :dc
    :service
  - :kv
    - *edit
- :nspace
  - :dc
    - :service
    - :kv
      - *edit
```

Given something like `/dc-1/kv/services` the router instead matches the
`nspace.dc.service` (nspace:dc-1, dc:kv, service:services) route first even
though the `dc.kv.edit` route should still match first.
Changing the `dc.kv.edit` route back to use a non-wildcard route
(:edit instead of *edit), returns the router to match the routes in the
correct order.

In order to work around this, we catch any incorrectly matched routes
(those being directed to the nspace Route but not having a `~`
character in the nspace parameter), and then recalculate the correct
route name and parameters. Lastly we use this recalculated route to
direct the user/app to the correct route.

This route recalcation requires walking up the route to gather up all of
the required route parameters, and although this feels like something
that could already exist in ember, it doesn't seem to. We had already
done a lot of this work a while ago when implementing our `href-mut`
helper. This commit therefore repurposes that work slighlty and externalizes
it outside of the helper itself into a more usable util so we can import
it where we need it. Tests have been added before refactoring it down
to make the code easier to follow.
2020-04-30 09:28:20 +01:00
..
adapters ui: Fix using 'ui-like' KVs when using an empty default nspace (#7734) 2020-04-30 09:28:20 +01:00
components ui: Discovery chain improvements (#7222) 2020-02-06 12:06:47 +00:00
computed ui: UI Release Merge (ui-staging merge) (#6527) 2019-09-30 14:47:49 +01:00
controllers ui: Use WithEventSource mixin on intentions to ensure cleanup (#7333) 2020-02-21 14:00:33 +00:00
forms ui: Namespace Support (#6639) 2019-12-18 12:26:47 +00:00
helpers ui: Fix using 'ui-like' KVs when using an empty default nspace (#7734) 2020-04-30 09:28:20 +01:00
initializers ui: Improve configuration/env/feature flag usage (also add build time year detection) (#7072) 2020-01-21 17:52:40 +00:00
instance-initializers ui: Add live updates/blocking queries to the Intention listing page (#7161) 2020-01-29 16:22:31 +00:00
mixins ui: Acceptance test improvements to prepare for more NS tests (#6980) 2020-01-24 12:26:28 +00:00
models ui: Discovery Chain (#6746) 2019-12-18 12:26:47 +00:00
routes ui: Fix using 'ui-like' KVs when using an empty default nspace (#7734) 2020-04-30 09:28:20 +01:00
routing ui: Namespace Support (#6639) 2019-12-18 12:26:47 +00:00
search/filters ui: Add ability to search nodes listing page with IP Address (#7204) 2020-02-04 10:45:25 -05:00
serializers Ensure ember-data ids are created correct taking into account the nspace (#6974) 2019-12-20 14:30:19 +00:00
services ui: Ensure blocking query configuration is passed through to findInstanceBySlug (#7543) 2020-03-30 15:23:06 +01:00
styles ui: Be more specific with the display toggling checkboxes (#7309) 2020-02-18 17:05:45 +00:00
templates ui: Use the `each key=""` parameter to force ember to reuse DOM (#7550) 2020-03-31 14:58:59 +01:00
utils ui: Fix using 'ui-like' KVs when using an empty default nspace (#7734) 2020-04-30 09:28:20 +01:00
validations ui: Namespace Support (#6639) 2019-12-18 12:26:47 +00:00
app.js UI V2 (#4086) 2018-05-10 19:52:53 +01:00
env.js ui: Improve configuration/env/feature flag usage (also add build time year detection) (#7072) 2020-01-21 17:52:40 +00:00
index.html ui: Keyboard navigation improvements (#7090) 2020-01-20 14:15:15 +00:00
resolver.js UI V2 (#4086) 2018-05-10 19:52:53 +01:00
router.js ui: Improve configuration/env/feature flag usage (also add build time year detection) (#7072) 2020-01-21 17:52:40 +00:00