Handled relative paths when loading and saving binary contents in projects.

7 files changed, 42 insertions, 20 deletions

diff --git a/src/analysis/content-int.h b/src/analysis/content-int.h
index 36168be..d9f9ef9 100644
--- a/src/analysis/content-int.h
+++ b/src/analysis/content-int.h
@@ -33,7 +33,7 @@
 typedef const char * (* describe_content_fc) (const GBinContent *, bool);
 
 /* Ecrit une sauvegarde de contenu binaire dans un fichier XML. */
-typedef bool (* save_content_fc) (const GBinContent *, xmlDocPtr, xmlXPathContextPtr, const char *);
+typedef bool (* save_content_fc) (const GBinContent *, xmlDocPtr, xmlXPathContextPtr, const char *, const char *);
 
 /* Fournit une empreinte unique (SHA256) pour les données. */
 typedef const gchar * (* get_checksum_fc) (GBinContent *);
diff --git a/src/analysis/content.c b/src/analysis/content.c
index d86a351..01c9c99 100644
--- a/src/analysis/content.c
+++ b/src/analysis/content.c
@@ -63,6 +63,7 @@ static void g_binary_content_default_init(GBinContentInterface *iface)
 *                                                                             *
 *  Paramètres  : context = contexte pour les recherches XPath.                *
 *                path    = chemin d'accès au noeud XML à lire.                *
+*                base    = référence au lieu d'enregistrement du projet.      *
 *                                                                             *
 *  Description : Charge en mémoire un contenu binaire à partir d'XML.         *
 *                                                                             *
@@ -72,7 +73,7 @@ static void g_binary_content_default_init(GBinContentInterface *iface)
 *                                                                             *
 ******************************************************************************/
 
-GBinContent *g_binary_content_new_from_xml(xmlXPathContextPtr context, const char *path)
+GBinContent *g_binary_content_new_from_xml(xmlXPathContextPtr context, const char *path, const char *base)
 {
     GBinContent *result;                    /* Contenu en place à renvoyer */
     char *type;                             /* Type de binaire à charger   */
@@ -82,7 +83,7 @@ GBinContent *g_binary_content_new_from_xml(xmlXPathContextPtr context, const cha
     type = get_node_prop_value(context, path, "type");
 
     if (strcmp(type, "file") == 0)
-        result = g_file_content_new_from_xml(context, path);
+        result = g_file_content_new_from_xml(context, path, base);
 
     free(type);
 
@@ -121,6 +122,7 @@ const char *g_binary_content_describe(const GBinContent *content, bool full)
 *                xdoc    = structure XML en cours d'édition.                  *
 *                context = contexte à utiliser pour les recherches.           *
 *                path    = chemin d'accès réservé au binaire.                 *
+*                base    = référence au lieu d'enregistrement du projet.      *
 *                                                                             *
 *  Description : Ecrit une sauvegarde de contenu binaire dans un fichier XML. *
 *                                                                             *
@@ -130,13 +132,13 @@ const char *g_binary_content_describe(const GBinContent *content, bool full)
 *                                                                             *
 ******************************************************************************/
 
-bool g_binary_content_save(const GBinContent *content, xmlDocPtr xdoc, xmlXPathContextPtr context, const char *path)
+bool g_binary_content_save(const GBinContent *content, xmlDocPtr xdoc, xmlXPathContextPtr context, const char *path, const char *base)
 {
     GBinContentIface *iface;                /* Interface utilisée          */
 
     iface = G_BIN_CONTENT_GET_IFACE(content);
 
-    return iface->save(content, xdoc, context, path);
+    return iface->save(content, xdoc, context, path, base);
 
 }
 
diff --git a/src/analysis/content.h b/src/analysis/content.h
index 591097d..1a61cb7 100644
--- a/src/analysis/content.h
+++ b/src/analysis/content.h
@@ -55,13 +55,13 @@ typedef struct _GBinContentIface GBinContentIface;
 GType g_binary_content_get_type(void) G_GNUC_CONST;
 
 /* Charge en mémoire un contenu binaire à partir d'XML. */
-GBinContent *g_binary_content_new_from_xml(xmlXPathContextPtr, const char *);
+GBinContent *g_binary_content_new_from_xml(xmlXPathContextPtr, const char *, const char *);
 
 /* Fournit le nom associé au contenu binaire. */
 const char *g_binary_content_describe(const GBinContent *, bool);
 
 /* Ecrit une sauvegarde de contenu binaire dans un fichier XML. */
-bool g_binary_content_save(const GBinContent *, xmlDocPtr, xmlXPathContextPtr, const char *);
+bool g_binary_content_save(const GBinContent *, xmlDocPtr, xmlXPathContextPtr, const char *, const char *);
 
 /* Fournit une empreinte unique (SHA256) pour les données. */
 const gchar *g_binary_content_get_cheksum(GBinContent *);
diff --git a/src/analysis/contents/file.c b/src/analysis/contents/file.c
index f24e930..8416348 100644
--- a/src/analysis/contents/file.c
+++ b/src/analysis/contents/file.c
@@ -35,6 +35,7 @@
 
 #include "../content-int.h"
 #include "../../common/extstr.h"
+#include "../../common/pathname.h"
 
 
 
@@ -80,7 +81,7 @@ static void g_file_content_finalize(GFileContent *);
 static const char *g_file_content_describe(const GFileContent *, bool);
 
 /* Ecrit une sauvegarde de contenu binaire dans un fichier XML. */
-static bool g_file_content_save(const GFileContent *, xmlDocPtr, xmlXPathContextPtr, const char *);
+static bool g_file_content_save(const GFileContent *, xmlDocPtr, xmlXPathContextPtr, const char *, const char *);
 
 /* Fournit une empreinte unique (SHA256) pour les données. */
 static const gchar *g_file_content_get_checksum(GFileContent *);
@@ -324,6 +325,7 @@ GBinContent *g_file_content_new(const char *filename)
 *                                                                             *
 *  Paramètres  : context = contexte pour les recherches XPath.                *
 *                path    = chemin d'accès au noeud XML à lire.                *
+*                base    = référence au lieu d'enregistrement du projet.      *
 *                                                                             *
 *  Description : Charge en mémoire le contenu d'un fichier à partir d'XML.    *
 *                                                                             *
@@ -333,11 +335,12 @@ GBinContent *g_file_content_new(const char *filename)
 *                                                                             *
 ******************************************************************************/
 
-GBinContent *g_file_content_new_from_xml(xmlXPathContextPtr context, const char *path)
+GBinContent *g_file_content_new_from_xml(xmlXPathContextPtr context, const char *path, const char *base)
 {
     GBinContent *result;                    /* Adresse à retourner         */
     char *access;                           /* Chemin pour une sous-config.*/
     char *filename;                         /* Chemin du binaire à charger */
+    char *absolute;                         /* Chemin absolu final         */
 
     result = NULL;
 
@@ -352,10 +355,14 @@ GBinContent *g_file_content_new_from_xml(xmlXPathContextPtr context, const char
 
     /* Chargement */
 
-    if (filename != NULL)
+    absolute = build_absolute_filename(base, filename);
+
+    if (filename != NULL) free(filename);
+
+    if (absolute != NULL)
     {
-        result = g_file_content_new(filename);
-        free(filename);
+        result = g_file_content_new(absolute);
+        free(absolute);
     }
 
     return result;
@@ -396,6 +403,7 @@ static const char *g_file_content_describe(const GFileContent *content, bool ful
 *                xdoc    = structure XML en cours d'édition.                  *
 *                context = contexte à utiliser pour les recherches.           *
 *                path    = chemin d'accès réservé au binaire.                 *
+*                base    = référence au lieu d'enregistrement du projet.      *
 *                                                                             *
 *  Description : Ecrit une sauvegarde de contenu binaire dans un fichier XML. *
 *                                                                             *
@@ -405,10 +413,11 @@ static const char *g_file_content_describe(const GFileContent *content, bool ful
 *                                                                             *
 ******************************************************************************/
 
-static bool g_file_content_save(const GFileContent *content, xmlDocPtr xdoc, xmlXPathContextPtr context, const char *path)
+static bool g_file_content_save(const GFileContent *content, xmlDocPtr xdoc, xmlXPathContextPtr context, const char *path, const char *base)
 {
     bool result;                            /* Bilan à faire remonter      */
     char *access;                           /* Chemin d'accès à un élément */
+    char *relative;                         /* Chemin d'accès relatif      */
 
     result = true;
 
@@ -421,8 +430,11 @@ static bool g_file_content_save(const GFileContent *content, xmlDocPtr xdoc, xml
     access = strdup(path);
     access = stradd(access, "/Filename");
 
-    result &= add_content_to_node(xdoc, context, access, content->filename);
+    relative = build_relative_filename(base, content->filename);
+
+    result &= add_content_to_node(xdoc, context, access, relative);
 
+    free(relative);
     free(access);
 
     return result;
diff --git a/src/analysis/contents/file.h b/src/analysis/contents/file.h
index 0985f6b..90e292e 100644
--- a/src/analysis/contents/file.h
+++ b/src/analysis/contents/file.h
@@ -54,7 +54,7 @@ GType g_file_content_get_type(void);
 GBinContent *g_file_content_new(const char *);
 
 /* Charge en mémoire le contenu d'un fichier à partir d'XML. */
-GBinContent *g_file_content_new_from_xml(xmlXPathContextPtr, const char *);
+GBinContent *g_file_content_new_from_xml(xmlXPathContextPtr, const char *, const char *);
 
 
 
diff --git a/src/analysis/project.c b/src/analysis/project.c
index ed60929..94b01a3 100644
--- a/src/analysis/project.c
+++ b/src/analysis/project.c
@@ -282,14 +282,20 @@ GStudyProject *g_study_project_open(GObject *ref, const char *filename)
         access = calloc(access_len, sizeof(char));
         snprintf(access, access_len, "/ChrysalideProject/Contents/Content[position()=%u]", i + 1);
 
-        content = g_binary_content_new_from_xml(context, access);
+        content = g_binary_content_new_from_xml(context, access, filename);
         status = get_node_prop_long_value(context, access, "state", &state);
 
         free(access);
 
+        if (content == NULL)
+        {
+            log_variadic_message(LMT_ERROR, _("Unable to load the binary content #%u ; skipping..."), i);
+            continue;
+        }
+
         if (!status)
         {
-            log_variadic_message(LMT_PROCESS, _("bad state for content '%s' ; skipping..."),
+            log_variadic_message(LMT_ERROR, _("Bad state for content '%s' ; skipping..."),
                                  g_binary_content_describe(content, true));
             continue;
         }
@@ -375,6 +381,7 @@ bool g_study_project_save(GStudyProject *project, const char *filename)
     bool result;                            /* Bilan à retourner           */
     xmlDocPtr xdoc;                         /* Document XML à créer        */
     xmlXPathContextPtr context;             /* Contexte pour les recherches*/
+    const char *final;                      /* Lieu d'enregistrement final */
     size_t i;                               /* Boucle de parcours          */
     size_t access_len;                      /* Taille d'un chemin interne  */
     char *access;                           /* Chemin pour une sous-config.*/
@@ -383,6 +390,8 @@ bool g_study_project_save(GStudyProject *project, const char *filename)
 
     result &= (ensure_node_exist(xdoc, context, "/ChrysalideProject") != NULL);
 
+    final = filename != NULL ? filename : project->filename;
+
     /* Enregistrement des contenus binaires attachés */
 
     for (i = 0; i < project->contents_count && result; i++)
@@ -394,7 +403,7 @@ bool g_study_project_save(GStudyProject *project, const char *filename)
         access = calloc(access_len, sizeof(char));
         snprintf(access, access_len, "/ChrysalideProject/Contents/Content[position()=%zu]", i + 1);
 
-        result = g_binary_content_save(project->contents[i].content, xdoc, context, access);
+        result = g_binary_content_save(project->contents[i].content, xdoc, context, access, final);
 
         if (result)
             result = add_long_attribute_to_node(xdoc, context, access, "state", project->contents[i].state);
@@ -420,7 +429,7 @@ bool g_study_project_save(GStudyProject *project, const char *filename)
 
     /* Sauvegarde finale */
 
-    result &= save_xml_file(xdoc, filename != NULL ? filename : project->filename);
+    result &= save_xml_file(xdoc, final);
 
     if (result && filename != NULL)
     {
diff --git a/src/common/xml.h b/src/common/xml.h
index 60891f7..674bfd4 100644
--- a/src/common/xml.h
+++ b/src/common/xml.h
@@ -34,7 +34,6 @@
 
 
 
-
 /* Crée un nouveau fichier XML. */
 bool create_new_xml_file(xmlDocPtr *, xmlXPathContextPtr *);