package net.wizardsoflua.lua.module.searcher;

import java.lang.reflect.UndeclaredThrowableException;
import java.net.URL;

import org.apache.commons.io.IOUtils;

import net.sandius.rembulan.Table;
import net.sandius.rembulan.Variable;
import net.sandius.rembulan.impl.NonsuspendableFunctionException;
import net.sandius.rembulan.runtime.AbstractFunction1;
import net.sandius.rembulan.runtime.ExecutionContext;
import net.sandius.rembulan.runtime.LuaFunction;
import net.sandius.rembulan.runtime.ResolvedControlThrowable;
import net.wizardsoflua.lua.compiler.ExtendedChunkLoader;
import net.wizardsoflua.lua.compiler.LuaFunctionBinary;

/**
 * The {@link ClasspathResourceSearcher} searches the Java classpath for Lua modules.
 * <p>
 * This searcher can find any classpath resource having a name ending with the '.lua' extension.
 * <p>
 * Please note, that according to the Lua conventions, the required module must be specified using
 * the dotted package notation but without the '.lua' extension, for example the resource
 * 'wol/Globals.lua' must be specified as 'wol.Globals'.
 */
public class ClasspathResourceSearcher {

  public static void installInto(Table env, ExtendedChunkLoader loader,
      LuaFunctionBinaryByUrlCache cache, ClassLoader classloader) {
    Object function = getFunction(env, loader, cache, classloader);
    Table pkg = (Table) env.rawget("package");
    Table searchers = (Table) pkg.rawget("searchers");
    long len = searchers.rawlen();
    searchers.rawset(len + 1, function);
  }

  private static Object getFunction(Table env, ExtendedChunkLoader loader,
      LuaFunctionBinaryByUrlCache cache, ClassLoader classloader) {
    return new LoaderFunction(env, loader, cache, classloader);
  }

  private static class LoaderFunction extends AbstractFunction1 {

    private final Table env;
    private ExtendedChunkLoader loader;
    private LuaFunctionBinaryByUrlCache cache;
    private ClassLoader classloader;

    public LoaderFunction(Table env, ExtendedChunkLoader loader, LuaFunctionBinaryByUrlCache cache,
        ClassLoader classloader) {
      this.env = env;
      this.loader = loader;
      this.cache = cache;
      this.classloader = classloader;
    }

    @Override
    public void invoke(ExecutionContext context, Object arg1) throws ResolvedControlThrowable {
      String moduleName = String.valueOf(arg1);
      if (moduleName == null) {
        throw new IllegalArgumentException(
            String.format("Expected module name, but got: %s", moduleName));
      }
      try {
        if (moduleName.endsWith(".lua")) {
          throw new IllegalArgumentException("Illegal module '" + moduleName
              + "': module must be specified without '.lua' extension!");
        }
        String name = moduleName.replaceAll("\\.", "/") + ".lua";
        URL resource = classloader.getResource(name);
        if (resource == null) {
          context.getReturnBuffer().setTo("no module with name '" + name + "' found in classpath");
          return;
        }
        LuaFunctionBinary fnBin = cache.get(resource);
        if (fnBin == null) {
          String src = IOUtils.toString(resource);
          fnBin = loader.compile(moduleName, src);
          cache.put(resource, fnBin);
        }
        LuaFunction fn = fnBin.loadInto(new Variable(env));
        context.getReturnBuffer().setTo(fn, moduleName);
      } catch (Exception e) {
        throw new UndeclaredThrowableException(e);
      }
    }

    @Override
    public void resume(ExecutionContext context, Object suspendedState)
        throws ResolvedControlThrowable {
      throw new NonsuspendableFunctionException();
    }
  }

}

@@ -116,16 +116,48 @@
 et.URL;%0A
+import java.nio.charset.Charset;
 %0Aimport 
@@ -187,17 +187,16 @@
 OUtils;%0A
-%0A
 import n
@@ -3173,32 +3173,58 @@
 oString(resource
+, Charset.defaultCharset()
 );%0A          fnB

Remove use of the deprecated method IOUtils.toString(URL)