mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-28 13:10:19 +00:00
Code Issues Projects Releases Packages Wiki Activity

multi-dive editing: don't change already set data for other dives

Browse source 
When editing multiple dives at the same time, don't change fields that
have already been set for a dive, unless the old field contents match
the currently selected ("master") dive.

So when you edit multiple dives, you can set the dive master or buddy
(or suit etc) for all of them in one go, but if one of them already has
that field set, it won't be modified just because you set the other
ones.

Acked-by: Dirk Hohndel <dirk@hohndel.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Linus Torvalds 2012-08-16 20:30:32 -07:00
parent f6dfb0094c
commit 2f773b97e0
1 changed files with 55 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

73
info.c
View file

@ -43,12 +43,42 @@ static int text_changed(const char *old, const char *new)
(!old && strcmp("",new));
}
static char *get_combo_box_entry_text(GtkComboBoxEntry *combo_box, char **textp)
static char *skip_space(char *str)
{
if (str) {
while (isspace(*str))
str++;
if (!*str)
str = NULL;
}
return str;
}
/*
* Get the string from a combo box.
*
* The "master" string is the string of the current dive - we only consider it
* changed if the old string is either empty, or matches that master string.
*/
static char *get_combo_box_entry_text(GtkComboBoxEntry *combo_box, char **textp, char *master)
{
char *old = *textp;
const gchar *new;
GtkEntry *entry;
old = skip_space(old);
master = skip_space(master);
/*
* If we had a master string, and it doesn't match our old
* string, we will always pick the old value (it means that
* we're editing another dive's info that already had a
* valid value).
*/
if (master && old)
if (strcmp(master, old))
return NULL;
entry = GTK_ENTRY(gtk_bin_get_child(GTK_BIN(combo_box)));
new = gtk_entry_get_text(entry);
while (isspace(*new))
@ -321,38 +351,38 @@ struct dive_info {
GtkTextView *notes;
};
static void save_dive_info_changes(struct dive *dive, struct dive_info *info)
static void save_dive_info_changes(struct dive *dive, struct dive *master, struct dive_info *info)
{
char *old_text, *new_text;
char *rating_string;
int changed = 0;
new_text = get_combo_box_entry_text(info->location, &dive->location);
new_text = get_combo_box_entry_text(info->location, &dive->location, master->location);
if (new_text) {
add_location(new_text);
changed = 1;
}
new_text = get_combo_box_entry_text(info->divemaster, &dive->divemaster);
new_text = get_combo_box_entry_text(info->divemaster, &dive->divemaster, master->divemaster);
if (new_text) {
add_people(new_text);
changed = 1;
}
new_text = get_combo_box_entry_text(info->buddy, &dive->buddy);
new_text = get_combo_box_entry_text(info->buddy, &dive->buddy, master->buddy);
if (new_text) {
add_people(new_text);
changed = 1;
}
new_text = get_combo_box_entry_text(info->suit, &dive->suit);
new_text = get_combo_box_entry_text(info->suit, &dive->suit, master->suit);
if (new_text) {
add_suit(new_text);
changed = 1;
}
rating_string = strdup(star_strings[dive->rating]);
new_text = get_combo_box_entry_text(info->rating, &rating_string);
new_text = get_combo_box_entry_text(info->rating, &rating_string, star_strings[master->rating]);
if (new_text) {
dive->rating = get_rating(rating_string);
free(rating_string);
@ -444,7 +474,7 @@ int edit_multi_dive_info(int nr, int *indices)
int success, i;
GtkWidget *dialog, *vbox;
struct dive_info info;
struct dive *dive;
struct dive *master;
if (!nr)
return 0;
@ -463,27 +493,34 @@ int edit_multi_dive_info(int nr, int *indices)
* data is used as the starting point for all selected dives
* I think it would be better to somehow collect and combine
* info from all the selected dives */
dive = current_dive;
dive_info_widget(vbox, dive, &info, (nr > 1));
show_dive_equipment(dive, W_IDX_SECONDARY);
save_equipment_data(dive);
master = current_dive;
dive_info_widget(vbox, master, &info, (nr > 1));
show_dive_equipment(master, W_IDX_SECONDARY);
save_equipment_data(master);
gtk_widget_show_all(dialog);
success = gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_ACCEPT;
if (success)
if (success) {
/* Update the other non-current dives first */
for (i = 0; i < nr; i++) {
int idx = indices[i];
struct dive *n = get_dive(idx);
struct dive *dive = get_dive(idx);
if (!n)
if (!dive || dive == master)
continue;
/* copy all "info" fields */
save_dive_info_changes(n, &info);
save_dive_info_changes(dive, master, &info);
/* copy the cylinders / weightsystems */
update_equipment_data(n, dive);
update_equipment_data(dive, master);
/* this is extremely inefficient... it loops through all
dives to find the right one - but we KNOW the index already */
flush_divelist(n);
flush_divelist(dive);
}
/* Update the master dive last! */
save_dive_info_changes(master, master, &info);
update_equipment_data(master, master);
flush_divelist(master);
}
gtk_widget_destroy(dialog);
return success;