php-worker does not work about brace HOT 6 OPEN

@beryllium, thanks, that worked for getting rid of the warning.

However, I needed the following to get the highlighting working (from #2542 (comment)):

session.setMode({path:"ace/mode/php", inline:true})

from brace.

The same is with xquery-worker. Please remove it from mode/xquery.js or create an empty worker.

from brace.

@thlorenz Is there any solution for this? I am also stuck with this.

from brace.

Nope, I wasn't able to properly convert those workers into blobs .. tried for a while, but they don't properly stringify.
However anyone is welcome to try and fix this I'd be very happy with a PR that does :)

from brace.

If I replace the reference to mod.id in index.js near line 18003 with just mod, the error goes away and the code seems to be highlighted. Could the fix be this simple, or am I missing something?

from brace.

Good catch @beryllium and @ozws, thank you! If it need to keep working for other languages alongside php, set line ~18003 to

workerUrl = workerUrl || config.moduleUrl(mod.id ? mod.id : mod, "worker");

I put the whole thing in a new file BraceWithBugFix.js and then used var ace = require('./BraceWithBugFix'); feels somewhat dirty though 😄

from brace.