pkp/pkp-lib#10906 Entering duplicate ROR affiliations are not filtered from search results

Author: GaziYucel

src/components/Form/fields/FieldAffiliations.vue

@@ -136,7 +136,10 @@
 									})
 								}}
 							</span>
-							<FieldAffiliationsRorAutoSuggest ref="autoSuggestRef" />
+							<FieldAffiliationsRorAutoSuggest
+								ref="autoSuggestRef"
+								:current-value="currentValue"
+							/>
 						</TableCell>
 						<TableCell>
 							<div v-if="showNewAffiliationForm">
@@ -248,6 +251,11 @@ const props = defineProps({
 		type: String,
 		default: null,
 	},
+	/** The ID of the form this field should appear in. This is passed down from the `Form`.  */
+	formId: {
+		type: String,
+		default: null,
+	},
 	/** Current value of the field */
 	value: {
 		type: Array,

src/components/Form/fields/FieldAffiliationsRorAutoSuggest.vue

@@ -60,6 +60,19 @@ import Icon from '@/components/Icon/Icon.vue';
 import {useFetch} from '@/composables/useFetch';
 import {useId} from '@/composables/useId';
 
+const props = defineProps({
+	currentValue: {
+		type: Array,
+		default: () => [],
+	},
+});
+const currentValueIds = computed(() => {
+	let ids = [];
+	props.currentValue.forEach((item) => {
+		if (item['ror']) ids.push(item['ror']);
+	});
+	return ids;
+});
 const {generateId} = useId();
 const autosuggestContainerId = generateId();
 const currentSelected = ref([]);
@@ -95,34 +108,41 @@ const autoSuggestProps = computed(() => ({
 }));
 
 const mappedSuggestions = computed(() => {
-	return suggestions.value?.items.map((item) => {
-		const displayLocale =
-			item.names?.find((i) => i.types.includes('ror_display'))?.lang !== null
-				? item.names?.find((i) => i.types.includes('ror_display'))?.lang
-				: noLangCode;
-
-		let names = {};
-		item.names?.forEach((name) => {
-			if (name.types.includes('label') || name.types.includes('ror_display')) {
-				const locale = name.lang !== null ? name.lang : noLangCode;
-				names[locale] = name.value;
-			}
-		});
-
-		return {
-			value: {
-				id: null,
-				ror: item.id,
-				displayLocale: displayLocale,
-				isActive: item.status === 'active' ? 1 : 0,
-				name: names,
-				_href: null,
-			},
-			label: names[displayLocale],
-			hasSlot: true,
-			href: item.id,
-		};
+	let newSuggestions = [];
+	suggestions.value?.items.map((item) => {
+		if (!currentValueIds.value?.includes(item.id)) {
+			const displayLocale =
+				item.names?.find((i) => i.types.includes('ror_display'))?.lang !== null
+					? item.names?.find((i) => i.types.includes('ror_display'))?.lang
+					: noLangCode;
+
+			let names = {};
+			item.names?.forEach((name) => {
+				if (
+					name.types.includes('label') ||
+					name.types.includes('ror_display')
+				) {
+					const locale = name.lang !== null ? name.lang : noLangCode;
+					names[locale] = name.value;
+				}
+			});
+
+			newSuggestions.push({
+				value: {
+					id: null,
+					ror: item.id,
+					displayLocale: displayLocale,
+					isActive: item.status === 'active' ? 1 : 0,
+					name: names,
+					_href: null,
+				},
+				label: names[displayLocale],
+				hasSlot: true,
+				href: item.id,
+			});
+		}
 	});
+	return newSuggestions;
 });
 
 watch(queryParams, () => {