Formit validator not working

119 Posts

[email protected] Reply #1, 11 years, 10 months ago

Hi,

I've written a simple validation script to check that two fields have the same value, but it's not working for some reason. Can anyone shed some light please?

From the formit call:

   
&validate=`email2:required:is_confirm_email,
name:required,
email:email:required,
address:required,
postcode:required`

Snippet contents:

<?php
  if ($value != $_POST['email']) {
    $validator->addError($key,'Email addreses must match');
    return false;
  }
  return true;

Studio Republic
http://www.studiorepublic.com
0845 226 3205
@christodhunter

119 Posts

[email protected] Reply #2, 11 years, 10 months ago

I've added a line to write to the error log and it doesn't look like it's firing the snippet at all

Studio Republic
http://www.studiorepublic.com
0845 226 3205
@christodhunter

☆ A M B ☆
3,141 Posts

Mark Hamstra Reply #3, 11 years, 10 months ago

Did you add it to the &customValidators property as well? Can't tell as you didn't show the entire snippet call.

Mark Hamstra • Developer spending his days working on Premium Extras and a MODX Site Dashboard with the ability to remotely upgrade MODX and extras to make the MODX world a little better.

Tweet me @mark_hamstra, check my infrequent blog at markhamstra.com, my slightly more frequent ramblings at MODX.today or see code at Github.

119 Posts

[email protected] Reply #4, 11 years, 10 months ago

Thanks Mark! That did the trick though it's not documented here http://rtfm.modx.com/extras/revo/formit/formit.validators

What's the best way to file a change request on the new rtfm?

Studio Republic
http://www.studiorepublic.com
0845 226 3205
@christodhunter

☆ A M B ☆
3,141 Posts

Mark Hamstra Reply #5, 11 years, 10 months ago

It's mentioned but very very easy to miss tongue

I'm not sure the best way to change the new RTFM is yet. I think the MODX team is still working on ironing out some rough edges and that they'll have more information on the new RTFM out in the next couple of days/week. There's the tracker for docs which I guess can be used for reporting content issues still: http://tracker.modx.com/projects/docs/issues

Mark Hamstra • Developer spending his days working on Premium Extras and a MODX Site Dashboard with the ability to remotely upgrade MODX and extras to make the MODX world a little better.

Tweet me @mark_hamstra, check my infrequent blog at markhamstra.com, my slightly more frequent ramblings at MODX.today or see code at Github.

375 Posts

gadgetto Reply #6, 11 years, 10 months ago

I think the best would be if they add a "Report error" button to each RTFM article! It would be even better if we could have a kind of commenting system to allow comments for everyone (probably moderated - but who should moderate this?).

Freelancer @bitego http://www.bitego.com
---
GoodNews - one of the most advanced and integrated Group Mailer premium add-ons for MODX Revolution!
More infos here: http://www.bitego.com/extras/goodnews/

712 Posts

jpdevries Reply #7, 11 years, 10 months ago

We are considering adding a report error button but ironically to busy fixing broken pages to do so at the moment. In the mean time, please report issues you notice to [email protected].

jpdevries

712 Posts

jpdevries Reply #8, 11 years, 10 months ago

There is now a temporary message at the top of documentation pages with info on how to report noticed issues.

jpdevries

119 Posts

[email protected] Reply #9, 11 years, 10 months ago

Nice one JP!

Studio Republic
http://www.studiorepublic.com
0845 226 3205
@christodhunter

712 Posts

jpdevries Reply #10, 11 years, 10 months ago

Well, congratulate me when it finally gets removed wink

jpdevries