Date: Tue, 22 Feb 2000 05:16:52 -0800
: yannickf@yahoo.com
Objet: Dir.php3
De: Dave Smith <dave.s@earthcorp.com>
Ok, I dunno if this is the sortof feedback you want, but
I'll suggest it anyway.
When using your script with PHP v3.0.5, there was a problem
with putting your "default:" statement on line 172. It
would seem that you can't put a default: before the rest of
your case statements.
I have some suggestions also
* Will you have some sort of Upload mechanism
* Rename
* Move (could be same as rename)
* Download
* Simple View
* Put atime, mtime, ctime as proper dates
* About information so that we can tell what version it is
and where to get further updates. You could just put it as
a comment in the source.
I think that is enough of my suggestions, just to say that
it looks very good and you could take it a very long way
Dave Smith
----------
Dave.S@Earthcorp.com
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Date: Tue, 22 Feb 2000 05:16:52 -0800
: yannickf@yahoo.com
Objet: Dir.php3
De: Dave Smith <dave.s@earthcorp.com>
Ok, I dunno if this is the sortof feedback you want, but
I'll suggest it anyway.
When using your script with PHP v3.0.5, there was a problem
with putting your "default:" statement on line 172. It
would seem that you can't put a default: before the rest of
your case statements.
I have some suggestions also
* Will you have some sort of Upload mechanism
* Rename
* Move (could be same as rename)
* Download
* Simple View
* Put atime, mtime, ctime as proper dates
* About information so that we can tell what version it is
and where to get further updates. You could just put it as
a comment in the source.
I think that is enough of my suggestions, just to say that
it looks very good and you could take it a very long way
Dave Smith
----------
Dave.S@Earthcorp.com