Jason Coward

By the logic in the code, both have to be true in order to uninstall an object.

Thanks for the issue. xPDO can handle compound keys, but the problem here is I never anticipated a case where a compound key would include an auto_increment value. I'll have...

> I'm kinda worried this introduces a breaking change where people may have relied on $response->response containing the output of the processor, although seeing just the file out of context...

> Hi @opengeek sorry to bug you on this one but it'd be epic to get a release out with this fix. > > People (like me and many other)...

I was unable to apply this to 3.0.x. If we want this fix on 3.0.x, I'll need the relevant changes submitted against that branch directly.

> It is nice to have NULL available in datetime columns. This is allowed with NO_ZERO_DATE and NO_ZERO_IN_DATE and in strict mode. Only `0000-00-00` is not permitted. > > MODX...

> The issue with the migration script can be invalid existing values in the column, if I am right. In that case the database column definition can't be changed and...

Thank you for your contribution! Before your pull request can be reviewed, please [sign the MODX Contributor License Agreement (CLA)](https://modx.com/community/contribute/cla/). This is to ensure MODX has your written permission to...

> > running on PHP 7.4 (or earlier). > > If you want to support PHP versions **< PHP 7.1**, then there is still a problem with the `void` return...

@halftrainedharry — did you report this in the phpthumb repository? If this does not get fixed upstream, there's not a lot we can do about it.