fix(util/ui): Display itins as list items of no itin active and url p…

…aram is full.

__tests__/util/ui.ts

@@ -11,15 +11,29 @@ describe('util > ui', () => {
         ItineraryView.LIST
       )
     })
-    it('returns a full itinerary view if URL contains ui_activeItinerary', () => {
+    it('returns a full itinerary view if URL contains ui_activeItinerary that is not -1', () => {
       expect(getItineraryView({ ui_activeItinerary: 2 })).toBe(
         ItineraryView.FULL
       )
     })
-    it('returns the specified view mode when set in URL', () => {
-      expect(getItineraryView({ ui_itineraryView: 'leg' })).toBe(
-        ItineraryView.LEG
+    it('returns an itinerary list view if URL contains ui_activeItinerary=-1 regardless of ui_itineraryView', () => {
+      expect(getItineraryView({ ui_activeItinerary: -1 })).toBe(
+        ItineraryView.LIST
       )
+      expect(
+        getItineraryView({
+          ui_activeItinerary: -1,
+          ui_itineraryView: ItineraryView.FULL
+        })
+      ).toBe(ItineraryView.LIST)
+    })
+    it('returns the specified view mode when set in URL', () => {
+      expect(
+        getItineraryView({
+          ui_activeItinerary: 0,
+          ui_itineraryView: ItineraryView.LEG
+        })
+      ).toBe(ItineraryView.LEG)
     })
   })
 })

lib/util/ui.ts

@@ -110,6 +110,10 @@ export function getItineraryView({
   ui_itineraryView
 }: UrlParams): ItineraryView {
   return (
+    ((ui_activeItinerary === null ||
+      ui_activeItinerary === undefined ||
+      `${ui_activeItinerary}` === '-1') &&
+      ItineraryView.LIST) ||
     ui_itineraryView ||
     (isDefinedAndNotEqual(ui_activeItinerary, -1) && ItineraryView.FULL) ||
     ItineraryView.LIST