Opened 8 years ago
Closed 6 years ago
#1430 closed Bug/Fehler (fixed)
Sprachkonstanten Versandmodul freeamount
| Reported by: | Owned by: | Torsten Riemer | |
|---|---|---|---|
| Priority: | niedrig | Milestone: | modified-shop-2.0.4.0 |
| Component: | Module | Version: | 2.0.3.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description (last modified by )
Hallo,
mir fiel gerade auf, dass im Versandkosten-Modul freeamount die Sprach-Konstanten "MODULE_SHIPPING_FREEAMOUNT_DISPLAY_TITLE" und "MODULE_SHIPPING_FREEAMOUNT_DISPLAY_DESC" doppelt definiert sind.
Vermutlich sind die zuletzt definierten die Richtigen.
Weiterhin kommt es zu unschönen Fehlern, wenn die Sprachdatei aus einem Script eingebunden wird, welches zuvor nicht die komplette Konfiguration des Shops in Konstanten umgewandelt hat, da dann die Konstante "MODULE_SHIPPING_FREEAMOUNT_NUMBER_ZONES" fehlt.
Der JTL-Connector beispielsweise liest nur die einzelne Sprachdatei der Versandart ein, ohne die Konfiguration geladen zu haben.
Ich empfehle in der Sprachdatei, auf das Vorhandensein der Konstante zu prüfen und diese wenn nicht vorhanden, mit dem Wert 1 zu belegen.
Viele Grüße
Andreas
Attachments (0)
Change History (12)
comment:1 by , 8 years ago
| Description: | modified (diff) |
|---|---|
| Milestone: | → modified-shop-2.0.4.0 |
| Version: | → 2.0.3.0 |
comment:2 by , 8 years ago
| Owner: | set to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:3 by , 8 years ago
Wir werden keine für uns unnötigen Korrekturen in unsere Dateien einbauen, damit uralte Schnittstellen weiterhin funktionieren, die man auch ganz einfach selber aktualisieren könnte mit:
// load configuration
define('TABLE_CONFIGURATION', 'configuration');
$configuration_query = xtc_db_query('SELECT configuration_key, configuration_value FROM '.TABLE_CONFIGURATION);
while ($configuration = xtc_db_fetch_array($configuration_query)) {
if (function_exists('extra_configuration')) extra_configuration();
defined($configuration['configuration_key']) OR define($configuration['configuration_key'], stripslashes($configuration['configuration_value']));
}
Im Falle von JTL fügt man in die Datei "/api/jtlwawi_connector/dbeS/syncinclude.php" ein nach:
define ('ES_ENABLE_LOGGING',0);
define ('ES_ATTRIBUTE_AN_BESCHREIBUNG_ANHAENGEN',1);
follow-up: 5 comment:4 by , 8 years ago
Danke.
So weit, so gut.
Deine Anleitung bezieht sich anscheinend auf eine unglaublich alte Version des Connectors.
Im neuen sieht es anders aus und die Definition der Number-Konstanten (wenn nicht vorhanden) in der Sprachdatei ist der einfachste Weg, die Fehler zu korrigieren, ohne den phar-Gepackten Connector anpassen zu müssen. ein weitere Vorteil ist, dass man die Sprachdateien auch dann fehlerfrei verwenden kann, wenn das Modul möglicherweise schon wieder deinstalliert wurde, aber in alten Bestellungen noch vorhanden ist.
Was den Connector betrifft, werde ich wohl im laufe des Jahres ein paar Plugins dafür veröffentlichen, sodass darüber auch die PayPal-Plus-Rechnungsinfos an die JTL-Wawi übertragen werden, oder Varianten-Kombinationen abgebildet werden können. Beides läuft seit Jahresbeginn stabil, ich muss es nur noch "schön" machen. Aber das wird nun etwas off-Topic.
Viele Grüße
comment:5 by , 8 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Replying to info@…:
[...] wenn das Modul möglicherweise schon wieder deinstalliert wurde, aber in alten Bestellungen noch vorhanden ist.
[...]
Das ist in der Tat zu prüfen.
comment:6 by , 8 years ago
| Milestone: | modified-shop-2.0.4.0 → modified-shop-2.0.4.1 |
|---|
comment:7 by , 7 years ago
| Milestone: | modified-shop-2.0.4.1 → modified-shop-2.0.5.0 |
|---|
comment:8 by , 7 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | reopened → closed |
Wenn das Modul in der Zwischenzeit deinstalliert wurde, werden nicht die Beschreibungen der einzelnen Zonen benötigt. Nach aktuellem Stand kann die Sprachdatei includiert werden, ohne dass es zu Fehlern kommt.
comment:9 by , 7 years ago
| Milestone: | modified-shop-2.0.5.0 |
|---|
comment:10 by , 7 years ago
| Milestone: | → modified-shop-2.0.4.0 |
|---|
comment:11 by , 6 years ago
| Resolution: | wontfix |
|---|---|
| Status: | closed → reopened |
comment:12 by , 6 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |

In 11108: