Fixed consume page location dropdown handling (fixes #2328)

changelog/74_UNRELEASED_xxxx-xx-xx.md

@@ -0,0 +1,57 @@
+> ⚠️ xxxBREAKING CHANGESxxx
+
+> ❗ xxxImportant upgrade informationXXX
+
+> 💡 xxxMinor upgrade informationXXX
+
+### New feature: xxxx
+
+- xxx
+
+### Stock
+
+- Fixed that the location dropdown on the consume page contained the same location multiple times if there are currently stock entries at multiple locations of the corresponding product
+
+### Shopping list
+
+- xxx
+
+### Recipes
+
+- xxx
+
+### Meal plan
+
+- xxx
+
+### Chores
+
+- xxx
+
+### Calendar
+
+- xxx
+
+### Tasks
+
+- xxx
+
+### Batteries
+
+- xxx
+
+### Equipment
+
+- xxx
+
+### Userfields
+
+- xxx
+
+### General
+
+- xxx
+
+### API
+
+- xxx

public/viewjs/consume.js

@@ -389,9 +389,12 @@ Grocy.Components.ProductPicker.GetPicker().on('change', function(e)
 					{
 						var setDefault = 0;
 						var stockAmountAtDefaultLocation = 0;
+						var addedLocations = [];
 						stockLocations.forEach(stockLocation =>
 						{
 							if (stockLocation.location_id == defaultLocationId)
+							{
+								if (!addedLocations.includes(stockLocation.location_id))
 								{
 									$("#location_id").append($("<option>", {
 										value: stockLocation.location_id,
@@ -400,15 +403,21 @@ Grocy.Components.ProductPicker.GetPicker().on('change', function(e)
 									$("#location_id").val(defaultLocationId);
 									$("#location_id").trigger('change');
 									setDefault = 1;
+								}
 								stockAmountAtDefaultLocation += stockLocation.amount;
 							}
 							else
+							{
+								if (!addedLocations.includes(stockLocation.location_id))
 								{
 									$("#location_id").append($("<option>", {
 										value: stockLocation.location_id,
 										text: stockLocation.location_name
 									}));
 								}
+							}
+
+							addedLocations.push(stockLocation.location_id);
 
 							if (setDefault == 0)
 							{