customEvent: error in code for building event.namespace_re

[tim@…]

Line 3151 of 1.7.2

event.namespace_re = event.namespace? new RegExp("(^{|
.)" + namespaces.join("
.(?:.*
.)?") + "(
.|$)") : null;}

When last array item in 'namespaces' ends with \ it fails with error (Chrome):

SyntaxError: Invalid regular expression: /(^{|\.)dynatree\.(?:.*\.)?file-browser-tree\.(?:.*\.)?u:\(\.|$)/: Unmatched ')'}

Example: ["dynatree", "file-browser-tree", "u:\"]

[dmethvin]

So your namespace has a backslash in it? Please provide a test case in jsFiddle.

[tim@…]

Sorry struggling to figure out how to code this in jsFiddle having never set up my own before, but this is the crux of it:

var self = {}; $('body').trigger("nodeLoaded.dynatree.file-browser-tree.u:
", [self, true]);​

It's an event generated by a 3rd party javascript library.

For time being I have changed to forward slash.

[tim@…]

Replying to dmethvin:

So your namespace has a backslash in it? Please provide a test case in jsFiddle.

The system stripped out two back slashes between u: and " e.g. \ \ but no space, ie escaped backslash

[dmethvin]

I don't think there is any need to change code, let's add some docs advising that event names should contain only alphanumerics, underscore, and colon. The dot is reserved for namespaces and we don't need a way to escape it.