- Improved file editing (especially if the file should be saved using a different...

[devedit.git] / modules / Command.pm

diff --git a/modules/Command.pm b/modules/Command.pm
index ad660c3310ab462fd9cdec38c5af3f5587ad0db2..cddb5ca37a2e3559fd8c1d7c060fb6cbb1fa905c 100644 (file)
--- a/modules/Command.pm
+++ b/modules/Command.pm
@@ -6,41 +6,64 @@ package Command;
 # Execute Dev-Editor's commands
 #
 # Author:        Patrick Canterino <patshaping@gmx.net>
 # Execute Dev-Editor's commands
 #
 # Author:        Patrick Canterino <patshaping@gmx.net>

-# Last modified: 09-23-2003
+# Last modified: 2004-02-23

 #
 
 use strict;
 
 #
 
 use strict;
 

-use vars qw(@EXPORT
-            $script);
-
-use CGI qw(header
-           redirect);
+use vars qw(@EXPORT);

 
 use File::Access;
 use File::Copy;
 
 use File::Access;
 use File::Copy;

+use File::Path;

 
 

-use HTML::Entities;
-use Output;

 use POSIX qw(strftime);
 use Tool;
 
 use POSIX qw(strftime);
 use Tool;
 

-$script = $ENV{'SCRIPT_NAME'};
+use CGI qw(header);
+use HTML::Entities;
+use Output;
+use Template;
+
+my $script = $ENV{'SCRIPT_NAME'};
+
+my %dispatch = ('show'       => \&exec_show,
+                'beginedit'  => \&exec_beginedit,
+                'canceledit' => \&exec_canceledit,
+                'endedit'    => \&exec_endedit,
+                'mkdir'      => \&exec_mkdir,
+                'mkfile'     => \&exec_mkfile,
+                'copy'       => \&exec_copy,
+                'rename'     => \&exec_rename,
+                'remove'     => \&exec_remove,
+                'unlock'     => \&exec_unlock
+               );

 
 ### Export ###
 
 use base qw(Exporter);
 
 
 ### Export ###
 
 use base qw(Exporter);
 

-@EXPORT = qw(exec_show
-             exec_beginedit
-             exec_endedit
-             exec_mkfile
-             exec_mkdir
-             exec_workwithfile
-             exec_copy
-             exec_rename
-             exec_remove
-             exec_unlock);
+@EXPORT = qw(exec_command);
+
+# exec_command()
+#
+# Execute the specified command
+#
+# Params: 1. Command to execute
+#         2. Reference to user input hash
+#         3. Reference to config hash
+#
+# Return: Output of the command (Scalar Reference)
+
+sub exec_command($$$)
+{
+ my ($command,$data,$config) = @_;
+
+ return error($config->{'err_cmd_unknown'},'/',{COMMAND => $command}) unless($dispatch{$command});
+
+ my $output = &{$dispatch{$command}}($data,$config);
+ return $output;
+}

 
 # exec_show()
 #
 
 # exec_show()
 #


@@ -51,142 +74,134 @@ use base qw(Exporter);
 #
 # Return: Output of the command (Scalar Reference)
 
 #
 # Return: Output of the command (Scalar Reference)
 

-sub exec_show($$$)
+sub exec_show($$)

 {
  my ($data,$config) = @_;
  my $physical       = $data->{'physical'};
  my $virtual        = $data->{'virtual'};
 {
  my ($data,$config) = @_;
  my $physical       = $data->{'physical'};
  my $virtual        = $data->{'virtual'};

- my $output;
+
+ my $tpl = new Template;

 
  if(-d $physical)
  {
   # Create directory listing
 
   my $direntries = dir_read($physical);
 
  if(-d $physical)
  {
   # Create directory listing
 
   my $direntries = dir_read($physical);

-  return error("Reading of directory $virtual failed") unless($direntries);
+  return error($config->{'dir_read_failed'},upper_path($virtual),{DIR => '$virtual'}) unless($direntries);

 
   my $files = $direntries->{'files'};
   my $dirs  = $direntries->{'dirs'};
 
 
   my $files = $direntries->{'files'};
   my $dirs  = $direntries->{'dirs'};
 

-  $output .= htmlhead("Directory listing of $virtual");
-  $output .= equal_url($config->{'httproot'},$virtual);
-  $output .=  "<hr>\n\n<pre>\n";
+  my $dirlist = "";

 
   # Create the link to the upper directory
   # (only if we are not in the root directory)
 
   unless($virtual eq "/")
   {
 
   # Create the link to the upper directory
   # (only if we are not in the root directory)
 
   unless($virtual eq "/")
   {

-   my $upper = $physical."/..";
-   my @stat  = stat($upper);
-
-   $output .= "  [SUBDIR]  ";
-   $output .= strftime("%d.%m.%Y %H:%M",localtime($stat[9]));
-   $output .= " " x 10;
-   $output .= "<a href=\"$script?command=show&file=".upper_path($virtual)."\">../</a>\n";
-  }
+   my @stat  = stat($physical."/..");

 
 

-  # Get the longest file/directory name
+   my $udtpl = new Template;
+   $udtpl->read_file($config->{'tpl_dirlist_up'});

 
 

-  my $max_name_len = 0;
+   $udtpl->fillin("UPPER_DIR",encode_entities(upper_path($virtual)));
+   $udtpl->fillin("DATE",strftime($config->{'timeformat'},localtime($stat[9])));

 
 

-  foreach(@$dirs,@$files)
-  {
-   my $length    = length($_);
-   $max_name_len = $length if($length > $max_name_len);
+   $dirlist .= $udtpl->get_template;

   }
 
   # Directories
 
   foreach my $dir(@$dirs)
   {
   }
 
   # Directories
 
   foreach my $dir(@$dirs)
   {

-   my @stat = stat($physical."/".$dir);
+   my @stat      = stat($physical."/".$dir);
+   my $virt_path = encode_entities($virtual.$dir."/");
+
+   my $dtpl = new Template;
+   $dtpl->read_file($config->{'tpl_dirlist_dir'});
+
+   $dtpl->fillin("DIR",$virt_path);
+   $dtpl->fillin("DIR_NAME",$dir);
+   $dtpl->fillin("DATE",strftime($config->{'timeformat'},localtime($stat[9])));

 
 

-   $output .= "  ";
-   $output .= "[SUBDIR]  ";
-   $output .= strftime("%d.%m.%Y %H:%M",localtime($stat[9]));
-   $output .= " " x 10;
-   $output .= "<a href=\"$script?command=show&file=$virtual$dir/\">".encode_entities($dir)."/</a>\n";
+   $dirlist .= $dtpl->get_template;

   }
 
   # Files
 
   foreach my $file(@$files)
   {
   }
 
   # Files
 
   foreach my $file(@$files)
   {

-   my $phys_path = $physical."/".$file; # Not exactly...
-   my $virt_path = $virtual.$file;
+   my $phys_path = $physical."/".$file;
+   my $virt_path = encode_entities($virtual.$file);

 
    my @stat      = stat($phys_path);
    my $in_use    = $data->{'uselist'}->in_use($virtual.$file);
 
 
    my @stat      = stat($phys_path);
    my $in_use    = $data->{'uselist'}->in_use($virtual.$file);
 

-   $output .= " " x (10 - length($stat[7]));
-   $output .= $stat[7];
-   $output .= "  ";
-   $output .= strftime("%d.%m.%Y %H:%M",localtime($stat[9]));
-   $output .= ($in_use) ? " (IN USE) " : (-B $phys_path) ? " (BINARY) " : " " x 10;
-   $output .= encode_entities($file);
-   $output .= " " x ($max_name_len - length($file))."\t  (";
+   my $ftpl = new Template;
+   $ftpl->read_file($config->{'tpl_dirlist_file'});

 
 

-   $output .= (-T $phys_path)
-              ? "<a href=\"$script?command=show&file=$virt_path\">View</a>"
-              : '<span style="color:#C0C0C0">View</span>';
+   $ftpl->fillin("FILE",$virt_path);
+   $ftpl->fillin("FILE_NAME",$file);
+   $ftpl->fillin("SIZE",$stat[7]);
+   $ftpl->fillin("DATE",strftime($config->{'timeformat'},localtime($stat[9])));

 
 

-   $output .= " | ";
+   $ftpl->parse_if_block("not_readable",not -r $phys_path);
+   $ftpl->parse_if_block("binary",-B $phys_path);
+   $ftpl->parse_if_block("readonly",not -w $phys_path);

 
 

-   $output .= ($in_use || -B $phys_path)
-              ? '<span style="color:#C0C0C0">Edit</span>'
-              : "<a href=\"$script?command=beginedit&file=$virt_path\">Edit</a>";
+   $ftpl->parse_if_block("viewable",-r $phys_path && -T $phys_path);
+   $ftpl->parse_if_block("editable",-w $phys_path && -r $phys_path && -T $phys_path && not $in_use);

 
 

-   $output .= " | <a href=\"$script?command=workwithfile&file=$virt_path\">Do other stuff</a>)\n";
+   $ftpl->parse_if_block("in_use",$in_use);
+   $ftpl->parse_if_block("unused",not $in_use);
+
+   $dirlist .= $ftpl->get_template;

   }
 
   }
 

-  $output .= "</pre>\n\n<hr>\n\n";
-  $output .= <<END;
-<table border="0">
-<tr>
-<td>Create new directory:</td>
-<td>$virtual <input type="text" name="newdirname"> <input type="submit" value="Create!"></td>
-</tr>
-<tr>
-<td>Create new file:</td>
-<td>$virtual <input type="text" name="newfilename"> <input type="submit" value="Create!"></td>
-</tr>
-</table>
-
-<hr>
-END
-  $output .= htmlfoot;
+  $tpl->read_file($config->{'tpl_dirlist'});
+
+  $tpl->fillin("DIRLIST",$dirlist);
+  $tpl->fillin("DIR",$virtual);
+  $tpl->fillin("SCRIPT",$script);
+  $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));

  }
  else
  {
   # View a file
 
  }
  else
  {
   # View a file
 

+  return error($config->{'err_noview'},upper_path($virtual)) unless(-r $physical);
+

   # Check on binary files
   # Check on binary files

+  # We have to do it in this way, or empty files
+  # will be recognized as binary files

 
 

-  if(-B $physical)
+  unless(-T $physical)

   {
    # Binary file
 
   {
    # Binary file
 

-   return error("This editor is not able to view/edit binary files.");
+   return error($config->{'err_binary'},upper_path($virtual));

   }
   else
   {
    # Text file
 
   }
   else
   {
    # Text file
 

-   $output  = htmlhead("Contents of file $virtual");
-   $output .= equal_url($config->{'httproot'},$virtual);
-   $output .= dir_link($virtual);
+   my $content =  file_read($physical);
+   $$content   =~ s/\015\012|\012|\015/\n/g;

 
 

-   $output .= '<div style="background-color:#FFFFE0;border:1px solid black;margin-top:10px;width:100%">'."\n";
-   $output .= '<pre style="color:#0000C0;">'."\n";
-   $output .= encode_entities(${file_read($physical)});
-   $output .= "\n</pre>\n</div>";
+   $tpl->read_file($config->{'tpl_viewfile'});

 
 

-   $output .= htmlfoot;
+   $tpl->fillin("FILE",$virtual);
+   $tpl->fillin("DIR",upper_path($virtual));
+   $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+   $tpl->fillin("SCRIPT",$script);
+   $tpl->fillin("CONTENT",encode_entities($$content));

   }
  }
 
   }
  }
 

- return \$output
+ my $output  = header(-type => "text/html");
+ $output    .= $tpl->get_template;
+
+ return \$output;

 }
 
 # exec_beginedit
 }
 
 # exec_beginedit


@@ -205,16 +220,17 @@ sub exec_beginedit($$)
  my $virtual        = $data->{'virtual'};
  my $uselist        = $data->{'uselist'};
 
  my $virtual        = $data->{'virtual'};
  my $uselist        = $data->{'uselist'};
 

- return error("You cannot edit directories.") if(-d $physical);
+ return error($config->{'err_editdir'},upper_path($virtual)) if(-d $physical);

  return error_in_use($virtual) if($uselist->in_use($virtual));
  return error_in_use($virtual) if($uselist->in_use($virtual));

+ return error($config->{'err_noedit'},upper_path($virtual)) unless(-r $physical && -w $physical);

 
  # Check on binary files
 
 
  # Check on binary files
 

- if(-B $physical)
+ unless(-T $physical)

  {
   # Binary file
 
  {
   # Binary file
 

-  return error("This editor is not able to view/edit binary files.");
+  return error($config->{'err_binary'},upper_path($virtual));

  }
  else
  {
  }
  else
  {


@@ -223,45 +239,43 @@ sub exec_beginedit($$)
   $uselist->add_file($virtual);
   $uselist->save;
 
   $uselist->add_file($virtual);
   $uselist->save;
 

-  my $dir     = upper_path($virtual);
-  my $content = encode_entities(${file_read($physical)});
-
-  my $output = htmlhead("Edit file $virtual");
-  $output   .= equal_url($config->{'httproot'},$virtual);
-  $output   .= <<END;
-<p><b style="color:#FF0000">Caution!</b> This file is locked for other users while you are editing it. To unlock it, click <i>Save and exit</i> or <i>Exit WITHOUT saving</i>. Please <b>don't</b> click the <i>Reload</i> button in your browser! This will confuse the editor.</p>
-
-<form action="$ENV{'SCRIPT_NAME'}" method="get">
-<input type="hidden" name="command" value="canceledit">
-<input type="hidden" name="file" value="$virtual">
-<p><input type="submit" value="Exit WITHOUT saving"></p>
-</form>
-
-<form action="$ENV{'SCRIPT_NAME'}" method="post">
-<input type="hidden" name="command" value="endedit">
-<input type="hidden" name="file" value="$virtual">
-
-<table width="100%" border="1">
-<tr>
-<td width="50%" align="center"><input type="checkbox" name="save_as_new_file" value="1"> Save as new file: $dir <input type=text name="new_filename" value=""></td>
-<td width="50%" align="center"><input type="checkbox" name="encode_iso" value="1"> Encode ISO-8859-1 special chars</td>
-</tr>
-<tr>
-<td align="center"><input type="reset" value="Reset form"></td>
-<td align="center"><input type="submit" value="Save and exit"></td>
-</tr>
-</table>
-
-<textarea name="filecontent" rows="25" cols="120">$content</textarea>
-</form>
-END
-
-  $output .= htmlfoot;
+  my $content =  file_read($physical);
+  $$content   =~ s/\015\012|\012|\015/\n/g;
+
+  my $tpl = new Template;
+  $tpl->read_file($config->{'tpl_editfile'});
+
+  $tpl->fillin("FILE",$virtual);
+  $tpl->fillin("DIR",upper_path($virtual));
+  $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+  $tpl->fillin("SCRIPT",$script);
+  $tpl->fillin("CONTENT",encode_entities($$content));
+
+  my $output = header(-type => "text/html");
+  $output   .= $tpl->get_template;

 
   return \$output;
  }
 }
 
 
   return \$output;
  }
 }
 

+# exec_canceledit()
+#
+# Abort file editing
+#
+# Params: 1. Reference to user input hash
+#         2. Reference to config hash
+#
+# Return: Output of the command (Scalar Reference)
+
+sub exec_canceledit($$)
+{
+ my ($data,$config) = @_;
+ my $virtual        = $data->{'virtual'};
+
+ file_unlock($data->{'uselist'},$virtual);
+ return devedit_reload({command => 'show', file => upper_path($virtual)});
+}
+

 # exec_endedit()
 #
 # Save a file, unlock it and return to directory view
 # exec_endedit()
 #
 # Save a file, unlock it and return to directory view


@@ -277,8 +291,11 @@ sub exec_endedit($$)
  my $physical       = $data->{'physical'};
  my $virtual        = $data->{'virtual'};
  my $content        = $data->{'cgi'}->param('filecontent');
  my $physical       = $data->{'physical'};
  my $virtual        = $data->{'virtual'};
  my $content        = $data->{'cgi'}->param('filecontent');

+ my $uselist        = $data->{'uselist'};

 
 

- return error("You cannot edit directories.") if(-d $physical);
+ # Normalize newlines
+
+ $content =~ s/\015\012|\012|\015/\n/g;

 
  if($data->{'cgi'}->param('encode_iso'))
  {
 
  if($data->{'cgi'}->param('encode_iso'))
  {


@@ -287,15 +304,35 @@ sub exec_endedit($$)
   $content = encode_entities($content,"\200-\377");
  }
 
   $content = encode_entities($content,"\200-\377");
  }
 

+ if($data->{'cgi'}->param('saveas'))
+ {
+  # Create the new filename
+
+  $physical = $data->{'new_physical'};
+  $virtual  = $data->{'new_virtual'};
+
+  # Check if someone else is editing the new file
+
+  return error_in_use($virtual) if($uselist->in_use($virtual));
+ }
+
+ return error($config->{'err_editdir'},upper_path($virtual)) if(-d $physical);
+ return error($config->{'err_noedit'}, upper_path($virtual)) unless(-r $physical && -w $physical);
+

  if(file_save($physical,\$content))
  {
  if(file_save($physical,\$content))
  {

-  # Saving of the file was successfull - so unlock it!
+  # Saving of the file was successful - so unlock it!
+
+  file_unlock($uselist,$data->{'virtual'});
+  #                    ^^^^^^^^^^^^^^^^^^
+  # Maybe the user saved the file using another filename...
+  # But we have to unlock the original file!

 
 

-  return exec_unlock($data,$config);
+  return devedit_reload({command => 'show', file => upper_path($virtual)});

  }
  else
  {
  }
  else
  {

-  return error("Saving of file '$virtual' failed'");
+  return error($config->{'err_edit_failed'},upper_path($virtual),{FILE => $virtual});

  }
 }
 
  }
 }
 


@@ -310,7 +347,16 @@ sub exec_endedit($$)
 
 sub exec_mkfile($$)
 {
 
 sub exec_mkfile($$)
 {

- 1;
+ my ($data,$config) = @_;
+ my $new_physical   = $data->{'new_physical'};
+ my $new_virtual    = $data->{'new_virtual'};
+ my $dir            = upper_path($new_virtual);
+ $new_virtual       = encode_entities($new_virtual);
+
+ return error($config->{'err_file_exists'},$dir,{FILE => $new_virtual}) if(-e $new_physical);
+
+ file_create($new_physical) or return error($config->{'err_mkfile_failed'},$dir,{FILE => $new_virtual});
+ return devedit_reload({command => 'show', file => $dir});

 }
 
 # exec_mkdir()
 }
 
 # exec_mkdir()


@@ -324,100 +370,89 @@ sub exec_mkfile($$)
 
 sub exec_mkdir($$)
 {
 
 sub exec_mkdir($$)
 {

- 1;
+ my ($data,$config) = @_;
+ my $new_physical   = $data->{'new_physical'};
+ my $new_virtual    = $data->{'new_virtual'};
+ my $dir            = upper_path($new_virtual);
+ $new_virtual       = encode_entities($new_virtual);
+
+ return error($config->{'err_file_exists'},$dir,{FILE => $new_virtual}) if(-e $new_physical);
+
+ mkdir($new_physical,0777) or return error($config->{'err_mkdir_failed'},$dir,{DIR => $new_virtual});
+ return devedit_reload({command => 'show', file => $dir});

 }
 
 }
 

-# exec_workwithfile()
+# exec_copy()

 #
 #

-# Display a form for renaming/copying/deleting/unlocking a file
+# Copy a file and return to directory view

 #
 # Params: 1. Reference to user input hash
 #         2. Reference to config hash
 #
 # Return: Output of the command (Scalar Reference)
 
 #
 # Params: 1. Reference to user input hash
 #         2. Reference to config hash
 #
 # Return: Output of the command (Scalar Reference)
 

-sub exec_workwithfile($$)
+sub exec_copy($$)

 {
  my ($data,$config) = @_;
  my $physical       = $data->{'physical'};
 {
  my ($data,$config) = @_;
  my $physical       = $data->{'physical'};

- my $virtual        = $data->{'virtual'};
- my $unused         = $data->{'uselist'}->unused($virtual);
-
- my $output = htmlhead("Work with file $virtual");
- $output   .= equal_url($config->{'httproot'},$virtual);
- $output   .= dir_link($virtual);
- $output   .= "<p><b>Note:</b> On UNIX systems, filenames are <b>case-sensitive</b>!</p>\n\n";
+ my $virtual        = encode_entities($data->{'virtual'});
+ my $new_physical   = $data->{'new_physical'};

 
 

- $output .= "<p>Someone else is currently editing this file. So not all features are available.</p>\n\n" unless($unused);
+ return error($config->{'err_nocopy'}) unless(-r $physical);

 
 

- $output .= <<END;
-<hr>
-
-<h2>Copy</h2>
-
-<p>Copy file '$virtual' to: <input type="text" name="newfilename" size="50"> <input type="submit" value="Copy!"></p>
-
-<hr>
-
-END
-
- if($unused)
+ if($new_physical)

  {
  {

-  # File is not locked
-  # Allow renaming and deleting the file
+  my $new_virtual = $data->{'new_virtual'};
+  my $dir         = upper_path($new_virtual);
+  $new_virtual    = encode_entities($new_virtual);

 
 

-  $output .= <<END;
-<h2>Move/rename</h2>
-
-<p>Move/Rename file '$virtual' to: <input type="text" name="newfilename" size="50"> <input type="submit" value="Move/Rename!"></p>
-
-<hr>
-
-<h2>Delete</h2>
+  if(-e $new_physical)
+  {
+   return error($config->{'err_exist_edited'},$dir,{FILE => $new_virtual}) if($data->{'uselist'}->in_use($data->{'new_virtual'}));
+
+   if(-d $new_physical)
+   {
+    return error($config->{'err_dircopy'});
+   }
+   elsif(not $data->{'cgi'}->param('confirmed'))
+   {
+    my $tpl = new Template;
+    $tpl->read_file($config->{'tpl_confirm_replace'});
+
+    $tpl->fillin("FILE",$virtual);
+    $tpl->fillin("NEW_FILE",$new_virtual);
+    $tpl->fillin("NEW_FILENAME",file_name($new_virtual));
+    $tpl->fillin("NEW_DIR",$dir);
+    $tpl->fillin("DIR",upper_path($virtual));
+    $tpl->fillin("COMMAND","copy");
+    $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+    $tpl->fillin("SCRIPT",$script);
+
+    my $output = header(-type => "text/html");
+    $output   .= $tpl->get_template;
+
+    return \$output;
+   }
+  }

 
 

-<form action="$script" method="get">
-<input type="hidden" name="file" value="$virtual">
-<input type="hidden" name="command" value="remove">
-<p><input type="submit" value="Delete file '$virtual'!"></p>
-</form>
-END
+  copy($physical,$new_physical) or return error($config->{'err_copy_failed'},upper_path($virtual),{FILE => $virtual, NEW_FILE => $new_virtual});
+  return devedit_reload({command => 'show', file => $dir});

  }
  else
  {
  }
  else
  {

-  # File is locked
-  # Just display a button for unlocking it
+  my $tpl = new Template;
+  $tpl->read_file($config->{'tpl_copyfile'});

 
 

-  $output .= <<END;
-<h2>Unlock file</h2>
+  $tpl->fillin("FILE",$virtual);
+  $tpl->fillin("DIR",upper_path($virtual));
+  $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+  $tpl->fillin("SCRIPT",$script);

 
 

-<p>Someone else is currently editing this file. At least, the file is marked so. Maybe, someone who was editing the file has forgotten to unlock it. In this case (and <b>only</b> in this case) you can unlock the file using this button:</p>
+  my $output = header(-type => "text/html");
+  $output   .= $tpl->get_template;

 
 

-<form action="$script" method="get">
-<input type="hidden" name="file" value="$virtual">
-<input type="hidden" name="command" value="unlock">
-<p><input type="submit" value="Unlock file '$virtual'"></p>
-</form>
-END
+  return \$output;

  }
  }

-
- $output .= "\n<hr>";
- $output .= htmlfoot;
-
- return \$output;
-}
-
-# exec_copy()
-#
-# Copy a file and return to directory view
-#
-# Params: 1. Reference to user input hash
-#         2. Reference to config hash
-#
-# Return: Output of the command (Scalar Reference)
-
-sub exec_copy($$)
-{
- 1;

 }
 
 # exec_rename()
 }
 
 # exec_rename()


@@ -431,12 +466,71 @@ sub exec_copy($$)
 
 sub exec_rename($$)
 {
 
 sub exec_rename($$)
 {

- 1;
+ my ($data,$config) = @_;
+ my $physical       = $data->{'physical'};
+ my $virtual        = $data->{'virtual'};
+ my $new_physical   = $data->{'new_physical'};
+
+ return error_in_use($virtual) if($data->{'uselist'}->in_use($virtual));
+
+ if($new_physical)
+ {
+  my $new_virtual = $data->{'new_virtual'};
+  my $dir         = upper_path($new_virtual);
+  $new_virtual    = encode_entities($new_virtual);
+
+  if(-e $new_physical)
+  {
+   return error($config->{'err_exist_edited'},$dir,{FILE => $new_virtual}) if($data->{'uselist'}->in_use($data->{'new_virtual'}));
+
+   if(-d $new_physical)
+   {
+    return error($config->{'err_dircopy'});
+   }
+   elsif(not $data->{'cgi'}->param('confirmed'))
+   {
+    my $tpl = new Template;
+    $tpl->read_file($config->{'tpl_confirm_replace'});
+
+    $tpl->fillin("FILE",$virtual);
+    $tpl->fillin("NEW_FILE",$new_virtual);
+    $tpl->fillin("NEW_FILENAME",file_name($new_virtual));
+    $tpl->fillin("NEW_DIR",$dir);
+    $tpl->fillin("DIR",upper_path($virtual));
+    $tpl->fillin("COMMAND","rename");
+    $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+    $tpl->fillin("SCRIPT",$script);
+
+    my $output = header(-type => "text/html");
+    $output   .= $tpl->get_template;
+
+    return \$output;
+   }
+  }
+
+  rename($physical,$new_physical) or return error($config->{'err_rename_failed'},upper_path($virtual),{FILE => $virtual, NEW_FILE => $new_virtual});
+  return devedit_reload({command => 'show', file => $dir});
+ }
+ else
+ {
+  my $tpl = new Template;
+  $tpl->read_file($config->{'tpl_renamefile'});
+
+  $tpl->fillin("FILE",$virtual);
+  $tpl->fillin("DIR",upper_path($virtual));
+  $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+  $tpl->fillin("SCRIPT",$script);
+
+  my $output = header(-type => "text/html");
+  $output   .= $tpl->get_template;
+
+  return \$output;
+ }

 }
 
 # exec_remove()
 #
 }
 
 # exec_remove()
 #

-# Remove a file and return to directory view
+# Remove a file or a directory and return to directory view

 #
 # Params: 1. Reference to user input hash
 #         2. Reference to config hash
 #
 # Params: 1. Reference to user input hash
 #         2. Reference to config hash


@@ -449,14 +543,58 @@ sub exec_remove($$)
  my $physical       = $data->{'physical'};
  my $virtual        = $data->{'virtual'};
 
  my $physical       = $data->{'physical'};
  my $virtual        = $data->{'virtual'};
 

- return error_in_use($virtual) if($data->{'uselist'}->in_use($virtual));
+ if(-d $physical)
+ {
+  # Remove a directory

 
 

- my $dir = upper_path($virtual);
+  if($data->{'cgi'}->param('confirmed'))
+  {
+   rmtree($physical);
+   return devedit_reload({command => 'show', file => upper_path($virtual)});
+  }
+  else
+  {
+   my $tpl = new Template;
+   $tpl->read_file($config->{'tpl_confirm_rmdir'});

 
 

- unlink($physical) or return error("Could not delete file '$virtual'.");
+   $tpl->fillin("DIR",$virtual);
+   $tpl->fillin("UPPER_DIR",upper_path($virtual));
+   $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+   $tpl->fillin("SCRIPT",$script);

 
 

- my $output = redirect("http://$ENV{'HTTP_HOST'}$script?command=show&file=$dir");
- return \$output;
+   my $output = header(-type => "text/html");
+   $output   .= $tpl->get_template;
+
+   return \$output;
+  }
+ }
+ else
+ {
+  # Remove a file
+
+  return error_in_use($virtual) if($data->{'uselist'}->in_use($virtual));
+
+  if($data->{'cgi'}->param('confirmed'))
+  {
+   unlink($physical) or return error($config->{'err_delete_failed'},upper_path($virtual),{FILE => $virtual});
+   return devedit_reload({command => 'show', file => upper_path($virtual)});
+  }
+  else
+  {
+   my $tpl = new Template;
+   $tpl->read_file($config->{'tpl_confirm_rmfile'});
+
+   $tpl->fillin("FILE",$virtual);
+   $tpl->fillin("DIR",upper_path($virtual));
+   $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+   $tpl->fillin("SCRIPT",$script);
+
+   my $output = header(-type => "text/html");
+   $output   .= $tpl->get_template;
+
+   return \$output;
+  }
+ }

 }
 
 # exec_unlock()
 }
 
 # exec_unlock()


@@ -472,17 +610,28 @@ sub exec_remove($$)
 sub exec_unlock($$)
 {
  my ($data,$config) = @_;
 sub exec_unlock($$)
 {
  my ($data,$config) = @_;

- my $physical       = $data->{'physical'};

  my $virtual        = $data->{'virtual'};
  my $virtual        = $data->{'virtual'};

- my $uselist        = $data->{'uselist'};

 
 

- my $dir = upper_path($virtual);
+ if($data->{'cgi'}->param('confirmed'))
+ {
+  file_unlock($data->{'uselist'},$virtual);
+  return devedit_reload({command => 'show', file => upper_path($virtual)});
+ }
+ else
+ {
+  my $tpl = new Template;
+  $tpl->read_file($config->{'tpl_confirm_unlock'});

 
 

- $uselist->remove_file($virtual);
- $uselist->save;
+  $tpl->fillin("FILE",$virtual);
+  $tpl->fillin("DIR",upper_path($virtual));
+  $tpl->fillin("URL",equal_url($config->{'httproot'},$virtual));
+  $tpl->fillin("SCRIPT",$script);

 
 

- my $output = redirect("http://$ENV{'HTTP_HOST'}$script?command=show&file=$dir");
- return \$output;
+  my $output = header(-type => "text/html");
+  $output   .= $tpl->get_template;
+
+  return \$output;
+ }

 }
 
 # it's true, baby ;-)
 }
 
 # it's true, baby ;-)