If the code in the question contains some construct, but the question isn't directly about it, it shouldn't contain the tag associated with it.
Although I usually view removing it as a minor edit, perhaps incorrectly so, and whether or not a question is actually about a construct is perhaps arguable in some cases.
I could be wrong, but this seems like a decent use of the foreach tag - it's specifically about trying to break from a foreach loop.
You could take another stance saying that any question which requires familiarity with something should have the associated tag, but then there's basically no point to having any of the following tags (to name a few): foreach, methods, class, for-loop, while-loop, break, continue, do-while, as many questions require familiarity with many such constructs, while a question can only have 5 tags, and it would be next to impossible to identify a question actually about one of these constructs (such questions would be my main argument for not getting rid of the above tags).