Fix up decimal migration_diff

Author: dereuromark

src/Command/BakeMigrationDiffCommand.php

@@ -311,6 +311,16 @@ protected function getColumns(): void
                         unset($changedAttributes['length']);
                     }
 
+                    // For decimal columns, ensure precision (scale) is included for proper conversion
+                    if (
+                        isset($column['type']) &&
+                        ($column['type'] === 'decimal' || $column['type'] === 'float') &&
+                        !isset($changedAttributes['precision']) &&
+                        isset($column['precision'])
+                    ) {
+                        $changedAttributes['precision'] = $column['precision'];
+                    }
+
                     $this->templateData[$table]['columns']['changed'][$columnName] = $changedAttributes;
                 }
             }

tests/TestCase/Command/BakeMigrationDiffCommandTest.php

@@ -241,6 +241,19 @@ public function testBakingDiffWithAutoIdIncompatibleUnsignedPrimaryKeys(): void
         $this->runDiffBakingTest('WithAutoIdIncompatibleUnsignedPrimaryKeys');
     }
 
+    /**
+     * Tests that baking a diff with decimal column changes includes precision
+     * This is a regression test for the fix that ensures precision is included for decimal/float columns
+     *
+     * @return void
+     */
+    public function testBakingDiffDecimalChange()
+    {
+        $this->skipIf(!env('DB_URL_COMPARE'));
+
+        $this->runDiffBakingTest('DecimalChange');
+    }
+
     /**
      * Tests that baking a diff with --plugin option only includes tables with Table classes
      */

tests/comparisons/Diff/decimalChange/the_diff_decimal_change_mysql.php

@@ -0,0 +1,48 @@
+<?php
+declare(strict_types=1);
+
+use Migrations\BaseMigration;
+
+class TheDiffDecimalChangeMysql extends BaseMigration
+{
+    /**
+     * Up Method.
+     *
+     * More information on this method is available here:
+     * https://book.cakephp.org/migrations/4/en/migrations.html#the-up-method
+     *
+     * @return void
+     */
+    public function up(): void
+    {
+        $this->table('products')
+            ->changeColumn('price', 'decimal', [
+                'default' => null,
+                'limit' => null,
+                'null' => false,
+                'precision' => 10,
+                'scale' => 2,
+            ])
+            ->update();
+    }
+
+    /**
+     * Down Method.
+     *
+     * More information on this method is available here:
+     * https://book.cakephp.org/phinx/0/en/migrations.html#the-down-method
+     *
+     * @return void
+     */
+    public function down(): void
+    {
+        $this->table('products')
+            ->changeColumn('price', 'decimal', [
+                'default' => null,
+                'null' => false,
+                'precision' => 8,
+                'scale' => 2,
+            ])
+            ->update();
+    }
+}

tests/comparisons/Diff/decimalChange/the_diff_decimal_change_pgsql.php

@@ -0,0 +1,48 @@
+<?php
+declare(strict_types=1);
+
+use Migrations\BaseMigration;
+
+class TheDiffDecimalChangePgsql extends BaseMigration
+{
+    /**
+     * Up Method.
+     *
+     * More information on this method is available here:
+     * https://book.cakephp.org/migrations/4/en/migrations.html#the-up-method
+     *
+     * @return void
+     */
+    public function up(): void
+    {
+        $this->table('products')
+            ->changeColumn('price', 'decimal', [
+                'default' => null,
+                'limit' => null,
+                'null' => false,
+                'precision' => 10,
+                'scale' => 2,
+            ])
+            ->update();
+    }
+
+    /**
+     * Down Method.
+     *
+     * More information on this method is available here:
+     * https://book.cakephp.org/phinx/0/en/migrations.html#the-down-method
+     *
+     * @return void
+     */
+    public function down(): void
+    {
+        $this->table('products')
+            ->changeColumn('price', 'decimal', [
+                'default' => null,
+                'null' => false,
+                'precision' => 8,
+                'scale' => 2,
+            ])
+            ->update();
+    }
+}