Egzaminer\Controller\QuestionDeleteController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	47
Duplicated Lines	48.94 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	deleteAction()	9	9	1
A	postDeleteAction()	14	14	2

<?php

namespace Egzaminer\Controller;

use Egzaminer\Model\QuestionDeleteModel;
use Egzaminer\Model\QuestionsModel;

class QuestionDeleteController extends AdminController
{
    /**
     * Delete question.
     *
     * GET /admin/exam/edit/[i:id]/question/del/[i:qid]
     *
     * @param int $examID
     * @param int $questionID
     *
     * @return string
     */
    public function deleteAction(int $examID, int $questionID): string

The parameter $examID is not used and could be removed.

This method seems to be duplicated in your project.

    {
        $question = (new QuestionsModel($this->get('dbh')))->getByQuestionId($questionID);

        return $this->render('admin/delete', [
            'title'   => 'Usuwanie pytania',
            'content' => 'Czy na pewno chcesz usunąć pytanie <i>'.$question['content'].'</i>?',
        ]);
    }

    /**
     * Delete question post action.
     *
     * POST /admin/exam/edit/[i:id]/question/del/[i:qid]
     *
     * @param int $examID
     * @param int $questionID
     *
     * @return void
     */
    public function postDeleteAction(int $examID, int $questionID)

This method seems to be duplicated in your project.

    {
        $delModel = new QuestionDeleteModel($this->get('dbh'));

        if ($delModel->delete($questionID)) {
            $this->setMessage('success', 'Usunięto pomyślnie!');
            $this->redirect('/admin/exam/edit/'.$examID);

            return;
        }

        $this->setMessage('warning', 'Coś się zepsuło!');
        $this->redirect('/admin/exam/edit/'.$examID.'/question/del/'.$questionID);
    }
}