fix(common): remove code duplication between entry-points (#51500)

devversion · thePunderWoman · commit 86c5e34601d7 · 2023-08-29T17:55:34.000Z
The common packages were duplicating a little bit of code due to relative imports between entry-points. This caused bundlers to inline shared functions twice in both FESM outputs.i PR Close #51500
diff --git a/packages/common/src/common.ts b/packages/common/src/common.ts
@@ -28,3 +28,4 @@ export {VERSION} from './version';
 export {ViewportScroller, NullViewportScroller as ɵNullViewportScroller} from './viewport_scroller';
 export {XhrFactory} from './xhr';
 export {IMAGE_CONFIG, ImageConfig, IMAGE_LOADER, ImageLoader, ImageLoaderConfig, NgOptimizedImage, PRECONNECT_CHECK_BLOCKLIST, provideCloudflareLoader, provideCloudinaryLoader, provideImageKitLoader, provideImgixLoader} from './directives/ng_optimized_image';
+export {normalizeQueryParams as ɵnormalizeQueryParams} from './location/util';
diff --git a/packages/common/testing/src/location_mock.ts b/packages/common/testing/src/location_mock.ts
@@ -6,12 +6,10 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {Location, LocationStrategy} from '@angular/common';
+import {Location, LocationStrategy, ɵnormalizeQueryParams as normalizeQueryParams} from '@angular/common';
 import {EventEmitter, Injectable} from '@angular/core';
 import {SubscriptionLike} from 'rxjs';
 
-import {normalizeQueryParams} from '../../src/location/util';
-
 /**
  * A spy for {@link Location} that allows tests to fire simulated location events.
  *
