I've posted some comments on the pull request on GitHub.
A quest rule is not out of the question (if you add one, please make sure it's set up consistently across 2.50.x and master, including its position in the bit field!!) but I don't think it's absolutely required here, unless you can think of a reason the gameplay of an old quest might be affected.
