Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DB-Tabellenname in db_settings.php definieren #43

Closed
fenglisch opened this issue Jul 19, 2021 · 4 comments
Closed

DB-Tabellenname in db_settings.php definieren #43

fenglisch opened this issue Jul 19, 2021 · 4 comments
Labels

Comments

@fenglisch
Copy link
Collaborator

Wenn die Datenbanktabelle anders als Results heißen soll, muss ich diese Änderung an mehreren Stellen eintragen, was fehleranfällig ist:

  • In der results_db.sql dreimal (lässt sich vermutlich nicht ändern, außer ich importiere die Tabelle als Results und benenne sie dann um)
  • In der vote_db.php einmal (relativ versteckt)
  • In der read_db_write_text.php einmal (ebenfalls versteckt)

Wäre es möglich, den Tabellennamen als Variable in der db_settings.php zu definieren, sodass andere Dateien darauf zugreifen?

@fenglisch fenglisch added the prio3 low priority label Jul 19, 2021
@msteudtn
Copy link
Owner

Jain.

Für den SQL-Import kann die Datenbank nur Text verarbeiten und keine Variablen.

Sobald der Tabellenname aber einmal feststeht (und dann in der db_settings.php drin steht), sollte man ihn aber auch als Variable definieren können.

@fenglisch
Copy link
Collaborator Author

Super! Das mit der SQL-Datei habe ich mir schon gedacht. Hier ist die nachträgliche Umbenennung (direkt im Anschluss an den Import) aber auch eine gute Methode. Wenn der Tabellenname in die anderen beiden Dateien übernommen werden könnte und die entsprechend gar nicht mehr angepasst werden müssten, wäre das schon super.

@msteudtn
Copy link
Owner

Die Möglichkeit den Tabellennamen anzugeben ist nun in den /extras/statistics/db_settings.php drin.

Die Standard-Tabelle heißt Results kann aber auch umbenannt werden. Wenn die Tabelle umbenannt wird (Zeilen 16, 27 und 33 in der results_db.sql), muss die db_settings.php entsprechend angepasst werden.

@fenglisch
Copy link
Collaborator Author

Funktioniert ausgezeichnet, vielen Dank!
Der komplette Prozess (SQL importieren und umbenennen, db_settings.php anpassen, definition.js anpassen, ein Dummy-Ergebnis erzeugen, Datenbank checken, read_db_write_text.php aufrufen, results.js anpassen und results.html aufrufen) waren jetzt nur wenige Minuten und (mit etwas Übung) auch nicht fehleranfällig.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants