twice the same code 2 lines apart - it hurts my eyes. verify_architecture_attribute!(p) would look so much easier