Deliverycost-calculation core-error


#1

After the update to version 4.7.10 there is an error in the deliverycost-calculation. For each delivery-category oxid amount the checksum. In my example there are two delivery-categories. One with deliverycost to xx€ and one from xx€. And for each category the amount ([I]$iAmount[/I]) would be increased, what is wrong.

sourcecode 4.7.10 model oxdelivery.php

foreach ( $aDeliveryCategories as $sCatId ) {
   if ( $oProduct->inCategory( $sCatId ) ) {
      $blUse = true;
      $iArtAmount = $this->getDeliveryAmount( $oContent );
      if ( $this->oxdelivery__oxfixed->value > 0 ) {
         if ( $this->_isForArticle( $oContent, $iArtAmount ) ) {
            $blForBasket = true;
         }
      }
      if (!$blForBasket) {
         $iAmount += $iArtAmount;
      }
   }
}

sourcecode 4.7.8 model oxdelivery.php

foreach ( $aDeliveryCategories as $sCatId ) {
   if ( $oProduct->inCategory( $sCatId ) ) {
      $blUse = true;
      $iArtAmount = $this->getDeliveryAmount( $oContent );
      if ( $this->oxdelivery__oxfixed->value > 0 ) {
         if ( $this->_isForArticle( $oContent, $iArtAmount ) ) {
             $blForBasket = true;
         }
      }
      break;
   }
}
if (!$blForBasket) {
   $iAmount += $iArtAmount;
}


#2

Hi,

could you please file a bug entry for this issue? Do you know how it works?

Cheers!


#3

The change was the fix for bug #4730: https://github.com/OXID-eSales/oxideshop_ce/commit/0f9ae806b29eed97edf07c3257bbceaa55c1bbd2


#4

No, i do not know how to

file a bug entry for this issue?

can you explain it?


#5

Sure. Go to the bugtracker and register yourself. Then press on “Report issue” - give as many information as possible, also @martin.wegele’s hint about #4730 and most important: how to reproduce this.

Cheers!


#6

Hi! Wir benutzen aktuell die CE4.9.7. Heute morgen erhielten wir eine Bestellung mit Artikeln, die u.a. in mehreren Shopkategorien zu finden sind. Und siehe da, die Versandkosten sind falsch kalkuliert. Ich dachte der Bug ist gefixt? Die Versandkostenregel ist ganz einfach: Bis 299,99kg per DHL Paketversand, ab 300kg Speditionsversand. Trotzdem hat ein Kunde Waren mit 335kg bestellt und der Shop hat den günstigen Paketversand gewählt. Ich kann das aber nicht reproduzieren…Lege ich mit meinem Account die Artikel in den Warenkorb bekomme ich auch den korrekten Speditionsversand angezeigt. Beide Versandkostenregeln gelten für alle Benutzergruppen. Keine Artikel- und Kategorienzuordnung vorhanden. Die Speditionsregel wird vor der Paketversandregel ausgeführt.