Compare commits

...

1 Commits

Author SHA1 Message Date
William MARTIN
f88e529b85 Update the form validation of IDF_Form_ReviewFileComment.
Now we can add a general comment without per file comment.

Fix issue 660
2011-04-06 13:07:19 +02:00

View File

@ -52,7 +52,7 @@ class IDF_Form_ReviewFileComment extends Pluf_Form
)); ));
} }
$this->fields['content'] = new Pluf_Form_Field_Varchar( $this->fields['content'] = new Pluf_Form_Field_Varchar(
array('required' => true, array('required' => false,
'label' => __('General comment'), 'label' => __('General comment'),
'initial' => '', 'initial' => '',
'widget' => 'Pluf_Form_Widget_TextareaInput', 'widget' => 'Pluf_Form_Widget_TextareaInput',
@ -94,23 +94,39 @@ class IDF_Form_ReviewFileComment extends Pluf_Form
*/ */
public function clean() public function clean()
{ {
$isOk = false;
foreach($this->files as $filename => $def) { foreach($this->files as $filename => $def) {
$this->cleaned_data[md5($filename)] = trim($this->cleaned_data[md5($filename)]);
if(!empty($this->cleaned_data[md5($filename)])) { if(!empty($this->cleaned_data[md5($filename)])) {
$isOk = true;
}
}
if(!empty($this->cleaned_data['content'])) {
$isOk = true;
}
if (!$isOk) {
throw new Pluf_Form_Invalid(__('You need to provide your general comment about the proposal, or comments on at least one file.'));
}
return $this->cleaned_data; return $this->cleaned_data;
} }
}
throw new Pluf_Form_Invalid(__('You need to provide comments on at least one file.'));
}
function clean_content() function clean_content()
{ {
$content = trim($this->cleaned_data['content']); $content = trim($this->cleaned_data['content']);
if (!$this->show_full and strlen($content) == 0) { if(empty($content)) {
throw new Pluf_Form_Invalid(__('You need to provide your general comment about the proposal.')); if ($this->fields['status']->initial != $this->fields['status']->value) {
return __('The status have been updated.');
} }
} else {
return $content; return $content;
} }
throw new Pluf_Form_Invalid(__('This field is required.'));
}
/** /**
* Save the model in the database. * Save the model in the database.