Project

General

Profile

GX-Bug #68427

ThemeContentView doesn't use the MainFactory to load Smarty / ThemeContentView nutzt nicht die MainFactory, um Smarty zu laden

Added by Timo Backhaus over 1 year ago. Updated 11 months ago.

Status:
Erledigt
Priority:
Hoch
Assignee:
Category:
Core
Target version:
% Done:

0%

Estimated time:
Steps to reproduce:
Betroffene Versionen:
4.3.3.1, 4.4.0.0
Release Notes Langtext:

ThemeContentView benutzt jetzt die MainFactory, um Smarty zu laden.


Description

In the Theme System you cannot overload Smarty because the ThemeContentView class creates Smarty directly (without MainFactory). This means that overloading of the Smarty class is not possible.

Steps to reproduce:

  1. Create an overload of GXSmarty
  2. Activate a theme
  3. Open the shop frontend
  4. Check, if the overload is loaded

Expected behavior: The overload is loaded

Actual behavior: The overload isn't loaded

Solution: Use the MainFactory to load the GXSmarty class
GXMainComponents/View/ThemeContentViews/core/ThemeContentView.inc.php

$this->v_coo_smarty = MainFactory::create('GXSmarty');

========================================================================================

Im Theme System kann man Smarty nicht überladen, weil die ThemeContentView Klasse Smarty direkt erzeugt (ohne MainFactory). Dadurch ist kein Overloading der Smarty Klasse möglich.

Schritte zum Reproduzieren:

  1. Erzeuge einen Overload der GXSmarty Klasse
  2. Aktiviere ein Theme
  3. Öffne den Shopbereich
  4. Prüfe, ob der Overload geladen wird

Erwartetes Verhalten: Der Overload wird geladen

Aktuelles Verhalten: Der Overload wird nicht geladen

Lösung: Verwende die MainFactory, um die GXSmarty Klasse zu laden
GXMainComponents/View/ThemeContentViews/core/ThemeContentView.inc.php

$this->v_coo_smarty = MainFactory::create('GXSmarty');

History

#1

Updated by Timo Backhaus over 1 year ago

  • Priority changed from Dringend to Hoch
#2

Updated by Ulrich Wenk over 1 year ago

  • Status changed from Gemeldet to Anerkannt
#3

Updated by Hilke Müller over 1 year ago

  • Tags set to 4.4.0.1
#4

Updated by Marvin Muxfeld over 1 year ago

  • Status changed from Anerkannt to In Bearbeitung
  • Assignee set to Marvin Muxfeld
#5

Updated by Marvin Muxfeld over 1 year ago

  • Status changed from In Bearbeitung to 30
  • Assignee changed from Marvin Muxfeld to Core-Entwickler
  • Target version changed from 133 to 4.4.0.1
  • Release Notes Langtext updated (diff)
  • Betroffene Versionen 4.4.0.0 added
#6

Updated by Ahmed Hamouda over 1 year ago

  • Assignee changed from Core-Entwickler to Ahmed Hamouda
#7

Updated by Ahmed Hamouda over 1 year ago

  • Status changed from 30 to Testing
#8

Updated by Ahmed Hamouda over 1 year ago

  • Assignee changed from Ahmed Hamouda to Core-Entwickler
#9

Updated by Moritz Bunjes about 1 year ago

  • Assignee changed from Core-Entwickler to Moritz Bunjes
#10

Updated by Moritz Bunjes about 1 year ago

  • Release Notes Langtext updated (diff)
#11

Updated by Moritz Bunjes about 1 year ago

  • Status changed from Testing to Erledigt
#12

Updated by Moritz Bunjes 11 months ago

  • Tags deleted (4.4.0.1)

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 5 MB)