DieSchittigs / recaptcha-bundle

reCAPTCHA replacement for Contao 4
ISC License
7 stars 6 forks source link

Datenbankupdate löscht Grid Spalten #20

Open BlackP999 opened 3 years ago

BlackP999 commented 3 years ago

Hi, nach der installation bei dem Datenbankupdate der Erweiterung werden folgende Spalten zum löschen angezeigt. Müssen diese gelöscht werden, bzw. Darf das Grid Element nicht genutzt werden mit der reCaptcha Erweiterung?

ALTER TABLE tl_form_field DROP bs_addInputGroup ALTER TABLE tl_form_field DROP bs_inputGroup ALTER TABLE tl_form_field DROP bs_grid ALTER TABLE tl_form_field DROP bs_grid_name ALTER TABLE tl_form_field DROP bs_grid_parent

amenk commented 3 years ago

Ähnliches Problem hier

amenk commented 3 years ago

Das Problem ist wohl das return; im DCA, das bringt Contao durcheinander.

Scheint im dev-master gefixt zu sein:

https://github.com/DieSchittigs/recaptcha-bundle/commit/afc9821f1bfc7d147112323f85991b8d86fedbf3#diff-398ad1d5b1a348c12dca295bd7d1d7ee46e8341beb4faa24f2f405da60422476

bennyborn commented 3 years ago

Das Issue ist seit Januar offen, das Problem ist ernst und nicht nur wir haben vermutlich gerade Stunden fürs Debugging verschwendet. Könntet ihr bitte einen Release machen?

amenk commented 3 years ago

@Lixus spricht etwas dagegen, dev-master zu taggen? Gibt es da noch Bugs? Wäre sehr nützlich, da wir in einem anderen Projekt wieder in das Problem gelaufen sind. Wenn wir irgendwo unterstützen können, bitte gerne melden.

Lixus commented 3 years ago

@amenk @bennyborn Ich habe einen Release getaggt. Sorry für die Inaktivität. Wir kommen leider aktuell nicht wirklich dazu das Repository zu pflegen.

amenk commented 3 years ago

Perfekt, danke