View Issue Details

IDProjectCategoryView StatusLast Update
0001659OXID eShop (all versions)1.03. Basket, checkout processpublic2012-12-10 14:13
Reporterleofonic 
PriorityhighSeverityfeatureReproducibilityalways
Status resolvedResolutionno change required 
Product Version4.2.0 revision 23610 
Target VersionFixed in Version 
Summary0001659: articles cannot be restricted to certain delivery set
DescriptionWhen an article has a restriction to a certain delivery set, eg. no delivery to foreign countries, this works as expected as long as it is the only article in basket. If another item is added to basket, which allows delivery to foreign countries, the restriction no longer works and both articles can be sent to foreign countries.
Additional InformationThe delivery set options in step 3 should be the intersection of the possible delivery sets for each article, and not the sum.
TagsNo tags attached.
Theme
BrowserAll
PHP Versionany
MySQL Versionany

Relationships

has duplicate 0001672 resolveddainius.bigelis Limited amount for products 
related to 0002149 resolvedarvydas_vapsva Higher shipping cost for certain articles does not work 

Activities

vilma_liorensaityte

2010-08-30 10:07

administrator   ~0003452

in oxdelivery::isForBasket() parameter $blUse will be set to false in foreach loop. If this delivery is not valid for some basket item, it will break the loop and return false.

Changed code from:
        // category & article check
        if ( $blHasCategories || $blHasArticles ) {
            $blUse = false;

            $aDeliveryArticles = $this->getArticles();
            $aDeliveryCategories = $this->getCategories();

            foreach ( $oBasket->getContents() as $oContent ) {
to:
// category & article check
        if ( $blHasCategories || $blHasArticles ) {
            $aDeliveryArticles = $this->getArticles();
            $aDeliveryCategories = $this->getCategories();

            foreach ( $oBasket->getContents() as $oContent ) {
                if ( !$blUse ) {
                    break;
                }
                $blUse = false;

dainius.bigelis

2011-03-21 08:16

reporter   ~0004237

As simple fix of this leads to newly occured issues, this case needs more complex solution and larger changes. As this is more a feature requests, this is moved to UserVoice:
http://oxid.uservoice.com/forums/31940-feature-requests/suggestions/1614091-restriction-for-articles-to-certain-delivery-set?ref=title