todo's

lib/providers/nutrition.dart

@@ -132,6 +132,7 @@ class NutritionPlansProvider with ChangeNotifier {
     try {
       plan = findById(planId);
     } on NoSuchEntryException {
+      // TODO: remove this useless call, because we will fetch all details below
       plan = await fetchAndSetPlanSparse(planId);
     }
 
@@ -145,6 +146,7 @@ class NutritionPlansProvider with ChangeNotifier {
       final List<MealItem> mealItems = [];
       final meal = Meal.fromJson(mealData);
 
+      // TODO: we should add these ingredients to the ingredient cache
       for (final mealItemData in mealData['meal_items']) {
         final mealItem = MealItem.fromJson(mealItemData);
 
@@ -376,9 +378,10 @@ class NutritionPlansProvider with ChangeNotifier {
 
     if (data['count'] == 0) {
       return null;
-    } else {
-      return Ingredient.fromJson(data['results'][0]);
     }
+    // TODO we should probably add it to ingredient cache.
+    // TODO: we could also use the ingredient cache for code searches
+    return Ingredient.fromJson(data['results'][0]);
   }
 
   /// Log meal to nutrition diary

lib/widgets/nutrition/forms.dart

@@ -367,6 +367,7 @@ class IngredientFormState extends State<IngredientForm> {
                       ) {
                         if (snapshot.hasData) {
                           _mealItem.ingredient = snapshot.data!;
+                          // here it could be nice to put the button to the popup
                           return MealItemValuesTile(
                             ingredient: _mealItem.ingredient,
                             nutritionalValues: _mealItem.nutritionalValues,

lib/widgets/nutrition/helpers.dart

@@ -107,6 +107,7 @@ void showIngredientDetails(BuildContext context, int id, {String? image}) {
 
   final url = context.read<NutritionPlansProvider>().baseProvider.auth.serverUrl;
 
+// TODO: display source name and source URL for an ingredient in the UI
   showDialog(
     context: context,
     builder: (context) => FutureBuilder<Ingredient>(

lib/widgets/nutrition/widgets.dart

@@ -224,6 +224,7 @@ class _IngredientTypeaheadState extends State<IngredientTypeahead> {
                     mainAxisSize: MainAxisSize.min,
                     children: [
                       Text(AppLocalizations.of(context).productFoundDescription(result.name)),
+                      // TODO replace with full view instead of small popup
                       MealItemValuesTile(
                         ingredient: result,
                         nutritionalValues: result.nutritionalValues,