Adhearsion Adhearsion

The use of 'eval' hides the true location of exceptions in helpers. For example, just try

telnet localhost 1337
GET /collab HTTP/1.1
<<blank line>>

The error given is:

[2008-04-03 15:58:49] ERROR TypeError: can't convert nil into String
        (eval):442:in `+'
        (eval):442:in `do_GET'

which doesn't identify which file the error occured in.

After the following patch is applied:

--- /usr/lib/ruby/gems/1.8/gems/adhearsion-0.7.7/ahn.orig       2008-04-03 14:21:39.000000000 +0100
+++ /usr/lib/ruby/gems/1.8/gems/adhearsion-0.7.7/ahn    2008-04-03 15:59:33.000000000 +0100
@@ -207,7 +207,7 @@
         if config.delete('enabled') != false
          log "Parsing helper #{name}"
           $HELPERS[name] = config
-          eval File.read(f)
+          load f
         end
       end
     end

then the exception becomes clearer:

[2008-04-03 16:00:18] ERROR TypeError: can't convert nil into String
        ./helpers/micromenus.rb:442:in `+'
        ./helpers/micromenus.rb:442:in `do_GET'

(Maybe there is some reason why eval File.read(f) is used instead of load(f), but I can't think of one)