[Symfony]Zamiast listy użytkowników w polu powinno być id zalogowanego uzytkownika Opcje

[nieraczek]

Mam tabelę w bazie danych:
id PK
id_uzytkownika FK
samochod

Relacja jest z tabelą sf_guard_user - jeden użytkownik może mieć wiele samochodów.
Wygenerowałem moduł i przy dodawaniu nowego elementu na stronie w polu 'id uzytkownika' jest lista wszystkich użytkowników do wyboru, bo symfony zrobił mi widget:
id_uzytkownika' => new sfWidgetFormDoctrineSelect(array('model' => 'sfGuardUser', 'add_empty' => false))
i walidator:
'id_uzytkownika' => new sfValidatorDoctrineChoice(array('model' => 'sfGuardUser'))

a to pole powinno być niewidoczne i powinno w nim być id zalogowanego użytkownika, a nie że użytkownik może sobie wybierać użytkowników z listy.

Jak temu zaradzić ? Mam usunąć $this['id_uzytkownika'], po czym utworzyć niewidoczny widget typu input i wpisać do niego id zalogowanego użytkownika wysyłane do formularza z actions.class.php

[Cysiaczek]

Hm, nie wiem jak w Doctrine, ale w Propelu ja bym użył opcji peer_method przekazanej do konstruktora formularza innej iż domyślne doSelect(). np. doSelectCurrentUser(). W walidatorze też bym to sprawdzał (osobny, Twój własny walidator). Inną metodą, chyba nawet bezpieczniejszą jest po prostu usunięcie tego pola i pobranie id usera bezpośrednio w kodzie akcji, bo zakładam, że jest to funkcja dla zalogowanych użytkowników.

Pozdrawiam

[nieraczek]

O tym drugim sposobie co go wspomniałeś myslałem wcześniej - w actions.class.php symfony mi stworzył

[PHP] pobierz, plaintext 
<?php
public function executeCreate(sfWebRequest $request)
  {
    $this->forward404Unless($request->isMethod('post'));
 
    $this->form = new SamochodyForm();
 
    $this->processForm($request, $this->form);
 
    $this->setTemplate('new');
  }
 
 
protected function processForm(sfWebRequest $request, sfForm $form)
  {
    $form->bind($request->getParameter($form->getName()));
    if ($form->isValid())
    {      
      $samochody = $form->save();
 
      $this->redirect('/profil/edit?id='.$samochody['id']);
    }
  }
?>
[PHP] pobierz, plaintext 

Usunąłem to pole:

[PHP] pobierz, plaintext 
<?php
unset
    (
      $this['id_uzytkownika']
    );
?>
[PHP] pobierz, plaintext 

Kombinowałem tak - na razie na sztywno ustawianie id_uzytkownika:

[PHP] pobierz, plaintext 
<?php
$form->setIdUzytkownika(2);
$samochody = $form->save();
?>
[PHP] pobierz, plaintext 

Czyli:

[PHP] pobierz, plaintext 
<?php
public function executeCreate(sfWebRequest $request)
  {
    $this->forward404Unless($request->isMethod('post'));
 
    $this->form = new SamochodyForm();
 
    $this->processForm($request, $this->form);
 
    $this->setTemplate('new');
  }
 
 
protected function processForm(sfWebRequest $request, sfForm $form)
  {
    $form->bind($request->getParameter($form->getName()));
    if ($form->isValid())
    {      
      $form->setIdUzytkownika(2);
      $samochody = $form->save();
 
      $this->redirect('/profil/edit?id='.$samochody['id']);
    }
  }
?>
[PHP] pobierz, plaintext 

Ale dostaję błąd: This request has been forwarded to a 404 error page by the action "profil/create".

[Cysiaczek]

Oczywiście, bo akcja create może być wywołana getem , co prawdpodobnie się dzieje

[nieraczek]

Nie, to akurat:

[PHP] pobierz, plaintext 
<?php
$this->forward404Unless($request->isMethod('post'));
?>
[PHP] pobierz, plaintext 

jest ok.

W tym miejscu się wywala:

[PHP] pobierz, plaintext 
<?php
$form->setIdUzytkownika(2);
$samochody = $form->save();
?>
[PHP] pobierz, plaintext 

Ten post edytował nieraczek 12.04.2009, 20:23:37

[michalg]

Dam Ci przykład, jak ja u siebie zrobiłem.

Metody klasy formularza:

[PHP] pobierz, plaintext 
<?php
public function configure()
    {
[...]
            unset($this['added_by']);
[...]
    }
 
 
    // TODO: In symfony 1.3 change this function into doUpdateObject
    public function updateObject($values = null)
    {
        $object = parent::updateObject($values);
 
        $user = sfContext::getInstance()->getUser();
[...]
                $object->setIdUserAddedBy($user->getId());
[...]
 
        return $object;
    }
?>
[PHP] pobierz, plaintext 

Czyli formularz jest odpowiedzialny za wpisanie użytkownika.

Ten post edytował michalg 14.04.2009, 16:48:08

[nieraczek]

Dziękuję, jednak znalazłem znacznie prostszy sposób - wystarczy użyć $this->form->getObject() czyli np.:

Kod

protected function processForm(sfWebRequest $request, sfForm $form)
  {
    $form->bind($request->getParameter($form->getName()));
    if ($form->isValid())
    {          

      $this->form->getObject()->setIdUzytkownika( $this->getUser()->getGuardUser()->getId() );

      $samochody = $form->save();

      $this->redirect('/profil/edit?id='.$samochody['id']);
    }
  }

[michalg]

Dziękuję, jednak znalazłem znacznie prostszy sposób - wystarczy użyć $this->form->getObject() czyli np.:

Też tak można, ale ja wolałem to robić w formularzu, żeby mieć spójność - za utworzenie obiektu odpowiedzialny jest formularz. I może on sam utworzyć pełny obiekt. A w Twoim rozwiązaniu obiekt jest częsciowo tworzony przez formularz, a częściowo przez akcję. Jeżeli będziesz chciał użyć tego formularza w innym module, to będziesz musiał duplikować kod.

Inna sprawa, że w moim przykładzie uzależniam formularz od klasy użytkownika, co też może nie jest idealnym rozwiązaniem.