Documentation aliasing doesn't work

Workaround (in _Events.groovy):

import grails.doc.DocEngine
eventRefdocsStart = {
    createConfig()
    DocEngine.ALIAS.putAll(config.grails.doc.alias)
}

1.3: http://github.com/grails/grails-core/commit/501e9942a6da63984a02a5e599451c408345ebdd
1.2: http://github.com/grails/grails-core/commit/0f3fd0aec2b08ffd24cd0ca2347fbbf448593b8c

Luke, there must be something wrong with this commit. If I do 'grails doc' with Grails 1.3.4.BUILD-SNAPSHOT, it fails since the import for grails.doc.DocEngine is missing in _GrailsDoc.groovy. After adding the import locally I get a classpath error, so I assume grails.doc.DocEngine is not on the classpath when 'grails doc' is executed. As a temporary workaround I commented out line 274 in _GrailsDoc.groovy.
I think this should be promoted to blocker, since it 'grails doc' fails, and consequently 'grails release-plugin' too.

Confirmed.

This is a really weird error: adding grails.doc.DocEngine as an import causes a compilation error, but grails.doc.DocPublisher does not. They're provided by the same JAR! I think it might be down to the way the classes are used and hence loaded.

Anyway, I'll commit a fix.

Actually, I don't have any aliases to test. Here's my suggested patch:

@@ -271,7 +272,8 @@ def readDocProperties(DocPublisher publisher) {
 }
 
 def configureAliases() {
-    DocEngine.ALIAS.putAll(config.grails.doc.alias)
+    def docEngineClass = classLoader.loadClass('grails.doc.DocEngine')
+    docEngineClass.ALIAS.putAll(config.grails.doc.alias)
 }
 
 private readIfSet(DocPublisher publisher,String prop) {

Fixes the error, but I don't know whether the aliases will be used by the doc engine (is the class loaded into the correct class loader?).

The suggested patch works for me, but I have no aliases here either.

Peter since Stefan has confirmed the fix can you apply it?

I have a project will aliases, so I'll test and apply Peter's patch.

This is a rather strange issue though. I definitely didn't get this when I applied this change.