Ardakilic / laravel-mutlucell-sms

Laravel için Mutlucell SMS Kütüphanesi
MIT License
45 stars 11 forks source link

generateXMLStringForSending strlen() hatası #16

Closed tuncaybahadir closed 4 years ago

tuncaybahadir commented 4 years ago

Merhabalar,

Öncelikle kütüphane için ellerinize sağlık. Laravel 6 sürümde karşılaştığım bir durumu iletmek istiyorum.

Mutlucell.php sınıfı içerisinde yer alan generateXMLStringForSending isimli metod içerisinde date parametresi'i strlen kontrolüne dahil edilmiş. Gelen data ise array tipli. Sanırım message request'i üzerinde olması gerekirken date değişkenine aktarım yapılmış.

İlgili sınıfta 145. satırda send metodu ve 115. satırda yer alan sendBulk metodlarında aşağıdaki şekilde çağırım yapılmış. Benmi yanlış kontrol ediyorum acaba ?

İyi çalışmalar.

resim

resim

resim

Ardakilic commented 4 years ago

Merhaba, parametreleri generateXMLStringForSending'e ters göndermişim evet, dikkatimden kaçmış teşekkürler :)

https://github.com/Ardakilic/laravel-mutlucell-sms/commit/2274613f5d2ecf403b40fb41cfa4122e3e494bd5

commit'inde düzelmiş olmalı, acaba dev-master ile kontrol edebilir misin?

Yeniden teşekkürler!

tuncaybahadir commented 4 years ago

👍 Siz bu kadar paylaştınız bizde bir zahmet bakalım :) Kontrol edip dönüş sağlayacağım.

tuncaybahadir commented 4 years ago

Ellerinize sağlık Arda Bey,

Sorun düzelmiş.

İyi çalışmalar.

Ardakilic commented 4 years ago

Rica ederim, yeni release geçiyorum birazdan 👍

Ardakilic commented 4 years ago

Yeni release geçtim, v3.0.1'de bu sorun artık olmayacak, composer.json dosyanızı eski haline döndürüp paket güncellemesi alırsanız güncel sürümü alabilirsiniz.