abstract use of sodipodi:modified

diff --git a/src/dialogs/export.cpp b/src/dialogs/export.cpp
index d26a5cde15dbc326528db9a67424ea45c94d87d8..d664ef729a5cf963c93882693f47425919f18fcf 100644 (file)
--- a/src/dialogs/export.cpp
+++ b/src/dialogs/export.cpp
@@ -1237,7 +1237,7 @@ sp_export_export_clicked (GtkButton */*button*/, GtkObject *base)
             }
 
             if (modified)
-                repr->setAttribute("sodipodi:modified", "TRUE");
+                doc->setModified();
             sp_document_set_undo_sensitive(doc, saved);
             break;
         }
@@ -1277,8 +1277,7 @@ sp_export_export_clicked (GtkButton */*button*/, GtkObject *base)
             }
 
             if (modified) {
-                Inkscape::XML::Node * repr = sp_document_repr_root(doc);
-                repr->setAttribute("sodipodi:modified", "TRUE");
+                doc->setModified();
             }
 
             sp_document_set_undo_sensitive(doc, saved);

diff --git a/src/document-undo.cpp b/src/document-undo.cpp
index 7556d2e8fed7df52a16ad5c00f551d8a006b62af..2acd740f2022b5e45216d38b1443fc71bf349818 100644 (file)
--- a/src/document-undo.cpp
+++ b/src/document-undo.cpp
@@ -66,6 +66,15 @@
 #include "debug/timestamp.h"
 #include "event.h"
 
+bool SPDocument::isModified() const {
+    return rroot ? rroot->attribute("sodipodi:modified") != NULL : false;
+}
+void SPDocument::setModified(bool modified) {
+    if (rroot) {
+        rroot->setAttribute("sodipodi:modified", "true");
+    }
+}
+
 
 /*
  * Undo & redo
@@ -199,9 +208,7 @@ sp_document_maybe_done (SPDocument *doc, const gchar *key, const unsigned int ev
        doc->actionkey = key;
 
        doc->virgin = FALSE;
-       if (!doc->rroot->attribute("sodipodi:modified")) {
-               doc->rroot->setAttribute("sodipodi:modified", "true");
-       }
+        doc->setModified();
 
        sp_repr_begin_transaction (doc->rdoc);
 
@@ -267,7 +274,7 @@ sp_document_undo (SPDocument *doc)
                sp_repr_undo_log (log->event);
                doc->priv->redo = g_slist_prepend (doc->priv->redo, log);
 
-               doc->rroot->setAttribute("sodipodi:modified", "true");
+                doc->setModified();
                 doc->priv->undoStackObservers.notifyUndoEvent(log);
 
                ret = TRUE;
@@ -313,7 +320,7 @@ sp_document_redo (SPDocument *doc)
                sp_repr_replay_log (log->event);
                doc->priv->undo = g_slist_prepend (doc->priv->undo, log);
 
-               doc->rroot->setAttribute("sodipodi:modified", "true");
+                doc->setModified();
                doc->priv->undoStackObservers.notifyRedoEvent(log);
 
                ret = TRUE;

diff --git a/src/document.h b/src/document.h
index 462d2a15a2b25b669483a998870c069090e4a022..6c9ea570d27d28bc2258cb561e6e8da563ec15a5 100644 (file)
--- a/src/document.h
+++ b/src/document.h
@@ -140,6 +140,9 @@ sigc::connection connectCommit(CommitSignal::slot_type slot);
     /// Are we currently in a transition between two "known good" states of the document?
     bool isSeeking() const;
 
+    bool isModified() const;
+    void setModified(bool modified=true);
+
 private:
     SPDocument(SPDocument const &); // no copy
     void operator=(SPDocument const &); // no assign

diff --git a/src/event-log.cpp b/src/event-log.cpp
index 4c2ef9c7add8a2a2d9a7c4d9dda76f6a071d0f29..54e58678a194eb303b099fd50a80060c30645649 100644 (file)
--- a/src/event-log.cpp
+++ b/src/event-log.cpp
@@ -364,10 +364,9 @@ void
 EventLog::checkForVirginity() {
     g_return_if_fail (_document);
     if (_curr_event == _last_saved) {
-        Inkscape::XML::Node *repr = sp_document_repr_root(_document);
         bool saved = sp_document_get_undo_sensitive(_document);
         sp_document_set_undo_sensitive(_document, false);
-        repr->setAttribute("sodipodi:modified", NULL);
+        _document->setModified(false);
         sp_document_set_undo_sensitive(_document, saved);
     }
 }

diff --git a/src/extension/system.cpp b/src/extension/system.cpp
index 0372e4aa051f242a61f7e24bcf9667fe0de82ab4..9b0c990104e438e9ceea186a210a9fe5fcaaef87 100644 (file)
--- a/src/extension/system.cpp
+++ b/src/extension/system.cpp
@@ -104,10 +104,9 @@ open(Extension *key, gchar const *filename)
 
     /* This kinda overkill as most of these are already set, but I want
        to make sure for this release -- TJG */
-    Inkscape::XML::Node *repr = sp_document_repr_root(doc);
     bool saved = sp_document_get_undo_sensitive(doc);
     sp_document_set_undo_sensitive(doc, false);
-    repr->setAttribute("sodipodi:modified", NULL);
+    doc->setModified(false);
     sp_document_set_undo_sensitive(doc, saved);
 
     sp_document_set_uri(doc, filename);
@@ -249,14 +248,12 @@ save(Extension *key, SPDocument *doc, gchar const *filename, bool setextension,
     Inkscape::XML::Node *repr = sp_document_repr_root(doc);
 
     // remember attributes in case this is an unofficial save
-    gchar *saved_sodipodi_modified = NULL;
+    bool saved_modified = false;
     gchar *saved_output_extension = NULL;
     gchar *saved_dataloss = NULL;
     gchar *saved_uri = NULL;
     if (!official) {
-        if (repr->attribute("sodipodi:modified")) {
-            saved_sodipodi_modified = g_strdup(repr->attribute("sodipodi:modified"));
-        }
+        saved_modified = doc->isModified();
         if (repr->attribute("inkscape:output_extension")) {
             saved_output_extension = g_strdup(repr->attribute("inkscape:output_extension"));
         }
@@ -271,7 +268,7 @@ save(Extension *key, SPDocument *doc, gchar const *filename, bool setextension,
     // update attributes:
     bool saved = sp_document_get_undo_sensitive(doc);
     sp_document_set_undo_sensitive (doc, false); 
-        repr->setAttribute("sodipodi:modified", NULL); 
+        doc->setModified(false);
         // save the filename for next use
         sp_document_set_uri(doc, fileName);
         // also save the extension for next use
@@ -289,14 +286,13 @@ save(Extension *key, SPDocument *doc, gchar const *filename, bool setextension,
     if ( !official) {
         saved = sp_document_get_undo_sensitive(doc);
         sp_document_set_undo_sensitive (doc, false);
-            repr->setAttribute("sodipodi:modified", saved_sodipodi_modified);
+            doc->setModified(saved_modified);
             repr->setAttribute("inkscape:output_extension", saved_output_extension);
             repr->setAttribute("inkscape:dataloss", saved_dataloss);
             sp_document_set_uri(doc, saved_uri);
         sp_document_set_undo_sensitive (doc, saved);
     }
     
-    if (saved_sodipodi_modified) g_free(saved_sodipodi_modified);
     if (saved_output_extension)  g_free(saved_output_extension);
     if (saved_dataloss)          g_free(saved_dataloss);
     if (saved_uri)               g_free(saved_uri);    

diff --git a/src/file.cpp b/src/file.cpp
index 355b1c50ab461dcdd215a6f8cfff3ffb8d047513..58ca463bf0eff55ea4e1666a48bf6c8fc87f31e7 100644 (file)
--- a/src/file.cpp
+++ b/src/file.cpp
@@ -249,7 +249,7 @@ sp_file_revert_dialog()
     }
 
     bool do_revert = true;
-    if (repr->attribute("sodipodi:modified") != NULL) {
+    if (doc->isModified()) {
         gchar *text = g_strdup_printf(_("Changes will be lost!  Are you sure you want to reload document %s?"), uri);
 
         bool response = desktop->warnDialog (text);
@@ -771,16 +771,14 @@ sp_file_save_document(Gtk::Window &parentWindow, SPDocument *doc)
 {
     bool success = true;
 
-    Inkscape::XML::Node *repr = sp_document_repr_root(doc);
-
-    gchar const *fn = repr->attribute("sodipodi:modified");
-    if (fn != NULL) {
+    if (doc->isModified()) {
+        Inkscape::XML::Node *repr = sp_document_repr_root(doc);
         if ( doc->uri == NULL
             || repr->attribute("inkscape:output_extension") == NULL )
         {
             return sp_file_save_dialog(parentWindow, doc, FALSE);
         } else {
-            fn = g_strdup(doc->uri);
+            gchar const *fn = g_strdup(doc->uri);
             gchar const *ext = repr->attribute("inkscape:output_extension");
             success = file_save(parentWindow, doc, fn, Inkscape::Extension::db.get(ext), FALSE, TRUE);
             g_free((void *) fn);
@@ -1187,13 +1185,11 @@ sp_file_export_to_ocal_dialog(Gtk::Window &parentWindow)
     static bool gotSuccess = false;
 
     Inkscape::XML::Node *repr = sp_document_repr_root(doc);
-    // Verify whether the document is saved, so save this as temporary
 
-    char *str = (char *) repr->attribute("sodipodi:modified");
-    if ((!doc->uri) && (!str))
+    if (!doc->uri && !doc->isModified())
         return false;
 
-   //  Get the default extension name
+    //  Get the default extension name
     Glib::ustring default_extension = "org.inkscape.output.svg.inkscape";
     char formatBuf[256];
 

diff --git a/src/inkscape.cpp b/src/inkscape.cpp
index 8b84416aa8c950e2ca11a6d1a7f1fac6a96cb11d..4f831e629f271ac964b3caa1a4fbfa59b1cf4b75 100644 (file)
--- a/src/inkscape.cpp
+++ b/src/inkscape.cpp
@@ -431,7 +431,7 @@ inkscape_crash_handler (int /*signum*/)
         Inkscape::XML::Node *repr;
         doc = (SPDocument *) l->data;
         repr = sp_document_repr_root (doc);
-        if (repr->attribute("sodipodi:modified")) {
+        if (doc->isModified()) {
             const gchar *docname, *d0, *d;
             gchar n[64], c[1024];
             FILE *file;

diff --git a/src/sp-namedview.cpp b/src/sp-namedview.cpp
index f738828e537c398de284bd4bb1e41f05a6c10bff..0368262dcc2eef5df8749aa4a08170564db81a9f 100644 (file)
--- a/src/sp-namedview.cpp
+++ b/src/sp-namedview.cpp
@@ -884,7 +884,7 @@ void sp_namedview_toggle_guides(SPDocument *doc, Inkscape::XML::Node *repr)
 
     sp_repr_set_boolean(repr, "showguides", v);
 
-    doc->rroot->setAttribute("sodipodi:modified", "true");
+    doc->setModified();
     sp_document_set_undo_sensitive(doc, saved);
 }
 
@@ -903,7 +903,7 @@ void sp_namedview_show_grids(SPNamedView * namedview, bool show, bool dirty_docu
     /* we don't want the document to get dirty on startup; that's when
        we call this function with dirty_document = false */
     if (dirty_document) {
-        doc->rroot->setAttribute("sodipodi:modified", "true");
+        doc->setModified();
     }
     sp_document_set_undo_sensitive(doc, saved);
 }

diff --git a/src/ui/view/edit-widget.cpp b/src/ui/view/edit-widget.cpp
index ff8b288fdc2dc2881dafe60b47b8d8418939a828..78cf76096267c95cb022716db74b6b87cb4a7c3a 100644 (file)
--- a/src/ui/view/edit-widget.cpp
+++ b/src/ui/view/edit-widget.cpp
@@ -1203,8 +1203,7 @@ EditWidget::shutdown()
         return false;
 
     SPDocument *doc = _desktop->doc();
-    if (sp_document_repr_root(doc)->attribute("sodipodi:modified") != NULL)
-    {
+    if (doc->isModified()) {
         gchar *markup;
         /// \todo FIXME !!! obviously this will have problems if the document
         /// name contains markup characters

diff --git a/src/ui/widget/registered-widget.cpp b/src/ui/widget/registered-widget.cpp
index aac9ff25ce8c47b50f5ed310a65aa90c569bde89..84d53663b818f2ac55396322b2e2842d3a1ffd42 100644 (file)
--- a/src/ui/widget/registered-widget.cpp
+++ b/src/ui/widget/registered-widget.cpp
@@ -69,7 +69,7 @@ RegisteredWidget::write_to_xml(const char * svgstr)
     sp_document_set_undo_sensitive (local_doc, false);
 
     if (!write_undo) local_repr->setAttribute(_key.c_str(), svgstr);
-    local_doc->rroot->setAttribute("sodipodi:modified", "true");
+    local_doc->setModified();
 
     sp_document_set_undo_sensitive (local_doc, saved);
     if (write_undo) {
@@ -378,7 +378,7 @@ RegisteredColorPicker::on_changed (guint32 rgba)
     sp_document_set_undo_sensitive (local_doc, false);
     local_repr->setAttribute(_ckey.c_str(), c);
     sp_repr_set_css_double(local_repr, _akey.c_str(), (rgba & 0xff) / 255.0);
-    local_doc->rroot->setAttribute("sodipodi:modified", "true");
+    local_doc->setModified();
     sp_document_set_undo_sensitive (local_doc, saved);
     sp_document_done (local_doc, SP_VERB_NONE,
                       /* TODO: annotate */ "registered-widget.cpp: RegisteredColorPicker::on_changed");

diff --git a/src/ui/widget/tolerance-slider.cpp b/src/ui/widget/tolerance-slider.cpp
index 4db82d88008872de509a0d1585d00eb7f7302214..1c5f9b52190d4d4fc8e47c1eb15d964422946385 100644 (file)
--- a/src/ui/widget/tolerance-slider.cpp
+++ b/src/ui/widget/tolerance-slider.cpp
@@ -179,7 +179,7 @@ ToleranceSlider::update (double val)
     sp_document_set_undo_sensitive (doc, false);
     Inkscape::XML::Node *repr = SP_OBJECT_REPR (sp_desktop_namedview(dt));
     repr->setAttribute(_key.c_str(), os.str().c_str());
-    doc->rroot->setAttribute("sodipodi:modified", "true");
+    doc->setModified();
     sp_document_set_undo_sensitive (doc, saved);
     
     _wr->setUpdating (false);

diff --git a/src/widgets/desktop-widget.cpp b/src/widgets/desktop-widget.cpp
index 4c2ebf21cda4797265221df657fb3fa99a0361ec..ca1121152945109c6d9a14679738c502b2e081cb 100644 (file)
--- a/src/widgets/desktop-widget.cpp
+++ b/src/widgets/desktop-widget.cpp
@@ -807,7 +807,7 @@ SPDesktopWidget::shutdown()
 
     if (inkscape_is_sole_desktop_for_document(*desktop)) {
         SPDocument *doc = desktop->doc();
-        if (sp_document_repr_root(doc)->attribute("sodipodi:modified") != NULL) {
+        if (doc->isModified()) {
             GtkWidget *dialog;
 
             /** \todo