Prio source obj's storage loc over org default loc in corrections (#5537)

Author: cassxw

app/helpers/application_helper.rb

@@ -126,6 +126,6 @@ def storage_location_for_source(source_object)
   end
 
   def default_location(source_object)
-    current_organization.default_storage_location || source_object.storage_location_id.presence || current_organization.intake_location
+    source_object.storage_location_id.presence || current_organization.default_storage_location || current_organization.intake_location
   end
 end

spec/helpers/application_helper_spec.rb

@@ -151,6 +151,14 @@ def current_organization; end
       it { is_expected.to eq(2) }
     end
 
+    context "returns source object's storage_location_id even when org has different default" do
+      let(:organization) { build(:organization, default_storage_location: 42) }
+      let(:purchase) { build(:purchase, storage_location_id: 1) }
+      subject { helper.default_location(purchase) }
+
+      it { is_expected.to eq(1) }
+    end
+
     context "returns current_organization intake_location if storage_location_id is not present" do
       let(:organization) { build(:organization, intake_location: 1) }
       let(:purchase) { build(:purchase, storage_location_id: nil) }