Fixing branch select onChange() function, hopefully fixes Bug 1957, Holiday Calendar broken in Internet Explorer

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

koha-tmpl/intranet-tmpl/prog/en/modules/tools/holidays.tmpl

@@ -88,7 +88,7 @@
 	}
 
 	function changeBranch () {
-		var branch = document.getElementById('branch').options[document.getElementById('branch').selectedIndex].value;
+		var branch = $("#branch option:selected").val();
 		location.href='/cgi-bin/koha/tools/holidays.pl?branch=' + branch;
 	}
 
@@ -108,6 +108,11 @@
 	function Help() {
 		newin=window.open("/cgi-bin/koha/help.pl","Koha Help",'width=600,height=600,toolbar=false,scrollbars=yes');
 	}
+	$(document).ready(function() {
+		$("#branch").change(function(){
+			changeBranch();
+		});
+	});
 //]]>
 </script>
 

tools/holidays.pl

@@ -60,8 +60,7 @@ my $branchesList = CGI::scrolling_list(-name => 'branch',
                                        -size => 1,
                                        -default => [$branch],
                                        -multiple => 0,
-                                       -id => "branch",
+                                       -id => "branch");
-                                       -onChange => "changeBranch()");
 
 $branches->finish;