Laravel 11 ve 12 desteği

[sdkakcy]

• Laravel 11 ve 12 desteği
• Currency class Enum dönüşümü
• Type hintler
• Laravel Pint ile code style fix

[sdkakcy]

Selamlar @GizemSever
Biliyorsun Laravel Laravel 10 desteğini sonlandıralı bir süre oluyor, ben de bir süredir başlama niyetinde olduğum sürüm yükseltme işini bakma fırsatı buldum. Paketi Laravel 11 ve 12 için destek verecek şekilde düzenleme yaparken php 8.2 bağımlılığından aldığım güce dayanarak kod tabanında da değişikliklere gittim. Parametre tipleri ve geri dönüş değerleri, constructer property promotion kullanımı ve daha fazlası var bu PR da. Olabildiğince dikkatli davranmaya çalıştım ama ikinci bir göz her zaman iyidir.

İhtilafa düştüğüm bir konu oldu PR ı açarken farkettim aslında; mevcut kullanımda class property olarak tanımlı olan Currency'ler artık Enum olarak tanımlı metodların aldığı ve döndüğü değerleri değiştirmeyim diyerek Currency::TRY->value şeklinde (string) alacak şekilde refactor ettim ama bu durumda güncelleme sonrası Currency kullanımının value gönderecek şekilde düzenlemek gerekiyor. Bu böyle kalsın breaking change php versiyonu var zaten diyorsan kalsın ama php'si zaten güncel vatandaş paketi güncelleyince kodunu düzenlemek zorunda kalmasın dersen en azından setter fonksiyonunu Currency enum alacak şekilde güncelleyebilirim.

Currency class → Enum dönüşümü

- class Currency
- {
-     public const TRY = 'TRY';
-     public const USD = 'USD';
-     public const EUR = 'EUR';
- }
+ enum Currency: string
+ {
+     case TRY = 'TRY';
+     case USD = 'USD';
+     case EUR = 'EUR';
+ }

- public function setCurrency(string $currency): static
- {
-     $this->currency = $currency;
-     return $this;
- }
+ public function setCurrency(Currency|string $currency): static
+ {
+     $this->currency = $currency instanceof Currency
+         ? $currency->value
+         : $currency;
+
+     return $this;
+ }

ya da her ikisini mi desteklemeli bilemedim mevcutta class kullanmayıp string geçenlerde olmuştur belki.

[GizemSever]

@sdkakcy currency ile ilgili kısmı orjinal hali ile bırakmanız mümkün müdür ?

[sdkakcy]

@GizemSever o daha mantıklı bir seçenek sanırım. Düzenleyeyim o şekilde.

[sdkakcy]

@GizemSever Currency.php yi mevcut haline geri çevirdim.

[sdkakcy]

@GizemSever