m-vo / contao-facebook-import

Facebook posts and events for Contao Open Source CMS
Other
11 stars 5 forks source link

Fix compatibility with strict db mode for boolean values #37

Closed rabauss closed 2 years ago

rabauss commented 2 years ago

Fix Issue: #36

Finally I found a solution for #36 ☺️

The 'default' => 0, needs to be an integer in the dca - boolean is not working! And as @fritzmg said we should add 'sql' => ['type' => 'boolean', 'default' => false] to the dca. The save_callback could be removed but I was not really sure - it's working without, too!

rabauss commented 2 years ago

@m-vo are there any problems to merge/review this PR?

EDIT: @rabauss need to run ECS!

rabauss commented 2 years ago

@m-vo now it should be ok

m-vo commented 2 years ago

Thanks a lot @rabauss!