Garry Nutting

Results 7 comments of Garry Nutting

Agreed, we should have First, Middle, Last with the current fullname being populated with the concat'ed versions of the new fields IMO.

Just get rid of MSSQL support, I doubt it'd be fully functional right now anyway ... I don't think the sqlsrv stuff has been maintained for a while now. That's...

In case it helps, I've tested this on a MODX 3.0.3 site and the following will fix the errors. Find the `$modx->getService()` line for `modFileHandler` in the following files: ```...

And apologies as I think this PR covers it (different approach, but same result) :): https://github.com/sergant210/AdminTools/pull/28

@scr34m I tested on my end and all is good here - the sigs were picked up.

Definitely one for MODX 3.0, it's a potential breaking change to change those variable names and any breaking changes need to happen in the next major release.