Opened 10 months ago

Closed 10 months ago

Last modified 10 months 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

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 Changed 10 months ago by GTB

  • Resolution set to fixed
  • Status changed from new to closed

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

Last edited 10 months ago by Tomcraft (previous) (diff)

Add Comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.