ACF Vs Railo Discrepency

[Greg Moser]

I just rolled ValidateThis into the Open Source eCommerce project I work on, and all was going very well until doing my tests on ACF.  However, when I flip over to Railo I get an error like:

The property brandName was not found in the object passed into the validation object

Obviously this is happening because ValidateThis isn't able to find the getter method for the property.  This is an ORM entity so we don't define getter / setters we just set persistent="true".  We also have accessors="true" on this particular entity but are still getting the error.

Has anyone ever seen this behavior before?  Any insights would be very helpful

-Greg

[Greg Moser]

I ended up digging deeper into this issue, and found out that is was actually because ValidateThis wasn't accepting that my object was in fact a CFC in Railo.  I came up with the fix that worked for me on this and issues a pull request (https://github.com/ValidateThis/ValidateThis/pull/65)

Huge thanks to everyone who has helped contribute to ValidateThis, I'm still new but loving every minute of it!

-Greg

[Bob Silverberg]

Thanks Gregg. I haven't tested VT on Railo in quite some time. I
need to get a copy installed on my dev machine so I can start running
my test suite against Railo as part of my release cycle.

I will check out your pull request and reply to it via GitHub.

Cheers,
Bob

> --
> You received this message because you are subscribed to the Google Groups
> "ValidateThis" group.
> To view this discussion on the web visit
> https://groups.google.com/d/msg/validatethis/-/I0vIN7mbrzoJ.
> To post to this group, send email to valida...@googlegroups.com.
> To unsubscribe from this group, send email to
> validatethis...@googlegroups.com.
> For more options, visit this group at
> http://groups.google.com/group/validatethis?hl=en.
>

--
Bob Silverberg
www.silverwareconsulting.com

[Matt Quackenbush]

Damn. No option to vote on pull requests.  Greg, thanks for that patch.  I would have definitely hit upon it in the very near future.

[Bob Silverberg]

I have pulled your change into the develop branch. Once I get a
chance to install Railo, run the tests on it, and address any bugs
that come up, I'll do another release to include this and any other
fixes.

Thanks again for the pull request, Greg.

Bob

--
Bob Silverberg
www.silverwareconsulting.com

[Greg Moser]

Not a problem at all.  Thanks guys!

[Chip Pinkston]

Just a quick heads up - this affects Lucee as well.  I'm not sure who's currently maintaining ValidateThis - but Greg's fix works shifting 'railo-context'  to 'lucee'

[Doug Roberson]

Could somebody give me a tip where to go to replicate Greg's fix? I'm still kind of new at this and am hip deep in a Lucee project that I had hoped to use ValidateThis with.

[John Whish]

This is Greg's fix:

https://github.com/teamcfadvance/ValidateThis/pull/65/files

You'll need to switch 'railo-context'  to 'lucee as Chip mentioned above.

HTH,

John

--

You received this message because you are subscribed to the Google Groups "ValidateThis" group.

To unsubscribe from this group and stop receiving emails from it, send an email to validatethis...@googlegroups.com.

To post to this group, send email to valida...@googlegroups.com.

Visit this group at http://groups.google.com/group/validatethis.
For more options, visit https://groups.google.com/d/optout.

[Doug Roberson]

Sir, I am sometimes reminded that I over-complicate things. Thank you for pulling me back to earth.

--
You received this message because you are subscribed to a topic in the Google Groups "ValidateThis" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/validatethis/BaAOMe0lS0Y/unsubscribe.
To unsubscribe from this group and all its topics, send an email to validatethis...@googlegroups.com.

[Chip Pinkston]

In case anyone from Team Advance is monitoring this list - found this on Github when I was looking to see how to get that change rolled in.  Looks like someone else has already made the change for Lucee and is waiting for it to get merged in.

https://github.com/teamcfadvance/ValidateThis/pull/89

[John Whish]

I had merge rights to the original repo but not the teamcfadvance one I'm afraid. Not sure if anyone with permissions is monitoring this list.

[Doug Roberson]

That's too bad :(

[Jason Durham]

I pinged someone from TeamCFAdvance. It should be merged now.

Jason Durham

[Doug Roberson]

Thank you, sir!