Just an observation: You may want to spend a few minutes and come up with some better variable names. Specifically, you're using both %site_data and @site_data which can be somewhat confusing for future maintenance--especially when both are used in the same subroutine.

...roboticus