[GH-ISSUE #18] SMS Doğrulaması Hata Kontrolü #7

Closed
opened 2026-03-02 04:06:12 +03:00 by kerem · 2 comments
Owner

Originally created by @brkygncn on GitHub (Jan 25, 2021).
Original GitHub issue: https://github.com/furkankadioglu/efatura/issues/18

Originally assigned to: @furkankadioglu on GitHub.

verifySMSCode fonskiyonunda dönen verinin checkError ile hata kontrolüne ek olarak $body["data"]["sonuc"] alanının da kontrolü gerekiyor.

Şu anki sistemde SMS şifresi hatalı girilse bile array üzerinde bir error kümesi dönmediği için işlem başarılı kabul edilip bağlı diğer işlemler devam ediyor. Şu sıralar pek vaktim olmadığı için pull request gönderemedim, kusura bakmayın.

$body ile gelen dönüşler hatalı ve başarılı sonuçlar için aşağıdaki şekilde:

Başarısız Dönüş

Array ( [data] => Array ( [msg] => SMS şifrenizi yanlış girdiniz [sonuc] => 0 ) [metadata] => Array ( [optime] => 20210116133633+0300 ) )

Başarısız Dönüş

Array ( [data] => Array ( [msg] => SMS şifreniz doğrulandı, işlem başarılı. [sonuc] => 1 ) [metadata] => Array ( [optime] => 20210116133544+0300 ) )

Originally created by @brkygncn on GitHub (Jan 25, 2021). Original GitHub issue: https://github.com/furkankadioglu/efatura/issues/18 Originally assigned to: @furkankadioglu on GitHub. verifySMSCode fonskiyonunda dönen verinin checkError ile hata kontrolüne ek olarak $body["data"]["sonuc"] alanının da kontrolü gerekiyor. Şu anki sistemde SMS şifresi hatalı girilse bile array üzerinde bir error kümesi dönmediği için işlem başarılı kabul edilip bağlı diğer işlemler devam ediyor. Şu sıralar pek vaktim olmadığı için pull request gönderemedim, kusura bakmayın. $body ile gelen dönüşler hatalı ve başarılı sonuçlar için aşağıdaki şekilde: # Başarısız Dönüş Array ( [data] => Array ( [msg] => SMS şifrenizi yanlış girdiniz [sonuc] => 0 ) [metadata] => Array ( [optime] => 20210116133633+0300 ) ) # Başarısız Dönüş Array ( [data] => Array ( [msg] => SMS şifreniz doğrulandı, işlem başarılı. [sonuc] => 1 ) [metadata] => Array ( [optime] => 20210116133544+0300 ) )
kerem closed this issue 2026-03-02 04:06:13 +03:00
Author
Owner

@furkankadioglu commented on GitHub (Feb 6, 2021):

Merhaba, gerekli güncellemeyi yapacağım. Belirttiğiniz için çok teşekkürler.

Katkı sağlayanlarda çıkmak isterseniz, pull request atarsanız da kabul ederim. :)

<!-- gh-comment-id:774477609 --> @furkankadioglu commented on GitHub (Feb 6, 2021): Merhaba, gerekli güncellemeyi yapacağım. Belirttiğiniz için çok teşekkürler. Katkı sağlayanlarda çıkmak isterseniz, pull request atarsanız da kabul ederim. :)
Author
Owner

@furkankadioglu commented on GitHub (Feb 7, 2021):

Eklendi. Siz de test ederseniz çok sevinirim.

<!-- gh-comment-id:774754225 --> @furkankadioglu commented on GitHub (Feb 7, 2021): Eklendi. Siz de test ederseniz çok sevinirim.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/efatura#7
No description provided.