LiteFarmOrg · Duncan-Brain · Oct 24, 2025 · Oct 10, 2025 · Oct 10, 2025 · Oct 10, 2025
@@ -79,6 +79,7 @@
   "QUANTITY": "Quantity",
   "REMOVE": "Remove",
   "REMOVE_ITEM": "Remove item",
+  "REMOVED": "Removed",
   "REQUIRED": "Required",
   "RETIRE": "Retire",
   "REVISION_INFO": "Revised on <strong>{{date}}</strong> <i>by</i> <strong>{{user}}</strong>",

@@ -13,13 +13,14 @@
  *  GNU General Public License for more details, see <https://www.gnu.org/licenses/>.
  */
 
-import { useEffect } from 'react';
+import { useEffect, useMemo } from 'react';
 import { useTranslation } from 'react-i18next';
 import { useFormContext } from 'react-hook-form';
 import Input, { getInputErrors } from '../../Form/Input';
 import ProductDetails, { type ProductDetailsProps } from '../../Form/ProductDetails';
 import { hookFormMaxCharsValidation } from '../../Form/hookformValidationUtils';
 import { getSoilAmendmentFormValues } from '../../Form/ProductDetails/utils';
+import { isLibraryProduct } from '../../../util/product';
 import { productDefaultValuesByType } from '../../../containers/ProductInventory/ProductForm/constants';
 import { TASK_TYPES } from '../../../containers/Task/constants';
 import { PRODUCT_FIELD_NAMES } from '../../Task/AddSoilAmendmentProducts/types';
@@ -79,7 +80,25 @@ const PureSoilAmendmentProductForm = ({
     }
   }, [mode]);
 
-  const productNames: SoilAmendmentProduct['name'][] = products.map(({ name }) => name);
+  const { customProductsInInventory } = useMemo(() => {
+    // LF-4963 - library product select options
+    const libraryProductsOutsideInventory: SoilAmendmentProduct[] = [];
+    const customProductsInInventory: SoilAmendmentProduct[] = [];
+
+    products.forEach((product) => {
+      if (isLibraryProduct(product)) {
+        if (!product.farm_id || product.removed) {
+          libraryProductsOutsideInventory.push(product);
+        }
+      } else if (!product.removed) {
+        customProductsInInventory.push(product);
+      }
+    });
+
+    return { libraryProductsOutsideInventory, customProductsInInventory };
+  }, [products]);
+
+  const customProductNames = customProductsInInventory.map(({ name }) => name);
 
   return (
     <div className={styles.soilAmendmentProductForm}>
@@ -95,7 +114,7 @@ const PureSoilAmendmentProductForm = ({
             // Allow duplicate check to pass if keeping the original name during edit
             if (
               !(mode === FormMode.EDIT && value === product?.name) &&
-              productNames.includes(value)
+              customProductNames.includes(value)
             ) {
               return t('ADD_TASK.DUPLICATE_NAME');
             }

@@ -33,6 +33,13 @@ import { hookFormMaxCharsValidation } from '../../../Form/hookformValidationUtil
 import { soilAmendmentProductDetailsDefaultValues } from '../../../../containers/ProductInventory/ProductForm/constants';
 import { getSoilAmendmentFormValues } from '../../../Form/ProductDetails/utils';
 
+const findProduct = (
+  products?: SoilAmendmentProduct[],
+  productId?: SoilAmendmentProduct['product_id'],
+) => {
+  return products?.find(({ product_id }) => product_id === productId);
+};
+
 export type ProductCardProps = ProductDetailsProps & {
   namePrefix: string;
   system: 'metric' | 'imperial';
@@ -126,9 +133,17 @@ const SoilAmendmentProductCard = ({
   const productId = getValues(PRODUCT_ID);
   const purposes = watch(PURPOSES);
 
+  const initialProductId = useRef(productId);
+
   const selectRef = useRef<SelectRef>(null);
-  const productOptions = products.map(({ product_id, name, ...rest }) => {
-    return { value: product_id, label: name, data: rest };
+  const productOptions = products.flatMap(({ product_id, name, ...rest }) => {
+    if (rest.removed && product_id !== initialProductId.current) {
+      return [];
+    }
+
+    const label = name + (rest.removed ? ` (${t('common:REMOVED')})` : '');
+
+    return { value: product_id, label, data: rest };
   });
 
   useEffect(() => {
@@ -137,6 +152,11 @@ const SoilAmendmentProductCard = ({
     }
   }, [otherPurposeId]);
 
+  useEffect(() => {
+    const selectedProduct = findProduct(products, productId);
+    nestedFormMethods.reset(getSoilAmendmentFormValues(selectedProduct));
+  }, []);
+
   return (
     <div className={styles.productCard}>
       {!isReadOnly && onRemove && (
@@ -154,7 +174,7 @@ const SoilAmendmentProductCard = ({
               options={productOptions}
               onChange={(e) => {
                 onChange(e?.value);
-                const selectedProduct = products.find(({ product_id }) => product_id === e?.value);
+                const selectedProduct = findProduct(products, e?.value);
                 nestedFormMethods.reset(getSoilAmendmentFormValues(selectedProduct));
               }}
               placeholder={t('ADD_PRODUCT.PRESS_ENTER')}

@@ -18,11 +18,13 @@ import { useGetSoilAmendmentFertiliserTypesQuery } from '../../../store/api/apiS
 import PureSoilAmendmentProductForm from '../../../components/ProductInventory/ProductForm/PureSoilAmendmentProductForm';
 import { userFarmSelector } from '../../userFarmSlice';
 import { certifierSurveySelector } from '../../OrganicCertifierSurvey/slice';
-import { productsSelector } from '../../productSlice';
+import { productsForTaskTypeSelector } from '../../productSlice';
 import { TASK_TYPES } from '../../Task/constants';
 import { FormMode } from '..';
 import { FormContentProps } from '.';
 
+const taskType = { task_translation_key: TASK_TYPES.SOIL_AMENDMENT.toUpperCase() };
+
 export default function SoilAmendmentProductForm({ mode, productId }: FormContentProps) {
   const { t } = useTranslation();
 
@@ -38,12 +40,9 @@ export default function SoilAmendmentProductForm({ mode, productId }: FormConten
     label: t(`ADD_PRODUCT.${key}_FERTILISER`),
   }));
 
-  const products = useSelector(productsSelector);
-
-  // TODO: Filter out removed products
-  const soilAmendmentCustomProducts = products.filter(
-    (product) => product.type === TASK_TYPES.SOIL_AMENDMENT,
-  );
+  const soilAmendmentProducts =
+    /* @ts-expect-error https://github.com/reduxjs/reselect/issues/550#issuecomment-999701108 */
+    useSelector((state) => productsForTaskTypeSelector(state, taskType)) || [];
 
   const isReadOnly = mode === FormMode.READ_ONLY;
 
@@ -53,7 +52,7 @@ export default function SoilAmendmentProductForm({ mode, productId }: FormConten
       isReadOnly={isReadOnly}
       farm={{ farm_id, interested, country_id }}
       fertiliserTypeOptions={fertiliserTypeOptions}
-      products={soilAmendmentCustomProducts}
+      products={soilAmendmentProducts}
       productId={productId}
     />
   );

@@ -24,7 +24,7 @@ import { ReactComponent as BookIcon } from '../../assets/images/book-closed.svg'
 import useSearchFilter from '../../containers/hooks/useSearchFilter';
 import PureProductInventory from '../../components/ProductInventory';
 import { getProducts } from '../Task/saga';
-import { productsSelector } from '../productSlice';
+import { productInventorySelector } from '../productSlice';
 import { Product, SoilAmendmentProduct } from '../../store/api/types';
 import { TASK_TYPES } from '../Task/constants';
 import { SearchProps } from '../../components/Animals/Inventory';
@@ -65,7 +65,7 @@ export default function ProductInventory() {
     dispatch(getProducts());
   }, []);
 
-  const productInventory = useSelector(productsSelector);
+  const productInventory = useSelector(productInventorySelector);
 
   const inventory = productInventory
     .filter((product) => product.type === TASK_TYPES.SOIL_AMENDMENT)

@@ -22,7 +22,7 @@ function TaskCompleteStepOne() {
   const { task_id } = useParams();
   const task = useSelector(taskWithProductSelector(task_id));
   const selectedTaskType = task?.taskType;
-  const products = useSelector(productsForTaskTypeSelector(selectedTaskType));
+  const products = useSelector((state) => productsForTaskTypeSelector(state, selectedTaskType));
   const persistedPaths = [`/tasks/${task_id}/complete`];
 
   const onContinue = (data) => {

@@ -28,7 +28,9 @@ function TaskDetails() {
   const { interested, farm_id } = useSelector(certifierSurveySelector, shallowEqual);
   const persistedFormData = useSelector(hookFormPersistSelector);
   const selectedTaskType = useSelector(taskTypeSelector(persistedFormData.task_type_id));
-  const products = useSelector(productsForTaskTypeSelector(selectedTaskType));
+  const products = useSelector((state) => productsForTaskTypeSelector(state, selectedTaskType));
+  const productsInInventory = products?.filter((product) => !product.removed);
+
   const managementPlanIds = persistedFormData.managementPlans?.map(
     ({ management_plan_id }) => management_plan_id,
   );
@@ -64,7 +66,7 @@ function TaskDetails() {
         persistedPaths={persistedPaths}
         selectedTaskType={selectedTaskType}
         system={system}
-        products={products}
+        products={productsInInventory}
         farm={{ farm_id, country_id, interested }}
         managementPlanByLocations={managementPlanByLocations}
         wildManagementPlanTiles={showWildCrops && wildManagementPlanTiles}

@@ -52,7 +52,7 @@ function TaskReadOnly() {
   const system = useSelector(measurementSelector);
   const task = useReadonlyTask(task_id);
   const selectedTaskType = task?.taskType;
-  const products = useSelector(productsForTaskTypeSelector(selectedTaskType));
+  const products = useSelector((state) => productsForTaskTypeSelector(state, selectedTaskType));
   const isIrrigationTaskWithExternalPrescription =
     isTaskType(selectedTaskType, 'IRRIGATION_TASK') &&
     task?.irrigation_task?.irrigation_prescription_external_id != null;

@@ -2,6 +2,7 @@ import { createEntityAdapter, createSlice } from '@reduxjs/toolkit';
 import { loginSelector, onLoadingFail, onLoadingStart } from './userFarmSlice';
 import { pick } from '../util/pick';
 import { createSelector } from 'reselect';
+import { isLibraryProduct } from '../util/product';
 
 export const getProduct = (obj) => {
   return pick(obj, [
@@ -55,24 +56,28 @@ const productSelectors = productAdapter.getSelectors(
   (state) => state.entitiesReducer[productSlice.name],
 );
 
+// Select farm products including removed ones
 export const productsSelector = createSelector(
   [productSelectors.selectAll, loginSelector],
   (products, { farm_id }) => {
     return products.filter((product) => product.farm_id === farm_id);
   },
 );
 
-export const productsForTaskTypeSelector = (taskType) => {
-  return createSelector([productSelectors.selectAll, loginSelector], (products, { farm_id }) => {
+// Select farm products for a given type including removed ones
+export const productsForTaskTypeSelector = createSelector(
+  [productsSelector, (_state, taskType) => taskType],
+  (products, taskType) => {
     if (taskType === undefined) {
       return undefined;
     }
-    return products.filter(
-      (product) =>
-        product.farm_id === farm_id && product.type === taskType.task_translation_key.toLowerCase(),
-    );
-  });
-};
+    return products.filter(({ type }) => type === taskType.task_translation_key.toLowerCase());
+  },
+);
+
+export const productInventorySelector = createSelector([productsSelector], (products) => {
+  return products.filter((product) => !product.removed);
+});
 
 export const productEntitiesSelector = productSelectors.selectEntities;
 

@@ -202,6 +202,7 @@ export interface Product {
     | typeof TASK_TYPES.PEST_CONTROL;
   farm_id?: string;
   on_permitted_substances_list?: 'YES' | 'NO' | 'NOT_SURE' | null;
+  removed: boolean;
 }
 
 export enum ElementalUnit {

@@ -30,6 +30,7 @@ export const products = [
       k: 3,
       elemental_unit: ElementalUnit.PERCENT,
     },
+    removed: false,
   },
   {
     product_id: 2,
@@ -45,6 +46,7 @@ export const products = [
       k: 3.3,
       elemental_unit: ElementalUnit.RATIO,
     },
+    removed: false,
   },
   {
     product_id: 3,
@@ -60,6 +62,7 @@ export const products = [
       k: undefined,
       elemental_unit: undefined,
     },
+    removed: false,
   },
   {
     product_id: 4,
@@ -75,6 +78,7 @@ export const products = [
       k: 1,
       elemental_unit: ElementalUnit.PERCENT,
     },
+    removed: false,
   },
   {
     product_id: 5,
@@ -90,6 +94,7 @@ export const products = [
       k: 1,
       elemental_unit: ElementalUnit.PERCENT,
     },
+    removed: false,
   },
   {
     product_id: 6,
@@ -105,5 +110,6 @@ export const products = [
       k: undefined,
       elemental_unit: ElementalUnit.RATIO,
     },
+    removed: false,
   },
 ];
@@ -0,0 +1,21 @@
+/*
+ *  Copyright 2025 LiteFarm.org
+ *  This file is part of LiteFarm.
+ *
+ *  LiteFarm is free software: you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation, either version 3 of the License, or
+ *  (at your option) any later version.
+ *
+ *  LiteFarm is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *  GNU General Public License for more details, see <https://www.gnu.org/licenses/>.
+ */
+
+import { Product } from '../store/api/types';
+
+// LF-4963 - confirm property that will distinguish custom from library products
+export const isLibraryProduct = (product: Product) => {
+  return !!product.product_translation_key;
+};