Skip to content
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

Payuclient ile official ALU php client arasındaki farklar #31

Open
edigu opened this issue Nov 3, 2015 · 1 comment
Open

Payuclient ile official ALU php client arasındaki farklar #31

edigu opened this issue Nov 3, 2015 · 1 comment

Comments

@edigu
Copy link
Contributor

edigu commented Nov 3, 2015

Selam, bir süre kullandıktan sonra bu kütüphanenin orjinal ALU Client ile hangi noktalarda niçin ayrıldığı konusunda kafamda soru işaretleri oluştu.

Her iki kodu da inceledikten sonra neden bağımlılık olarak orjinal ALU'nun tercih edilmediğini merak ettim, gördüğüm kadarıyla official ALU pek maintain edilmese de payuclient tarafında bazı benzer içerikli tekrar eden sınıfların olduğunu gördüm. (Card, Merchant, Billing v.s.)

Bununla ilgili bir dökümantasyon ihtiyacı var gibi geldi bana, en azından aşağıdaki maddeler hakkında birkaç satır yazmak/düşünmek gerekebilir

  • payuclient hangi noktalarda alu-client-php kütüphanesi ile ayrışır
  • neden official kütüphane composer üzerinden dependency olarak alınıp ihtiyaç olan sınıfların override edilmesi yerine tüm yapı baştan yaratılma yoluna gitmeyi tercih eder?
  • Bu durum biraz zahmetli de olsa refactor sürecine girilip değiştirilebilir mi? Refactor etmek doğru tercih olur mu?

Ne düşünüyorsunuz?

@ibrahimgunduz34
Copy link
Collaborator

Maintein edilebilirlik anlaminda karsilastirdigimda zaten soyleyecek cok fazla birsey yok. :)

Kullanilabilirlik anlaminda karsilastirdigimda payuclient i digerinden farkli kilacak vazgecilmez bir ozelik goremedim acikcasi. Belki istegi olusturan componentlerin tek tek elle insa edilmek yerine builderlar ile ayakaldiriliyor olmasi kullanicilara kullanim kolayligi sagliyor olabilir. Bir de bu haftaki release le gelecek olan transaction eventleri loglama veya araya hook filan atma konusunda gelistiriclere arti saglayabilecegini dusunuyorum.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants