[glade/glade-3-8] Fixed bug 584381 "Saving as an existing file name except the extension overwrites without confirmati



commit dc02d1b7c05433e1ea6cfc939809a2995e79df1e
Author: Juan Pablo Ugarte <juanpablougarte gmail com>
Date:   Thu Sep 12 17:21:08 2013 -0300

    Fixed bug 584381 "Saving as an existing file name except the extension overwrites without confirmation."
    
    The problem was that if the selected filename did not had an extension we appended append a .glade 
extension
    but did not check if the new filename existed or not.
    Now we fixed this problem by selecting the file if it exist and running the dialog again, giving it a 
chance
    to promt the user about overwriting.

 src/glade-window.c |   63 ++++++++++++++++++++++++++++++++++++---------------
 1 files changed, 44 insertions(+), 19 deletions(-)
---
diff --git a/src/glade-window.c b/src/glade-window.c
index 5bbb352..f244c55 100644
--- a/src/glade-window.c
+++ b/src/glade-window.c
@@ -1133,6 +1133,15 @@ save (GladeWindow *window, GladeProject *project, const gchar *path)
        g_free (display_name);
 }
 
+static gboolean
+path_has_extension (const gchar *path)
+{
+  gchar *basename = g_path_get_basename (path);
+  gboolean retval = g_utf8_strrchr (basename, -1, '.') != NULL;
+  g_free (basename);
+  return retval;
+}
+
 static void
 save_as (GladeWindow *window)
 {
@@ -1140,7 +1149,7 @@ save_as (GladeWindow *window)
        GtkWidget    *filechooser;
        GtkWidget    *dialog;
        gchar *path = NULL;
-       gchar *real_path, *ch, *project_name;
+       gchar *project_name;
        
        project = glade_design_view_get_project (window->priv->active_view);
        
@@ -1172,31 +1181,47 @@ save_as (GladeWindow *window)
                g_free (project_name);
        }
        
-       if (gtk_dialog_run (GTK_DIALOG(filechooser)) == GTK_RESPONSE_OK)
-               path = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filechooser));
+  while (gtk_dialog_run (GTK_DIALOG (filechooser)) == GTK_RESPONSE_OK)
+    {
+      path = gtk_file_chooser_get_filename (GTK_FILE_CHOOSER (filechooser));
+
+      /* Check if selected filename has an extension or not */
+      if (!path_has_extension (path))
+        {
+          gchar *real_path = g_strconcat (path, ".glade", NULL);
+
+          g_free (path);
+          path = real_path;
+
+          /* We added .glade extension!,
+           * check if file exist to avoid overwriting a file without asking
+           */
+          if (g_file_test (path, G_FILE_TEST_EXISTS))
+            {
+              /* Set existing filename and let filechooser ask about overwriting */
+              gtk_file_chooser_set_filename (GTK_FILE_CHOOSER (filechooser), path);
+              g_free (path);
+              path = NULL;
+              continue;
+            }
+        }
+      break;
+    }
        
        gtk_widget_destroy (filechooser);
  
        if (!path)
                return;
 
-       ch = strrchr (path, '.');
-       if (!ch || strchr (ch, G_DIR_SEPARATOR))
-               real_path = g_strconcat (path, ".glade", NULL);
-       else 
-               real_path = g_strdup (path);
-       
-       g_free (path);
-
        /* checks if selected path is actually writable */
-       if (glade_util_file_is_writeable (real_path) == FALSE)
+       if (glade_util_file_is_writeable (path) == FALSE)
        {
                dialog = gtk_message_dialog_new (GTK_WINDOW (window),
                                                 GTK_DIALOG_MODAL,
                                                 GTK_MESSAGE_ERROR,
                                                 GTK_BUTTONS_OK,
                                                 _("Could not save the file %s"),
-                                                real_path);
+                                                path);
                                                 
                gtk_message_dialog_format_secondary_text (GTK_MESSAGE_DIALOG (dialog),
                                                          _("You do not have the permissions "
@@ -1209,29 +1234,29 @@ save_as (GladeWindow *window)
                                          dialog);
 
                gtk_widget_show (dialog);
-               g_free (real_path);
+               g_free (path);
                return;
        }
 
        /* checks if another open project is using selected path */
-       if ((another_project = glade_app_get_project_by_path (real_path)) != NULL)
+       if ((another_project = glade_app_get_project_by_path (path)) != NULL)
        {
                if (project != another_project) {
 
                        glade_util_ui_message (GTK_WIDGET (window), 
                                               GLADE_UI_ERROR, NULL,
                                               _("Could not save file %s. Another project with that path is 
open."), 
-                                              real_path);
+                                              path);
 
-                       g_free (real_path);
+                       g_free (path);
                        return;
                }
 
        }
 
-       save (window, project, real_path);
+       save (window, project, path);
 
-       g_free (real_path);
+       g_free (path);
 }
 
 static void


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]