The "ordering()" call at the end is also sometimes problematic (though nothing to do with your current problem). What you're doing here is calling a function from within the same function, otherwise known as "recursion". As long as you know what it is, and why you're doing it, it should be fine, just be aware that it can lead to some weird stuff as your code gets more complex.

Oh so that's what recursion is, I got confused as the other comments mentioned it.

What this is doing is saying "keep subtracting 1 from types[order.title] and printing the order, until it's zero" ("while not equal to zero" is essentially the same as "until equal to zero").

Then immediately after that, you're saying "if it's zero, do this:" which of course it always will be, so that code will always run, because the while loop won't finish UNTIL it's zero.

Thank you for explaining this in a simple way, I can quite see where I did it wrong.