Ver Mensaje Individual
  #4 (permalink)  
Antiguo 27/07/2004, 07:39
Cluster
O_O
 
Fecha de Ingreso: enero-2002
Ubicación: Santiago - Chile
Mensajes: 34.417
Antigüedad: 22 años, 3 meses
Puntos: 129
Bueno .. ahí van mis comentarios:

Documentación:
- Demasiado "sutilmente" mencionas que dicho script/aplicación usa las extensiones PDF lib .. de hecho dices textualmente: "ste script necesita el modulo php_pdf cargado en el php.ini". No estaría de más decir que lo que usa el script son dichas extensiones y un link al manual oficial de PHP .. por qué no todo el mundo disfruta de esas extensioenes (sobre todo en servicios de hosting es raro verla). Y que requiere de su instalación en caso de no tenerla (y que lo pueden comprobar con un phpinfo() ..)

- La aplicación crea archivos y directorios .. en ningún momento hablas sobre permisos de escritura y demás .. Eso les va a dar problemas a usuarios de S.O. Linux, y algunas versiones de Windows.

- Licencia .. No está de más indicar algún tipo de normas para su uso o licencia .. Poco màs que digas que es un proyecto bajo GPL sobraría (de libre distribución, uso y modificaciones) salvo que especifiques algo concreto...

- Ortografía xD .. Hay que cuidarla ¬¬

Código
- Uso de variables externas. En general mezclas el uso de arrays superglobales como $_POST .. con definición de "globales" (global $nombre,$apellido,$email,$tel,$productos,$con,$dia ,$dia2,$mes,$ano,$hora,$horamala,$horareg;) lo cual no tiene sentido si usas ya los arrays superglobales que es lo que debes usar. Tambien usas getenv() para acceder a variables de servidor que ya puedes acceder por $_SERVER como las $_POST y demás ...

Este uso de los arrays superglobales ya te limita a versiones de PHP 4.1.0 en adelante .. otro detalle a avisar en la documentación.

- Cuando creas directorios (mkdir() ... ) no defines permisos de lectura/escritura .. deberías definirlos usando chmod() ..

- Crear esos registros que haces de archivos sin extensión o con nombre "un MD5()" .. no tendría que ser tanto problema de "seguridad" si colocas (creas) tus archivos fuera del "Document Root" de ese servidor HTTP. Una configuración del servidor HTTP (Apache por ejemplo) tipo "Indexes" podría mostrar igualmente esos archivos que contienen tus PDF's o registros y verlos/descargarlos sin problemas .. Por eso lo ideal es dejarlo fuera del Document Root o proteger su acceso por un .htacces limitando que IP accede (sólo locales = tus scripts PHP y siendo estos quien entregue el archivo previa pequeña autentificación).

- Esa fecha que generas con la "traducción" de "ingles/español" .. te resulta más óptimo tomando el nº de mes con: date("n") y usando un array con los nombres de tus meses ( y lo mismo para los días de la semana si quieres) para usar como indice dicho dato.

De todas formas .. seguro que a alguien le servirá como ejemplo para ver como usar PDF lib y hasta el pequeño codiguito para enviar archivos adjuntos ..

Personalmente para esas taréas prefiero usar classes como PHP Mailer para la gestión del mail con archivos adjuntos y demás ... y para generar PDF's uso la classe R&OS, no porqué sea más óptimo en ese caso (nada que comparar una "classe" con una extensión de PHP) .. pero por lo menos no me hace depender de mi proveedor de Hosting, donde como ya te comenté no es muy común ver a PDF lib y no hay posibilidad de realizar ninguna instalación de una extensión por nuestra parte.

Un saludo,