Only one criticism: you must choose either the elemental or outsider type. The outsider type "trumps" the elemental type (i.e. if it is an outsider, it overrides the other types). For example, the efreeti has the fire subtype, but is an outsider instead of an elemental. The fire subtype is noted on the creature, so it would just be an outsider (nothing stopping you from giving it fire elemental abilities and weaknesses, though...

). For reference on the "stacking" of types, see the Savage Species book (they really do a good job of spelling out how to mix types and subtypes). This is just to let you know; otherwise it is a good job! Is your campaign that high level that you will be using the creature right away? If not, take your time to play test it, especially considering that it is such a high status creature (servant of god, etc.). You'll be able to tweak out the bad stuff before you introduce the creature (after it's in the game, it's a little too late).