On 30 Jan 2011, at 09:21, Pezhman Lali wrote:
>  Faxter is an opensource email to fax gateway, 
> please check it, let me know if any bug.

Only bug i can see is the attitude of the developer... 

As for the bugs, having the config variables liberally scattered throughout the 
script makes it's use (and then subsequent update) near impossible. There are 
even context names towards the end of the file. Ideally you'd want a separate 
config.php which you then include from your main script. A readme would then 
document what you'd put in here (and their default values if you dont). The 
tabbing is pretty random, and the commented out test data is pretty decorative.

chmod($save_dir.$filename,0777);

Is a slightly interesting idea.

Not actually run it to see if it works, wouldn't know how..

S


--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
New to Asterisk? Join us for a live introductory webinar every Thurs:
               http://www.asterisk.org/hello

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users

Reply via email to