OOP-MVC-CRUD-PDO

I'm curious as to why the vendor folder has been put on github? Normally the user would just run composer install or composer dump-autoload

And I would personally move index.php into the public directory. I get that it's easier with it in the base folder, but it allows anyone to access any file in all folders

Apart from these things it looks pretty good. I assume it's just something to base a project on and not a framework?

@Sinnbeck Done. Thank you for your valuable feedback. It's not for a project or framework. Just trying to learn OOP PHP and MVC patterns.

@feralam well it's a really great start. Can I suggest looking at the auto load scheme in laravel for inspiration

https://github.com/laravel/laravel/blob/9.x/composer.json#L25

@Sinnbeck if I add the below code in composer then show ERROR, can you pls tell me the problem?

"autoload": {
        "files": [
            "app/helpers.php"
        ],
        "psr-4": {
            "App\": "app/",
            "Database\": "database/"
        }
    }

ERROR: Fatal error: Uncaught Error: Class 'App\Database\Connection' not found

Worked with the below code:

"autoload": {
        "files": [
            "app/helpers.php"
        ],
        "classmap": [
            "./"
        ]
    }

@feralam That’s because the class App\Database\Connection isn’t located where Composer is expecting it.

Note that in PSR-4, you tell it where to look for classes based on their top-level namespace. When you do "Database\\": "database/", you’re saying that the class Database\Connection should be at /database/Connection.php – but in the actual class itself, you use the namespace App\Database, which falls under App, so Composer expects that class to be located at /app/Database/Connection.php.

If you fix the namespace in the Connection class, it should work.

@kokoshneta I am using double, laracast removed one

@feralam Yeah, I noticed that after posting; I looked at the source and updated the comment (you have the wrong namespace in your class).

Working. Thank you

@feralam You can use the “Set Best Answer” option on the answer that best answered the question to mark the problem as solved and get it off the unsolved list.