-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Kur kodu eşleştirmelerinin kod tabanından çıkartılması #50
Comments
Adaptörlerin farklı implementasyonlarında farklı kur kodları mı kullanılıyor? Geliştirici her adaptör için hangi kura karşılık hangi kodu vereceğini bilmek zorunda mı? |
Pos implementasyonunun içinde hardcoded sabitlerin olması biraz çirkin göründü. Amacım sadece eşleştirme verisini kodun dışına biryerlere almaktı. Aslında konfigürasyon senin de deiğin gibi kullanıcı tarafından doldurulan birşey olduğundan pek de mapin barındırılması gereken bir yer değil gibi gerçekten. Belki bir base konfigürasyon kullanılabilir bu tarz mapping işleri buralarda saklanabilir ve kullanıcının pos spesifik girdiği konfigürasyon verileri bu bilgileri miras alabilir. Öte yandan uzun vadede ben o konfigürasyondan zaten tamamen kurtulmayı planlıyorum. Kullanıcının o bilgileri nerede nasıl tuttuğu ile ilgili hiçbirşey kod tabanında kalmamalı. İsterse PHP dizisi olarak saklasın, isterse JSON veya XML olarak saklasın. Bunun için de konfigürasyona ara bir katmandan ulaşılması ve katmanın beklenen değerleri valide etmesi gerekiyor. Katman hem kullanıcı tarafından gönderilen değişken verileri hem de benim sabit olarak saklamak istediğim verileri birleştirip pos implementasyonuna sunabilmeli şeklinde düşünüyorum. Sen ne dersin ? |
Son durumda $currencyCodes dizisi ile ISO kodları AbstractAdapter'da, özel implementasyonlar ise adapterlar içinde (bkz. posnet) map ediliyor. Bu hali uygun ise bu issue'yu kapatabiliriz. |
Kur kodlarıyla ilgili sabitler ve sağlayıcı tarafından beklenen kur kodlarıyla ilgili değişken ve sabitlerin ödeme adaptörlerinden temizlenerek konfigürasyona taşınması.
The text was updated successfully, but these errors were encountered: