Only require strict check as opt-in

Author: dereuromark

src/Decimal.php

@@ -65,14 +65,15 @@ class Decimal implements JsonSerializable, Stringable
     /**
      * @param object|string|float|int $value
      * @param int|null $scale Decimal places in the result. Omit to enable auto-detection.
+     * @param bool $strict If scale should be strictly checked to avoid accidental precision loss.
      */
-    public function __construct(object|string|float|int $value, ?int $scale = null)
+    public function __construct(object|string|float|int $value, ?int $scale = null, bool $strict = false)
     {
         $value = $this->parseValue($value);
         $value = $this->normalizeValue($value);
 
         $this->setValue($value, $scale);
-        $this->setScale($scale);
+        $this->setScale($scale, $strict);
     }
 
     /**
@@ -146,16 +147,17 @@ protected function normalizeValue(string $value): string
      *
      * @param object|string|float|int $value
      * @param int|null $scale Decimal places in the result. Omit to enable auto-detection.
+     * @param bool $strict If scale should be strictly checked to avoid accidental precision loss.
      *
      * @return static
      */
-    public static function create(object|string|float|int $value, ?int $scale = null): static
+    public static function create(object|string|float|int $value, ?int $scale = null, bool $strict = false): static
     {
         if ($scale === null && $value instanceof static) {
             return clone $value;
         }
 
-        return new static($value, $scale);
+        return new static($value, $scale, $strict);
     }
 
     /**
@@ -737,7 +739,7 @@ protected function copy(?string $integerPart = null, ?string $decimalPart = null
         }
         if ($decimalPart !== null) {
             $clone->fractionalPart = $decimalPart;
-            $clone->setScale(null);
+            $clone->setScale(null, false);
         }
         if ($negative !== null) {
             $clone->negative = $negative;
@@ -860,15 +862,16 @@ protected function fromScientific(string $value, ?int $scale): void
 
     /**
      * @param int|null $scale
+     * @param bool $strict
      *
      * @throws \InvalidArgumentException
      *
      * @return void
      */
-    protected function setScale(?int $scale): void
+    protected function setScale(?int $scale, bool $strict): void
     {
         $calculatedScale = strlen($this->fractionalPart);
-        if ($scale && $calculatedScale > $scale) {
+        if ($strict && $scale && $calculatedScale > $scale) {
             throw new InvalidArgumentException('Loss of precision detected. Detected scale `' . $calculatedScale . '` > `' . $scale . '` as defined.');
         }
 

tests/DecimalTest.php

@@ -1036,16 +1036,16 @@ public function testDebugInfo(): void
      */
     public function testPrecisionLossProtection(): void
     {
-        $a = Decimal::create('0.1', 50);
+        $a = Decimal::create('0.1', 50, true);
         $this->assertSame(50, $a->scale());
 
-        $b = Decimal::create($a);
+        $b = Decimal::create($a, null, true);
         $this->assertSame(50, $b->scale());
 
-        $c = Decimal::create($b, 6); // Not 50 if manually overwritten
+        $c = Decimal::create($b, 6, true); // Not 50 if manually overwritten
         $this->assertSame(6, $c->scale());
 
-        $d = Decimal::create($c, 64);
+        $d = Decimal::create($c, 64, true);
         $this->assertSame(64, $d->scale());
     }
 
@@ -1057,6 +1057,6 @@ public function testPrecisionLossFail(): void
         $this->expectException(InvalidArgumentException::class);
         $this->expectExceptionMessage('Loss of precision detected');
 
-        Decimal::create('0.123', 2);
+        Decimal::create('0.123', 2, true);
     }
 }