I've tested this, and compared to master I can see the change and it works but I have found some strange behaviour:

• This will return a 404 if the news ID doesn't exist.
• This will return a 404 if the news ID exists and there is no image for that item.
• But if there is an image, and you type in a different image number to the one associated with the image in the source of the News Item page, it will still access the image and not 404. Master behaves the same, so this is not introducing a new issue.

So - if the PR is just about non-existent news ID, then I think it does what it is supposed to do, but if it is about non-existent images, then it doesn't because it allows any image number as if it were the correct one as long as the news ID is correct.

e.g.
if image is at: news/id/699/image/166081/

Then news/id/69/image/166081/ will 404 (assuming no other item 69 exists)
But news/id/699/image/166/ will still access the image as if it were news/id/699/image/166081/

I am approving this but leaving it up to the originator to consider the above and decide whether it is working as intended or not, or whether that is a separate issue etc.