Project

General

Profile

GX-Bug #51090

Nach dem Login zurück auf die vorher aufgerufene Seite

Added by Torben Wark about 5 years ago. Updated 11 months ago.

Status:
Erledigt
Priority:
Normal
Assignee:
Category:
Frontend
Target version:
% Done:

100%

Estimated time:
Steps to reproduce:
Betroffene Versionen:
3.10.0.0, 3.10.0.0 beta1, 3.10.0.1, 3.4.1.0, 3.4.1.1, 3.4.2.0, 3.4.2.0 beta1, 3.4.2.0 beta2, 3.4.3.0, 3.4.3.0 beta1, 3.5.1.0, 3.5.1.0 beta1, 3.5.1.0 beta2, 3.5.1.0 beta3, 3.5.1.0 beta4, 3.5.1.0 rc1, 3.5.1.1, 3.5.1.2, 3.5.2.0, 3.5.2.0 beta1, 3.5.2.0 beta2, 3.5.3.0, 3.5.3.0 beta1, 3.5.3.1, 3.6.0.0, 3.6.0.0 beta1, 3.6.0.1, 3.6.0.2, 3.6.1.0, 3.6.1.0 beta1, 3.6.1.0 beta2, 3.6.2.0, 3.6.2.0 beta1, 3.6.3.0, 3.6.3.0 beta1, 3.7.1.0, 3.7.1.0 RC1, 3.7.1.0 beta1, 3.7.1.0 beta2, 3.7.1.1, 3.7.1.2, 3.7.2.0, 3.7.2.0 beta1, 3.7.3.0, 3.7.3.0 beta1, 3.8.0.0, 3.8.0.0 beta1, 3.8.0.0 beta2, 3.8.0.0 beta3, 3.8.0.1, 3.8.0.2, 3.8.0.3, 3.8.0.4, 3.8.1.0, 3.8.1.0 beta1, 3.8.1.0 beta2, 3.8.1.0 beta3, 3.8.1.1, 3.8.1.2, 3.8.2.0, 3.8.2.1, 3.8.2.2, 3.8.3.0, 3.8.3.0 beta1, 3.8.3.1, 3.9.1.0, 3.9.1.0 beta1, 3.9.1.0 beta2, 3.9.1.0 beta3, 3.9.1.0 rc1, 3.9.1.0 rc2, 3.9.1.1, 3.9.1.2, 3.9.2.0, 3.9.2.0 beta1, 3.9.2.1, 3.9.2.2, 3.9.3.0, 3.9.3.0 beta1, 3.9.3.1
Release Notes Langtext:

Nach dem Login wird zurück auf die vorher aufgerufene Seite geleitet


Description

Wenn sich ein Kunde im Shop einloggt, landet er auf seiner Accountseite. Um hier dasShopping-Erlebnis zu beschleunigen, könnte es sinnvoll sein, den Kunden nach dem Login direkt auf die Seite zurückzuleiten, von der er den Login veranlasst hat. Dafür muss die LoginContentControl::proceed() angepasst werden. Hier finden sich folgende Zeilen:

if($_SESSION['cart']->count_contents() > 0)
                    {
                        if(isset($this->v_data_array['GET']['checkout_started'])
                           && $this->v_data_array['GET']['checkout_started'] == 1
                        )
                        {
                            $this->set_redirect_url(xtc_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'));
                        }
                        else
                        {
                            $this->set_redirect_url(xtc_href_link(FILENAME_ACCOUNT, '', 'SSL'));
                        }
                    }
                    else
                    {
                        $this->set_redirect_url(xtc_href_link(FILENAME_DEFAULT));
                    }

Wenn man dies nun zu folgendem ändert, hat man das gewünschte Verhalten:

if($_SESSION['cart']->count_contents() > 0)
                    {
                        if(isset($this->v_data_array['GET']['checkout_started'])
                           && $this->v_data_array['GET']['checkout_started'] == 1
                        )
                        {
                            $this->set_redirect_url(xtc_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'));
                        }
                        else
                        {
                            $this->set_redirect_url(xtc_href_link($_SERVER['HTTP_REFERER'], '', 'SSL', true, true, true));
                        }
                    }
                    else
                    {
                        $this->set_redirect_url(xtc_href_link($_SERVER['HTTP_REFERER'], '', 'NONSSL', true, true, true));
                    }

History

#1

Updated by Torben Wark about 5 years ago

  • Description updated (diff)
#2

Updated by Marco Bruchmann almost 5 years ago

$_SERVER['HTTP_REFERER'] ist als Datenquelle möglicherweise keine gute Idee. Es gibt etliche Privacy-Tools (Browsererweiterungen, AV-Software, …), die das Senden des Referrer entweder ganz unterbinden oder den Referrer-URL irgendwie verstümmeln.

#3

Updated by Marco Bruchmann almost 4 years ago

  • Tags set to MR
  • Tracker changed from GX-Feature to GX-Bug
  • Target version changed from 58 to 133
  • Betroffene Versionen 3.10.0.0, 3.10.0.0 beta1, 3.10.0.1, 3.4.1.1, 3.4.2.0, 3.4.2.0 beta1, 3.4.2.0 beta2, 3.4.3.0, 3.4.3.0 beta1, 3.5.1.0, 3.5.1.0 beta1, 3.5.1.0 beta2, 3.5.1.0 beta3, 3.5.1.0 beta4, 3.5.1.0 rc1, 3.5.1.1, 3.5.1.2, 3.5.2.0, 3.5.2.0 beta1, 3.5.2.0 beta2, 3.5.3.0, 3.5.3.0 beta1, 3.5.3.1, 3.6.0.0, 3.6.0.0 beta1, 3.6.0.1, 3.6.0.2, 3.6.1.0, 3.6.1.0 beta1, 3.6.1.0 beta2, 3.6.2.0, 3.6.2.0 beta1, 3.6.3.0, 3.6.3.0 beta1, 3.7.1.0, 3.7.1.0 RC1, 3.7.1.0 beta1, 3.7.1.0 beta2, 3.7.1.1, 3.7.1.2, 3.7.2.0, 3.7.2.0 beta1, 3.7.3.0, 3.7.3.0 beta1, 3.8.0.0, 3.8.0.0 beta1, 3.8.0.0 beta2, 3.8.0.0 beta3, 3.8.0.1, 3.8.0.2, 3.8.0.3, 3.8.0.4, 3.8.1.0, 3.8.1.0 beta1, 3.8.1.0 beta2, 3.8.1.0 beta3, 3.8.1.1, 3.8.1.2, 3.8.2.0, 3.8.2.1, 3.8.2.2, 3.8.3.0, 3.8.3.0 beta1, 3.8.3.1, 3.9.1.0, 3.9.1.0 beta1, 3.9.1.0 beta2, 3.9.1.0 beta3, 3.9.1.0 rc1, 3.9.1.0 rc2, 3.9.1.1, 3.9.1.2, 3.9.2.0, 3.9.2.0 beta1, 3.9.2.1, 3.9.2.2, 3.9.3.0, 3.9.3.0 beta1, 3.9.3.1 added
#4

Updated by Torben Wark almost 4 years ago

  • Status changed from Gemeldet to Testing
  • Assignee set to Torben Wark
  • Target version changed from 133 to 3.11.1.0 beta1
  • Release Notes Langtext updated (diff)
#5

Updated by Torben Wark almost 4 years ago

  • Status changed from Testing to Erledigt
  • % Done changed from 0 to 100
#6

Updated by Moritz Bunjes 11 months ago

  • Tags deleted (MR)

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 5 MB)