correct, this should have been leisure=model_railway, I updated the paragraph.

IMHO the tag tourism=attraction is not a semantic tag that describes the nature of the thing, more a kind of a placeholder for the feature tag (or to be used additionally) and somehow stating “this is important”