Laravel broadcastAs and Echo listen mismatch

Hi,

Laravel gives the possibility to name event’s broadcast, however, Echo assumes that the event’s name is always a namespace. Following this documentation:

public function broadcastAs()
{
    return 'server.created';
}

I was trying to broadcast the event via Pusher, and had this code in my Javascript:

window.Echo = new Echo({
    // settings
});
window.Echo.private(`App.User.${user.id}`)
.listen('.server.created', (e) => {
    console.log('notif arrived', e);
})

But wasn’t receiving any notifications in the front end, the problem was in Echo’s event-formatter, the format method replaces all . with \\

format(event: string): string {
        if (this.namespace) {
            if (event.charAt(0) != '\\' && event.charAt(0) != '.') {
                event = this.namespace + '.' + event;
            } else {
                event = event.substr(1);
            }
        }
        // Why is this here?
        return event.replace(/\./g, '\\');
    }

Now I’m not sure if the problem is:

1. is it in Laravel Event broadcastAs method that should format the event’s name similar to Echo?
2. or Echo shouldn’t always format the event name and allow the developer to choose? which seems to be the right way to do it, unless I’m missing something. I can PR this.
3. or Laravel documentation is incorrect?
4. or I’m missing something.

Thanks

[ellisio]

Seeing as how namespace is exposed in Echo’s options, it might be worth changing the documentation to reflect that doing the following will prevent App\\Events\\ from being prepended.

let Echo = new Echo({
    namespace: null
});

Then changing the formatter to the following:

format(event: string): string {
    if (this.namespace) {
        event = this.namespace + '\\' + event;
    }

    return event;
}

Finally, change connector.ts’s namespace: 'App.Events' to namespace: 'App\\Events' since Laravel’s BroadcastEvent does get_class($this->event) here.

With those changes you could have the following configuration:

Dot Syntax

// PHP
broadcastAs() {
    return 'server.created';
}

// JS
let Echo = new Echo({
    namespace: null
});

Echo.listen('server.created', () => {});

Default Laravel Namespacing

// PHP
namespace App\Events;
class ServerCreated extends Event { }

// JS
let Echo = new Echo;
Echo.listen('App\\Events\\ServerCreated', () => {});

[hernandev]

made me crazy for a couple of hours.