terminal42 / contao-leads

Leads extension for Contao Open Source CMS; Store and manage form data with ease!
GNU Lesser General Public License v3.0
35 stars 24 forks source link

Load form object by model due to problems if $dc->id is null #91

Closed bildschirmartist closed 4 years ago

bildschirmartist commented 4 years ago

I got an exception in my setup when the modifyPalette-method has been called without a valid $dc->id (for some reasons it was null). So I changed the objForm Loading to use the belonging FormModel.

bildschirmartist commented 4 years ago

I just realized, that this problem has been fixed in Contao 4.4.50 (see https://github.com/contao/contao/issues/1867). So this fix is optional.

aschempp commented 4 years ago

Thanks! I still think validating the record is correct.