Modify

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#1677 closed Bug/Fehler (fixed)

product_reviews_info.php - Fehler in Prüfung auf Pflicht GET Parameter

Reported by: web-looks <info@…> Owned by: somebody
Priority: hoch Milestone: modified-shop-2.0.5.0
Component: Shop Version: 2.0.4.2
Keywords: Cc:
Blocked By: Blocking:

Description

Am Anfang der product_reviews_info.php wird geprüft ob 2 GET Parameter vorhanden sind, die später beide in einer DB Query genutzt werden. Die Umleitung auf die Haupt-Bewertungsseite erfolgt nur, wenn beide Parameter fehlen. Da beide Parameter aber für die Ausführung der Query erforderlich sind, muss die if-Bedingung geändert werden.

Suchen nach:

if (!isset($_GET['reviews_id']) && !isset($_GET['products_id'])) {
	xtc_redirect(xtc_href_link(FILENAME_REVIEWS, '', 'NONSSL'));
}

Ersetzen mit:

if (empty($_GET['reviews_id']) || empty($_GET['products_id'])) {
	xtc_redirect(xtc_href_link(FILENAME_REVIEWS, '', 'NONSSL'));
}

Attachments (0)

Change History (1)

comment:1 by Gerhard Waldemair, 6 years ago

Resolution: fixed
Status: newclosed

Das wurde bereits in r12026 für die kommende Shopversion korrigiert.

Last edited 6 years ago by Torsten Riemer (previous) (diff)

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.