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

E-Mail Doğrulama İşlemi

takix · 2022-06-21T16:06:37+00:00

Merhaba,Sitenizi aktif olarak kullanmaktayım ve hatalarım karşısında siz bilen insanlara sormaktan çekinmiyorum.Sizler iyiki varsınız.Bir başka sorun ile kar...

takix

mgsmus şuanda hemen hemen tüm kontroller bu konuda var

mgsmus

takix EmailVerificationPromptController içerisinde redirect()->intended(RouteServiceProvider::HOME) yapmışsınız. Kullanıcının gitmek istediği bir rota varsa oraya yönlendir, yoksa RouteServiceProvider::HOME değerine yönlendir demişsiniz. Demek ki save_like işleminden sonra bir yönlendirme yapmadığınız için intended boş oluyor, o yüzden HOME değerine gidiyor.

takix

mgsmus peki değerli hocam save like ve favorite işlemi gelirse eğer kullanıcı e-mail onayla değilse site.com/verifiy-email kısmına nasıl yönlendirebilirim.Eğer onayla ise beğensin veya gönderiyi kaydedilenlere kayıt edebilsin.Çok minnetarım bu arada çok yardımlarınız dokundu.

mgsmus

takix ['auth', 'verified'] şeklinde iki middleware içine almadınız mı zaten?

takix

mgsmus

<?php

use App\Http\Controllers\HomeController;
use Illuminate\Support\Facades\Route;

Route::get('/', [HomeController::class, 'index'])
                ->name('index');
                
Route::get('viral', [HomeController::class, 'viral'])
                ->name('viral');

Route::get('random', [HomeController::class, 'random'])
                ->name('random');

Route::get('search', [HomeController::class, 'search'])
                ->name('search');

Route::get('view/{id}/{slug}', [HomeController::class, 'show'])
                ->name('show');

Route::group(['middleware' => ['auth','verified']], function () {
    
    Route::post('write', [HomeController::class, 'store'])
                ->middleware('verified')
                ->name('store');
    
    Route::post('save_like', [HomeController::class, 'save_like'])
                ->middleware('verified')
                ->name('save_like');
    
    Route::post('save_favorite', [HomeController::class, 'save_favorite'])
                ->middleware('verified')
                ->name('save_favorite');
    
    Route::get('post/delete/{id}', [HomeController::class, 'delete_user_post'])
                ->middleware('verified')
                ->name('delete_user_post');
    
    // report
    Route::get('report/{id}', [HomeController::class, 'report'])
                ->middleware('verified')
                ->name('report');
    
});

require __DIR__.'/gender.php';
require __DIR__.'/points.php';
require __DIR__.'/comments.php';
require __DIR__.'/categories.php';
require __DIR__.'/pages.php';
require __DIR__.'/tags.php';
require __DIR__.'/admin.php';
require __DIR__.'/auth.php';

takix

mgsmus

takix Zaten verified middleware içine almışsınız, içerideki rotalara tek tek middleware('verified') yapmanıza gerek yok.

takix

mgsmus save_like ve _save_favorite kısımları çalışınca yani anasayfada bir kart üzerinden gönderiyi beğenmeye çalıştığımda otomatik anasayfaya atıyor.Story ve report doğru çalışıp site.com/verify-email kısmına atıyor.Ama diğer ikisi çalışmıyor nedense

takix

mgsmus hepsini kontrol ettim save_like ve save_favorite dışında hepsi çalışıyor

mgsmus

takix Size üstte yazdım. save_like işlemi sonrası eğer return redirect() vs yapmadan işlemi bırakıyorsanız ana sayfaya atar, intended boş kalıyor. Eğer bu işlemi ajax ile yapıyorsanız yöntem içerisinde Redirect::setIntendedUrl('save_like işlemi sonrası gidilecek url') yapmanız lazım. Olaya çözüm odaklı bakmıyorsunuz.

takix

mgsmus
Hocam sadece basit bir soru sorucam cevaplarsanız sevinirim.

Redirect::setIntendedUrl('save_like işlemi sonrası gidilecek url')

bunu bu kodlara mı entegre etmem gerekiyor?

function save_like(Request $request){
        
        $item = Items::findOrFail($request->item);
        
        if(Auth::user()->hasLiked($item)) {
            
            Auth::user()->unlike($item);
            
            return response()->json([
                'bool'=>false
            ]);

        } else {
            
            $like = Auth::user()->like($item);
            
            $create = Notifications::create([
                'sender_id' => Auth::id(),
                'recipient_id' => $item->user_id,
                'item_id' => $item->id,
                'notification_type' => "like",
                'like_id' => Auth::user()->likes()->with('likeable')->first()->id,
                'seen' => 2,
            ]);
            
            if($create == true){
                if(Settings::find('status_points')->value == 1){
                    if(Settings::find('status_points_like')->value == 1){
                        Points::create([
                            'user_id' => Auth::id(),
                            'point_type' => "like",
                            'score' => Settings::find('points_like')->value,
                            'item_id' => $item->id,
                            'like_id' => Auth::user()->likes()->with('likeable')->first()->id,
                        ]);
                    }
                }
            }
            
            return response()->json([
                'bool' => true
            ]);
            
        }
    }

deathisonitsway

takix

nacizane tavsiyelerimi soyliyim size..
Yazdıgınız homeController store methodu 10 numara fuzuli şeylerle dolu.

1.cisi:

if(Settings::find('active_upload')->value == 0){ toastr()->warning(__('main.new_entries_paused')); return redirect('/'); }
Bu nedir? burada ne işi var...
Controllera gelmeden bu işi yapabilirsiniz.Laravelde alamayacağınız veri yok...
Yada illa yapcanız her controller Controller.php gibi bomboş bir sınıfla genişletiliyor..Yazın oraya..
her bir haltı Controller a yazmak zorunda değilsiniz...

En net kural : CONTROLLER sınıfları olabildiğince tertemiz olmak zorundalar..

$rules = [
            'title' => 'required|string|max:255',
            'story' => 'required|string|min:'.Settings::find('minimum_characters')->value.'|max:'.Settings::find('maximum_characters')->value,
            'tags' => 'nullable',
            'category_id' => 'required',
            'genders_id' => 'required',
            'age' => 'required',
            'photo' => 'nullable|image|mimes:jpeg,png,jpg|max:2048'
        ];

Sıfır puan...Hiç yazılmayacak yer.
Validationlar controller içinde yazılaaaa--maaaa--zzz...Nokta..yazılır diyenin alnını karıslarım..
Request sınıfı var oraya yazın..Methoda inject edin...

php artisan make:request FooRequest yaparsanız requestler sizin yerinize olusur laravel tarafından...
2.kural..Database kaydı yapıyorsunuz alt tarafı..required kuralını database veriyor size..ben olsam yazarım bir console command bunlarla da ugrasmam...Minimum efor maximum verim...

Bu sekil insaat ameleliğimizden farkımız yok..Aynı şekil çalışan sistemi kurallaştırmayı ve makineye yaptırmayı bilmiyoruz.Yada uğraşmıyoruz...Tavsiyem olabildiğince ama olabildiğince generatorlere yığın sisteminizi..Yazın command heryerde rahat edersiniz.

Validator::make($request->all(), $rules,[
			'title.required' => 'Bir Başlık Giriniz.',
            'title.max' => 'Başlık 255 Karakterden Uzun Olamaz.',

            'story.required' => 'Boş bırakılmamalıdır.',
            'story.min' => 'Girilen Metin '.Settings::find('minimum_characters')->value.' Karakterden Fazla Olmalıdır.',
            'story.max' => 'Girilen Metin ' .Settings::find('maximum_characters')->value.' Karakterden Az Olmalıdır.',

            'category_id.required' => 'Kategori Seçmeniz Gerekiyor.',

            'photo.image' => 'Lütfen Resim Dosyası Yükleyiniz.',
            'photo.mimes' => 'Dosya Biçimi jpg,png veya jpeg olmalıdır.',
			
			
		
		])->validateWithBag('write');

Farkettin mi bilmiyorum ama iğrenç görünüyor.
Clean code kitabını okuyun..Bakarken keyif almadığınız kod kötü koddur..
Bunun burda işi yok..az once bahsettiğim request sınıfı icinde bu tanımlanabilir...

Ve diğer bir kural...Abiler: türkçe yazılım olmaz..
Madem yapacaksınız..dil dosyalarını kullanın...

Static cümle yazmayın..Sonra uğraşırsınız..
Dil dosyalarına yazın..bırakın sonrasını tercumanlar ugraşsın...

// create item
    	$create_item = Items::create([
            'title' => $request->title,
            'story' => $string,
            'slug' => Str::slug($request->title),
            'status' => Settings::find('new_entries')->value,
            'user_id' => Auth::id(),
            'category_id' => $request->category_id,
            'genders_id' => $request->genders_id,
            'age' => $request->age,
        ]);

Yukarda tanımladıgın validationu adam gibi tanımlarsan.. böyle dizi ile create methoduna değer geçmek zorunda kalmazsınız..

Items::create($request->validated());

diyerek kısaltırdım ben bu kodu..

if(Settings::find('new_entries')->value == 1){
            
                toastr()->success(__('main.toast_your_post_has_been_posted'));
                return redirect('/');
                
            } else {
                
                toastr()->warning(__('main.toast_post_in_moderation'));
                return redirect('/');
                
            }

En nefret ettiğim şey..return yazıyorsanız..Else methoduna neden ihtiyaç duyuyorsunuz anlamak mümkün değil..

if(Settings::find('new_entries')->value == 1){
       toastr()->success(__('main.toast_your_post_has_been_posted'));
       return redirect('/');
                
} 

toastr()->warning(__('main.toast_post_in_moderation'));
return redirect('/');

daha güzel görünüyor..gereksiz else bloklarından kaçının..if şartı birşey donderecekse..elseye gerek yok...

Sağlıcakla...

MehmetCanYUMUTUTAN

deathisonitsway ben olsam yazarım bir console command bunlarla da ugrasmam

Merhaba. Açıklamalarınızı dikkatle okuyorum. Alıntı yaptığım cümlenizde ne anlatmak istediğinize dair örnek verebilir misiniz? Tam olarak anlayamadım nasıl bir yöntemden bahsettiğinizi. Araştırmam gereken anahtar kelimeler bile verseniz benim için yeterli olur. Teşekkürler.

deathisonitsway

MehmetCanYUMUTUTAN

dostum çok basit..

SELECT * FROM information_schema.columns WHERE table_schema = burayaDatabaseName && table_name = burayaTableName

bu sql komutu size database de sütunların ne olduğunu veriyor..

if (is_null($column->COLUMN_DEFAULT) && $column->IS_NULLABLE == 'NO' && $column->COLUMN_NAME !== 'id') {
                    $list['required_columns'][] = '"' . $column->COLUMN_NAME . '"';
                }

soyle bir komut yazarsanız..required columnları bir diziye aktarır..bunu da bir json dosyasına yazarsanız request sınıfların da bu json dosyasını alın kurallarınıza ekleyin...on numara iş görür...aynı şekilde maximum uzunluguda alabilirsiniz..falan feşmekan işte...zırt pırt uğraşmazsınız..migration olusturdugunuzda bir tane servis yazarsınız bunun icin..calıstır abi komutu..yaz bir yere..requestleriniz de otomatik olusur....daha doğrusu json listesine tum kolonları yazardım..

atıyorum soyle bir şey yazardım...

['name' => 'required|integer|max:55'];

bu sekil bir liste yapardım ben olsam..kaydederdim bir json dosyasına..onu okurdum request sınıflarımda..
mis gibi iş işte...

takix

deathisonitsway kardeşim biraz yardım aldım biraz hazır kod kullandım.Amacım hem öğrenmek hem de bir site yapmaktı biraz fazla ağır eleştirmemişin mi sence?

mgsmus

takix Evet ama ben hala neden emaili onaylanmamış bir kullanıcı login olup save_like aşamasına kadar gelebiliyor anlamış değilim. Kullanıcı login olduğunda emaili onaylı değilse direkt email onaylamaya yönlendirseniz, hiç bunlarla uğraşmasanız?

takix

mgsmus atıyorum tamamen örnek veriyorum kızlarsoruyor.com da soruyu üyeliksiz okuyabiliyorsunuz bende öyle bir şey tasarladım

« Önceki Sayfa