Hilfe MMuM upgrade php auf 8.1

Hier kommt alles rein, was woanders keinen Platz hat.
Antworten
Benutzeravatar
muldengold
Posting Freak
Posting Freak
Beiträge: 1434
Registriert: Montag 25. Oktober 2010, 16:54

Hilfe MMuM upgrade php auf 8.1

#1

Beitrag von muldengold »

Hallo an die php-Experten unter euch,

ich brauche mal professionelle Hilfe hinsichtlich der MMuM-Seite:

ein upgrade von php von derzeit 7.4 auf 8.1 erzeugt folgenden Fehler:

Fatal error: Uncaught Error: Undefined constant "id" in /homepages/0/d436750884/htdocs/maischemalzundmehr/lr.php:327 Stack trace: #0 /homepages/0/d436750884/htdocs/maischemalzundmehr/index.php(201): include() #1 {main} thrown in /homepages/0/d436750884/htdocs/maischemalzundmehr/lr.php on line 327

Der betreffende code auf line 327 ist eigentlich eine ganz normale Datenbankabrage in einer while-Schleife:

Code: Alles auswählen

while($dbsatz = mysqli_fetch_assoc($res)) {
  $id=$dbsatz[id];
Hab jetzt wieder auf 7.4 zurückgestellt, irgendwann muss ich aber upgraden. Kann mir da evtl. jemand weiterhelfen?? Danke schon mal.

LG
Sandro
Wo ein Wille ist, da ist auch ein Bier! :Wink
IronHosch
Posting Senior
Posting Senior
Beiträge: 368
Registriert: Mittwoch 9. März 2022, 10:45

Re: Hilfe MMuM upgrade php auf 8.1

#2

Beitrag von IronHosch »

Müssen da nicht Anführungszeichen hin?

Code: Alles auswählen

  $id=$dbsatz["id"];
ludiwawa
Posting Klettermax
Posting Klettermax
Beiträge: 119
Registriert: Montag 19. Dezember 2022, 18:10

Re: Hilfe MMuM upgrade php auf 8.1

#3

Beitrag von ludiwawa »

ja, die Anführungszeichen fehlen, ausserdem auf uninitialisierte Variablen achten und define(CONST) muss jetzt auch define("CONST") heissen.
php-migration is pain in the ass
Benutzeravatar
muldengold
Posting Freak
Posting Freak
Beiträge: 1434
Registriert: Montag 25. Oktober 2010, 16:54

Re: Hilfe MMuM upgrade php auf 8.1

#4

Beitrag von muldengold »

Mhhh, danke Leute. Evtl. wurden die fehlenden Gänsefüßchen bis php 7.4 toleriert? Die entsprechenden Programmteile sind tlw. viele Jahre alt. Mir schwant da Schlimmes, da der code voll ist mit fehlenden Gänsefüßchen.
Wo ein Wille ist, da ist auch ein Bier! :Wink
ludiwawa
Posting Klettermax
Posting Klettermax
Beiträge: 119
Registriert: Montag 19. Dezember 2022, 18:10

Re: Hilfe MMuM upgrade php auf 8.1

#5

Beitrag von ludiwawa »

muldengold hat geschrieben: Donnerstag 9. Februar 2023, 13:25 Mhhh, danke Leute. Evtl. wurden die fehlenden Gänsefüßchen bis php 7.4 toleriert? Die entsprechenden Programmteile sind tlw. viele Jahre alt. Mir schwant da Schlimmes, da der code voll ist mit fehlenden Gänsefüßchen.
Ja, 7.x hat noch ne menge verziehen - ab 8.1 wirds strikter. Auf gehts - Coding-Mucke auf die Ohren und Fleissarbeit machen :D
Und bei einer leckeren Flasche selbstgebrautem mal die Versionsinfos zu php9 lesen - da kann man ggf schonmal vorarbeiten.
hölsch
Posting Klettermax
Posting Klettermax
Beiträge: 196
Registriert: Freitag 5. November 2021, 12:04

Re: Hilfe MMuM upgrade php auf 8.1

#6

Beitrag von hölsch »

Ich helfe gerne, gibts nen Repo / Development Umgebung die sich flott aufsetzen lässt?
Dann kann ich gerne kommende Woche mal Abends bisschen Fleißarbeit machen.

LG
LG
Jan
Sude: 26 ~ Equipment: 3.5kW Yato und 34L+57L Topf. Letzter Sud: House Ale v3 Nächster Sud: noch keine Ahnung.
Benutzeravatar
muldengold
Posting Freak
Posting Freak
Beiträge: 1434
Registriert: Montag 25. Oktober 2010, 16:54

Re: Hilfe MMuM upgrade php auf 8.1

#7

Beitrag von muldengold »

ludiwawa hat geschrieben: Donnerstag 9. Februar 2023, 13:44 Ja, 7.x hat noch ne menge verziehen - ab 8.1 wirds strikter. Auf gehts - Coding-Mucke auf die Ohren und Fleissarbeit machen :D
Und bei einer leckeren Flasche selbstgebrautem mal die Versionsinfos zu php9 lesen - da kann man ggf schonmal vorarbeiten.
:Smile
hölsch hat geschrieben: Donnerstag 9. Februar 2023, 14:10 Ich helfe gerne, gibts nen Repo / Development Umgebung die sich flott aufsetzen lässt?
Dann kann ich gerne kommende Woche mal Abends bisschen Fleißarbeit machen.
Ich könnte Dir gern z.B. Programmteile zukommen lassen, wäre total nett. Wenn da mal jemand mit Sachverstand über meinen Laien-Code drüber schaut wäre das nicht schlecht. Kannst mir ja mal eine PM (DM?) schicken wenn Du Zeit dafür hast. Danke!
Wo ein Wille ist, da ist auch ein Bier! :Wink
Benutzeravatar
muldengold
Posting Freak
Posting Freak
Beiträge: 1434
Registriert: Montag 25. Oktober 2010, 16:54

Re: Hilfe MMuM upgrade php auf 8.1

#8

Beitrag von muldengold »

Ich habe jetzt auf php 8.2 umgestellt und offenbar scheint alles zu laufen. php 8.2 support läuft bis 12 2025, sprich da haben wir erst einmal knapp drei Jahre Ruhe. Ich hab viel gelernt und wirklich viel Hilfe im Hintergrund erfahren - vielen, vielen Dank Leute!!!

Bei offensichtlichen Fehlern, die mit der Umstellung zu tun haben könnten, (sprich, alles was z.B. mit "Fatal error: Uncaught Error: " anfängt, gern hier.

Sandro
Wo ein Wille ist, da ist auch ein Bier! :Wink
HubertBräu
Posting Freak
Posting Freak
Beiträge: 1480
Registriert: Montag 27. März 2017, 11:56
Wohnort: Berlin

Re: Hilfe MMuM upgrade php auf 8.1

#9

Beitrag von HubertBräu »

Ich wollte gerade die IBUs für ein neues Rezept durchrechnen. Da kommt diese Fehlermeldung:
2023-03-05 17_50_00-MaischeMalzundMehr und 2 weitere Seiten - Profil 1 – Microsoft​ Edge.jpg
Gruß & Danke.
Hubert
Benutzeravatar
Braufex
Posting Freak
Posting Freak
Beiträge: 2512
Registriert: Montag 6. August 2018, 22:22
Wohnort: Kreis Augsburg

Re: Hilfe MMuM upgrade php auf 8.1

#10

Beitrag von Braufex »

muldengold hat geschrieben: Sonntag 5. März 2023, 17:19 Bei offensichtlichen Fehlern, die mit der Umstellung zu tun haben könnten, (sprich, alles was z.B. mit "Fatal error: Uncaught Error: " anfängt, gern hier.
Servus Sandro,
auch beim Aufruf des Wasserrechners:
Fatal error: Uncaught TypeError: Unsupported operand types: string - string in /homepages/0/d436750884/htdocs/maischemalzundmehr/tools_wasserrechner.php:79 Stack trace: #0 /homepages/0/d436750884/htdocs/maischemalzundmehr/index.php(229): include() #1 {main} thrown in /homepages/0/d436750884/htdocs/maischemalzundmehr/tools_wasserrechner.php on line 79
Beim Aufruf der Kontodaten:
Fatal error: Uncaught Error: Undefined constant "email" in /homepages ...
Viel Erfolg bei der Fehlersuche
Gruß Erwin
lonestar
Posting Junior
Posting Junior
Beiträge: 55
Registriert: Dienstag 31. Mai 2022, 09:15

Re: Hilfe MMuM upgrade php auf 8.1

#11

Beitrag von lonestar »

Ja. PHP 8.2 ist da ziemlich strikt. Muss bestimmt gecastet werden nach Int.
Benutzeravatar
muldengold
Posting Freak
Posting Freak
Beiträge: 1434
Registriert: Montag 25. Oktober 2010, 16:54

Re: Hilfe MMuM upgrade php auf 8.1

#12

Beitrag von muldengold »

Kontodaten funktioniert wieder. Der IBU-Rechner treibt mich in den Wahnsinn - leider. Ich kümmere mich zeitnah.
Wo ein Wille ist, da ist auch ein Bier! :Wink
HubertBräu
Posting Freak
Posting Freak
Beiträge: 1480
Registriert: Montag 27. März 2017, 11:56
Wohnort: Berlin

Re: Hilfe MMuM upgrade php auf 8.1

#13

Beitrag von HubertBräu »

Danke für deinen Einsatz :Drink
HubertBräu
Posting Freak
Posting Freak
Beiträge: 1480
Registriert: Montag 27. März 2017, 11:56
Wohnort: Berlin

Re: Hilfe MMuM upgrade php auf 8.1

#14

Beitrag von HubertBräu »

muldengold hat geschrieben: Sonntag 5. März 2023, 20:13 Kontodaten funktioniert wieder. Der IBU-Rechner treibt mich in den Wahnsinn - leider. Ich kümmere mich zeitnah.
Die Fehlermeldung kommt jetzt nicht mehr.
Aber egal welche Werte man eingibt, es kommt immer 0 IBU bei der Berechnung raus. Bis Du da noch dran?

Gruß
Hubert
lonestar
Posting Junior
Posting Junior
Beiträge: 55
Registriert: Dienstag 31. Mai 2022, 09:15

Re: Hilfe MMuM upgrade php auf 8.1

#15

Beitrag von lonestar »

muldengold hat geschrieben: Sonntag 5. März 2023, 20:13 Kontodaten funktioniert wieder. Der IBU-Rechner treibt mich in den Wahnsinn - leider. Ich kümmere mich zeitnah.
Wenn nicht, schick mal den Code. Dann schau ich mal drüber
Benutzeravatar
muldengold
Posting Freak
Posting Freak
Beiträge: 1434
Registriert: Montag 25. Oktober 2010, 16:54

Re: Hilfe MMuM upgrade php auf 8.1

#16

Beitrag von muldengold »

lonestar hat geschrieben: Dienstag 7. März 2023, 11:45
muldengold hat geschrieben: Sonntag 5. März 2023, 20:13 Kontodaten funktioniert wieder. Der IBU-Rechner treibt mich in den Wahnsinn - leider. Ich kümmere mich zeitnah.
Danke. Komme soweit zurecht, manche Sachen dauern leider etwas.
Wenn nicht, schick mal den Code. Dann schau ich mal drüber
Die Seite ging deshalb kurzeitig wieder, da ich zwischenzeitlich auf php 7.4 zurückgestellt hatte. Bin jetzt wieder auf 8.2 und bis auf die Exportfunktion sollte eigentlich (füge Bet-emoji ein) alles, einschließlich der IBU und Wasserrechner, klappen. Um die Exportfunktion kümmere ich mich zeitnah.

Sandro
Wo ein Wille ist, da ist auch ein Bier! :Wink
HubertBräu
Posting Freak
Posting Freak
Beiträge: 1480
Registriert: Montag 27. März 2017, 11:56
Wohnort: Berlin

Re: Hilfe MMuM upgrade php auf 8.1

#17

Beitrag von HubertBräu »

Vielen Dank Sandro :thumbup
Antworten