MichalOravec

You have to create a new migration

https://laravel.com/docs/8.x/migrations#updating-column-attributes

To create model and migration at once:

php artisan make:model Post -m

Docs: https://laravel.com/docs/8.x/eloquent#generating-model-classes

Use upsert

https://laravel.com/docs/8.x/eloquent#upserts

Use upsert

https://laravel.com/docs/8.x/eloquent#upserts

@foreach ($books as $book)
       <div class="form-check">
          <input type="checkbox" value="{{ $book->id }}" {{ $author->books->pluck('id')->contains($book->id) ? 'checked' : '' }}>

          <label>{{ $book->name }}</label>
      </div>
@endforeach

or with in_array it would be

{{ in_array($book->id, $author->books->pluck('id')) ? 'checked' : '' }}

@foreach ($books as $book)
       <div class="form-check">
          <input type="checkbox" value="{{ $book->id }}" {{ $author->books->pluck('id')->contains($book->id) ? 'checked' : '' }}>

          <label>{{ $book->name }}</label>
      </div>
@endforeach

or with in_array it would be

{{ in_array($book->id, $author->books->pluck('id')) ? 'checked' : '' }}

@chaudigv People (or beginners) usually don't think about a code, what is the biggest mistake about programming. They just copy the code and if it works, they use it all the time.

So when you gave them a code which works they don't take any effort how to do it better, or try to understand what's going on.

Which means that they stay on the same level for very long time.

So you are thinking it's better for them, but it's not.

Obviously not.

@chaudigv The word obviously is on the right place in your case and don't take it as an insult.

Your code is basically like this logic

$condition = true;

if ($condition == true) {
    $result = true;
} else {
    $result = false;
}

It's same as

$result = $condition;

In the end if somebody has a problem to understand to that logic, then programming is not for him.

Yeah I know, has() method return true or false.

But obviously people don't want to write a better code, like @chaudigv

$computer->is_deletion_required = $request->has('is_deletion_required');

@rokka I really don't understand what is different what I posted above...

You need to use whereHas because you want to filter categories by your relationships

https://laravel.com/docs/8.x/eloquent-relationships#querying-relationship-existence

Also you need to specify additional query conditions for the eager loading query, otherwise you get all data from relationships.

https://laravel.com/docs/8.x/eloquent-relationships#constraining-eager-loads

Since you have there where with or orWhereHas in this case, you need to use logical grouping because you have there an another where condition->where('user_id', $store->id)

https://laravel.com/docs/8.x/queries#logical-grouping

$categories = Category::with(['childrenCategories' => $childrenClosure = function ($query) use ($id) {
        $query->where('user_id', $id);
    }, 'products' => $productsClosure = function ($query) use ($id) {
        $query->where('auth_id', $id);
    }])
    ->where('user_id', $store->id)
    ->where(function ($query) use ($childrenClosure, $productsClosure) {
        $query->whereHas('childrenCategories', $childrenClosure)
            ->orWhereHas('products', $productsClosure);
    })
    ->latest()
    ->get();

$categorySlugs = [
    'current-affairs-2', 'current-events-2'
];

return Post::whereHas('categories', function ($query) use ($categorySlugs) {
        $query->whereIn('slug', $categorySlugs);
    })
     ->where('post_status', 'publish')
     ->whereYear('post_date','2019')
     ->whereMonth('post_date', '10')
     ->newest()
     ->paginate(10);

$categories = Category::with(['childrenCategories' => $childrenClosure = function ($query) use ($id) {
        $query->where('user_id', $id);
    }, 'products' => $productsClosure = function ($query) use ($id) {
        $query->where('auth_id', $id);
    }])
    ->where('user_id', $store->id)
    ->where(function ($query) use ($childrenClosure, $productsClosure) {
        $query->whereHas('childrenCategories', $childrenClosure)
            ->orWhereHas('products', $productsClosure);
    })
    ->latest()
    ->get();

 return $cat->posts()
    ->where('status', 'publish')
    ->whereDate('post_date','2021-03-03')
    ->paginate(10);

You don't need to have eager loading for posts on category.

So why do you have it here?

>with(['posts'=>function($q) {
                $q->where('status', 'publish')
					->whereDate('post_date','2021-03-03');
            }])-

Are you sure?

For checkbox you need to use has()

public function update(UpdateComputerRequest $request, Computer $computer) 
{
    $copmuter->update($request->except('is_deletion_required') + [
        'is_deletion_required' => $request->has('is_deletion_required')
    ]);
}

Docs: https://laravel.com/docs/8.x/requests#determining-if-input-is-present

 return $cat->posts()
    ->where('status', 'publish')
    ->whereDate('post_date','2021-03-03')
    ->paginate(10);

You don't need to have eager loading for posts on category.

With null coalescing operator it could be just

public static function findOrFail($slug)
{
    return static::find($slug) ?? abort(404);
}

Docs: https://www.php.net/manual/en/migration70.new-features.php#migration70.new-features.null-coalesce-op

Have you ever read a documentation?

https://laravel.com/docs/8.x/requests#files

https://laravel.com/docs/8.x/filesystem

You file input doesn't have a name attribute.

<input type="file" id="file_picture_input" name="picture_input">

and remove this hidden input.

<input type="hidden" id="picture_input" name="picture_input">

How do you import css? With asset()? Use public_path() instead.

You can still use Laravel/UI and it will work in all future new versions of Laravel.

<head> and </head> tags as well.

<form action="{{ route('chart') }}" method="GET">
    //
</form>

->whereMonth('created_at', $request->month)

<form action="{{ route('chart') }}" method="GET">
    //
</form>

->whereMonth('created_at', $request->month)

https://laravel.com/docs/8.x/eloquent#soft-deleting

https://laravel.com/docs/8.x/eloquent#querying-soft-deleted-models

Soft deleted models will automatically be excluded from query results.

<select name="month">
    @foreach (range(1, 12) as $month)
        <option value="{{ $month }}">{{ date("F", mktime(0, 0, 0, $month, 1)) }}</option>
    @endforeach
</select>

->whereMonth('created_at', $request->month)

https://laravel.com/docs/8.x/eloquent#soft-deleting

https://laravel.com/docs/8.x/eloquent#querying-soft-deleted-models

Soft deleted models will automatically be excluded from query results.

Remove that row from database and create it again.

Always write a code in English and database tables and columns as well.

Why do you have in the select velos?

select * from velos where id = '53ca6f30-5a39-11ea-9fd6-d930c5b6158c';

And what is this?

Route::get('allartworks', '[email protected]');

It has to be

Route::resource('allartworks', 'ArtworkController');

or if you want only one route then

Route::get('allartworks', '[email protected]');

$quotations = Quotation::doesntHave('bicycle')->get();

So with that you retrieve a quotation or quotations with bicycle_id which doesn't exist in bicycles table in column id of course.

It's not a problem with relationship, but with your data.

That's very strange because all datas are ok in the database.

Obviously they are not ok.

$quotations = Quotation::doesntHave('bicycle')->get();

// or

$quotations = Quotation::whereNull('bicycle_id')->get();

With that you will see what is wrong and then fix your data in the database.

No, this is enough

@include('semesters.partials.form', [
    'years' => ['Please Choose A Year'] + array_combine($years = range(now()->year, now()->year + 5), $years)
])

No, this is enough

@include('semesters.partials.form', [
    'years' => ['Please Choose A Year'] + array_combine($years = range(now()->year, now()->year + 5), $years)
])

[
    'years' => ['Please Choose A Year'] + array_combine($years = range(now()->year, now()->year + 5), $years)
]

What does years have to do with other arrays?

Why?

Just change your blade file, that's it.

And use $loop variable to detect the first item in the array.

[
    'years' => ['Please Choose A Year'] + range(now()->year, now()->year + 5)
]

<select name="year">
    <option>Please Choose A Year</option>

    @foreach (range(now()->year, now()->year + 5) as $year)
        <option value="{{ $year }}">{{ $year }}</option>
    @endforeach
</select

@datlechin

<img src="{{ asset("storage/{$file->path}") }}" alt="Image">

In $file->path has to be stored only images/filename.png.

And mark this thread as solved. Thanks.

$request->file('image')->storeAs('images', 'filename.png', 'public');

Or make public disk as default, so set FILESYSTEM_DRIVER in .env

FILESYSTEM_DRIVER=public

https://github.com/laravel/laravel/blob/8.x/config/filesystems.php#L16

https://laravel.com/docs/8.x/filesystem#the-public-disk

@atif123 Do you know the "N + 1" query problem?

It seems to me that no.

https://laravel.com/docs/8.x/eloquent-relationships#eager-loading

public function showhealer()
{
    $healers = ReikiHealer::with('sell')->get();

    return view('user.browsehealer.index', compact('healers'));
}

{{ $healer->sell->title ?? 'Something default' }}

Or if you want to get healers which have sell then

$healers = ReikiHealer::with('sell')->has('sell')->get();

It can be just

$multiplied = collect($article->json)->reject(function ($item) {
    return item['layout'] === 'section';
});

or with map

$multiplied = collect($article->json)->map(function ($item) {
    return $item['layout'] === 'section' ? null : $item;
})->filter();

Maybe you need to use values() to reset keys somewhere in your code

Docs: https://laravel.com/docs/8.x/collections#method-values