Teşekkürler, yavaş yavaş alışacağız artık github standartlarına.
Bu konuda size ufak bir yardımım dokunsun diye kodunuzu forklayıp üzerinde biraz çalıştım. Ardından pull request gönderdim. Açıklama şu şekilde:
https://github.com/oktayc0/SplitImagePHP/pull/1
Öncelikle composer initialization ekledim. packagist.org a submit edilirse composer require oktayc0/split-image-php şeklinde kurulabilir.
Ardından psr-1 uyarlaması yaptım.
Sonrasında ise kodu php@8'e göre revize ederek ufak bir iki hatayı da giderdim.
Örneğin ilk gözüme çarpan eksiklik 57. satırdaki imagejpeg fonksiyonu ile görselleri oluşturma işlemi. Dosya tipine bakılmaksızın jpeg oluşturuluyor. Her ne kadar uzantı png ya da gif olsa da mime jpeg olarak giriyor. Bunu düzelttim.
İkincisi ise 18.satırdaki öntanımsız değişken. Onu da php@8 ile gelen match fonksiyonunu kullanarak düzenledim. Bu sayede altındaki switch-case'den kurtulmuş oldu.
Üçüncü olarak dosya uzantısı kontrol edilse de görselin MIME'i kontrol edilmiyordu, onu da kontrole ekledim. Bu sayede mime type'ı sınırlayarak farklı dosya üzerinde işlem yapılması durumunda fatal error ihtimalini eledim.
Dördüncü olarak hedef klasörün varlığını ve yazılabilirliğini kontrol ettirdim. (Doğrudan klasör de oluşturulabilir ama bunu yapmadım. Kullanıcıya bırakmak lazım onu)
Son olarak output değişkenine sadece dosyaların yolunu içeren array'i attım. Arraydaki KesiSayisi anahtarını kaldırdım. Array count ile bu zaten elde edilebilir bir veri.
Niyetim "ilk defa" yaptığınız open source katkısına ufak da olsa bir katkıda bulunmak. Naçizane düzenlemeleri Pull Request ile de gönderdim. Onaylarsanız reponuza otomatik dahil edilecek.
birazda paylaşımda bulunmak istiyordum uzun süredir ama işten güçten hiç vakit bulamıyordum, bir müşterime yaptığım yapı için oluşturduğum en boy koruyarak resim bölme fonksiyonudur.
