Jak mogę w symfony 1.2 ustawić setError ?

Jak mogę zrobić walidacje wysokosci i szerokosci obrazka ? dopisac do sfFileValidator kod odpowiedzialny za to czy w jakis inny sposób ?

[PHP] pobierz, plaintext 
<?php
$this->setValidators(array(
            'tytul'    =>    new sfValidatorString(
                array('min_length'=>3, 'max_length' => 140, 'required' => true),
                array('min_length'=>'za krótki tytuł', 'max_length'=>'za długi tytuł', 'required'=>'brak tytułu')
            )
        ));
?>
[PHP] pobierz, plaintext 

walidacje obrazka musisz zrobić samemu.
wydaje mi się, że można to zrobić tak:
tworzysz swoją klasę walidacji np. myValidatorFile
tam nadpisujesz metodę doclean.
w niej wykonujesz nadrzędną metodę doclean, sprawdzasz zapisany plik i jego rozmiary, jeżeli są za duże czy nie dozwolone to zwracasz wyjątek
throw new sfValidatorError

Znalazlem gotowa klase ale nie dla symfony 1.2 moze to przerobie

http://snippets.symfony-project.org/snippet/259

właśnie, tak jak napisałem;)

[PHP] pobierz, plaintext 
<?php
class sfFileImageValidator extends sfFileValidator
{
  function doClean($value){
    if(parent::doClean($value)){
...
    }
  }
}
?>
[PHP] pobierz, plaintext 

trzeba się przyjrzeć jak to wszystko jest zrobione i później już fajnie się z tego korzysta.

Nie wiem czemu jak próbuje tego użyc to wywala błąd :
sfFileImageValidator does not support the following options: 'max_width'.

[PHP] pobierz, plaintext 
<?php
'file_path'  => new sfFileImageValidator(array('path' => sfConfig::get('sf_upload_dir').'/gallery/temp','required' => true,'mime_types' => array('image/jpeg','image/pjpeg'),'max_size' => '261120','max_width' =>'200'), array('mime_types' =>'Nieprawidłowy format pliku','max_size'=>'Za duży rozmiar pliku. Limit to 255 Kb','required'=>'Wybierz zdjęcie')  )
?>
[PHP] pobierz, plaintext 

To dlatego, że walidator podczas konfigurowania nie dodaje (rejestruje) takiej opcji. Musisz w metodzie sfFileImageValidator::configure(), dodać te opcje:

[PHP] pobierz, plaintext 
<?php
class sfFileImageValidator extends sfValidatorFile
{
  protected function configure($options = array(), $messages = array())
  {
    $this->addOption('mime_types');
    //a jesli chcesz dodac opcje wymagana, to:
    $this->addRequiredOption('mime_types');
  }
}
?>
[PHP] pobierz, plaintext 

Tak samo jest z wiadomościami.

Faktycznie dzięki - zrobiłem jak piszesz tyle że walidator nie działa - nie wyświetla komunikatu o błędzie

opcje dodałem do klasy sfValidatorFile

Po części mój błąd, bo domyślam się, że tego nie zrobiłeś: parent::configure($options, $messages). Jeśli nadal nie działa jak powinno, to bez kodu nic nie zrobimy.
Po drugie, nie wiem czemu akurat dodałeś opcje do sfValidatorFile.

Zrobiłem ale też nie działo :/ walidator wygląda tak :

[PHP] pobierz, plaintext 
<?php
class sfFileImageValidator extends sfValidatorFile
{
  /**
   * Executes this validator.
   *
   * @param mixed A file or parameter value/array
   * @param error An error message reference
   *
   * @return bool true, if this validator executes successfully, otherwise false
   */
  protected function configure($options = array(), $messages = array())
  {
    parent::configure($options, $messages);
 
    $this->addOption('max_width');
    $this->addMessage('max_width', 'tttt');
  } 
 
 
  public function execute(&$value, &$error)
  {
    if (parent::execute($value, $error))
    {
      list($width, $height) = @getimagesize($value['tmp_name']);
 
      // File is not a square
      $is_square = $this->getParameter('is_square');
      if ($is_square && $width != $height)
      {
        $error = $this->getParameter('is_square_error');
 
        return false;
      }
 
      // File height too large
      $max_height = $this->getParameter('max_height');
      if ($max_height !== null && $max_height < $height)
      {
        $error = $this->getParameter('max_height_error');
 
        return false;
      }
 
      // File width too large
      $max_width = $this->getParameter('max_width');
      if ($max_width !== null && $max_width < $width)
      {
        $error = $this->getParameter('max_width_error');
 
        return false;
      }
 
      // File height too small
      $min_height = $this->getParameter('min_height');
      if ($min_height !== null && $min_height > $height)
      {
        $error = $this->getParameter('min_height_error');
 
        return false;
      }
 
      // File width too small
      $min_width = $this->getParameter('min_width');
      if ($min_width !== null && $min_width > $width)
      {
        $error = $this->getParameter('min_width_error');
 
        return false;
      }
 
      return true;
    }
  }
 
  /**
   * Initializes this validator.
   *
   * @param sfContext The current application context
   * @param array   An associative array of initialization parameters
   *
   * @return bool true, if initialization completes successfully, otherwise false
   */
  public function initialize($context, $parameters = null)
  {
    // initialize parent
    parent::initialize($context, $parameters);
 
    // set defaults
    $this->getParameterHolder()->set('max_height',        null);
    $this->getParameterHolder()->set('max_height_error',  'The file height is too large');
    $this->getParameterHolder()->set('max_width',         null);
    $this->getParameterHolder()->set('max_width_error',   'The file width is too large');
    $this->getParameterHolder()->set('min_height',        null);
    $this->getParameterHolder()->set('min_height_error',  'The file height is too small');
    $this->getParameterHolder()->set('min_width',         null);
    $this->getParameterHolder()->set('min_width_error',   'The file width is too small');
    $this->getParameterHolder()->set('is_square',         false);
    $this->getParameterHolder()->set('is_square_error',   'The file is not a square');
 
    $this->getParameterHolder()->add($parameters);
 
    return true;
  }
}
?>
[PHP] pobierz, plaintext 

walidacja

[PHP] pobierz, plaintext 
<?php
$this->setValidators(array(
        'photo_id'    => new sfValidatorPropelChoice(array('model' => 'Photo', 'column' => 'photo_id', 'required' => false)),
        'gallery_id'  => new sfValidatorChoice(array('choices' => array_keys(Gallery::getValuesToSelectGallery1()))),
        'title'       => new sfValidatorString(array('min_length' => 1), array('required' => 'Wpisz tytuł zdjęcia')),
        'file_patch'  => new sfFileImageValidator(array('path' => sfConfig::get('sf_upload_dir').'/gallery/temp','required' => true,'mime_types' => array('image/jpeg','image/pjpeg'),'max_size' => '261120','max_width' =>'200'), array('mime_types' =>'Nieprawidłowy format pliku','max_size'=>'Za duży rozmiar pliku. Limit to 255 Kb','required'=>'Wybierz zdjęcie')  ),
        'description' => new sfValidatorString(array('required' => false)),
        'tags'        => new sfValidatorString(array('required' => false)),
      ));
?>
[PHP] pobierz, plaintext 

Wrzucajac zdjecie na serwer w ogole nie ma walidacji szerokosci tego pliku

Jako, że tytuł wątku i pośrednio tematyka bardzo mi pasuje to się podłączę. Jak przekazać do formularza dodatkowe błędy?

Sytuacja jest następująca. Mam formularz rozszerzający sfFormPropel. Po wykonaniu validacji formularza chciałbym wykonać trochę logiki w kontrolerze, w którym jest on zawarty.
Robię sobie zwykłe $form->save(). Jeśli zapis wywoła wyjątek to sfFormPropel::save() propaguje ten wyjątek. A ja chciałbym jego treść przekazać jako jeden z błędów formularza.

[PHP] pobierz, plaintext 
<?php
 
// Utworzenie formularza i...
$this->adminForm = new AdminForm();
// ...przekazanie go do widoku.
$this->setVar('adminForm', $this->adminForm);
 
// Jeśli żądania jest typu POST i przetwarzanie formularza powiodło się...
if ($request->getMethod() === sfRequest::POST
    && $this->processForm($request, $this->adminForm)) {
 
    // ...to dokonywana jest próba zapisu
    try {
        $this->adminForm->save();
 
        /* @var $admin Admin */
        $admin = $this->adminForm->getObject();
 
        // Ustawienie komunikatu
        $this->getUser()->setFlash('create', "Utworzono nowego administratora ($admin)");
        // Przekierowanie na widok główny modułu zarządzania administratorami
        $this->redirect('@admins');
    } catch (Exception $e) {
        // Logowanie błędu
        sfContext::getInstance()->getLogger()->err($e->getMessage());
        // Ustawienie komunikatu o niepowodzeniu
        $this->errors = array('Nie udało się utworzyć nowego administratora');
    }
}
 
?>
[PHP] pobierz, plaintext 

W 28 linii jestem zmuszony przekazywać błąd do widoku poza formularzem. Wiem, że w zasadzie błędy formularza to logicznie co innego niż błędy wynikające z logiki biznesowej ale chciałbym użyć mechanizmów formularza żeby te błędy wyświetlić. Ot taka neichęc do dublowania kodu.

No własnie w symfony 1.0, 1.1 było prosto setError i chyba tyle :/

Wyrzucenie błędów z sfRequest to akurat dobra decyzja. Było to wygodne ale jakoś mnie to kuło w oczy.
Pozostała jednak luka bo błędy związane z formularzami zostały załatwione. Na błędy związane z logiką nie ma pomysłu.

@Lonas, walidator nie działa z prostej przyczyny. Metoda execute nigdy nie zostanie wykonana. Zajrzyj do książki o formularzach, powinno coś byc.

@mike, co do Twojego problemu to jest to dłuższa historia. Nie wiem jak wygląda u Ciebie metoda processForm, domyślam się, że standardowo tylko wtedy powstaje problem. Ty masz wynik tej metody w warunku i po prostu nie wiem. Wiec zakładam, że jest to metoda wygenerowana:

[PHP] pobierz, plaintext 
<?php
// Utworzenie formularza i...
$this->adminForm = new AdminForm();
// ...przekazanie go do widoku.
$this->setVar('adminForm', $this->adminForm);
 
// Jeśli żądania jest typu POST
if ($request->getMethod() === sfRequest::POST) 
{
 
   // ...to dokonywana jest próba zapisu
    try 
    {
        $this->processForm($request, $this->adminForm);
 
        /* @var $admin Admin */
        $admin = $this->adminForm->getObject();
 
        // Ustawienie komunikatu
        $this->getUser()->setFlash('create', "Utworzono nowego administratora ($admin)");
        // Przekierowanie na widok główny modułu zarządzania administratorami
        $this->redirect('@admins');
    }
    catch (Exception $e) 
    {
        // Logowanie błędu
        sfContext::getInstance()->getLogger()->err($e->getMessage());
        // Ustawienie komunikatu o niepowodzeniu
        $this->adminForm->getErrorSchema()->addError(
            new sfValidatorError(new sfValidatorInteger(), 'min', array('value' => 15, 'min' => 18))
        );
   }
}
?>
[PHP] pobierz, plaintext 

Szkorzystałem z sfValidatorInteger, ponieważ nie wiem jaki błąd masz, pewnie jakiś od bazy ale to już sobie sam rozwiążesz. Najlepiej chyba będzie nadpisać klase sfValidatorError i to tak, żeby w konstruktorze nie trzeba było podawać obiektu walidatora.

Bindowanie parametrów żądania z polami formularza i wykonanie isValid(), które to jest zwracane jako rezultat działania funkcji.
1. Błęd to powiedzmy: baza strzeliła focha :-)
2. No właśnie te validatory w konstruktorze całkowici mi są nie na rękę :-) Faktycznie napisanie swojej klasy błędów będzie niezłym pomysłem a do konstruktora będę pchał neutralnego sfValidatorPass z komunikatem invalid.

Dzięki ~destroyerr. Kliknąłbym Ci pomógł ale i nie mój wątek, i uprawnień moda się wybyłem :-)

To ja klikne za Ciebie ;-)

destroyerr : Na stronie autor tej klasy podaje że wystarczy utworzyć plik yml ? chyba że jest jakaś różnica w działaniu tego walidatora w symfony 1.1 i 12 ?
http://snippets.symfony-project.org/snippet/259

Jest. Podany przez Ciebie validator został napisany pod 1.0. Dzięki kompatybilności wstecznej działa w 1.1 ale w 1.2 już nie zadziała.

Czyli dokumentacja niby do 1.2 też nieaktualna ?
http://www.symfony-project.org/book/1_2/10...ustom_validator

Niestety tak.

Rozwiązanie mojego problemu to 3 klasy:

[PHP] pobierz, plaintext 
<?php
 
/**
 * @author Michał Mech
 */
class sfFormProcessable extends sfForm {
 
    public function addProcessError(sfFormProcessError $error) {
        $this->getErrorSchema()->addError($error);
    }
}
 
/**
 * @author Michał Mech <michal.mech@k2.pl>
 */
abstract class sfFormPropelProcessable extends sfFormPropel {
 
    public function addProcessError(sfFormProcessError $error) {
        $this->getErrorSchema()->addError($error);
    }
}
 
 
/**
 * @author Michał Mech <michal.mech@k2.pl>
 */
class sfFormProcessError extends sfValidatorError {
 
    /**
     * @var sfValidatorBase
     */
    protected $validator;
 
    public function __construct($message = null) {
        $this->validator = new sfValidatorPass();
        $this->validator->setMessage('invalid', $message);
 
        parent::__construct($this->validator, 'invalid');
    }
}
 
?>
[PHP] pobierz, plaintext 

Obecnie moje klasy formularzy dziedziczą po sfFormProcessable (lub po sfFormPropelProcessable ) dzięki temu mogę zrobić po prostu:

[PHP] pobierz, plaintext 
<?php
 
$form = SampleForm();
try {
    $form->save()
} catch (Exception $e) {
    $form->addProcessError(
        new sfFormProcessError($e->getMessage());
    );
}
 
?>
[PHP] pobierz, plaintext 

A błędy wyświetlą się w formularzu.

Czy to powinno być coś w ten deseń ?

[PHP] pobierz, plaintext 
<?php
class sfFileImageValidator extends sfValidatorFile
{
 
  protected function configure($options = array(), $messages = array())
  {
    parent::configure($options, $messages);
 
    $this->addOption('max_width');
    $this->addMessage('max_width', 'tttt');
  } 
 
 
  protected function doClean($value)
  {
    list($width, $height) = getimagesize($value['tmp_name']);
    
 
    if ($width >  100)
    {
      throw new sfException('My exception message');
    }
 
    return $value;
  } 
 }
?>
[PHP] pobierz, plaintext 

Mniej więcej coś takiego, tylko klasa wyjątku to sfValidatorError. Zerknij na inne walidatory, to będziesz wiedział jak to zrobić.

Walidator działa - wyglada to narazie tak :

[PHP] pobierz, plaintext 
<?php
class sfFileImageValidator extends sfValidatorFile
{
 
  protected function configure($options = array(), $messages = array())
  {
    parent::configure($options, $messages);
 
    $this->addOption('max_width');
    $this->addMessage('max_width', 'tttt');
  } 
 
 
  protected function doClean($value)
  {
    
    list($width, $height) = getimagesize($value['tmp_name']);
    
 
    if ($this->hasOption('max_width') && $width > $this->getOption('max_width'))
    {
      throw new sfValidatorError($this, 'max_width', array('value' => $value, 'max_width' => $this->getOption('max_width')));
    }
 
    return $value;
  } 
 
}
?>
[PHP] pobierz, plaintext 

Jak będzie gotowy wrzuce pełen kod, nie działa mi natatomiast w sytuacji gdy walidacja się powiedzie - walidator nie zwraca obiektu
w sfValidatorFile zwracany jest obiekt

[PHP] pobierz, plaintext 
<?php
$class = $this->getOption('validated_file_class');
 
    return new $class($value['name'], $mimeType, $value['tmp_name'], $value['size'], $this->getOption('path'));
?>
[PHP] pobierz, plaintext 

Fatal error: Call to a member function save() on a non-object in C:\Program Files\WebServ\httpd\sf_gallery\lib\symfony\plugins\sfPropelPlugin\lib\form\sfFormPropel.class.php on line 508

jak to powinno być w moim przypadku ?