On Feb 28, 4:44 pm, sarurobi <vittorio.balles...@gmail.com> wrote:

Yeah, last time I looked you could attach a file to a pool of files on
the group itself, but not to individual posts or 'Discussions'.  Seems
crazy.

Thank you very much.  Your patch seems to be backwards but I can see
what you were doing.  I've patched up cometd.js files, and also
reload.js and ReloadExtension.js since I use that.  Other extensions
would need patching accordingly.

However, my ready() events then fail to work, which seems to be a dojo
loader bug.

I'm not sure your code is correct.  Where you have e.g.:

    define("org/cometd",[],function() {

at the top of org/cometd.js, I'm used to syntax like this:

    define( [ "dojo/_base/declare", .. list of dependencies .. ],
function(declare, .. other deps ..) {
        return declare("org.cometd", null, { .. body here .. })
    })

Something alone those lines, anyway.  I've tried updating the code to
match this syntax, but I got lost converting the cometd objects to a
format that's compatible with "return declare", and I have a feeling
this is a wasted effort anyway since Simone said he was working toward
an official AMD patch (though I've seen no word for almost a month on
that).

Due to the loader bug and ready() event not working, this simple AMD
conversion is a non-starter for me.  I'm going to have to leave
async:false in the data-dojo-config, and hope the cometd project comes
through with an AMD compliant version before too long.

Nick