Merge branch 'master' into chriskrycho/unneeded-setFactoryFor-this-this

emberjs · Mar 16, 2022 · 74b77a9 · 74b77a9
2 parents 0a1b4e4 + 86f8dad
commit 74b77a9
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 14 deletions.
diff --git a/packages/@ember/-internals/container/lib/container.ts b/packages/@ember/-internals/container/lib/container.ts
@@ -480,10 +480,6 @@ export class FactoryManager<T, C extends FactoryClass | object = FactoryClass> {
     this.normalizedName = normalizedName;
     this.madeToString = undefined;
     this.injections = undefined;
-
-    if (isInstantiatable(container, fullName)) {
-      setFactoryFor(factory, this);
-    }
   }
 
   toString(): string {
@@ -503,14 +499,10 @@ export class FactoryManager<T, C extends FactoryClass | object = FactoryClass> {
       );
     }
 
-    let props = {};
+    let props = options ? { ...options } : {};
     setOwner(props, container.owner!);
     setFactoryFor(props, this);
 
-    if (options !== undefined) {
-      props = Object.assign({}, props, options);
-    }
-
     if (DEBUG) {
       let lazyInjections;
       let validationCache = this.container.validationCache;

diff --git a/packages/@ember/-internals/routing/lib/services/routing.ts b/packages/@ember/-internals/routing/lib/services/routing.ts
@@ -12,7 +12,7 @@ import { Route } from '../..';
 import EmberRouter from '../system/router';
 import RouterState from '../system/router_state';
 
-const ROUTER = (symbol('ROUTER') as unknown) as string;
+const ROUTER = symbol('ROUTER');
 
 /**
   The Routing service is used by LinkTo, and provides facilities for

diff --git a/packages/@ember/-internals/routing/lib/system/router.ts b/packages/@ember/-internals/routing/lib/system/router.ts
@@ -341,7 +341,7 @@ class EmberRouter<R extends Route = Route> extends EmberObject.extend(Evented) i
 
         assert('Route is unexpectedly missing an owner', routeOwner);
 
-        let route = routeOwner.lookup(fullRouteName) as R;
+        let route = routeOwner.lookup(fullRouteName) as R | undefined;
 
         if (seen[name]) {
           assert('seen routes should exist', route);
@@ -351,6 +351,8 @@ class EmberRouter<R extends Route = Route> extends EmberObject.extend(Evented) i
         seen[name] = true;
 
         if (!route) {
+          // SAFETY: this is configured in `commonSetupRegistry` in the
+          // `@ember/application/lib` package.
           let DefaultRoute: any = routeOwner.factoryFor('route:basic')!.class;
           routeOwner.register(fullRouteName, DefaultRoute.extend());
           route = routeOwner.lookup(fullRouteName) as R;
@@ -362,15 +364,15 @@ class EmberRouter<R extends Route = Route> extends EmberObject.extend(Evented) i
           }
         }
 
-        route!._setRouteName(routeName);
+        route._setRouteName(routeName);
 
-        if (engineInfo && !hasDefaultSerialize(route!)) {
+        if (engineInfo && !hasDefaultSerialize(route)) {
           throw new Error(
             'Defining a custom serialize method on an Engine route is not supported.'
           );
         }
 
-        return route!;
+        return route;
       }
 
       getSerializer(name: string) {