[ic] New commit

David Christensen david at endpoint.com
Wed Jun 14 15:35:18 UTC 2017


This was on my IC branch as a bug fix for a client but I wasn't sure if there might be other implications here; anyone think this would break things for them?


commit 2163a7c222a9b93fc50d925e77770353c186f477
Author: David Christensen <david at endpoint.com>
Date:   Fri Feb 24 16:08:32 2017 -0600

    Early exit from find_options_type if no sku provided

diff --git a/lib/Vend/Options.pm b/lib/Vend/Options.pm
index 3d0d898..1de47ab 100644
--- a/lib/Vend/Options.pm
+++ b/lib/Vend/Options.pm
@@ -68,6 +68,8 @@ sub find_options_type {

        my $sku = $item->{mv_sku} || $item->{code};

+       return unless defined $sku && $sku ne '';
        $opt = get_option_hash($opt);

        my $module;
David Christensen
End Point Corporation
david at endpoint.com

More information about the interchange-users mailing list