Laravel Türkiye Discord Kanalı Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz?

  • AArda

      Seviye 28
    • Gönderi #1
    • Düzenlendi
    Merhaba arkadaşlar,

    Tamamen ihtiyaçtan, Mutlucell firmasına ait SMS hizmetini Laravel 5 ve 4 üzerinde kullanmak için bir API hazırladım.

    Bu api ile
    * Tekli veya çoklu SMS gönderebilir,
    * Bu çoklu sms leri farklı kişilere gönderebilir,
    * İleri tarihe sms gönderimi ayarlayabilir
    * Bakiye bilgisi alabilir,
    * Farklı originatörler kullanabilirsiniz.

    http://arda.pw/laravel-4-ile-sms-gonderme-mutlucell-sms/

    GitHub üzerinden de aşağıdaki linkten erişebilirsiniz:
    https://github.com/Ardakilic/laravel-mutlucell-sms
    Eline sağlık @Arda,
    Birkaç kod iyileştirme önerim, bir de bug bildirimim olacak.

    Servis sağlayıcılarda $defer özelliğini false ayarladığımızda provides metodu tanımlamamıza gerek yok. Eğer ertelemişsek, o zaman IoC konteynerinin hangi anahtarın sağlandığını bilmesi için provides metodu tanımlanması gerekiyor.

    Her üç sms gönderme metodunda da senderId, dateStr, xml'nin ilk ve son satırları tekrarlanıyor. Hatta ara kısmı da bir kısmında döngü içinde olmakla birlikte aynıdır. Bu işlemlerin private veya protected bir fonksiyonda yapılması daha uygun olabilir. Hatta Laravel'in Route ekleme metoduna baktığımızda üç gönderme metodunun tek metoda indirilmesi bile düşünülebilir. Sadece send. Send metodu parametreyi kontrol ederek üç private metoddan birini seçilebilir.

    sendBulk metodunun ilk parametresi dizi değilse 101 döndürüyor. Ama bu parametrenin dizi olup olmadığı kontrol edilmeden önce üç işlem yapılıyor. Öyleyse metodun en başında dizi değilse 101 döndürmek daha uygun olacaktır.
    Öte yandan bu recipents parametresinin dizi olma şartı neden gerekli anlayamadım. Zira xml oluşturulurken ", " ile implode edilerek bir string oluşturuluyor. O zaman bu parametre string ise de yine gönderme yapılabilmelidir.

    Her gönderme için ayrıca bir de echo parseOutput satırından kurtulmak yararlı olabilir (gönderme metodlarındaki return $this->postXML($xml); yerine return $this->parseOutput($this->postXML($xml));). Tabii bu da 100, 101, 102 gibi return değerlerinin de doğrudan değil parseOutput ile döndürülmesini gerektirecek.

    Bug: src/Ardakilic/Mutlucell/Mutlucell.php dosyasında 163. satır: '<nums>'.$recipentsString.'</nums>'. yerine '<nums>'.$number.'</nums>'. olarak okunmalıdır.

    Bir de bir sorum olacak; 2 ay kadar önce @sineld Türkiye SMS için bir paket çıkartmıştı. O, mesajı CDATA içine almış, siz de stripText adını verdiğiniz bir fonksiyonla dönüştürmüşsünüz. CDATA içine alınmış mesaj metinlerinin de StripText gibi bir metoddan geçirilmesi gerekli midir? XML yapısını bozmak dışında güvenlikle ilgili bir sorun da olabilir mi?

    • AArda

        Seviye 28
      • Gönderi #3
      • Düzenlendi
      @Sergin yapıcı yorumun için teşekkürler,

      Mesajın üzerine birkaç düzenlemeye gittim, şu commit'ten görebilirsin. Deneyen olursa sevinirim:
      https://github.com/Ardakilic/laravel-mutlucell-sms/commit/1add1a1f53a8e19182d1c808d34fb1ec94ae75e8

      sendBulk'ı string kabul edecek şekilde ayarladım, sanırım sorun olmayacak artık

      Ya ben orada hem XML'den dönen veriyi, hem de okunaklı bir mesaj geri döndürmek istedim. Boolean için getStatus gibi bir metod yazdım. True/false verecek gönderim durumuna göre.

      $1234567#1.0'ı düzgün yakalayabilmek için ayrıca bir regex yazdım.

      163'de de haklısın, düzelttim, teşekkürler.

      CDATA'yı denedim, denemem mi :-/ CDATA ile wraplayınca nedenini bilemediğim bir sebepten mutlucell'in APIsi hatalı xml hatası veriyor. o yüzden stripText diye bir şey yazdım. Bir de güvenlik sorunu olabilir, belki de ondan kapamışlardır, sonuçta CDATA ile sms'i bozacak bir syntax bile basılabilir. & > < gibi karakterleri bile kabul etmiyorken sms cdata basıp ne var ne yoksa yollamak pek akıl kârı gelmedi bana.
      2 ay sonra
      Arkadaşlar kütüphaneyi güncelledim.

      Queue özelliği getirdim. Özellikle toplu işlem yapanların işine yarayacağını düşünüyorum.

      Ayrıca kod girintilerini vs. de düzelttim.

      An itibari ile son commitimden bu güncellemeyi inceleyebilirsiniz.
      7 ay sonra
      Çok güzel bir çalışma. Elinize sağlık
      3 yıl sonra
      Merhaba bu eklentiyi kullanmak istiyorum fakat laravel hakkında hiç bir bilgim yok bu verdiğiniz dosyaları ftp'ye atsam direkt olarak çalıştıramaz mıyım ?