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

[Çözüldü] Restful controller varsayılan method

m.erin

Merhaba arkadaşlar,
Restful controller kullanarak yönlendirme yaptığımda varsayılan method yani getIndex metodu çalıştırılmıyor, mutlaka adres satırına ismini eklemem gerekiyor.
Yani adres satırına app.dev/test yazdığımda yönlendirme gerçekleşmiyor ama app.dev/test/Index yazarak denediğimde gerçekleşiyor.
Sanıyorum bu durum son güncellemelerden sonra oluştu. Benzer sorun yaşayan ve ya çözümü bilen var mıdır?

# /app/routes.php
...
Route::controller('test', 'TestController');
...

# /app/controllers/TestController.php
class TestController extends BaseController {
	public function getIndex()
	{
 return View::make('hello');
	}
}

pellempus

index olmadan çalışmaması bir bug değil aslında. Bir kaç ay önce bir bug'u kapatmak için bu değişikliği yaptılar, yani index o adreste olmak zorunda.
Ancak HTML::link('test') şeklinde link verdiğinde bir problem olmaması lazım, yani index'ten kurtulabiliyorsun.

Resource ile denedin mi? Restful için daha kullanışlı ve index'ten kurtulmanın kesin yoludur.

Route::resource('test', 'TestController');

ya da sadece istediğin metodlar için

Route::resource('test', 'TestController', array('only' => array('index', 'show', 'update', 'destroy')));

Restful contoller (Route::controller) ile Resource (Route::resource) arasında farkı bildiğim kadarıyla açıklamaya çalışayım.
Restful kullandığınızda laravel metodu tahmin etmeye çalışır ya da manuel olarak route ekleyebilrsiniz. Ben bunun tam olarak Restful olduğunu düşünmüyorum.
Resource metodunda ise tüm CRUD fonksiyonları öntanımlıdır, 7 adet metod için (index, create, store, show, update, edit, destroy) route oluşturur, tahmin yapmaz. Gerçek anlamda restfuldur, Laravel'e Rails'ten gelen bir özelliktir.
Her iki route kullanımında linklerin nasıl oluşturulduğunu ve farkı görmek için komut penceresinde

php artisan routes

yazarak oluşturulan link listesini alabilirsiniz.

Ben resource kullanımı seviyorum, kötü yanı bütün metodlar için route yaratıyor olması.. Örneğin ben 7 metodu da kullanmak istemiyorum ya da destroy kullanmak istemiyorum, ne yapacağım? İşte bunu da güzel bir yöntemle çözmüşler array('only' => array('index', 'show' .... ile sadece index ve show için oto route yaratır.

m.erin

@pellempus, cevap için teşekkürler. Restful kullanmam gerekiyor, resouce işimi görmüyor. Güncelleme öncesinde projede yazdığım backend kısmı tamamen bu yapı üzerine kuruldu

Ayrıca restful içerisindeki tüm methodlara test/method şeklinde ulaşabilirken index methoduna test/index degilde test/Index şeklinde ilk harfi büyük yazmam gerekiyor.

$ php artisan routes
+--------+------------------------------------------------------------+------------------+-------------------------------+----------------+---------------+
| Domain | URI | Name | Action | Before Filters | After Filters |
+--------+------------------------------------------------------------+------------------+-------------------------------+----------------+---------------+
| | GET|HEAD / | | Closure | | |
| | GET|HEAD test/Index/{one?}/{two?}/{three?}/{four?}/{five?} | | \TestController@getIndex | | |
| | GET|HEAD test/sub/{one?}/{two?}/{three?}/{four?}/{five?} | | \TestController@getSub | | |
| | GET|HEAD|POST|PUT|PATCH|DELETE test/{_missing} | | \TestController@missingMethod | | |
| | GET|HEAD products | products.index | ProductsController@index | | |
| | GET|HEAD products/create | products.create | ProductsController@create | | |
| | POST products | products.store | ProductsController@store | | |
| | GET|HEAD products/{products} | products.show | ProductsController@show | | |
| | GET|HEAD products/{products}/edit | products.edit | ProductsController@edit | | |
| | PUT products/{products} | products.update | ProductsController@update | | |
| | PATCH products/{products} | | ProductsController@update | | |
| | DELETE products/{products} | products.destroy | ProductsController@destroy | | |
+--------+------------------------------------------------------------+------------------+-------------------------------+----------------+---------------+

pellempus

"php artisan routes" komutundan sonra listede
GET|HEAD test/Index/{one?}/{two?}/{three?}/{four?}/{five?} olan kısmın normalde

GET|HEAD test/{one?}/{two?}/{three?}/{four?}/{five?}

şeklinde çıkması lazım. O yüzden büyük harfle Index yazmak zorunda kalıyorsun ama route çıktısının böyle çıkması ilginç. Buyuk I ile çıkması daha da ilginç. Laravel güncelleştirmeden kaynaklı bir bug olabilir mi? Uzun zamandır laravel kurmuyorum, son sürümü kurup deneyerek bir route çıktısı alayım bakalım bende de aynı şeyi yapacak mı, sonucu burada yazarım

m.erin

Evet maalasef yazdığınız şekilde olması gerek. Siz de deneyip yazabilirseniz sevinirim. Projeleri nginx/php-fpm ikilisine taşıdım taşıyalı benzer sorunlar yaşıyorum. Apache üzerinde de denemek gerek.

pellempus

Apache üzerinde Laravel 4.1.18 kurdum.
Route çıktısı olması gerektiği gibi çıktı Yukarıda yanlışlık yapmışım hemen düzelteyim

GET|HEAD test/index/{one?}/{two?}/{three?}/{four?}/{five?}
]GET|HEAD test

şeklinde iki çıktı vermesi lazım. Ben de öyle çıkıyor, her şey normal.

http://localhost/denemem/public/test
yazarak index yazmama gerek kalmadan hello sayfasonı açtım.
Sorun laravel'den kaynaklanmıyor.
Yanlış hatırlamıyorsam senin linux sürümüne php-fpm kurarken de sorunların olmuştu, hala ayarlarında bir sorun olabilir.

m.erin

@pellempus, ilgin için teşekkürler.
Aynen kurulumda sorunlar olmuştu onları aştık şükür. Şu an ilginçtir

GET|HEAD test/Index/{one?}/{two?}/{three?}/{four?}/{five?} | | TestController@getIndex

Şeklinde görünüyor ve hiçbir güncelleme ve değişiklik yapmadığım halde app.dev/test şeklinde erişebiliyorum. Normalde routes, senin yazdığın şekilde görünmeli:

GET|HEAD test/index/{one?}/{two?}/{three?}/{four?}/{five?} | | TestController@getIndex

Sanıyorum case sensitive ile alakalı bir durum.
Nasıl çözüldüğünü bilmesem de konuyu çözüldü olarak işaretliyorum

pellempus

Ben de şimdi case sensitive konusunda araştırıyordum, windowsta denediğim içim küçük büyük harf farketmiyor ama linux olunca iş değişiyor. Sorunun çözülmesine secindim

sergin

Bunun anlaşılması için Route::controller metodunun ne iş yaptığına göz atarak başlayabiliriz.
Normalde routes.php dosyasındaki metodların birçoğu route koleksiyonuna rotalar eklenmesi işini yapar. Eklenen bir rota tanımında rotanın istek metodu, rota adı, istek uri'si ve çalıştırılacak metod yer alır. Sonra, Laravel istek yaşam döngüsünde en son işlem(lerden birisi) olarak isteğe karşılık gelen rota çağrılır. artisan routes diye çağırdığımızda dökümü yapılan bu rota koleksiyonunun elemanlarıdır.

controller metodu şöyle çalışır. İkinci parametrede yer alan Controller sınıfı gözden geçirilir. İçerisinde get, post, put, delete vb kelimelerle başlayan metod isimleri bulunur. Sonra bu snake_case'e çevrilir. Dolayısıyla "postGetir" diye bir metod varsa bu "post_getir" olur. Sonra bu "_" ile bölünür. İlk kısmı HTTP metod olarak, gerikalanı ise "_" lar "-"ye çevrilerek ön uri kısmını oluşturur. Uri kısmının tamamı ise controller metodu tanımında verilen ilk parametreye "/" ile birlikte eklenir. Son olarak Controller sınıfındaki metodların parametreleri olabileceği dikkate alınarak /{one?}/{two?}/{three?}/{four?}/{five?} eklenir (bunların hepsinin isteğe bağlı olduğuna dikkat edelim) ve böylece oluşturulan rota rota koleksiyonuna eklenir.

Dolayısıyla Route::controller('aka', 'TakaController') demişsek ve TakaController sınıfında postGetir adında bir metod varsa rota koleksiyonuna

GET|HEAD aka/getir/{one?}/{two?}/{three?}/{four?}/{five?} | TakaController@getGetir

eklenir (4.1 sürümünde get çağrılarında aynı zamanda head'de oluşturulur.)
Buraya kadar zaten bildik şeyler. Ancak getIndex özel bir metod olarak ele alınır ve Controller'in sanki varsayılan metodu gibi kullanılır. Bunu sağlamak için yukarıdaki bahsedilen dünüştürmelerden sonra eğer (henüz --one/two... kısmı eklenmeden önce) "aka/index" (örneğimizde ilk parametre "aka" olduğu için) mi diye kontrol yapılır, eğer böyleyse diğer metodları için yapılanın aynısı yapılmakla kalmaz aynı zamanda sadece "aka" için de bir get rotası TakaController@getGetir'e bağlanır. Bu yüzden m.erin'in verdiği örnekte getIndex metodu için iki rota eklenmiş olur (olağan durumda).

Birincisi GET|HEAD test/index/{one?}/{two?}/{three?}/{four?}/{five?} | TestController@getIndex
İkincisi GET|HEAD test | TestController@getIndex

Şimdi @m.erin'in sorununa gelince, bu ikinci rotanın eklenmediğini, ama ilkinin de "index" yerine "Index" olarak eklendiğini görüyoruz.
snake_case dönüştürmesi şöyle yapıldığına göre,


 return strtolower(preg_replace('/(.)([A-Z])/', '$1'.'_'.'$2', 'getIndex'));
// normalde 'get_index' sonucu vermesi lazım.

Demek ki @m.erin'in bilgisayarında strtolower fonksiyonu "I" harfini "i" harfine dönüştüremiyor ve sonuç "get_Index" oluyor. Dolayısıyla "test\Index" olarak uri oluşuyor. "test\Index" == "test\index" sınaması doğru olmadığından bunu diğer metodlar için olan gibi "GET|HEAD test/Index/{one?}/{two?..." rotası olarak ekliyor ama index metodu için gerekli "GET|HEAD test" metodunu ekleyemiyor. Strtolower fonksiyonunun İ, Ş, Ü vs küçük harfe çevirmediğini biliyorduk ama I harfini çevirmeyen bir dil ayarı ile karşılaşmış oluyoruz.

m.erin

@sergin ayrıntılı açıklama için teşekkürler. Evet sorunun case sensitive olduğunu bulduk ama şu an ilginç bir şekilde routes test/Index görünmesine rağmen çalışıyor, üstelik hiçbir değişiklik yapmadım. Sistem yerelimde de herhangi bir problem yok (tr_TR.UTF-8). strlower kullanılarak İ,Ş,Ü,Ö çevrilemez tabi ama mb_strlower ile bu sorun aşılabilir. Zaten Laravel'de de sanıyorum salt strlower kullanılmamıştır. Bunun bir string dönüştürme hatası değil de sunucu problemi olduğunu düşünüyorum. Nginx'le daha pek anlaşamadık sanırım

sergin

Larevel kodlarında uriyi şöyle buluyor:

//ControllerInspector sınıfı
return $prefix.'/'.implode('-', array_slice(explode('_', snake_case($name)), 1));

Buradaki snake_case helper metodu Str::snake'yi çağırıyor o da şöyle:

public static function snake($value, $delimiter = '_')
	{
 $replace = '$1'.$delimiter.'$2';

 return ctype_lower($value) ? $value : strtolower(preg_replace('/(.)([A-Z])/', $replace, $value));
	}
// Route::controller('test', 'testController'); yazıldığında getIndex metodu için
// çıktı test/index olacak.

mb_ fonksiyonları kullanılmıyor ve uri olarak türkçe karakterler kullanılamıyor.

m.erin

@sergin, bende bu fonksiyona getIndex parametresi verildiğinde söylediğin şekilde get_index dönüyor. İki ayrı sistemde denedim: Debian - Nginx/php-fpm ve Arch Linux - Nginx/php-fpm. Apache üzerinde denemedim ve windows üzerinde deneme şansım yok. Neyse ki şu an çalışıyor, dokunmaya gerek yok

m.erin

Nginx ve php-fpm servislerini yeniden başlattım ve sorun tekrar hortladı. Sorun %99.9 sunucu yapılandırmasından gibi görünüyor. Varsayılan methodu getIndex yerine get_index şeklinde kullanarak mutlu-mesut çalışmaya devam ediyorum

Her ihtimale karşı bu şekilde kullanmak mantıklı sanırım.

Vedat

pellempus yazdıBen de şimdi case sensitive konusunda araştırıyordum, windowsta denediğim içim küçük büyük harf farketmiyor ama linux olunca iş değişiyor. Sorunun çözülmesine secindim :)

$post = Post::whereRaw('BINARY slug = ?', array($slug))->where('type', '=','xxx')->where('status', '=','1')->firstOrFail();

şeklinde sorgulayarak case sensitive olarak sonuç almıştım.