id mit autoincrement mit in die Tabelle aufnehmen

[rkemmere]

Es wäre super, wenn die ID mit autoincrement mit in die Tabelle aufgenommen werden kann. Dann hat man die Möglichkeit das per YFORM auch als Tabelle zu migrieren und im Backend anzuzeigen und Datensätze zu löschen.

[alxndr-w]

Was wäre der konkrete Nutzen? Wenn sich der Entwickler ausgesperrt hat oder geht es um etwas anderes?

[rkemmere]

Wir hatten jetzt schon mehrmals das Kunden die mehrere Teilnehmer über ein Formular anmelden, ausgesperrt wurden. Und da wäre es schön wenn man die wieder freigeben könnte.

[alxndr-w]

Dann sind imho die Einstellungen nicht optimal gewählt - in diesem Szenario sollte man die Toleranzwerte erhöhen oder dieses pro Tabelle festlegen können. Eine ID-Spalte ist dafür nicht nötig.

[alxndr-w]

Nochmal darüber nachgedacht - als Entwickler hätte ich es auch ganz praktisch gefunden, habe mich aber per Adminer beholfen.

Eine Ausgabe per rex_form im Addon mit Möglichkeit, einzelne Einträge zu löschen oder die Tabelle zu leeren, wäre eine Alternative. Wenn du es brauchst, dann könntest du ja auch den Ansatz verfolgen/umsetzen.

[alxndr-w]

@rkemmere löst #46 dein Problem oder brauchst du noch das ID-Feld? Wenn ja, der PR ist schnell gemacht:

1. im Adminer in die Tabelle egehn
2. id-Feld mit AI anlegen
3. Passende rex_sql_table-Definition rauskopieren und in
4. die install.php übernehmen (und ggf. update.php)

[rkemmere]

Soll der Pull dann hier rein? Also wollen wir das in das initiale Release mit aufnehmen. Man kann die YFORM Tabelle ja initial ausblenden.

[alxndr-w]

Die Diskussion darüber dauert länger, als es zu machen. Ich hab's jetzt einfach gemacht.

[alxndr-w]

@rkemmere magst du mal schauen, ob 2.0 für dich so passt?

#59

Ein 2. paar Auge würde meiner Überarbeitung gut tun.

[rkemmere]

Hmm habe in einer R5 mit YFORM 4.2 das vorhandene spam_protection deinstalliert und das aus dem GIT 2.0 RC1 installiert.

Bekomme im Formular beim Absenden den Fehler:

`TypeError: Return value of spam_protection::getConfig() must be an instance of mixed, int returned
File: redaxo/src/addons/yform_spam_protection/lib/spam_protection.php
Line: 177

Stacktrace

Function	File	Line
spam_protection::getConfig	redaxo/src/addons/yform_spam_protection/lib/spam_protection.php	59
spam_protection::check	redaxo/src/addons/yform_spam_protection/lib/yform/value/spam_protection.php	23
rex_yform_value_spam_protection->postValidateAction	redaxo/src/addons/yform/lib/yform.php	352
rex_yform->executeFields	redaxo/src/addons/yform/lib/yform.php	257
rex_yform->getForm	theme/private/inc/form/contact_formular.php	91
include_once	redaxo/src/addons/xoutputfilter/lib/xoutputfilter_util.php(671) : eval()'d code	2
eval	redaxo/src/addons/xoutputfilter/lib/xoutputfilter_util.php	671
xoutputfilter_util::evalphp	redaxo/src/addons/xoutputfilter/lib/xoutputfilter.php	399
xoutputfilter::frontendReplace	redaxo/src/core/lib/extension.php	52
rex_extension::{closure}	redaxo/src/core/lib/util/timer.php	63
rex_timer::measure	redaxo/src/core/lib/extension.php	63
rex_extension::registerPoint	redaxo/src/core/lib/response.php	248
rex_response::sendPage	redaxo/src/addons/structure/plugins/content/boot.php	75
rex_package::{closure}	redaxo/src/core/lib/extension.php	52
rex_extension::{closure}	redaxo/src/core/lib/util/timer.php	63
rex_timer::measure	redaxo/src/core/lib/extension.php	63
rex_extension::registerPoint	redaxo/src/core/frontend.php	26
require	redaxo/src/core/boot.php	143
require	index.php	9

System report (REDAXO 5.13.2, PHP 7.4.28, MySQL 5.7.37)

REDAXO
Version	5.13.2

Packages

| yform | 4.0.2 |
| yform/email | 4.0.2 |
| yform/manager | 4.0.2 |
| yform_spam_protection | 2.0-beta1 |
| yform_usability | 2.0.3 |

`

[alxndr-w]

Ist korrigiert...

[rkemmere]

Ok, das klappt jetzt.

Woher kommen diese Meldungen und wie kann ich die ändern?

{{ yform_spam_protection_warning }} {{ already }}
{{ yform_spam_protection_warning }} {{ form_microtime }}

[alxndr-w]

Die kommen, wenn eine der Regeln verletzt wurde, also Spam vermutet wird Anpassen kann man diese über Sprog oder den Output-Filter, aber ich werde besser Felder für die Einstellungsseite liefern...

[rkemmere]

Ja entweder Felder auf der Einstellungsseite oder über die Langdateien. Sprog ist zwar schön, aber wenn man anstelle von {{ ## verwendet klappt es schon nicht mehr.