Symbols fix

[Profesor08]

Description

Symbol message is converted to string, this behavior make uselles using symbols. Because symbols are not strings, and every symbol is unique Symbol("message") !== Symbol("message").

What is the purpose of this pull request?

• Bug fix

Reproduction

Next code will call func1 twice. This is unexpecte behavior, func1 must be called only once.

        var MESSAGE1 = Symbol("MESSAGE");
        var MESSAGE2 = Symbol("MESSAGE");

        var func1 = function (msg) {
            return undefined;
        };

        PubSub.subscribe(MESSAGE1, func1);

        PubSub.publish(MESSAGE1);
        PubSub.publish(MESSAGE2);

[Profesor08]

@mroderick some attention please

[fezhengjin]

@mroderick same to me

[verekia]

Is there something we can help with to get this merged?

Maybe removing 'MESSAGE' from the test and 'MY_TOPIC' from the docs?

https://github.com/mroderick/PubSubJS/blob/master/README.md?plain=1#L217

I added a comment on the PR that introduced Symbol support: #141 (comment)

[mroderick]

This looks good to me. Is there anything you'd add/change @verekia ?

[verekia]

Looks good to me too :)

[dikaso]

Any news here? Are you going to release it anytime soon?