Code

Krzysztof's patch for 388257
[inkscape.git] / src / document.cpp
index 2a9014e3e3a91893d6fefce6aa7b917768d139d8..d406f3712d4ba2cf7a483efe3bf2449b0839c0d4 100644 (file)
@@ -61,6 +61,7 @@
 #include "transf_mat_3x4.h"
 #include "unit-constants.h"
 #include "xml/repr.h"
+#include "xml/rebase-hrefs.h"
 
 #define SP_DOCUMENT_UPDATE_PRIORITY (G_PRIORITY_HIGH_IDLE - 1)
 
@@ -223,6 +224,14 @@ void SPDocument::remove_persp3d (Persp3D * const /*persp*/)
     g_print ("Please implement deletion of perspectives here.\n");
 }
 
+void SPDocument::initialize_current_persp3d()
+{
+    this->current_persp3d = persp3d_document_first_persp(this);
+    if (!this->current_persp3d) {
+        this->current_persp3d = persp3d_create_xml_element(this);
+    }
+}
+
 unsigned long SPDocument::serial() const {
     return priv->serial;
 }
@@ -262,7 +271,6 @@ sp_document_create(Inkscape::XML::Document *rdoc,
 {
     SPDocument *document;
     Inkscape::XML::Node *rroot;
-    Inkscape::Version sodipodi_version;
     Inkscape::Preferences *prefs = Inkscape::Preferences::get();
 
     rroot = rdoc->root();
@@ -275,7 +283,7 @@ sp_document_create(Inkscape::XML::Document *rdoc,
     document->rroot = rroot;
 
 #ifndef WIN32
-    prepend_current_dir_if_relative(&(document->uri), uri);
+    document->uri = prepend_current_dir_if_relative(uri);
 #else
     // FIXME: it may be that prepend_current_dir_if_relative works OK on windows too, test!
     document->uri = uri? g_strdup(uri) : NULL;
@@ -292,10 +300,7 @@ sp_document_create(Inkscape::XML::Document *rdoc,
 
     document->root = sp_object_repr_build_tree(document, rroot);
 
-    sodipodi_version = SP_ROOT(document->root)->version.sodipodi;
-
     /* fixme: Not sure about this, but lets assume ::build updates */
-    rroot->setAttribute("sodipodi:version", SODIPODI_VERSION);
     rroot->setAttribute("inkscape:version", Inkscape::version_string);
     /* fixme: Again, I moved these here to allow version determining in ::build (Lauris) */
 
@@ -312,7 +317,7 @@ sp_document_create(Inkscape::XML::Document *rdoc,
 
     /* Eliminate obsolete sodipodi:docbase, for privacy reasons */
     rroot->setAttribute("sodipodi:docbase", NULL);
-    
+
     /* Eliminate any claim to adhere to a profile, as we don't try to */
     rroot->setAttribute("baseProfile", NULL);
 
@@ -320,7 +325,7 @@ sp_document_create(Inkscape::XML::Document *rdoc,
     if (!sp_item_group_get_child_by_name((SPGroup *) document->root, NULL, "sodipodi:namedview")) {
         // if there's none in the document already,
         Inkscape::XML::Node *rnew = NULL;
-        
+
         rnew = rdoc->createElement("sodipodi:namedview");
         //rnew->setAttribute("id", "base");
 
@@ -350,7 +355,7 @@ sp_document_create(Inkscape::XML::Document *rdoc,
             prefs->getInt("/template/base/inkscape:window-width", 640));
         sp_repr_set_int(rnew, "inkscape:window-height",
             prefs->getInt("/template/base/inkscape:window-height", 480));
-        
+
         // insert into the document
         rroot->addChild(rnew, NULL);
         // clean up
@@ -376,11 +381,7 @@ sp_document_create(Inkscape::XML::Document *rdoc,
     // Remark: Here, we used to create a "currentpersp3d" element in the document defs.
     // But this is probably a bad idea since we need to adapt it for every change of selection, which will
     // completely clutter the undo history. Maybe rather save it to prefs on exit and re-read it on startup?
-
-    document->current_persp3d = persp3d_document_first_persp(document);
-    if (!document->current_persp3d) {
-        document->current_persp3d = persp3d_create_xml_element (document);
-    }
+    document->initialize_current_persp3d();
 
     sp_document_set_undo_sensitive(document, true);
 
@@ -608,54 +609,83 @@ void SPDocument::fitToRect(Geom::Rect const &rect)
     }
 }
 
-void sp_document_set_uri(SPDocument *document, gchar const *uri)
+static void
+do_change_uri(SPDocument *const document, gchar const *const filename, bool const rebase)
 {
     g_return_if_fail(document != NULL);
 
-    if (document->name) {
-        g_free(document->name);
-        document->name = NULL;
-    }
-    if (document->base) {
-        g_free(document->base);
-        document->base = NULL;
-    }
-    if (document->uri) {
-        g_free(document->uri);
-        document->uri = NULL;
-    }
-
-    if (uri) {
+    gchar *new_base;
+    gchar *new_name;
+    gchar *new_uri;
+    if (filename) {
 
 #ifndef WIN32
-        prepend_current_dir_if_relative(&(document->uri), uri);
+        new_uri = prepend_current_dir_if_relative(filename);
 #else
         // FIXME: it may be that prepend_current_dir_if_relative works OK on windows too, test!
-        document->uri = g_strdup(uri);
+        new_uri = g_strdup(filename);
 #endif
 
-        /* fixme: Think, what this means for images (Lauris) */
-        document->base = g_path_get_dirname(document->uri);
-        document->name = g_path_get_basename(document->uri);
-
+        new_base = g_path_get_dirname(new_uri);
+        new_name = g_path_get_basename(new_uri);
     } else {
-        document->uri = g_strdup_printf(_("Unnamed document %d"), ++doc_count);
-        document->base = NULL;
-        document->name = g_strdup(document->uri);
+        new_uri = g_strdup_printf(_("Unnamed document %d"), ++doc_count);
+        new_base = NULL;
+        new_name = g_strdup(document->uri);
     }
 
     // Update saveable repr attributes.
     Inkscape::XML::Node *repr = sp_document_repr_root(document);
-    // changing uri in the document repr must not be not undoable
-    bool saved = sp_document_get_undo_sensitive(document);
+
+    // Changing uri in the document repr must not be not undoable.
+    bool const saved = sp_document_get_undo_sensitive(document);
     sp_document_set_undo_sensitive(document, false);
 
+    if (rebase) {
+        Inkscape::XML::rebase_hrefs(document, new_base, true);
+    }
+
     repr->setAttribute("sodipodi:docname", document->name);
     sp_document_set_undo_sensitive(document, saved);
 
+
+    g_free(document->name);
+    g_free(document->base);
+    g_free(document->uri);
+    document->name = new_name;
+    document->base = new_base;
+    document->uri = new_uri;
+
     document->priv->uri_set_signal.emit(document->uri);
 }
 
+/**
+ * Sets base, name and uri members of \a document.  Doesn't update
+ * any relative hrefs in the document: thus, this is primarily for
+ * newly-created documents.
+ *
+ * \see sp_document_change_uri_and_hrefs
+ */
+void sp_document_set_uri(SPDocument *document, gchar const *filename)
+{
+    g_return_if_fail(document != NULL);
+
+    do_change_uri(document, filename, false);
+}
+
+/**
+ * Changes the base, name and uri members of \a document, and updates any
+ * relative hrefs in the document to be relative to the new base.
+ *
+ * \see sp_document_set_uri
+ */
+void sp_document_change_uri_and_hrefs(SPDocument *document, gchar const *filename)
+{
+    g_return_if_fail(document != NULL);
+
+    do_change_uri(document, filename, true);
+}
+
 void
 sp_document_resized_signal_emit(SPDocument *doc, gdouble width, gdouble height)
 {
@@ -704,6 +734,10 @@ SPDocument::emitReconstructionFinish(void)
 {
     // printf("Finishing Reconstruction\n");
     priv->_reconstruction_finish_signal.emit();
+    
+    // Reference to the old persp3d object is invalid after reconstruction.
+    initialize_current_persp3d();
+    
     return;
 }
 
@@ -874,16 +908,16 @@ SPDocument::_updateDocument()
  * Repeatedly works on getting the document updated, since sometimes
  * it takes more than one pass to get the document updated.  But it
  * usually should not take more than a few loops, and certainly never
- * more than 32 iterations.  So we bail out if we hit 32 iterations,
+ * more than 64 iterations.  So we bail out if we hit 64 iterations,
  * since this typically indicates we're stuck in an update loop.
  */
 gint
 sp_document_ensure_up_to_date(SPDocument *doc)
 {
-    int counter = 32;
+    int counter = 64;
     while (!doc->_updateDocument()) {
         if (counter == 0) {
-            g_warning("More than 32 iteration while updating document '%s'", doc->uri);
+            g_warning("More than 64 iteration while updating document '%s'", doc->uri? doc->uri:"<unknown URI, probably clipboard>");
             break;
         }
         counter--;