Merge pull request #658 from no-sec-marko/fix_hotkey_bug

fix hotkey bug in test view

Author: aaronweaver

dojo/templates/dojo/view_test.html

@@ -636,6 +636,7 @@ <h3>Notes</h3>
     </div>
 {% endblock %}
 {% block postscript %}
+    <script type="application/javascript" src="{% static "jquery.hotkeys/jquery.hotkeys.js" %}"></script>
     <script type="text/javascript" src="{% static "jquery-highlight/jquery.highlight.js" %}"></script>
     <script type="text/javascript">
         var checkbox_count = 0;
@@ -655,18 +656,20 @@ <h3>Notes</h3>
               $('div#bulk_edit_menu').removeClass('hidden');
           }
         }
+
+        $(document).on('keypress', null, 'e', function () {
+            window.location.assign('{% url 'edit_test' test.id %}');
+        });
+
+        $(document).on('keypress', null, 'a', function () {
+            window.location.assign('{% url 'add_findings' test.id %}');
+        });
+
         $(function () {
             $(document).ready(function(){
               $('[data-toggle="tooltip"]').tooltip();
             });
-            $(document).on('keypress', function(e) {
-                var key = String.fromCharCode(e.which);
-            if (key =='e') {
-                window.location.assign('{% url 'edit_test' test.id %}');
-            } else if (key === 'a') {
-                window.location.assign('{% url 'add_findings' test.id %}');
-            }
-            });
+
             check_checked_finding();
             $('#id_status').on('click', function (e) {
                 var checked = this.checked;