From a02b957b60faa7d5390df19cf8d9593e95e230b3 Mon Sep 17 00:00:00 2001 From: Matthieu Riegler Date: Sat, 1 Nov 2025 17:22:01 +0100 Subject: [PATCH] refactor(core): remove resource flag. This was used to migrate G3 and is no longer necessary. (cherry picked from commit d1ab73dd8771e11f5a363b66ac24015ed2407af0) --- packages/core/src/resource/resource.ts | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/packages/core/src/resource/resource.ts b/packages/core/src/resource/resource.ts index e15a7116f9c..04e3bdcf858 100644 --- a/packages/core/src/resource/resource.ts +++ b/packages/core/src/resource/resource.ts @@ -30,13 +30,6 @@ import {PendingTasks} from '../pending_tasks'; import {linkedSignal} from '../render3/reactivity/linked_signal'; import {DestroyRef} from '../linker/destroy_ref'; -/** - * Whether a `Resource.value()` should throw an error when the resource is in the error state. - * - * This internal flag is being used to gradually roll out this behavior. - */ -let RESOURCE_VALUE_THROWS_ERRORS_DEFAULT = true; - /** * Constructs a `Resource` that projects a reactive request to an asynchronous operation defined by * a loader function, which exposes the result of the loading operation via signals. @@ -77,7 +70,6 @@ export function resource(options: ResourceOptions): ResourceRef extends BaseWritableResource implements Resou defaultValue: T, private readonly equal: ValueEqualityFn | undefined, injector: Injector, - throwErrorsFromValue: boolean = RESOURCE_VALUE_THROWS_ERRORS_DEFAULT, ) { super( // Feed a computed signal for the value to `BaseWritableResource`, which will upgrade it to a @@ -196,11 +187,7 @@ export class ResourceImpl extends BaseWritableResource implements Resou } if (!isResolved(streamValue)) { - if (throwErrorsFromValue) { - throw new ResourceValueError(this.error()!); - } else { - return defaultValue; - } + throw new ResourceValueError(this.error()!); } return streamValue.value;