So personally I would vote for this if you used a default attribute (in attributes/default.rb) to set the default port to 80 instead of using an if/else statement right in the template.  It follows Chef conventions much more closely and makes the template cleaner.

Otherwise, upvote.