From 7da1049bcd69ea20e80ef59bbdf427ffd6d131cc Mon Sep 17 00:00:00 2001 From: Hannes Giesenow Date: Thu, 28 Aug 2025 10:03:28 +0200 Subject: [PATCH] Do not rely on assert() in production mode. Fixes #162 --- src/CompilerPass/AddingTypesToAdjustmentClearerPass.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/CompilerPass/AddingTypesToAdjustmentClearerPass.php b/src/CompilerPass/AddingTypesToAdjustmentClearerPass.php index 5df7002e..0de35446 100644 --- a/src/CompilerPass/AddingTypesToAdjustmentClearerPass.php +++ b/src/CompilerPass/AddingTypesToAdjustmentClearerPass.php @@ -35,7 +35,9 @@ public function process(ContainerBuilder $container): void private function addClearerToContainerParameter(ContainerBuilder $container, string $parameterName): void { - assert(1 === preg_match('/^%(.*)%$/', $parameterName, $matches)); + if (1 !== preg_match('/^%(.*)%$/', $parameterName, $matches)) { + throw new \RuntimeException(sprintf('Could not match placeholders in parameter name %s', $parameterName)); + } $parameterName = $matches[1]; $listOfAdjustmentsToClear = $container->getParameter($parameterName); $listOfAdjustmentsToClear[] = CustomerOptionRecalculator::CUSTOMER_OPTION_ADJUSTMENT;