rdelaage
/
castopod
mirror of https://code.castopod.org/adaures/castopod.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix(ux): redirect user to install page on database error in home page 

When first visiting the home page, users would see an error message that was normal because Castopod
Host wasn't installed yet. From now on, the error is caught to redirect users directly to install
page, preventing them seeing the error and thus, resulting in a better user experience.
This commit is contained in:
Yassine Doghri 2021-05-26 11:37:41 +00:00
parent 0a66de3e6c
commit 9017e30bf4
No known key found for this signature in database
GPG Key ID: 3E7F89498B960C9F
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
app/Controllers/HomeController.php
View File

@ -10,14 +10,19 @@ namespace App\Controllers;
use App\Models\PodcastModel;
use CodeIgniter\HTTP\RedirectResponse;
use mysqli_sql_exception;
class HomeController extends BaseController
{
public function index(): RedirectResponse | string
{
$model = new PodcastModel();
$allPodcasts = $model->findAll();
try {
$allPodcasts = (new PodcastModel())->findAll();
} catch (mysqli_sql_exception) {
// An error was caught when retrieving the podcasts from the database.
// Redirecting to install page because it is likely that Castopod Host has not been installed yet.
return redirect()->route('install');
}
// check if there's only one podcast to redirect user to it
if (count($allPodcasts) === 1) {