akshaykhale1992 / TinyPHP

public/index.php

<?php

/**
 * Loading composer Libraries
 */
require __DIR__ .'/../vendor/autoload.php';


/**
 * Loading the Environment
 * setting option
 */
$dotenv = new Dotenv\Dotenv(__DIR__.'/../');
$dotenv->load();

/**
 * Loading main Controller Class and
 * helpers file to load helper function
 * and to load Controller
 * Once can skip helpers file if
 * don't want the helper functions
 * in the framework.
 */
require __DIR__.'/../helpers.php';

require __DIR__.'/../'.getenv('CONTROLLERS_DIRECTORY','controllers').'/Controller.php';

'controllers' of type string is incompatible with the type boolean expected by parameter $local_only of getenv().

$paths = array(__DIR__ .'/../'.getenv('VIEWS_DIRECTORY','views'));
$cache_path = array('cache_path' => __DIR__ .'/../'.getenv('CACHE_DIRECTORY','cache'));

$path = explode('/', trim(strtolower($_SERVER['REQUEST_URI']), '/'));
$path[count($path)-1] = explode('?', end($path))[0]; // Removing get parameters from the Request URL

$default_pages = ['index','home','default']; //Default pages of the website

if (empty($path[0]) || in_array($path[0], $default_pages)) {
	$method = 'index'; // Loading default index method from the controller for homepage
} else {
	$method = implode('_', $path); // Calling method from the URL
}

$controller = new Controller($paths, $cache_path);
if (method_exists($controller, $method)) {
	echo $controller->{ $method }(); // Calling the method from the Controller
} else {
	echo $controller->notFound(); // Calling 404 Page if method not found

Are you sure $controller->notFound() of type View can be used in echo? Consider adding a __toString()-method.

}

exit();