Modify

Opened 8 years ago

Last modified 8 years ago

#1410 new Bug/Fehler

application_top-php generell mit require einbinden

Reported by: Volker Strähle Owned by: somebody
Priority: normal Milestone:
Component: Sicherheit Version: 2.0.3.0
Keywords: Cc:
Blocked By: Blocking:

Description

Ich sehe, dass die Einbindung der application_top.php nicht einheitlich ist.

in der index.php wird das meiner Meinung nach richtig mit require eingebunden.

In checkout_confirmation.php dagegen nur mit include.

Mit Hinweis auf https://www.modified-shop.org/forum/index.php?topic=36934.msg348731#msg348731 wäre ich generell dafür eine so wichtige Datei mit require_once einzubinden.

Ein Fehlen oder ein Ladefehler der application_top sollte zum Programmabbruch führen und nicht dazu das der Code trotzdem weiter ausgeführt werden kann.

Die checkout_confirmation ist übrigens nicht die einzige bei der das nur mit include eingebunden wird.

Attachments (0)

Change History (1)

comment:1 by Torsten Riemer, 8 years ago

Milestone: modified-shop-2.0.4.0

Modify Ticket

Action
as new The owner will remain somebody.

Add Comment


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