Fixed darn IE9 bug with history

Author: sparkica

static/js/faceted-search-events.js

@@ -19,26 +19,56 @@ var Codeweek = window.Codeweek || {};
 				var data = $('#faceted-search-events').serialize();
 				var url = $('#faceted-search-events').attr('action');
 
-				$.get(url, data, function(fragment) {
-					History.replaceState({}, document.title, url+'?'+data);
-					container.empty();
-					container.html(fragment);
-					var no_results = $('body').data('all_results');
-					var result_string = '';
-
-					if (no_results > 0 ) {
-						result_string = (no_results > 1 ? ' events match ' : ' event matches ');
-						search_counter_container.html(no_results + result_string + 'your search criteria: ');
-						$('#all-search-results').show();
-					} else {
-						$('#all-search-results').hide();
-					}
-
-				});
+				if (is_modern_browser()) {
+					$.get(url, data, function(fragment) {
+						history.replaceState({}, document.title, '?' + data);
+						container.empty();
+						container.html(fragment);
+						var no_results = $('body').data('all_results');
+						var result_string = '';
+
+						if (no_results > 0 ) {
+							result_string = (no_results > 1 ? ' events match ' : ' event matches ');
+							search_counter_container.html(no_results + result_string + 'your search criteria: ');
+							search_counter_container.show();
+						} else {
+							search_counter_container.hide();
+						}
+
+					});
+				} else {
+					document.getElementById('faceted-search-events').submit();
+				}
+
 			});
 		});
 	};
 
+	function is_modern_browser() {
+
+		var ie_version = msieversion();
+
+		if('querySelector' in document
+		     && 'localStorage' in window
+		     && 'addEventListener' in window
+		     && ( ie_version==0 || ie_version > 9)) {
+		     // bootstrap the javascript application
+		 	return true;
+		}
+
+		return false;
+	}
+
+	function msieversion() {
+
+		var ua = window.navigator.userAgent;
+		var msie = ua.indexOf("MSIE ");
+
+		if (msie > 0 || !!navigator.userAgent.match(/Trident.*rv\:11\./))      // If Internet Explorer, return version number
+			return parseInt(ua.substring(msie + 5, ua.indexOf(".", msie)));
+		return 0;
+	}
+
 	Codeweek.FacetedSearch = {};
 	Codeweek.FacetedSearch.init = init;
 

static/js/history.js/scripts/bundled/html4+html5/dojo.history.js

@@ -43,7 +43,7 @@
 			<div class="clearfix">
 				<div id="search-events-link">
 					<a class="btn btn-primary btn-lg"
-					   href="{% url 'web.search_events' %}?country_code={{ country.country_code }}&amp;past={{ past }}">
+					   href="{% url 'web.search_events' %}?country={{ country.country_code }}&amp;past={{ past }}">
 						<i class="fa fa-list"></i> List all events {% if country %}in
 						<span id="country"> {{ country.country_name }}</span>{% endif %}
 					</a>

static/js/history.js/scripts/bundled/html4+html5/extjs.history.js

@@ -58,7 +58,6 @@
 
 {% block custom_js %}
 {{ block.super }}
-<script type="text/javascript" src="{% static "js/history.js/scripts/bundled/html4+html5/jquery.history.js" %}" ></script>
 <script type="text/javascript" src="{% static "js/endless_pagination/endless-pagination.js" %}" ></script>
 <script type="text/javascript" src="{% static "js/faceted-search-events.js" %}" ></script>
 <script type="text/javascript">

static/js/history.js/scripts/bundled/html4+html5/jquery.history.js

@@ -131,7 +131,7 @@ def test_search_with_theme_multiple_events(self):
 	def test_search_with_theme_multiple_events_all_countries(self):
 		approved1 = ApprovedEventFactory.create(title="Event Arglebargle - Approved", country="SI")
 		approved2 = ApprovedEventFactory.create(title="Event Arglebargle - Approved", country="AT")
-		response = self.client.get(reverse('web.search_events'), {'country_code':'00', 'theme':1}, REMOTE_ADDR='93.103.53.11')
+		response = self.client.get(reverse('web.search_events'), {'country':'00', 'theme':1}, REMOTE_ADDR='93.103.53.11')
 
 		self.assertEquals(2,response.context['events'].count())
 
@@ -152,7 +152,7 @@ def test_search_events_with_search_query_all_countries_multiple_results(self):
 		approved1 = ApprovedEventFactory.create(title="Event Arglebargle - Approved", country="SI")
 		approved2 = ApprovedEventFactory.create(title="Event Arglebargle - Approved", country="AT")
 
-		response = self.client.get(reverse('web.search_events'), {'q':'arglebargle', 'country_code':'00'}, REMOTE_ADDR='93.103.53.11')
+		response = self.client.get(reverse('web.search_events'), {'q':'arglebargle', 'country':'00'}, REMOTE_ADDR='93.103.53.11')
 
 		self.assertEquals(2,response.context['events'].count())
 

static/js/history.js/scripts/bundled/html4+html5/mootools.history.js

@@ -253,10 +253,7 @@ def created_events(request):
 
 def search_events(request):
 
-		country_filter = request.GET.get('country_code', None)
-
-		if not country_filter:
-			country_filter = request.GET.get('country', None)
+		country_filter = request.GET.get('country', None)
 
 		if not country_filter :
 			user_ip = get_client_ip(forwarded=request.META.get('HTTP_X_FORWARDED_FOR'),
@@ -274,18 +271,7 @@ def search_events(request):
 
 		template = 'pages/search_events.html'
 
-		if request.method == 'POST':
-			form = SearchEventForm(request.POST)
-
-			if form.is_valid():
-				search_filter = form.cleaned_data.get('q', None)
-				country_filter = form.cleaned_data.get('country', None)
-				theme_filter = form.cleaned_data.get('theme', None)
-				audience_filter = form.cleaned_data.get('audience', None)
-				past_events = form.cleaned_data.get('past',None)
-
-				events = get_filtered_events(search_filter, country_filter, theme_filter, audience_filter, past_events)
-		else:
+		if request.method == 'GET':
 			theme_filter = request.GET.getlist('theme', None)
 			audience_filter =request.GET.getlist('audience', None)