From 0a9ad49f0aaf45a4d89c4ed90ce2c5c6fab154d5 Mon Sep 17 00:00:00 2001
From: Dirk Hohndel <dirk@hohndel.org>
Date: Wed, 10 Apr 2013 10:04:50 -0700
Subject: [PATCH] Fix selection handling for invalid dives

This is the downside of having to track our own selection state. We now
have a class of dives that potentially isn't known to Gtk as we aren't
adding them to the TreeView. So we need to make sure that their selection
state is consistent by deselecting them (either when they are marked as
invalid OR when we change preferences to no longer show invalid dives).

And because Gtk sends out another set of selection events when clearing
the TreeView (not sure why), we also need to make sure that their
selection state isn't reset to selected by mistake when the divelist is
recreated.

A bit of a pain, but it seems to work now and be consistent.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
---
 divelist-gtk.c | 11 ++++++++---
 divelist.c     |  3 +++
 gtk-gui.c      | 14 ++++++++++++++
 info.c         |  7 +++++++
 4 files changed, 32 insertions(+), 3 deletions(-)

diff --git a/divelist-gtk.c b/divelist-gtk.c
index f5ed73eb6..ad0f5efe8 100644
--- a/divelist-gtk.c
+++ b/divelist-gtk.c
@@ -1074,9 +1074,6 @@ static void invalid_dives_cb(GtkWidget *menuitem, GtkTreePath *path)
 		return;
 	/* walk the dive list in chronological order */
 	for_each_dive(i, dive) {
-		dive = get_dive(i);
-		if (!dive)
-			continue;
 		if (!dive->selected)
 			continue;
 		/* now swap the invalid tag if just 1 dive was selected
@@ -1093,7 +1090,15 @@ static void invalid_dives_cb(GtkWidget *menuitem, GtkTreePath *path)
 				changed = 1;
 			}
                 }
+		/* if invalid dives aren't shown they need to be
+		 * de-selected here to avoid confusion */
+		if (dive->selected && !prefs.display_invalid_dives) {
+			dive->selected = 0;
+			amount_selected--;
+		}
 	}
+	if (amount_selected == 0)
+		selected_dive = -1;
 	if (changed) {
 		dive_list_update_dives();
 		mark_divelist_changed(TRUE);
diff --git a/divelist.c b/divelist.c
index 13ed5113f..d105da52e 100644
--- a/divelist.c
+++ b/divelist.c
@@ -852,6 +852,9 @@ void select_dive(int idx)
 {
 	struct dive *dive = get_dive(idx);
 	if (dive && !dive->selected) {
+		/* never select an invalid dive that isn't displayed */
+		if (dive->dive_tags & DTAG_INVALID && !prefs.display_invalid_dives)
+			return;
 		dive->selected = 1;
 		amount_selected++;
 		selected_dive = idx;
diff --git a/gtk-gui.c b/gtk-gui.c
index eb07807b0..1dd73b21d 100644
--- a/gtk-gui.c
+++ b/gtk-gui.c
@@ -1027,9 +1027,23 @@ static void preferences_dialog(GtkWidget *w, gpointer data)
 	result = gtk_dialog_run(GTK_DIALOG(dialog));
 	if (result == GTK_RESPONSE_ACCEPT) {
 		const char *po2_threshold_text, *pn2_threshold_text, *phe_threshold_text, *mod_text, *gflow_text, *gfhigh_text;
+		int j;
+		struct dive *d;
+
 		/* Make sure to flush any modified old dive data with old units */
 		update_dive(NULL);
 
+		/* if we turned off displaying invalid dives. de-select all
+		 * invalid dives that were selected before hiding them */
+		if (oldprefs.display_invalid_dives && !prefs.display_invalid_dives) {
+			for_each_dive(j, d)
+				if (d->selected && d->dive_tags && DTAG_INVALID) {
+					d->selected = 0;
+					amount_selected--;
+				}
+			if (amount_selected == 0)
+				selected_dive = -1;
+		}
 		prefs.divelist_font = strdup(gtk_font_button_get_font_name(GTK_FONT_BUTTON(font)));
 		set_divelist_font(prefs.divelist_font);
 		po2_threshold_text = gtk_entry_get_text(GTK_ENTRY(entry_po2));
diff --git a/info.c b/info.c
index 724c1dcfd..c5f918e9d 100644
--- a/info.c
+++ b/info.c
@@ -609,6 +609,13 @@ static void save_dive_info_changes(struct dive *dive, struct dive *master, struc
 	if (tags_shown && dive->dive_tags != master->dive_tags) {
 		changed = 1;
 		dive->dive_tags = master->dive_tags;
+		/* if a dive is selected and we aren't showing invalid dives and it is
+		 * now marked as invalid we need to deselect it to keep the selection
+		 * state consistent */
+		if (!prefs.display_invalid_dives && dive->selected && dive->dive_tags & DTAG_INVALID) {
+			dive->selected = 0;
+			amount_selected--;
+		}
 	}
 	if (changed) {
 		mark_divelist_changed(TRUE);