SOLVED: Validation on php form plugin

[dimoss]

Hi

I use the following code on a php form plugin to validate the record insertion on a table onAfterProcess:

$etos = $formModel->getElementData('fab_mitroo_elegktwn___etos', true);if (is_array($etos)) {$etos = $etos[0];}
$minas = $formModel->getElementData('fab_mitroo_elegktwn___minas', true);if (is_array($minas)) {$minas = $minas[0];}
$db = JFactory::getDbo();
$query = $db->getQuery(true);
$query->select('COUNT(*)');
$query->from('fab_table');
$query->where('etos = '.$etos);
$query->where('minas = '.$minas);
$db->setQuery($query);
$number = $db->loadResult();
if ($number > 0)
{
return false;
}
else
{
return true;
}

However it seems that something is wrong as the real query should return > 0 which means 'false'.
Any help is appreciated.

Thanks

 

[rob]

if you do :

$db->setQuery($query);
echo $query->dump();
exit;

what is the query that is generated?

 

[dimoss]

Hi Rob

This is the query produced:

SELECT COUNT(*)
FROM fab_table
WHERE etos =  AND minas =

This is incorrect as it doesn't pass the variables' values.

 

[troester]

It looks like $etos and $minas are empty, check with var_dump

What do you want to validate? onAfterProcess the record is already stored.

 

[dimoss]

Hi Troester

The query was empty because it was m y mistake with the naming...
You are right with onAfterProcess. I did it with onBeforeProcess and worked which means it didn't record but I got the normal message that the record added/updated.
Why is that? Is there any way to hide this message and get a pop up or something which says that record is not added blah...blah.... ?
Thanks!

 

[troester]

You must set the error message yourself
http://fabrikar.com/forums/index.php?wiki/php-form-plugin/#stopping-form-submission

As far as I can remember the main message is displayed only if there's an element error message, so you have to set at least one, too

 

[dimoss]

Thanks Troester
However in which position I need to put it?
After the if statement?
Sorry if it's a stupid question!

 

[troester]

Yes, before return false;

 

[dimoss]

Hi Troester

It works ok with the message appeared as expected.
I put it before 'return false'
Thanks a lot!