Merge pull request #347 from ins0/fix/view

Fix: Changed escape to correct escape attr helper

Author: localheinz

module/User/view/user/helper/new-users.phtml

@@ -1,33 +1,24 @@
 <div class="block">
     <div class="row">
         <div class="span4">
-            <h4>
-                New Developers
-            </h4>
+            <h4>New Developers</h4>
             <hr>
         </div>
     </div>
-
-    <?php
-    $count = 0;
-    foreach ($this->users as $user) {
-        if ($count % 4 == 0) {
-            ?>
+    <?php $count = 0; ?>
+    <?php foreach ($this->users as $user): ?>
+        <?php if ($count % 4 == 0): ?>
             <div class="row-fluid" style="margin-bottom:10px;">
                 <div class="span12">
-                <?php } ?>
-                <div class="span3">
-                    <a href="https://github.com/<?php echo $this->escapeHtml($user->getUserName()) ?>" data-delay="0" rel="tooltip" title="<?php echo $this->escapeHtml($user->getUserName()) ?>" class="thumbnail">
-                        <img src="<?php echo $user->getPhotoUrl() ?>" alt="<?php echo $this->escapeHtml($user->getUserName()) ?>">
-                    </a>
-                </div>
-                <?php if ($count % 4 == 3) { ?>
+        <?php endif; ?>
+                    <div class="span3">
+                        <a href="https://github.com/<?php echo $this->escapeHtmlAttr($user->getUserName()); ?>" data-delay="0" rel="tooltip" title="<?php echo $this->escapeHtmlAttr($user->getUserName()); ?>" class="thumbnail">
+                            <img src="<?php echo $this->escapeHtmlAttr($user->getPhotoUrl()); ?>" alt="<?php echo $this->escapeHtmlAttr($user->getUserName()); ?>">
+                        </a>
+                    </div>
+        <?php if ($count++ % 4 == 3): ?>
                 </div>
             </div>
-            <?php
-        }
-        $count++;
-    }
-    ?>
-
+        <?php endif; ?>
+    <?php endforeach; ?>
 </div>