Modify

Opened 3 years ago

Closed 3 years ago

#2397 closed Bug/Fehler (duplicate)

Aktivieren/Deaktivieren eines Banners/Sliders führt zu warning mit PHP 7.4

Reported by: Timm Owned by: somebody
Priority: normal Milestone:
Component: Admin Version: 2.0.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

Folgendes admin_warning.log entsteht, wenn man auf den grünen oder roten Statusbutton eines Banners/Sliders klickt im Banner Manager.

[2022-11-26 19:21:51]	[warning]	[modified]	[pid:9499]	imagefilledpolygon() expects exactly 4 parameters, 3 given in File: /admin/includes/classes/phplot.php on Line: 6068	{}	{}
[2022-11-26 19:21:51]	[warning]	[modified]	[pid:9499]	Backtrace #0 - /admin/includes/graphs/banner_infobox.php called at Line 50	{}	{}
[2022-11-26 19:21:51]	[warning]	[modified]	[pid:9499]	Backtrace #1 - /dadmin/banner_manager.php called at Line 624	{}	{}

Der Fehler entsteht unter PHP 7.4 jedoch nicht unter PHP 8.1.

Da PHP 7.4 EOL ist wäre die Frage, ob man das noch fixed oder so lässt.

Attachments (0)

Change History (5)

comment:1 by Torsten Riemer, 3 years ago

Priority: niedrignormal

Ich kann mir nicht vorstellen, dass der Fehler mit PHP 8.1 nicht mehr auftreten soll. Fehler der Art "expects exactly 4 parameters, 3 given in File" gehören immer korrigiert.

comment:2 by noRiddle, 3 years ago

In diesem Fall kann das schon sein, wenn man die für dieses Problem nicht gerade verständlich beschriebene Problematik im PHP-Manual: imagefilledpolygon liest.
Wenn ich das richtig verstehe gilt ab PHP 8.1.0 die Version mit vier Parametern als veraltet.
Bereits für PHP 8.0 heißt es im Manual über den dritten Parameter "num_points"
"Der Parameter num_points ist veraltet und sollte nicht mehr verwendet werden."

Ich denke es lohnt sich nicht da wegen eines Warnings noch einen Switch mit Abfrage der PHP-Version einzubauen.

Gruß,
noRiddle

comment:3 by Gerhard Waldemair, 3 years ago

Milestone: modified-shop-2.0.8.0
Resolution: wontfix
Status: newclosed

wie noRiddle beschrieben hat, wäre hier eine Routine für frühere PHP Versionen notwendig was wir hier unterlassen und auf PHP8 setzen.

comment:4 by Torsten Riemer, 3 years ago

Resolution: wontfix
Status: closedreopened

comment:5 by Torsten Riemer, 3 years ago

Resolution: duplicate
Status: reopenedclosed

Duplikat von Ticket #2334.

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.