media: move addition of pictures out of create_picture()

If we want to make addition of pictures undoable, then create_picture()
must not add directly to the dive. Instead, return the dive to which the
picture should be added and let the caller perform the addition.

This means that the picture-test has to be adapted.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

2 files changed, 20 insertions, 19 deletions

diff --git a/core/dive.c b/core/dive.c
index 8ed9995b8..5ac008e8f 100644
--- a/core/dive.c
+++ b/core/dive.c
@@ -3556,7 +3556,7 @@ bool picture_check_valid_time(timestamp_t timestamp, int shift_time)
 	return false;
 }
 
-static void dive_set_geodata_from_picture(struct dive *dive, struct picture *picture, struct dive_site_table *table)
+void dive_set_geodata_from_picture(struct dive *dive, struct picture *picture, struct dive_site_table *table)
 {
 	struct dive_site *ds = dive->dive_site;
 	if (!dive_site_has_gps_location(ds) && has_location(&picture->location)) {
@@ -3570,31 +3570,31 @@ static void dive_set_geodata_from_picture(struct dive *dive, struct picture *pic
 	}
 }
 
-void create_picture(const char *filename, int shift_time, bool match_all)
+/* Creates a picture and indicates the dive to which this picture should be added.
+ * The caller is responsible for actually adding the picture to the dive.
+ * If no appropriate dive was found, no picture is created and NULL is returned.
+ */
+struct picture *create_picture(const char *filename, int shift_time, bool match_all, struct dive **dive)
 {
 	struct metadata metadata;
-	struct dive *dive;
 	timestamp_t timestamp;
 
 	get_metadata(filename, &metadata);
 	timestamp = metadata.timestamp + shift_time;
-	dive = nearest_selected_dive(timestamp);
-
-	if (!dive)
-		return;
-	if (get_picture_idx(&dive->pictures, filename) >= 0)
-		return;
-	if (!match_all && !dive_check_picture_time(dive, timestamp))
-		return;
+	*dive = nearest_selected_dive(timestamp);
 
-	struct picture picture;
-	picture.filename = strdup(filename);
-	picture.offset.seconds = metadata.timestamp - dive->when + shift_time;
-	picture.location = metadata.location;
+	if (!*dive)
+		return NULL;
+	if (get_picture_idx(&(*dive)->pictures, filename) >= 0)
+		return NULL;
+	if (!match_all && !dive_check_picture_time(*dive, timestamp))
+		return NULL;
 
-	add_picture(&dive->pictures, picture);
-	dive_set_geodata_from_picture(dive, &picture, &dive_site_table);
-	invalidate_dive_cache(dive);
+	struct picture *picture = malloc(sizeof(struct picture));
+	picture->filename = strdup(filename);
+	picture->offset.seconds = metadata.timestamp - (*dive)->when + shift_time;
+	picture->location = metadata.location;
+	return picture;
 }
 
 /* clones a dive and moves given dive computer to front */
diff --git a/core/dive.h b/core/dive.h
index 33c7c8b0f..8e9ee280a 100644
--- a/core/dive.h
+++ b/core/dive.h
@@ -210,8 +210,9 @@ extern enum divemode_t get_divemode_at_time(const struct divecomputer *dc, int d
 		for (struct picture *picture = (_dive)->pictures.pictures;		\
 		     picture < (_dive)->pictures.pictures + (_dive)->pictures.nr;	\
 		     picture++)
-extern void create_picture(const char *filename, int shift_time, bool match_all);
+extern struct picture *create_picture(const char *filename, int shift_time, bool match_all, struct dive **dive);
 extern bool picture_check_valid_time(timestamp_t timestamp, int shift_time);
+extern void dive_set_geodata_from_picture(struct dive *dive, struct picture *picture, struct dive_site_table *table);
 
 extern bool has_gaschange_event(const struct dive *dive, const struct divecomputer *dc, int idx);
 extern int explicit_first_cylinder(const struct dive *dive, const struct divecomputer *dc);