Add option to log complete nutritional plan

[rolandgeider]

At the moment, it is possible to log all entries for a meal. It should also be possible to log the calories for all meals for a day.

This is really simple, instead of just adding the ingredients for a meal, we need to loop through all meals in a plan and add those (since this will duplicade code, a bit of refactoring will be needed as well).

[oconnelc]

If this is still open, I'd like to take a crack at it.

[oconnelc]

Is the plan to place a button on the "Nutrition plan" view to log the entire plan? If so, how does the total calories get logged?

I see similar logic in the NutritionPlan's get_nutritional_values method; it appears that the method iterates over the meals and sums up all of the values. Is this doing what this ticket is describing or is this similar code that needs to be refactored?

[rolandgeider]

Yes it should be placed there. The calories are summed by the log and the plan, you only need to add the ingredients and units (e.g. 100g of bread, 2 slices of cheese, etc)

Take a look at

wger/wger/nutrition/views/log.py

Line 117 in 88c28bf

def log_meal(request, meal_pk):

to see how it is done for a single meal. For the whole plan you'd need to iterate over each meal in it

[oconnelc]

Awsome! I'll take a look at it today.

[oconnelc]

I just wanted to make sure I'm going in the right direction. I've added a link at the bottom of the plan to log the entire plan:

Right now it's just "Log this Plan".

If this looks good, I'll go ahead and hook it up to the back end and push up a PR when I'm done.

[oconnelc]

I think I have what this ticket is asking for. I've included a gif screenshot of how to log an entire plan. If this is what is desired, then all I have left is to update the unit test. If there's anything else that needs to be done to complete this ticket, please let me know.

[rolandgeider]

Nice work! You can open a PR after updating the tests

[oconnelc]

Pull request #540 is now available

[oconnelc]

I'm participating in Hacktoberfest. According to the site this project is not participating. I was wondering if a maintainer could add the "hacktoberfest-accepted" label to the PR. This way the whole project doesn't have to participate and I can get credit.

If you'd like to have the project participate, a maintainer can add the topic "hacktoberfest" to the project

If not, no worries, I was happy to help with this project.

[rolandgeider]

Hacktoberfest, I completely forgot about that! I'll clean up the issues a bit and add the tag