Modify

Opened 8 years ago

Closed 6 years ago

#1430 closed Bug/Fehler (fixed)

Sprachkonstanten Versandmodul freeamount

Reported by: info@… 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 Torsten Riemer)

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 Torsten Riemer, 8 years ago

Description: modified (diff)
Milestone: modified-shop-2.0.4.0
Version: 2.0.3.0

comment:2 by Torsten Riemer, 8 years ago

Owner: set to Torsten Riemer
Resolution: fixed
Status: newclosed

In 11108:

fix #1430

comment:3 by Torsten Riemer, 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);
Last edited 8 years ago by Torsten Riemer (previous) (diff)

comment:4 by info@…, 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

in reply to:  4 comment:5 by Torsten Riemer, 8 years ago

Resolution: fixed
Status: closedreopened

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 Torsten Riemer, 8 years ago

Milestone: modified-shop-2.0.4.0modified-shop-2.0.4.1

comment:7 by Torsten Riemer, 7 years ago

Milestone: modified-shop-2.0.4.1modified-shop-2.0.5.0

comment:8 by Gerhard Waldemair, 7 years ago

Resolution: wontfix
Status: reopenedclosed

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 Gerhard Waldemair, 7 years ago

Milestone: modified-shop-2.0.5.0

comment:10 by Torsten Riemer, 7 years ago

Milestone: modified-shop-2.0.4.0

comment:11 by Torsten Riemer, 6 years ago

Resolution: wontfix
Status: closedreopened

comment:12 by Torsten Riemer, 6 years ago

Resolution: fixed
Status: reopenedclosed

Modify Ticket

Action
as closed The owner will remain Torsten Riemer.
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.