[GH-ISSUE #123] VakifBankPos 'Invalid installment' hatası #66

New issue

Closed

opened 2026-03-03 15:42:38 +03:00 by kerem · 3 comments

kerem commented

2026-03-03 15:42:38 +03:00

Owner

Originally created by @FikretCin on GitHub (May 2, 2023).
Original GitHub issue: https://github.com/mewebstudio/pos/issues/123

Information	Description
Kütüphane versiyonu	0.12.1
PHP versiyonu	7.4
Gateway ismi	VakifBankPos
Banka ismi	Vakıfbank

Merhabalar;

Hatanın özeti

Paket içerisindeki ödeme geçitlerinin verileri eşleyen sınıflarında ({Gateway}RequestDataMapper), siparişe atanan taksit sayısını düzenleyen mapInstallment metodu taksit sayısı 1 olarak gönderildiğinde gayet düzgün çalışmakta. Fakat VakifBankPosRequestDataMapper ın içerisindeki mapInstallment'a 1 gönderildiğinde bankadan geçersiz taksit (Invalid installment
) hatası alınmakta. Regular ile test edildiğinde bu işlem problem çıkarmıyorken 3d işleminde sorun yaratıyor.

Sorunu çözmek için belirttiğim dosyadanın belirtilen satırındaki operatörü >= ile değiştirdiğimde problem çözüldü.

https://github.com/mewebstudio/pos/blob/master/src/DataMapper/VakifBankPosRequestDataMapper.php#L275

return $installment > 1 ? (string) $installment : '0';
return $installment >= 1 ? (string) $installment : '0';

Çözüm yöntemim doğru ise paketi günceller misiniz ? Eğer kullanım yöntemimde bir sorun var ise örneğin peşin fiyat için 1 değil de 0 gönderdiğimde tüm ödeme geçitleri sorunsuz çalışacak ise projemi revize edeceğim.

Originally created by @FikretCin on GitHub (May 2, 2023). Original GitHub issue: https://github.com/mewebstudio/pos/issues/123 | Information | Description | |--------------|---------| | Kütüphane versiyonu | 0.12.1 | | PHP versiyonu | 7.4 | | Gateway ismi| VakifBankPos| | Banka ismi| Vakıfbank | Merhabalar; ### Hatanın özeti Paket içerisindeki ödeme geçitlerinin verileri eşleyen sınıflarında ({Gateway}RequestDataMapper), siparişe atanan taksit sayısını düzenleyen mapInstallment metodu taksit sayısı 1 olarak gönderildiğinde gayet düzgün çalışmakta. Fakat VakifBankPosRequestDataMapper ın içerisindeki mapInstallment'a 1 gönderildiğinde bankadan geçersiz taksit (Invalid installment ) hatası alınmakta. Regular ile test edildiğinde bu işlem problem çıkarmıyorken 3d işleminde sorun yaratıyor. Sorunu çözmek için belirttiğim dosyadanın belirtilen satırındaki operatörü >= ile değiştirdiğimde problem çözüldü. https://github.com/mewebstudio/pos/blob/master/src/DataMapper/VakifBankPosRequestDataMapper.php#L275 `return $installment > 1 ? (string) $installment : '0';` `return $installment >= 1 ? (string) $installment : '0';` Çözüm yöntemim doğru ise paketi günceller misiniz ? Eğer kullanım yöntemimde bir sorun var ise örneğin peşin fiyat için 1 değil de 0 gönderdiğimde tüm ödeme geçitleri sorunsuz çalışacak ise projemi revize edeceğim.

kerem closed this issue

2026-03-03 15:42:39 +03:00

kerem commented

2026-03-03 15:42:39 +03:00

Author

Owner

@nuryagdym commented on GitHub (May 2, 2023):

Merhabalar, bugun icinde kodu belirttiginiz gibi guncellemeye calirisirim.
Ama genel olarak, taksit miktari 1 verilmemesi gerekiyor. ya 0 ya da > 1 sayi olmasi gerekiyor.

Kutuphanede taksit degeri saglanmadiginda taksit'i 0 olarak atar https://github.com/mewebstudio/pos/blob/master/src/Gateways/VakifBankPos.php#L281

Ayrinca vakifbank'in Common Payment diye yeni API destegi eklendi. VPOS 7/24 eski API, yeni Common Payment ise yeni API'lari diye biliyorum.
https://github.com/mewebstudio/pos/blob/master/config/pos.php#L163

@nuryagdym commented on GitHub (May 2, 2023): Merhabalar, bugun icinde kodu belirttiginiz gibi guncellemeye calirisirim. Ama genel olarak, taksit miktari 1 verilmemesi gerekiyor. ya 0 ya da > 1 sayi olmasi gerekiyor. Kutuphanede taksit degeri saglanmadiginda taksit'i 0 olarak atar https://github.com/mewebstudio/pos/blob/master/src/Gateways/VakifBankPos.php#L281 Ayrinca vakifbank'in Common Payment diye yeni API destegi eklendi. VPOS 7/24 eski API, yeni Common Payment ise yeni API'lari diye biliyorum. https://github.com/mewebstudio/pos/blob/master/config/pos.php#L163

kerem commented

2026-03-03 15:42:39 +03:00

Author

Owner

@FikretCin commented on GitHub (May 2, 2023):

Bilgilendirme ve güncelleme için teşekkürler.

@FikretCin commented on GitHub (May 2, 2023): Bilgilendirme ve güncelleme için teşekkürler.

kerem commented

2026-03-03 15:42:39 +03:00

Author

Owner

@nuryagdym commented on GitHub (May 13, 2023):

sorunu kapatiyorum,
cozum olarak taksit miktari ya 0, ya da >= 2 deger verilince sorun olmuyor.

@nuryagdym commented on GitHub (May 13, 2023): sorunu kapatiyorum, cozum olarak taksit miktari ya 0, ya da >= 2 deger verilince sorun olmuyor.

kerem referenced this issue

2026-03-03 15:43:52 +03:00

[PR #66] [MERGED] Refactor methods for better testing #189