Updated version number to 2.3.2

Merged changes of revision 1.76 of modules/Command.pm and revision 1.33 of
modules/Tool.pm to this branch

This commit was manufactured by cvs2svn to create branch 'version_2_3_x'.

Form element "file" was defined twice

Updated to version 1.4a
See http://www.patshaping.de/redirector/redirect.php?page=template-class

Using the move() function from the File::Copy module is more reliable than using
the rename() function

Don't create an incomplete HTML table if the root directory is empty or none of
the files in the root directory matches against the current wildcard

Now, the root directory must not be a symbolic link.
Sorry for this, I really would have liked to allow the root directory to be a
symbolic link, but if I would allow this, Dev-Editor would not be able to create
files in the root directory. I'm looking for an alternative...

http://forum.de.selfhtml.org/?t=101375&m=622582
This posting in a big german web developer forum sounds very convincing

The file saving process was aborted if the user wanted to write text data using
the edit function into a file that does not exists.
The reason was that Dev-Editor thought that this non-existing file was a binary
file.

Small code cleaning

Aha!

- Handle broken symbolic links properly
- It was possible to create files above the virtual root directory using
  symbolic links
- Updated version number to 2.3.1

Windows drive letters are case-insensitive!

- For security reasons, I completely changed the handling of symbolic links:
  Symbolic links are now treated as files, no matter where they point to. The
  user is just allowed to see the path where the link points to, he is allowed
  to delete the link, to rename it and to overwrite it by renaming an other
  file. Nothing else is allowed.
  Accessing a symbolic link pointing to a directory caused also a very strange
  effect: It was possible to access this directory, but you could not access the
  objects in it.
  I had to do it in this way because of the very stupid behaviour of abs_path()
  from the Cwd module: This function is just able to detect the absolute path of
  directories.
  Hard links are not affected by this, because it is not possible to detect them
  (at least I don't know how to detect them).
- Changed the names of two error messages:
  dircopy -> dir_copy
  editdir -> dir_edit
- Fixed various errors in some template files

- Check if unlocking of the list of files in use was successful
- Check if a file could be successfully added to the list of files in use
- Check if a file could be successfully removed from the list of files in use

file_read() is now able to open a file in binary mode (maybe we will need it)

Updated to version 1.4
See http://www.patshaping.de/redirector/redirect.php?page=template-class

Changed some comments

ARGH!

Parse the {IF} block "gmt" only once

- Allow to use Greenwich Mean Time in date strings (maybe someone needs it)
- Changed the order of the configuration options a little bit

Whoops...
Missing line break at the end of the file

- NOW check_path() should work properly
  I removed my self-written code to resolve "..", "...", "...." and so on and
  replaced it by a single call to abs_path(). That is more reliable!
- file_name() and upper_path() now take care of a
  "multiple-slash-root-directory" and Windows drive letters

Updated version number and copyright year

Small cleanup

Let's hope, check_path() is now working properly.
Maybe we should switch to Win32::GetFullPathName() on Windows...

- Fixed a bug occuring if the "curdir" CGI parameter had no trailing slash and
  the "newfile" parameter had no leading slash
- On Windows systems it was possible to access the directories above the root
  directory by accessing a path ending with "..." or "...." and so on

- When composing the temporary virtual path for a new file, don't call
  clean_path(). It is unnecessary and it also caused a problem if this path
  would begin with /../, because on UNIX systems, canonpath() removes /../ at
  the beginning of a path. So if a user wanted to create the file /../file.ext
  (which he wasn't allowed to), he created /file.ext.
- file_name() and upper_path() now remove multiple trailing slashes
- Improved configuration file parser:
  - Allow configuration options with empty values
  - If a option is defined twice, the line number is shown in the error message
- Static values are now surrounded by single quotes. Maybe it helps to increase
  the speed of Dev-Editor, because Perl doesn't have to try to interpolate
  variables in the values.

- Removed error message "file_in_use"
- Synchronized exec_mkfile() and exec_mkdir() in Command.pm

- In the virtual directory string, check_path() converts backslashes to slashes
  before removing slashes at the beginning (not really necessary, but I want to
  be sure)
- check_path() and equal_url() now remove multiple disturbing slashes in strings

- Format HTTP root as hyperlink
- "User"  -> "Process user"
  "Group" -> "Process group"

- Check if the CGI module was able to create a valid handle for a file upload
- Changed the error message "lock_failed" a little bit

When parsing a configuration file, ignore lines like that:
   = Value

If locking of the uselist file failed, get the template variables directly from
the File::UseList object, because the name of the lock file could have been
composed by File::UseList.

Changed some error messages

Cleaned exec_show() and exec_beginedit()

- Allow to filter directory listing using DOS-style wildcards (maybe it still
  needs work for being more usable)
- Fixed a typo in File::Access

- While processing a file upload, check if the user wants to replace a directory
  or overwrite a write-protected file
- If the user wants to copy or rename a file, check if he wants to overwrite a
  write-protected file

Some small changes

- Dev-Editor now uses flock(). flock() is called using a wrapper function which
  checks if it is really available.
- file_save() in File::Access is now able to open a file in binary mode. Thus,
  the file saving process in the file upload code in Command.pm has been
  replaced by a single file_save() call.

Removed useless "use Tool;" in Output.pm

- Entities in some more strings are now encoded
- The Hash Reference for devedit_reload() is now optional
- Removed "use Data::Dumper;" in Command.pm. I forgot to remove it after debugging.

- exec_chprop() should have a proper coding style, but it made the routine hard
  to maintain. Thus, I changed its structure.
- Don't allow to change the properties of a file in use
- Don't allow to upload a file if a file with the same virtual name is currently
  in use

The "Edit" link of files in use were not greyed in directory listing.
I hope it works properly now...

Do not allow to change the properties of the virtual root directory

Some cleanings

Version 1.3:
- Replaced open() by sysopen()
- Added method remove_all() to remove all files from the list (maybe we will
  need it...)
- Changed my email address

File uploads did not work. Fixed in Dev-Editor 2.2a.

This file testing order before saving a file makes more sense

Dev-Editor 2.2

Changed my email address (patshaping@gmx.net -> patrick@patshaping.de) in all
files except the Template and the File::UseList modules (they are seperate
software).

- Don't display the file upload link in directory listing if Dev-Editor has no
  write access to this directory.
- Wrong variable "{DIRECTORY}" in error message "dir_no_create" - it should be
  "{DIR}".

- Check if we have enough permissions to write in directory at the following
  points:
  - Before executing a command which creates a new file
  - Before uploading a file
  - Before removing or moving/renaming a file or directory
- oct() does not care about leading zeros, the resulting value is always the
  same.
- The name of the variable which will be replaced by a directory to go back to
  in error.htm was renamed from "{DIR}" to "{BACK}". "{DIR}" caused some
  problems.
- Changed the names of some error messages:
  nocopy -> no_copy
  noedit -> no_edit
  noview -> no_view

Some small changes

- Check if the user really wants to upload a file to a directory
- Cleaned exec_show()

- Added new if blocks to the templates about.htm and chprop.htm: The blocks are
  named "group_detected" and "user_detected" and show if Dev-Editor could
  successfully convert a group or user ID into the corresponding name.
- Cleaned exec_beginedit() and exec_endedit()
- More proper encoding of HTML entities at some points (still needs work)

Added single quotes around the "COMMAND" variable in the "cmd_unknown" error
message and around the "FILE" variable in the "not_owner" error message.

The path the user wants to access must now BEGIN with the root directory. The
old check made it possible to access a directory structure outside the root
directory similar to the root directory.

- The abort() routine from the Output module is now also able to display a link
  at the bottom of the error message. It is a direct wrapper to the error()
  routine.
  Thus, the following error messages now contain a link at the bottom:
  - File not found
  - Accessing above the virtual root directory
  - Creating an object above the virtual root directory
  - Directory where Dev-Editor should create an object does not exist
- It was possible to access the directory containing the virtual root directory.
  Well, it was only read-access.

Another bug in the same line!
I should work more properly...

Trying to fight against the chaos in errors.dat and templates.dat by sorting the
entries alphabetically.

Stupid bug

Check if the root directory exists

- Dev-Editor now checks if it is able to access a directory: Directories that
  are not accessible are greyed in directory listing. Accessing such a directory
  causes an error message.
- Dev-Editor checks if it has read access to the virtual root directory. Having
  no access to this directory would cause Dev-Editor to show the contents of the
  real root directory!

- There is no need to load the POSIX module twice
- Dev-Editor also shows the current umask of the process in the about dialogue

- There is no need to load the POSIX module twice
- Dev-Editor also shows the current umask of the process in the about dialogue

Because of the ugly behaviour of $(, I'm now using the getgid() function from
the POSIX module to detect the process GID. For being consistent, I'm now using
getuid() (also from the POSIX module) to detect the process UID.

In file edit and file upload dialogue, I added <label> tags for the checkboxes
and radio buttons.

Changed a comment

Removing one and more whitespace makes more sense than removing no, one and more whitespaces.

We are now using the `chgrp` system command to change the group of a file
(see bug [1052335])

When viewing a file, insert the contents of the file into the template AFTER
parsing the "editable" {IF}-block.
(Ever tried to view the viewfile.htm template file using Dev-Editor? Oh my god!)

Dev-Editor 2.1a

Instead of absolute paths in error_file and template_file, I use relative ones.
That's better for me ;-)

Now we already unlock the file at the begining of exec_endedit(), because if an
error occurs during saving, the file keeps locked.
This is not a perfect solution, there are still problems. I have to figure out
another one.

Dev-Editor 2.1

Removed the note which said that existing files can't be overwritten

Changed some comments

exec_unlock() now checks if the file is really in use. Otherwise, we return to
directory view (maybe we change this behaviour and display an error message...).

Check if the user wants to write text data into a binary file when saving a file

Improved file editing using another filename:
- If the user just entered spaces as filename (detection works for other
  commands using the "newfile" CGI parameter too), we use the original filename
- Saving even works if the file has to be created first

Improved file editing using another filename:
- If the user just entered spaces as filename (detection works for other
  commands using the "newfile" CGI parameter too), we use the original filename
- Saving even works, if the file has to be created first

Ooops...?
What was that?

Something definitely went wrong when I remade the copy and the rename functions.
They printed a wrong error message if the destination already existed and if it
was a file.
Dev-Editor also didn't deny copying a directory.

Added an option for overwriting files during file upload if they already exist

Dev-Editor 2.0

- Added some notes for changing mode and group
- The formular fields are no longer filled with the old values

Removed the race condition in file_create() which could set a file to 0 if it
already exist.
I wanted to use O_EXCL, but `perldoc -f sysopen` doesn't say that O_EXCL is
avaiable in _every_ system. So I use O_RDONLY and O_CREAT.
Additionally, I replaced all the open() calls in this module by sysopen() calls.

Some small changes

Allow to change the group of a file using the Group ID
(Let's hope nobody will define a group composed only of numbers - let's hope it
isn't allowed at all - I should install Linux at home...)

Replaced the code of mode_string() by some self-written code that I also
understand
(The previous code was grabbed from Sven Rautenberg's SELFHTML Developer
internal PHP Dev-Editor)

Dev-Editor is now able to change the mode and the group of files

Commands are now case-insensitive

Do not allow removing or renaming the root directory

Files and directories are now sorted case-insensitively

Typo

When viewing a file and if the file is editable, there is a new link to edit this file.

The Artistic License is now embedded in the about dialogue

Replaced "&" by "&amp;" in query strings

Small enhancement for devedit_reload()