Opened 3 years ago

Closed 2 years ago

#1430 closed Bug/Fehler (fixed)

Sprachkonstanten Versandmodul freeamount

Reported by: info@… Owned by: Tomcraft
Priority: niedrig Milestone: modified-shop-2.0.4.0
Component: Module Version: 2.0.3.0

Description (last modified by Tomcraft)

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 Changed 3 years ago by Tomcraft

  • Description modified (diff)
  • Milestone set to modified-shop-2.0.4.0
  • Version set to 2.0.3.0

comment:2 Changed 3 years ago by Tomcraft

  • Owner set to Tomcraft
  • Resolution set to fixed
  • Status changed from new to closed

In 11108:

fix #1430

comment:3 Changed 3 years ago by Tomcraft

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 3 years ago by Tomcraft (previous) (diff)

comment:4 follow-up: Changed 3 years ago by info@…

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 in reply to: ↑ 4 Changed 3 years ago by Tomcraft

  • Resolution fixed deleted
  • Status changed from closed to 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 Changed 3 years ago by Tomcraft

  • Milestone changed from modified-shop-2.0.4.0 to modified-shop-2.0.4.1

comment:7 Changed 3 years ago by Tomcraft

  • Milestone changed from modified-shop-2.0.4.1 to modified-shop-2.0.5.0

comment:8 Changed 2 years ago by GTB

  • Resolution set to wontfix
  • Status changed from reopened to 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 Changed 2 years ago by GTB

  • Milestone modified-shop-2.0.5.0 deleted

comment:10 Changed 2 years ago by Tomcraft

  • Milestone set to modified-shop-2.0.4.0

comment:11 Changed 2 years ago by Tomcraft

  • Resolution wontfix deleted
  • Status changed from closed to reopened

comment:12 Changed 2 years ago by Tomcraft

  • Resolution set to fixed
  • Status changed from reopened to closed

Add Comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.