Changeset 10298

Timestamp:

12/13/10 16:45:51 (2 years ago)

Author:

OliverKrueger

Message:

Item10152: ChartPlugin fails to create directories for sub webs

Location:

trunk/ChartPlugin/lib/Foswiki/Plugins

Files:

• ChartPlugin.pm (modified) (1 diff)
• ChartPlugin/DEPENDENCIES (modified) (1 diff)

trunk/ChartPlugin/lib/Foswiki/Plugins/ChartPlugin.pm

@@ -214 +214 @@
 
     # before save, create directories if they don't exist.
-    # If the top level "pub/$web" directory doesn't exist, create it.
-    my $dir = Foswiki::Func::getPubDir() . "/$web";
-    if( ! -e "$dir" ) {
-        umask( 002 );
-        mkdir( $dir, 0775 );
-    }
-    # If the top level "pub/$web/$topic" directory doesn't exist, create
-    # it.
-    my $tempPath = "$dir/$topic";
-    if( ! -e "$tempPath" ) {
-        umask( 002 );
-        mkdir( $tempPath, 0775 );
-    }
+    my $tempPath = Foswiki::Func::getPubDir() . "/$web/$topic";
+    use File::Path;
+    eval { File::Path::mkpath( $tempPath, 0, $Foswiki::cfg{RCS}{dirPermission} ); };
+
     # Return both the directory and the filename
     return ($tempPath, $fullname);

trunk/ChartPlugin/lib/Foswiki/Plugins/ChartPlugin/DEPENDENCIES

@@ -2 +2 @@
 GD,>=0.01,cpan,Required.  Available from CPAN:GD
 POSIX,>0,cpan,Required.  Available from CPAN:POSIX
+File::Path,>0,cpan,Required