From d986496131f4c36db5c4ec361615bb9858e15426 Mon Sep 17 00:00:00 2001
From: Colin Kuskie <colink@perldreamer.com>
Date: Fri, 30 Apr 2010 08:47:27 -0700
Subject: [PATCH] Code tweaks and comments for clarity.

---
 lib/WebGUI/Shop/Cart.pm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/WebGUI/Shop/Cart.pm b/lib/WebGUI/Shop/Cart.pm
index 6d9bc0cf9..e6ce6201d 100644
--- a/lib/WebGUI/Shop/Cart.pm
+++ b/lib/WebGUI/Shop/Cart.pm
@@ -751,13 +751,13 @@ sub updateFromForm {
     $self->update($cartProperties);
 
     if ($self->requiresShipping) {
-        my %shippingData = $book->processAddressForm('shipping_');
-        my @missingShippingFields = $book->missingFields(\%shippingData);
-        my $shippingAddressId = $form->process('shippingAddressId');
         if ($form->process('sameShippingAsBilling', 'yesNo')) {
             $cartProperties->{shippingAddressId} = $self->get('billingAddressId');
         }
         else {
+            my %shippingData = $book->processAddressForm('shipping_');
+            my @missingShippingFields = $book->missingFields(\%shippingData);
+            my $shippingAddressId = $form->process('shippingAddressId');
             ##No missing shipping fields, if we set to the same as the billing fields
             if (@missingShippingFields) {
                 $self->error('missing shipping '.$missingShippingFields[0]);
@@ -905,6 +905,7 @@ sub www_update {
     my $session = $self->session;
     $self->updateFromForm;
     if ($session->form->get('checkout')) {
+        ##Setting a shipping address greatly simplifies the Transaction
         if (! $self->requiresShipping && ! $self->get('shippingAddressId')) {
             $self->update({shippingAddressId => $self->get('billingAddressId')});
         }