modified eCommerce Shopsoftware 2.0.5.0 rev 12487 veröffentlicht

[karsta.de]

FIX zum Fix aus Ticket #1492 - Lieferadresse bei Selbstabholung
Leider ist bei der Versandart Selbstabholung in der neuen Shopversion noch ein kleiner Fehler drin.
In der checkout_confirmation wird zwar Abholadresse als Überschrift geschrieben, aber die Adresse des Kunden und nicht die wirkliche Abholadresse angezeigt.
Mein Vorschlag:
Im shoproot/lang/SPRACHE/extra Ordner eine neue Datei "selfpickup.php" mit der Variable SELFPICKUP_LABEL hinterlegen, diese kann dann im Template verwendet kann. Ausgabe ist die Adresse, die der Shopbetreiber im Versandmodul Selbstabholung hinterlegt hat.

NEU: selfpickup.php

Code: PHP  [Auswählen]

<?php
if (MODULE_SHIPPING_SELFPICKUP_SUBURB != ''){
$adresszusatz = '<br />'.MODULE_SHIPPING_SELFPICKUP_SUBURB;
}
require_once(DIR_FS_INC . 'xtc_get_country_name.inc.php');
 define('SELFPICKUP_LABEL', MODULE_SHIPPING_SELFPICKUP_COMPANY.'<br />'.MODULE_SHIPPING_SELFPICKUP_FIRSTNAME.' '.MODULE_SHIPPING_SELFPICKUP_LASTNAME.'<br />'.MODULE_SHIPPING_SELFPICKUP_STREET_ADDRESS.$adresszusatz.MODULE_SHIPPING_SELFPICKUP_SUBURB.'<br />'.MODULE_SHIPPING_SELFPICKUP_POSTCODE.' '.MODULE_SHIPPING_SELFPICKUP_CITY. '<br />' .xtc_get_country_name(MODULE_SHIPPING_SELFPICKUP_COUNTRY));

in der root/templates/MEIN_TEMP/module/checkout_confirmation.html
Suchen:

Code: PHP  [Auswählen]

<h4>{if isset($SHIPPING_CLASS) && strstr($SHIPPING_CLASS, 'selfpickup')}{#text_pickup_address#}{else}{#text_shipping_address#} <a class="midi" href="{$SHIPPING_ADDRESS_EDIT}">{#text_edit#}</a>{/if}</h4>
   {$DELIVERY_LABEL}

Ersetzen:

Code: PHP  [Auswählen]

{if isset($SHIPPING_CLASS) && strstr($SHIPPING_CLASS, 'selfpickup')}
<h4>{#text_pickup_address#}</h4>
{$smarty.const.SELFPICKUP_LABEL}
{else}
<h4>{#text_shipping_address#} <a class="midi" href="{$SHIPPING_ADDRESS_EDIT}">{#text_edit#}</a></h4>
{$DELIVERY_LABEL}
{/if}

BG Karsta

[Tomcraft]

Stimmt, da klemmt es noch! Habe ich in Ticket #1492 ergänzt und es erneut geöffnet.
Stimmt es sonst an allen anderen Stellen im Shop und in den Bestell-Mails?

Grüße

Torsten

[karsta.de]

In den Bestellmails und auch im Admin wird alles korrekt übergeben.
Bisher konnte ich dazu keine weiteren Fehler entdecken.

BG Karsta

[Tomcraft]

Und in der Bestell-Historie der Kunden unter "Mein Konto"?

Grüße

Torsten

[karsta.de]

Auch korrekt.

[Tomcraft]

Danke dir für's testen!

Grüße

Torsten

[Andymax55]

Hallo,

Kann mir jemand helfen bei Update auf die Version 2.0.5.0 bekomme ich folgende Fehlermeldung:

Code: PHP  [Auswählen]

Fatal error: require_once(): Failed opening required '/www/htdocs/w00f1615/ratgeber-ebook-shop.de/includes/external/phpfastcache/phpfastcache.php' (include_path='.:/usr/share/php:..') in /www/htdocs/w00f1615/ratgeber-ebook-shop.de/includes/external/smarty/smarty_3/sysplugins/smarty_cacheresource_phpfastcache.php on line 17

Code: PHP  [Auswählen]

<?php

/**
 * phpFastCache CacheResource
 *
 * CacheResource Implementation based on the KeyValueStore API to use
 * phpFastCache as the storage resource for Smarty's output caching.
 * *
 * @package CacheResource-examples
 */
class Smarty_CacheResource_Phpfastcache extends Smarty_CacheResource_KeyValueStore {

    protected $phpfastcache = null;

    public function __construct()
    {
        require_once (DIR_FS_EXTERNAL . 'phpfastcache/phpfastcache.php');
        $this->phpfastcache = phpFastCache();        
    }

    /**
     * Read values for a set of keys from cache
     *
     * @param array $keys list of keys to fetch
     * @return array list of values with the given keys used as indexes
     * @return boolean true on success, false on failure
     */
    protected function read(array $keys)
    {
        $_keys = $_res = array();
        foreach ($keys as $k) {
            $_k = sha1($k);
            $_res[$k] = $this->phpfastcache->get($_k);
        }

        return $_res;
    }

    /**
     * Save values for a set of keys to cache
     *
     * @param array $keys list of values to save
     * @param int $expire expiration time
     * @return boolean true on success, false on failure
     */
    protected function write(array $keys, $expire=null)
    {
       foreach ($keys as $k => $v) {
            $k = sha1($k);
            $this->phpfastcache->set($k, $v, $expire);
        }
        return true;
    }

    /**
     * Remove values from cache
     *
     * @param array $keys list of keys to delete
     * @return boolean true on success, false on failure
     */
    protected function delete(array $keys)
    {
        foreach ($keys as $k) {
            $this->phpfastcache->delete($k);
        }
        return true;
    }

    /**
     * Remove *all* values from cache
     *
     * @return boolean true on success, false on failure
     */
    protected function purge()
    {
        return $this->phpfastcache->clean();
    }
}

[EDIT Tomcraft 06.01.2020: Fehlermeldung & Code formatiert.]

[Timm]

Sieh dir mal die Frage und darauf folgenden Antworten aus Antwort #82 an.

Gruß Timm

[karsta.de]

HINWEIS:
Bei einem Shop-Update von Version 2.02.1 auf Version 2.0.5.0 ist aufgefallen, dass die Einstellung durch das Update zur USt-IdNr.  > Kundengruppe nach USt-IdNr. Check anpassen? > auf "JA" gestellt wurde.

Für alle die diese Einstellungen nicht möchten, unbedingt nach dem Update kontrollieren.

BG Karsta

[Tomcraft]

Ganz sicher nicht! Such mal im Shopordner nach "ACCOUNT_COMPANY_VAT_GROUP". Es werden keinerlei SQL-Befehle ausgeführt, die diese Einstellung von "Nein" auf "Ja" ändern würden!

Grüße

Torsten

[karsta.de]

Stimmt, Du hast Recht. Ich bin den Installer mal durchgegangen.
Der Shopbetreiber meinte, er hätte die Einstellung ganz bewußt vorher ausgestellt gehabt und nach dem Update sei es dann an gewesen. Aber irren ist menschlich.

BG Karsta

[karsta.de]

@modified Team
Ich hab nun die Zeile in der includes/external/compactor/compactor.php gefunden. Mit der Datei aus 2.0.4.2 geht es ja mit eingeschalteter CSS und JS Komprimierung und dann hab ich einfach mal alle Stellen nacheinander auskommentiert und immer wieder leere tpl_plugin.min.css und js Dateien hochgeladen. Sonst sah man keinen Unterschied.

Es ist die vorletzte neue Änderung in Zeile 454

Code: PHP  [Auswählen]

//  Removes single line '//' comments, treats blank characters
      $html = preg_replace('![ \t]*[^:]//.*[ \t]*[\r\n]!', '', $html);

wenn man das ändert zu

Code: PHP  [Auswählen]

//  Removes single line '//' comments, treats blank characters
      $html = preg_replace('!^[ \t]*[^:]//.*[ \t]*[\r\n]!', '', $html);

dann wird die Seite nicht mehr zerstört bei eingeschalteter CSS und JS Komprimierung. Allerdings hab ich keinen Plan, was dieses zusätzliche

Code: PHP  [Auswählen]

^

Dach macht. Just try and error und vergleichen mit anderen Zeilen. Vielleicht hilft es ja dennoch.

Gruß Timm

Leider gab es immer wieder beim Komprimieren der Seite mit der CSS-Komprimierung Probleme. Deshalb habe ich die von dir erwähnte Zeile jetzt auskommentiert.

Code: PHP  [Auswählen]

//  Removes single line '//' comments, treats blank characters
     // $html = preg_replace('![ \t]*[^:]//.*[ \t]*[\r\n]!', '', $html);

Nun passt es auch mit diversen Templates.

BG Karsta

[Timm]

@Karsta
GTB hat die compactor.php in meinem Testshop gefixed und es funktioniert nun wie es soll. Das Ticket dazu wurde auch auf fixed gesetzt. Ich gehe davon aus, dass der Fix zeitnah hier gepostet wird oder die 2.0.5.1 erscheint.

Gruß Timm

[Timm]

@modified Team
Wenn man sich auf einer paginierten Seite des Forums befindet und oben auf Community klickt, dann wird man nicht auf die Startseite des Forums geleitet wie früher, sondern die paginierte Seite selbst wird erneut geladen. Wäre schön, wenn das wieder so wie früher wäre. Vor allem mobil ist das etwas umständlich zur Startseite zu gelangen.

Auch wäre es mobil von Vorteil, wenn es einen to Top Pfeil gäbe.

Gruß Timm

[karsta.de]

@Karsta
GTB hat die compactor.php in meinem Testshop gefixed und es funktioniert nun wie es soll. Das Ticket dazu wurde auch auf fixed gesetzt.

Das ist schön für dich und den Projektplan. Das hatte ich auch im Bug-Tracker gesehen, leider ohne Lösungsvorschlag.
Deshalb mein Vorschlag.

BG Karsta