Editace formuláře – InvalidArgumentException

Jan Mikeš napsal(a):

Je důležité porozumět příčině problému – nette vyhazuje výjimku, v případě, že nastane pokus o nastavení výchozí hodnoty u choice controlu (selectbox, radio) a hodnota, kterou se pokusíš nastavit jako výchozí není v seznamu povolených hodnot (setItems(...)), pak nette vyhodnotí situaci jako pokus o útok → podstrčení nepovolené hodnoty.

Musíš si být jist, že ve chvíli, kdy se snažíš nastavit hodnotu prvku, tak prvek má už načtené hodnoty v závislosti na hodnotě předchozího inputu.

Ukaž prosím v jaké fázi životního cyklu presenteru nastavuješ ->setItems() prvku, kterému chceš nastavit výchozí hodnotu.

Opět děkuji za popíchnutí, dává mi to smysl. Přidal jsem tedy setItems ke zmíněnému selectboxu Prj_ID viz

protected function createComponentTimesheetForm()
{
        $form = new Form;
        $form->addHidden('Tsk_ID');
        $form->addSelect('Cmp_ID', 'Firma', $this->timesheetManager->getCompanies()->fetchPairs('Cmp_ID', 'CmpName1'))
            ->setPrompt('Vybrat firmu')->setRequired();
        $form->addSelect('Prj_ID', 'Projekt')
            ->setItems($this->timesheetManager->getProjects()->fetchPairs('Prj_ID', 'PrjName'))
            ->setPrompt('Vybrat projekt');
        $form->addText('TskName', 'Obor zájmu')->setRequired();
        $form->addText('TskDescription1', 'Popis úkolu')->setRequired();
        $form->addText('TskDateStart', 'Datum zahájení')->setRequired();
        $form->addText('TskDateEnd', 'Datum ukončení')->setRequired();
        $form->addText('TskTaskmaster', 'Zadavatel')->setRequired();
        $form->addText('TskWorked', 'Odpracováno (hod.)')->setRequired()->addRule(Form::INTEGER, 'Zadejte číselnou hodnotu');
        $form->addSelect('TskUser', 'Provedl', $employees = array(
            "user1" => 'user1',
            "user2" => 'user2',
            "user3" => 'user3',
            "user4" => 'user4',
            "user5" => 'user5',
            "user6" => 'user6',
        ))->setPrompt('Vybrat zaměstnance')->setRequired();
        $form->addSubmit('submit', 'Uložit');
        $form->onSuccess[] = [$this, 'timesheetFormSucceeded'];
        return $form;
}

Problém je nyní ten, že selectbox hned nabízí veškeré možnosti, které v databázi jsou (jak v čistém formuláři tak při jeho editaci), což však nechci. Já je vlastně potřebuji mít načtené, ale nechci, aby byli vidět. Zobrazit by se v editovaném formuláři měli právě podle IDčka projektu, které se natáhne ze záznamu, jež chci editovat že. Čili stále mi uniká korektní cesta, jak problém vyřešit. :D

EDIT: Znovu jsem zabrouzdal na jedno téma, https://pla.nette.org/…cni-formular, nyní po tvém vysvětlení mě napadá, že toto by mohlo vyřešít můj problém, ne? Pro načtení dat si vytvořit vlastní funkci a samotné loadování hodnot provést až v action nikoliv v továrničce na formulář.

Editoval iNyxLadis (29. 11. 2016 15:40)

Jan Mikeš napsal(a):

Vycházel jsi z tohoto návodu?
Zde se data loadují v handle metodách, všimni si, je tam 2× setItems() podle toho jestli hodnota byla nalezena nebo ne.

Ano, vycházím z tohoto. Problém tedy bude někde v kódu, kde se snažím z databáze tahat projekty na základě ID firmy. Přikládám kód:

Presenter

public function handleTimesheetFormChange($value)
{
    if ($value) {
            $resultPrj = $this->timesheetManager->getProjectSelect($value);
            $secondItems = $resultPrj->fetchPairs('Prj_ID', 'PrjName');

            $this['timesheetForm']['Prj_ID']->setPrompt('Vybrat projekt')
                ->setItems($secondItems);

    } else {
            $this['timesheetForm']['Prj_ID']->setPrompt('Nejprve zvolit firmu')
                ->setItems(array());
    }

    $this->redrawControl('wrapper');
    $this->redrawControl('projectSnippet');
}

Model:

const
        TABLE_TASK = 'TblTask',
        TABLE_TIME = 'TblTime',
        TABLE_COMPANY = 'TblCompany',
        TABLE_PROJECT = 'TblProject',
        TABLE_USER = 'TblUser',
        TASK_DATEEND = 'TskDateEnd',
        TASK_COLUMN_ID = 'Tsk_ID',
        PROJECT_COMPANY_ID = 'Cmp_ID'; // tabulka TblProject, sloupec Cmp_ID

public function getProjectSelect($value)
{
        return $this->database->table(self::TABLE_PROJECT)->where(self::PROJECT_COMPANY_ID, $value);
}

Ahoj, ještě jednou prosím o pomoct při řešení invalidArgumentException. Stále nemohu přijít na chybu.

Můj presenter

class TimesheetPresenter extends BaseFrontPresenter
{
        /** Instance třídy modelu pro práci s úkoly. */

        protected $timesheetManager;

        /**
         * Konstruktor s injektovaným modelem pro práci s úkoly.
         */
        public function __construct(TimesheetManager $timesheetManager)
        {
                parent::__construct();
                $this->timesheetManager = $timesheetManager;
        }

        /**
         * Výpis úkolů z tabulky.
         */
        public function renderDefault()
        {
                $this->template->tasks = $this->timesheetManager->getTasks();
        }

        public function handleTimesheetFormChange($value)
        {
            if ($value) {
                    $resultPrj = $this->timesheetManager->getProjectSelect($value);
                    $secondItems = $resultPrj->fetchPairs('Prj_ID', 'PrjName');

                    $this['timesheetForm']['Prj_ID']->setPrompt('Vybrat projekt')
                        ->setItems($secondItems);

            } else {
                    $this['timesheetForm']['Prj_ID']->setPrompt('Nejprve zvolit firmu')
                        ->setItems(array());
            }

            $this->redrawControl('wrapper');
            $this->redrawControl('projectSnippet');
        }

        public function actionEdit($id)
        {
            if ($id) {
            ($task = $this->timesheetManager->getTask($id)) ? $this['timesheetForm']->setDefaults($task) : $this->flashMessage('Úkol nenalezen.');}
        }

        public function actionRemove($id)
        {
                $this->timesheetManager->removeTask($id);
                $this->flashMessage('Úkol byl odstraněn');
                $this->redirect(':Front:Timesheet:');
        }

        protected function createComponentTimesheetForm()
        {
                $form = new Form;
                $form->addHidden('Tsk_ID');
                $form->addSelect('Cmp_ID', 'Firma', $this->timesheetManager->getCompanies()->fetchPairs('Cmp_ID', 'CmpName1'))
                    ->setPrompt('Vybrat firmu')->setRequired();
                $form->addSelect('Prj_ID', 'Projekt')
                    ->setPrompt('Vybrat projekt');
                $form->addText('TskName', 'Obor zájmu')->setRequired();
                $form->addText('TskDescription1', 'Popis úkolu')->setRequired();
                $form->addText('TskDateStart', 'Datum zahájení')->setRequired();
                $form->addText('TskDateEnd', 'Datum ukončení')->setRequired();
                $form->addText('TskTaskmaster', 'Zadavatel')->setRequired();
                $form->addText('TskWorked', 'Odpracováno (hod.)')->setRequired()->addRule(Form::INTEGER, 'Zadejte číselnou hodnotu');
                $form->addSelect('TskUser', 'Provedl', $employees = array(
                    "user1" => 'user1',
                    "user2" => 'user2',
                    "user3" => 'user3',
                    "user4" => 'user4',
                    "user5" => 'user5',
                    "user6" => 'user6',
                ))->setPrompt('Vybrat zaměstnance')->setRequired();
                $form->addSubmit('submit', 'Uložit');
                $form->onSuccess[] = [$this, 'timesheetFormSucceeded'];
                return $form;
        }

        public function timesheetFormSucceeded($form)
        {

                $values = $form->getHttpData();
                unset($values['_submit']);
                unset($values['_do']);
                $this->timesheetManager->saveTask($values);
                $this->flashMessage('Záznam byl uložen.');
                $this->redirect(':Front:Timesheet:');
        }
}

Děkuji

Editoval iNyxLadis (7. 12. 2016 10:37)

Action se provádí dříve než Handle, takže problém by mohl být že v actionEdit nastavuješ výchozí hodnoty, které dodává až Handle

Editoval Mortisson (8. 12. 2016 16:40)

Vyřešeno následujícím způsobem:

public function actionEdit($taskId, $cmpId)
{
    $task = $this->timesheetManager->getTask($taskId);
    if (!$task) {
        $this->error('Úkol nenalezen');
    } else
    {

    if (!$this->isAjax()) {         // pro nastaveni pocatecni hodnoty selectboxu, ve chvíli obsluhy selectboxu se o zbytek stará handle
          $selectedProjects = $this->timesheetManager->getProjectsForSelectbox($cmpId);
          $this['timesheetForm']['Prj_ID']->setItems($selectedProjects);
          $this['timesheetForm']->setDefaults($task);
        }
    }
    }