Theming a slider view using templates
Apart from your template needed some serious code cleanup (to follow the coding standards for PHP Template), I would say your issue lies with the fact that you have overridden the wrong views template.

It seems like your views template is doing the iterations around the nodes itself (from the foreach), and I bet your view is also doing the same (hence the double up).

My suggestions:


Read https://drupal.org/coding-standards - thoroughly
Also look at the default implementations in the views templates, these will give your excellent examples on how your code should look


As for your problem, either:


override a higher level views template (i.e. one that only gets rendered once per view)
or alternatively, use the same template, and leave out the foreach loop (as the view is essentially doing this. I would say this is the 'cleaner' option. All the fields you have pulled out in your view will be available to be used without having to access the global $view object.


Consider installing the devel module to help with outputting variables, so you know what is available to be themed (using the dsm() function)