compileVirtual should not depend on `File` object to template

[unoexperto]

I think TwirlCompiler.compileVirtual should not depend on File object to template because internally it's used only to generate template name. This prevent clients from compiling twirl templates on the fly. In my case I'd like admins of web-service to be able to update templates via API which are stored in DB.

[mkurz]

@unoexperto How about a pull request? 😄

[mkurz]

+1
When I was writing compiler tests I had to create a temporary template file, I was wondering why it is not possible to compile a template without having to create a file:

twirl/compiler/src/test/scala/play/twirl/compiler/test/CompilerSpec.scala

Line 26 in c6e9ee2

private val tmpTemplateFile = new File(sourceDir, "temporary.scala.html")

twirl/compiler/src/test/scala/play/twirl/compiler/test/CompilerSpec.scala

Lines 391 to 403 in c6e9ee2

	"using pure code blocks" in {
	TwirlIO.writeStringToFile(
	tmpTemplateFile,
	"""@var field = @{ "bar1" }
	|@field = @{ "bar2" }
	|@field""".stripMargin
	)
	val helper = newCompilerHelper
	val compiled = helper.compile[(() => Html)]("temporary.scala.html", "html.temporary")
	compiled.static().toString.trim.replace("\n", "").replaceAll("\\s+", "") must be(
	"""bar2"""
	)
	}

twirl/compiler/src/test/scala/play/twirl/compiler/test/CompilerSpec.scala

Lines 55 to 58 in c6e9ee2

	override def afterEach(): Unit = {
	tmpTemplateFile.delete()
	super.afterEach()
	}

[mkurz]

• Related: Sbt Setting for package name  #64