Mal kurz geschaut und einige nicht so schöne sachen gefunden.
Meine Meinung
id="eanhinweis13stelligmodalschliessen"
Ist elaubt , kann man aber so sehr schlecht lesen.Bitte kürzere id wählen , oder so machen
id="ean_hinweis_13_stellig_modal_schliessen"
Inline-style versuchen zu vemeiden
<div id="eanhinweis13stelligmodal" class="modal" style="display: block;">
besser mit klassen
<style>
.d_block{
display: block;
}
</style>
<div id="eanhinweis13stelligmodal" class="modal d_block">
Du hast hier
<div id="eanhinweis13stelligmodal" class="modal" style="display: block;">
und hier
var eanhinweis13stelligmodal = document.querySelector('#eanhinweis13stelligmodal');
eine ID , die auch als Variable benutzt wird.
Das geht fast immer schief.
Nutze da irgendwelchen anderen Namen, nur nie die gleichen, auch Funktionen sollten nicht unbedingt wie eine ID oder Variable heißen.
Da hatten schon mehrere mit Probleme.
De Code
document.querySelector('#hinzufuegenbutton').onclick = function() {
,,,,
Erzeugt ein Error, wenn kein Button erzeugt wurde.
Mach es anders. Lass dir ein Template einfallen, wo die Buttons beim Seitenladen alle im Element, stehen ( display:none).
Oder
Erzeuge das Javascript nur, wenn es den Button auch gibt.
Das hast du bei anderen Elementen auch so.
Dann würde in noch onclick in addEventListener ändern.
Dass du so viele einzelne Datei nutzt , verstehe ich nicht ganz. Ich finde es zu viel.
Die PHP Profis können da bestimmt mehr zu sagen, ob das überhaupt sinnvoll ist zu viele Einzel Dateien einzubinden.
Das war es erstmal,
das dürfte dann schon reichen -
Bitte den Hinweis von tk1234 auch beachten