search

FriendsOfREDAXO / poll

Erstellt und verwaltet Umfragen in REDAXO 5, bei Bedarf mit E-Mailbestätigung.
MIT License
36 stars 2 forks source link

Fehler bei der Installation -> SQL Error: module_id ist NULL #17

Closed greatif closed 4 years ago

greatif commented 4 years ago

Bei der (Neu-)Installation von Poll 1.4 (kein Update) auf REX 5.10.1 u. YForm 3.3.1 kommt es zur Fehlermeldung:

AddOn poll konnte aus folgendem Grund nicht installiert werden: SQL error: Error while executing statement "insert into rex_module set name = :name, input = :input, output = :output, updatedate = :updatedate, updateuser = :updateuser where id = :module_id" using params {":name":"translate:poll_module",":input":"find();\n\n$select = new rex_select();\n$select->setName('REX_INPUT_VALUE[1]');\n\nforeach ($polls as $poll) {\n $select->addOption($poll->title, $poll->id);\n}\n\n$select->setSelected('REXVALUE[1]');\n\necho $select->get();\n",":output":"setVar('poll', $poll);\n echo $fragment->parse('addons\/poll\/poll.php');\n}\n",":updatedate":"2020-05-26 14:57:48",":updateuser":"poll-addon",**":moduleid":null**}! SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'where id = NULL' at line 1

Siehe Screenshot: poll

Die module_id ist NULL.

tyrant88 commented 4 years ago

Kann ich bestätigen.

dergel commented 4 years ago

ja da habe ich richtig MIst gebaut und übereilt rumgefummelt. weil ich Anpassungen brauchte.. Ich habe deswegen die Version rausgenommen und baue lokal an der Version 2.0 die einfach flexibler ist. Aber erst wenn das live auch alles geht werde ich die Änderungen deployen

tyrant88 commented 4 years ago

Dann bin ich mal gespannt! Das mit dem zusätzlichem "comment" ist schon mal cool!

Die Umstellung auf sprog hatte ich bei mir lokal in den Projekten auch immer gemacht, es macht das AddOn aber erstmal unbenutzbar für Leute ohne sprog. Dafür ist mir auch noch keine richtig gute Lösung eingefallen.

Zwei Kleinigkeiten habe ich noch in einen PR ( https://github.com/FriendsOfREDAXO/poll/pull/18 ) - vielleicht magst du das noch einarbeiten, wenn du dazukommst.

cedrus86 commented 4 years ago

ja da habe ich richtig MIst gebaut und übereilt rumgefummelt. weil ich Anpassungen brauchte.. Ich habe deswegen die Version rausgenommen und baue lokal an der Version 2.0 die einfach flexibler ist. Aber erst wenn das live auch alles geht werde ich die Änderungen deployen

Heisst das, der Issue kann geschlossen werden?

tyrant88 commented 4 years ago

Ja, das Issue kann geschlossen werden. Ich hatte gedacht wir führen noch eine Diskussion bzgl sprog.