kima

@SWIMMER - It depends on your implementation. how do you call it?

Are there situations when you have no Menu model for this $url? If no, do you check that the getAll() result is an array before you iterate it? if you iterate it at all

I'm not familiar with the mysqlnd extension, but I check the php.net page and noticed this:

The MySQL database extensions must be configured to use the MySQL Client Library. In order to use the MySQL Native Driver, PHP needs to be built specifying that the MySQL database extensions are compiled with MySQL Native Driver support. This is done through configuration options prior to building the PHP source code.

As I can see, you have no other DB extention, mysqli for example

I noticed some issues in StackOF, all of them pointing to the mysqlnd extension. Please check your php-mysqlnd extension is updated to your php version, and loaded properly by Apache. Restart Apache if needed.

Are you using a debugger?

Please set the DB_CONNECTION value in your env file to lower case, as there is no 'MySql' key in your connections array

You need to use ->first(), to actually fetch the record from the database, but with no ->toArray() function. Please try this:

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class content extends Model
{
    static public function getAll($url){
        if($menu = Menu::where('url', '=', $url)->first()) {
            $content = Content::where('menu_id', '=',$menu->id)->get()->toArray();

            return $content;
        }
        
        return [];
    }
}

I don't understand the logic here, but it seems like you try to do a Menu existing validation, but the validation breaks when there is no Menu and the first() function returns null. so please try to remove the first toArray()

@CHRISTOPHHARMS - @keizah Both databases should work with no backticks at all:

$neighbours = Category::where('left', '>', $category->right);

$neighbours->update([
    'left' => DB::raw('left+2'),
    'right' => DB::raw('right+2'),
]);

@ChristophHarms This is not a Collection, the $neighbours variable is still a QueryBuilder as it didn't use the get() method.

why not using

ModelClass::query()->select

?

the when clause is for PHP boolean and not being interpreted in SQL. Please try to set the DB::raw inside the the Join clause

->join("person_fields", function(\Illuminate\Database\Query\JoinClause $join){
                        $join->on('person_fields.name','Name')
                             ->where('person_fields.person_type','Family Member')
                             ->whereRaw('person_vatas.member_id IS NOT NULL');
                        });

please fix the 'id_skill' to 'is_skill' in the validator

Did you make sure the routes files are not cached? Please run 'php artisan route:clear' and try again

You have 2 options.

You can set a variable that will reference to the QueryBuilder, do your logic and then apply more clauses on the variable and get the result

$builder = Ad::query()
            ->whereHas('book', function (\Illuminate\Database\Eloquent\Builder $query) use ($book_id) {
                $query->where('book_id', $book_id);
            });
        
        if (true) {
            $builder->whereHas('authors', function (\Illuminate\Database\Eloquent\Builder $query) use ($checkbox) {
                $query->whereIn('ad_service.author_id', $checkbox);
            });
        }
        
        $result = $builder->get();

Another option is to use the when function

            ->whereHas('book', function (\Illuminate\Database\Eloquent\Builder $query) use ($book_id) {
                $query->where('book_id', $book_id);
            })->when("some condition", function (\Illuminate\Database\Eloquent\Builder $query) use ($checkbox) {
                $query->whereHas('authors', function (\Illuminate\Database\Eloquent\Builder $query) use ($checkbox) {
                    $query->whereIn('ad_service.author_id', $checkbox);
                });
            })->get();```

how does $factory getting instantiated?

It uses Laravel dependency injection. It is important to read this article to understand: https://laravel.com/docs/5.7/container

Faker\Generator and Eloquent\Factory have to be constructed with required parameters, and Laravel doing this for you in a singleton pattern. When you just use 'new Faker()' you are not set these parameters!

Laravel registers (instantiated) the Faker\Generator and Eloquent\Factory in DatabaseServiceProvider::registerEloquentFactory() function. Whenever the application needs a Faker\Generator, it would inject the instance that was set.

How does the closure getting called?

As you see, the $factory->define() function saving the $attributes (actually an anonymous function) in the singleton under 'definitions' member (and already has 'faker' member)

When you using the factory() helper, it uses the Illuminate/Database/Eloquent\Factory::of() function. in the end, it is calling Illuminate/Database/Eloquent/FactoryBuilder::getRawAttributes(). this function calls your anonymous function with the 'faker' member, which is the Faker\Generator singleton

It is not an Automatic Injection

The $factory is an Eloquent\Factory class, that has a 'faker' member that was set in the constructor. When your app is trying to resolve the defined class, it injects the 'faker' member to the anonymous function.

The default faker is set in DatabaseServiceProvider

If you just want a faker instance, you can use app(\Faker\Generator::class);

See this post https://laracasts.com/discuss/channels/eloquent/eloquent-wherenotin

Receive the Model table name by $table member with fallback by the class name:

with(new static)->getTable() When static is a Model

@MOENCHFRACHT - I don't see it part of a service provider and not using guards to detect the user, as I assume that the system admin should reach this storage although he is not logged in to the specific user, or using multiple user's storage paths in one session

in this case, I would just add it in the User class or creating a factory for this.

public function getStorageInstance()
    {
        $dir = storage_path('app/public/'.$this->name.'/');
        // Make sure the storage path exists and writeable
        if (!is_writable($dir)) {
            mkdir($dir, 0777, true);
        }

        return Storage::createLocalDriver(["root" => $dir]);
    }

I prefer to not use variables in the config files as I cache them in Production environments.

Try using Storage::createLocalDriver(["root" => storage_path('app/public/'.\App\Auth::user()->name.'/')]);

You might create a function that returns you a storage instance for a user. the function can handle the directory creation and directory permissions also.