Added test on user to determine if the module should show or not

[brianteeman]

This is an admin menu module. How can there ever be a situation that a user is not logged in?

[obuisard]

It is to avoid this:

[brianteeman]

I thought that might be what you meant. Its not the correct solution and yes I see you did it this way the core guided yours module. ;)

A better way is to check if the user has permissions for j2store and only display the module then. thats how core should be doing it. something like

        if (
            !$this->getApplication()->getIdentity()->authorise('core.manage', 'com_j2store')
        ) {
            return;
        `}`

[j2commerce]

Ok, I understand, thanks Brian @brianteeman. It does also cover the case where someone should not have access to the module in the console, once logged in.

[brianteeman]

@j2commerce perfect