Use String cast in pickup location comparison to avoid weak comparison.

themes/bootstrap3/js/hold.js

@@ -24,8 +24,8 @@ function setUpHoldRequestForm(recordId) {
       var defaultValue = $('#pickUpLocation').data('default');
       $.each(response.data.locations, function holdPickupLocationEach() {
         var option = $('<option></option>').attr('value', this.locationID).text(this.locationDisplay);
-        // Weak comparison between locationID and defaultValue since locationID may be an integer
-        if (this.locationID === defaultValue || (defaultValue === '' && this.isDefault && $emptyOption.length === 0)) {
+        // Make sure to compare locationID and defaultValue as Strings since locationID may be an integer
+        if (String(this.locationID) === String(defaultValue) || (defaultValue === '' && this.isDefault && $emptyOption.length === 0)) {
           option.attr('selected', 'selected');
         }
         $('#pickUpLocation').append(option);