fix(router-store): remove circular dependency in serializers (#1904)

Closes #1902

Author: timdeschryver

modules/router-store/src/actions.ts

@@ -6,10 +6,8 @@ import {
   RoutesRecognized,
 } from '@angular/router';
 
-import {
-  BaseRouterStoreState,
-  SerializedRouterStateSnapshot,
-} from './serializers';
+import { BaseRouterStoreState } from './serializers/base';
+import { SerializedRouterStateSnapshot } from './serializers/default_serializer';
 
 /**
  * An action dispatched when a router navigation request is fired.

modules/router-store/src/index.ts

@@ -28,11 +28,15 @@ export {
 } from './router_store_module';
 export {
   RouterStateSerializer,
+  BaseRouterStoreState,
+} from './serializers/base';
+export {
   DefaultRouterStateSerializer,
   SerializedRouterStateSnapshot,
-  BaseRouterStoreState,
+} from './serializers/default_serializer';
+export {
   MinimalActivatedRouteSnapshot,
   MinimalRouterStateSnapshot,
   MinimalRouterStateSerializer,
-} from './serializers';
+} from './serializers/minimal_serializer';
 export { getSelectors } from './router_selectors';

modules/router-store/src/reducer.ts

@@ -5,10 +5,8 @@ import {
   ROUTER_NAVIGATION,
   RouterAction,
 } from './actions';
-import {
-  BaseRouterStoreState,
-  SerializedRouterStateSnapshot,
-} from './serializers';
+import { BaseRouterStoreState } from './serializers/base';
+import { SerializedRouterStateSnapshot } from './serializers/default_serializer';
 
 export type RouterReducerState<
   T extends BaseRouterStoreState = SerializedRouterStateSnapshot

modules/router-store/src/router_selectors.ts

@@ -1,6 +1,6 @@
-import { RouterReducerState } from '@ngrx/router-store';
 import { createSelector } from '@ngrx/store';
 import { RouterStateSelectors } from './models';
+import { RouterReducerState } from './reducer';
 
 export function getSelectors<V>(
   selectState: (state: V) => RouterReducerState<any>

modules/router-store/src/router_store_module.ts

@@ -27,12 +27,14 @@ import {
 } from './actions';
 import { RouterReducerState } from './reducer';
 import {
-  DefaultRouterStateSerializer,
   RouterStateSerializer,
-  SerializedRouterStateSnapshot,
   BaseRouterStoreState,
-  MinimalRouterStateSerializer,
-} from './serializers';
+} from './serializers/base';
+import {
+  DefaultRouterStateSerializer,
+  SerializedRouterStateSnapshot,
+} from './serializers/default_serializer';
+import { MinimalRouterStateSerializer } from './serializers/minimal_serializer';
 
 export type StateKeyOrSelector<
   T extends BaseRouterStoreState = SerializedRouterStateSnapshot

…s/router-store/src/serializers/shared.ts …les/router-store/src/serializers/base.tsmodules/router-store/src/serializers/shared.ts renamed to modules/router-store/src/serializers/base.ts modules/router-store/src/serializers/shared.ts renamed to modules/router-store/src/serializers/base.ts

@@ -1,5 +1,5 @@
 import { ActivatedRouteSnapshot, RouterStateSnapshot } from '@angular/router';
-import { BaseRouterStoreState, RouterStateSerializer } from './shared';
+import { BaseRouterStoreState, RouterStateSerializer } from './base';
 
 export interface SerializedRouterStateSnapshot extends BaseRouterStoreState {
   root: ActivatedRouteSnapshot;

modules/router-store/src/serializers/default_serializer.ts

@@ -1,5 +1,5 @@
 import { RouterStateSnapshot, ActivatedRouteSnapshot } from '@angular/router';
-import { BaseRouterStoreState, RouterStateSerializer } from './shared';
+import { BaseRouterStoreState, RouterStateSerializer } from './base';
 
 export interface MinimalActivatedRouteSnapshot {
   routeConfig: ActivatedRouteSnapshot['routeConfig'];