The episode is set in 1913, and as the Doctor enters the village dance, he's asked for a donation for 'veterans of the Crimean war'. This war ended in 1856, and since the average age of a fighting soldier during it was 25, it's unlikely there would be any extant veterans to collect on behalf of.
In the opening shot of the scene where the schoolboys are practicing firing their machine gun, a large white modern articulated lorry can be seen moving from right to left in the background behind the targets.
Tubular metal scaffolding (of the kind that the Doctor dislodges with a thrown cricket ball in order to save the woman and child from a falling piano) was not in widespread use in the UK until the 1930s. Before that, wooden scaffolding poles were employed, lashed together/into position with rope or something similar.
School boys are shown firing a .303 Vickers machine gun, which had a range of 4,500 yards. An outdoor firing range would always have a slope of earth or sand behind the targets to stop dead all bullets fired down the range. Not only is there no such slope behind the boys targets, but we can see they're firing down into a valley full of buildings, well within the 4.5km (2.8mi) range of the gun, jeopardizing the lives of the villagers!