Modify ↓

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#2191 closed Bug/Fehler (fixed)

Parameter-Reihenfolge korrigieren in /inc/product_redirect.inc.php

Reported by: noRiddle Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.7.0
Component: Shop Version: 2.0.6.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Torsten Riemer)

Wie auch in Ticket #2175 für eine andere Stelle angesprochen gibt es in /inc/product_redirect.inc.php eine deprecated Parameter-Reihenfolge in der Funktion.

function product_redirect_link($products_id = false, $current_link, $categories_id = 0) {

$current_link sollte als erster Parameter stehen, weil er nicht optionaler Parameter ist.

Ich bekomme da auf 8.0.17 (Zend: 4.0.17) eine Deprecated-Meldung im Browser angezeigt, allerdings nur beim allerersten Aufruf (strange). Kann an der Server-Config liegen.

 Deprecated: Required parameter $current_link follows optional parameter $products_id in PATH/inc/product_redirect.inc.php on line 17

Jedenfalls sollte man die Reihenfolge fixen.
Im oben verlinkten Ticket habe ich dazu die PHP-Doku zitiert.

Gruß,
noRiddle

Attachments (0)

Change History (3)

comment:1 by Torsten Riemer, 4 years ago

Description: modified (diff)
Milestone: → modified-shop-2.0.6.1

comment:2 by Gerhard Waldemair, 4 years ago

Resolution: → fixed
Status: new → closed

fixed in r13729

comment:3 by Torsten Riemer, 4 years ago

Die Reihenfolge wurde nicht verändert, aber der zweite Parameter $current_link ist nun auch optional, wodurch es zu keiner Meldung mehr kommt.

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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.