[Traverso-commit] traverso/src core/ProjectManager.cpp traverso/d...

[Remon Sijrier]

CVSROOT:        /sources/traverso
Module name:    traverso
Changes by:     Remon Sijrier <r_sijrier>       07/10/29 09:38:20

Modified files:
        src/core       : ProjectManager.cpp 
        src/traverso/dialogs/project: OpenProjectDialog.cpp 

Log message:
        * check if a newly chosen project directory is writable by the user, 
and warn if not writable

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/traverso/src/core/ProjectManager.cpp?cvsroot=traverso&r1=1.46&r2=1.47
http://cvs.savannah.gnu.org/viewcvs/traverso/src/traverso/dialogs/project/OpenProjectDialog.cpp?cvsroot=traverso&r1=1.12&r2=1.13

Patches:
Index: core/ProjectManager.cpp
===================================================================
RCS file: /sources/traverso/traverso/src/core/ProjectManager.cpp,v
retrieving revision 1.46
retrieving revision 1.47
diff -u -b -r1.46 -r1.47
--- core/ProjectManager.cpp     24 Oct 2007 18:11:40 -0000      1.46
+++ core/ProjectManager.cpp     29 Oct 2007 09:38:20 -0000      1.47
@@ -293,12 +293,23 @@
 
        QDir dir;
        if ( (projects_path.isEmpty()) || (!dir.exists(projects_path)) ) {
-               if (projects_path.isEmpty())
+               if (projects_path.isEmpty()) {
                        projects_path = QDir::homePath();
+               }
 
                QString newPath = QFileDialog::getExistingDirectory(0,
                                tr("Choose an existing or create a new Project 
Directory"),
                                   projects_path );
+               
+               QFileInfo fi(newPath);
+               if (dir.exists(newPath) && !fi.isWritable()) {
+                       QMessageBox::warning( 0, tr("Traverso - Warning"), 
+                                             tr("This directory is not 
writable by you! \n") +
+                                             tr("Please check permission for 
this directory or "
+                                               "choose another one:\n\n 
%1").arg(newPath) );
+                       return;
+               } 
+               
                if (dir.exists(newPath)) {
                        info().information(tr("Using existing Project 
directory: %1\n").arg(newPath));
                } else if (!dir.mkpath(newPath)) {

Index: traverso/dialogs/project/OpenProjectDialog.cpp
===================================================================
RCS file: 
/sources/traverso/traverso/src/traverso/dialogs/project/OpenProjectDialog.cpp,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -b -r1.12 -r1.13
--- traverso/dialogs/project/OpenProjectDialog.cpp      6 Oct 2007 14:17:59 
-0000       1.12
+++ traverso/dialogs/project/OpenProjectDialog.cpp      29 Oct 2007 09:38:20 
-0000      1.13
@@ -249,6 +249,16 @@
        
        QDir dir;
        
+       QFileInfo fi(newPath);
+       if (dir.exists(newPath) && !fi.isWritable()) {
+               QMessageBox::warning( 0, tr("Traverso - Warning"), 
+                                     tr("This directory is not writable by 
you! \n") +
+                                       tr("Please check permission for this 
directory or "
+                                       "choose another one:\n\n 
%1").arg(newPath) );
+               return;
+       } 
+
+       
        if (dir.exists(newPath)) {
 //             QMessageBox::information( interface, tr("Traverso - 
Information"), tr("Using existing Project directory: %1\n").arg(newPath), "OK", 
0 );
        } else if (!dir.mkpath(newPath)) {