# HG changeset patch
# User nadvornik
# Date 1205610813 0
# Node ID 9eafc4957f1ad46c0c756d25a09973aad3425fd9
# Parent  bd3fc1aa7fe939d8e985ee7d653241d3e5146dd2
write support in Exiv2 wrapper; for now only string values

diff -r bd3fc1aa7fe9 -r 9eafc4957f1a src/exif.c
--- a/src/exif.c	Sun Feb 24 18:55:19 2008 +0000
+++ b/src/exif.c	Sat Mar 15 19:53:33 2008 +0000
@@ -1608,5 +1608,28 @@
 	fprintf(f, "----------------------------------------------------\n");
 }
 
+int exif_write(ExifData *exif)
+{
+	printf("Not compiled with EXIF write support");
+	return 0;
+}
+
+ExifItem *exif_add_item(ExifData *exif, const gchar *key)
+{
+	return NULL;
+}
+
+int exif_item_delete(ExifData *exif, ExifItem *item)
+{
+	return 0;
+}
+
+int exif_item_set_string(ExifItem *item, const char *str)
+{
+	return 0;
+}
+
+
+
 #endif 
 /* not HAVE_EXIV2 */
diff -r bd3fc1aa7fe9 -r 9eafc4957f1a src/exif.h
--- a/src/exif.h	Sun Feb 24 18:55:19 2008 +0000
+++ b/src/exif.h	Sat Mar 15 19:53:33 2008 +0000
@@ -106,6 +106,7 @@
  */
 
 ExifData *exif_read(gchar *path, gint parse_color_profile);
+int exif_write(ExifData *exif);
 void exif_free(ExifData *exif);
 
 gchar *exif_get_data_as_text(ExifData *exif, const gchar *key);
@@ -115,9 +116,11 @@
 double exif_get_rational_as_double(ExifData *exif, const gchar *key);
 
 ExifItem *exif_get_item(ExifData *exif, const gchar *key);
+ExifItem *exif_add_item(ExifData *exif, const gchar *key);
 ExifItem *exif_get_first_item(ExifData *exif);
 ExifItem *exif_get_next_item(ExifData *exif);
 
+
 char *exif_item_get_tag_name(ExifItem *item);
 guint exif_item_get_tag_id(ExifItem *item);
 guint exif_item_get_elements(ExifItem *item);
@@ -134,6 +137,10 @@
 
 gchar *exif_get_formatted_by_key(ExifData *exif, const gchar *key, gint *key_valid);
 
+int exif_item_delete(ExifData *exif, ExifItem *item);
+int exif_item_set_string(ExifItem *item, const char *str);
+
+
 gint format_raw_img_exif_offsets_fd(int fd, const gchar *path,
 				    unsigned char *header_data, const guint header_len,
 				    guint *image_offset, guint *exif_offset);
diff -r bd3fc1aa7fe9 -r 9eafc4957f1a src/exiv2.cc
--- a/src/exiv2.cc	Sun Feb 24 18:55:19 2008 +0000
+++ b/src/exiv2.cc	Sat Mar 15 19:53:33 2008 +0000
@@ -34,22 +34,23 @@
 
 struct _ExifData
 {
-	Exiv2::ExifData exifData;
+	Exiv2::Image::AutoPtr image;
 	Exiv2::ExifData::const_iterator exifIter; /* for exif_get_next_item */
-	Exiv2::IptcData iptcData;
 	Exiv2::IptcData::const_iterator iptcIter; /* for exif_get_next_item */
-	Exiv2::XmpData xmpData;
 	Exiv2::XmpData::const_iterator xmpIter; /* for exif_get_next_item */
 
 	_ExifData(gchar *path, gint parse_color_profile)
 	{
-		Exiv2::Image::AutoPtr image = Exiv2::ImageFactory::open(path);
-		g_assert (image.get() != 0);
+		image = Exiv2::ImageFactory::open(path);
+//		g_assert (image.get() != 0);
 		image->readMetadata();
-		exifData = image->exifData();
-		iptcData = image->iptcData();
-		xmpData = image->xmpData();
 	}
+	
+	void writeMetadata()
+	{
+		image->writeMetadata();
+	}
+	
 
 };
 
@@ -68,6 +69,20 @@
 	
 }
 
+int exif_write(ExifData *exif)
+{
+	try {
+		exif->writeMetadata();
+		return 1;
+	}
+	catch (Exiv2::AnyError& e) {
+		std::cout << "Caught Exiv2 exception '" << e << "'\n";
+		return 0;
+	}
+	
+}
+
+
 void exif_free(ExifData *exif)
 {
 	
@@ -80,21 +95,56 @@
 		Exiv2::Metadatum *item;
 		try {
 			Exiv2::ExifKey ekey(key);
-			Exiv2::ExifData::iterator pos = exif->exifData.findKey(ekey);
-			if (pos == exif->exifData.end()) return NULL;
+			Exiv2::ExifData::iterator pos = exif->image->exifData().findKey(ekey);
+			if (pos == exif->image->exifData().end()) return NULL;
 			item = &*pos;
 		}
 		catch (Exiv2::AnyError& e) {
 			try {
 				Exiv2::IptcKey ekey(key);
-				Exiv2::IptcData::iterator pos = exif->iptcData.findKey(ekey);
-				if (pos == exif->iptcData.end()) return NULL;
+				Exiv2::IptcData::iterator pos = exif->image->iptcData().findKey(ekey);
+				if (pos == exif->image->iptcData().end()) return NULL;
 				item = &*pos;
 			}
 			catch (Exiv2::AnyError& e) {
 				Exiv2::XmpKey ekey(key);
-				Exiv2::XmpData::iterator pos = exif->xmpData.findKey(ekey);
-				if (pos == exif->xmpData.end()) return NULL;
+				Exiv2::XmpData::iterator pos = exif->image->xmpData().findKey(ekey);
+				if (pos == exif->image->xmpData().end()) return NULL;
+				item = &*pos;
+			}
+		}
+		return (ExifItem *)item;
+	}
+	catch (Exiv2::AnyError& e) {
+		std::cout << "Caught Exiv2 exception '" << e << "'\n";
+		return NULL;
+	}
+}
+
+ExifItem *exif_add_item(ExifData *exif, const gchar *key)
+{
+	try {
+		Exiv2::Metadatum *item;
+		try {
+			Exiv2::ExifKey ekey(key);
+			exif->image->exifData().add(ekey, NULL);
+			Exiv2::ExifData::iterator pos = exif->image->exifData().end(); // a hack, there should be a better way to get the currently added item
+			pos--;
+			item = &*pos;
+		}
+		catch (Exiv2::AnyError& e) {
+			try {
+				Exiv2::IptcKey ekey(key);
+				exif->image->iptcData().add(ekey, NULL);
+				Exiv2::IptcData::iterator pos = exif->image->iptcData().end();
+				pos--;
+				item = &*pos;
+			}
+			catch (Exiv2::AnyError& e) {
+				Exiv2::XmpKey ekey(key);
+				exif->image->xmpData().add(ekey, NULL);
+				Exiv2::XmpData::iterator pos = exif->image->xmpData().end();
+				pos--;
 				item = &*pos;
 			}
 		}
@@ -110,22 +160,22 @@
 ExifItem *exif_get_first_item(ExifData *exif)
 {
 	try {
-		exif->exifIter = exif->exifData.begin();
-		exif->iptcIter = exif->iptcData.begin();
-		exif->xmpIter = exif->xmpData.begin();
-		if (exif->exifIter != exif->exifData.end()) 
+		exif->exifIter = exif->image->exifData().begin();
+		exif->iptcIter = exif->image->iptcData().begin();
+		exif->xmpIter = exif->image->xmpData().begin();
+		if (exif->exifIter != exif->image->exifData().end()) 
 			{
 			const Exiv2::Metadatum *item = &*exif->exifIter;
 			exif->exifIter++;
 			return (ExifItem *)item;
 			}
-		if (exif->iptcIter != exif->iptcData.end()) 
+		if (exif->iptcIter != exif->image->iptcData().end()) 
 			{
 			const Exiv2::Metadatum *item = &*exif->iptcIter;
 			exif->iptcIter++;
 			return (ExifItem *)item;
 			}
-		if (exif->xmpIter != exif->xmpData.end()) 
+		if (exif->xmpIter != exif->image->xmpData().end()) 
 			{
 			const Exiv2::Metadatum *item = &*exif->xmpIter;
 			exif->xmpIter++;
@@ -143,19 +193,19 @@
 ExifItem *exif_get_next_item(ExifData *exif)
 {
 	try {
-		if (exif->exifIter != exif->exifData.end())
+		if (exif->exifIter != exif->image->exifData().end())
 			{
 			const Exiv2::Metadatum *item = &*exif->exifIter;
 			exif->exifIter++;
 			return (ExifItem *)item;
 		}
-		if (exif->iptcIter != exif->iptcData.end())
+		if (exif->iptcIter != exif->image->iptcData().end())
 			{
 			const Exiv2::Metadatum *item = &*exif->iptcIter;
 			exif->iptcIter++;
 			return (ExifItem *)item;
 		}
-		if (exif->xmpIter != exif->xmpData.end())
+		if (exif->xmpIter != exif->image->xmpData().end())
 			{
 			const Exiv2::Metadatum *item = &*exif->xmpIter;
 			exif->xmpIter++;
@@ -337,6 +387,49 @@
 	}
 }
 
+int exif_item_set_string(ExifItem *item, const char *str)
+{
+	try {
+		if (!item) return 0;
+		((Exiv2::Metadatum *)item)->setValue(std::string(str));
+		return 1;
+	}
+	catch (Exiv2::AnyError& e) {
+		return 0;
+	}
+}
+
+int exif_item_delete(ExifData *exif, ExifItem *item)
+{
+	try {
+		if (!item) return 0;
+		for (Exiv2::ExifData::iterator i = exif->image->exifData().begin(); i != exif->image->exifData().end(); ++i) {
+			if (((Exiv2::Metadatum *)item) == &*i) {
+				i = exif->image->exifData().erase(i);
+				return 1;
+			}
+		}
+		for (Exiv2::IptcData::iterator i = exif->image->iptcData().begin(); i != exif->image->iptcData().end(); ++i) {
+			if (((Exiv2::Metadatum *)item) == &*i) {
+				i = exif->image->iptcData().erase(i);
+				return 1;
+			}
+		}
+		for (Exiv2::XmpData::iterator i = exif->image->xmpData().begin(); i != exif->image->xmpData().end(); ++i) {
+			if (((Exiv2::Metadatum *)item) == &*i) {
+				i = exif->image->xmpData().erase(i);
+				return 1;
+			}
+		}
+		
+		return 0;
+	}
+	catch (Exiv2::AnyError& e) {
+		return 0;
+	}
+}
+
+
 
 }