From eb50665323976bfa7ebdf3767ac1e9ef726300d8 Mon Sep 17 00:00:00 2001
From: Kentaro Ohkouchi <nanasess@fsm.ne.jp>
Date: Thu, 4 Jun 2026 16:58:30 +0900
Subject: [PATCH] =?UTF-8?q?fix:=20Shipping=20=E3=82=A8=E3=83=B3=E3=83=86?=
 =?UTF-8?q?=E3=82=A3=E3=83=86=E3=82=A3=E3=81=AE=20nullable=20=E3=82=AB?=
 =?UTF-8?q?=E3=83=A9=E3=83=A0=E3=81=A8=E5=9E=8B=E5=AE=9A=E7=BE=A9=E3=81=AE?=
 =?UTF-8?q?=E4=B8=8D=E6=95=B4=E5=90=88=E3=82=92=E4=BF=AE=E6=AD=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

dtb_shipping の kana01 / kana02 カラムは nullable: true だが、
getKana01() / getKana02() の戻り型と setKana01() / setKana02() の
引数型が非null (string) になっており、DB に NULL が格納されている
Shipping を読み込んで getter を呼ぶと TypeError が発生する状態だった。

同様に tracking_number カラムも nullable だが setTrackingNumber() の
引数型のみ非null になっていたため修正する。

Customer / CustomerAddress の同名カラムは既に ?string で定義されており、
本変更で住所系エンティティ間の型定義が揃う。型を nullable に広げる
後方互換な変更で、PHPStan も問題なく通過する。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 src/Eccube/Entity/Shipping.php | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/Eccube/Entity/Shipping.php b/src/Eccube/Entity/Shipping.php
index ea61226beea..52b0ff3c9e4 100644
--- a/src/Eccube/Entity/Shipping.php
+++ b/src/Eccube/Entity/Shipping.php
@@ -239,7 +239,7 @@ public function getName02(): string
         /**
          * Set kana01.
          */
-        public function setKana01(string $kana01): Shipping
+        public function setKana01(?string $kana01): Shipping
         {
             $this->kana01 = $kana01;
 
@@ -249,7 +249,7 @@ public function setKana01(string $kana01): Shipping
         /**
          * Get kana01.
          */
-        public function getKana01(): string
+        public function getKana01(): ?string
         {
             return $this->kana01;
         }
@@ -257,7 +257,7 @@ public function getKana01(): string
         /**
          * Set kana02.
          */
-        public function setKana02(string $kana02): Shipping
+        public function setKana02(?string $kana02): Shipping
         {
             $this->kana02 = $kana02;
 
@@ -267,7 +267,7 @@ public function setKana02(string $kana02): Shipping
         /**
          * Get kana02.
          */
-        public function getKana02(): string
+        public function getKana02(): ?string
         {
             return $this->kana02;
         }
@@ -643,7 +643,7 @@ public function getOrder(): Order
         /**
          * Set trackingNumber
          */
-        public function setTrackingNumber(string $trackingNumber): Shipping
+        public function setTrackingNumber(?string $trackingNumber): Shipping
         {
             $this->tracking_number = $trackingNumber;