Modify ↓

Opened 10 years ago

Closed 9 years ago

#1172 closed Bug/Fehler (fixed)

Grundsätzlich nur PHP Zeitfunktionen verwenden

Reported by: Ronald Parcinski Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.3.0
Component: Shop Version: 2.0.2.2
Keywords: Cc:
Blocked By: Blocking:

Description

Hier wird der Vorschlag beschrieben:

Problem mit Passwort zurücksetzen Link.

​https://www.modified-shop.org/forum/index.php?topic=35614.msg337166#msg337166

Ich habe den SQL-Befehl now() einfach mit PHP date() ersetzt!
Dies erscheint mir dir sinnvollste lösung da der Zeitstempel unten ja auch wieder per PHP mit der Systemzeit verglichen wird!

In: password_double_opt.php
suchen nach:
password_request_time = now()
ersetzen mit:
password_request_time = '".date('Y-m-d H:i:00')."'

Attachments (0)

Change History (5)

comment:1 by Volker Strähle, 10 years ago

Das halte ich für falsch. Einzig richtig ist in meinen Augen ausschließlich mit UTC zu arbeiten. Hierfür eine zentrale Funktion z. B. in "xtc_datetime.inc.php" definieren und diese überall im Code nutzen. Die Datums/Zeitangaben für die Benutzeroberfläche kann man korrekt über ein Zeitzoneneinstellung (evtl. sogar Kundenabhängig) steuern.

comment:2 by Ronald Parcinski, 10 years ago

Problem ist doch MySQL Funktion now() und PHP Funktion date() wenn der Datenbankserver eine andere Zeit wie der PHP Server hat.

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

comment:3 by Volker Strähle, 10 years ago

Statt now muss man natürlich utc_timestamp nutzen.

Im übrigen kann man auch now nutzen und ens SQL-Server alle Daten liefern lassen:

SELECT `password_request_time`, now() as currentservertime FROM `customers`

Beide Werte vergleichen und fertig.

comment:4 by Ronald Parcinski, 9 years ago

Da für password_request_time nur die Zeitdifferenz ausschlaggebend ist, muss sich die Berechnung auf die gleiche Basis beziehen, egal welche das auch ist.

Den obigen vorgeschlagenen FIX für password_double_opt.php habe ich jetzt übernommen.

comment:5 by Ronald Parcinski, 9 years ago

Resolution: → fixed
Status: new → closed

In 10753:

fix #1172

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.