Allow specifying "global-overview" as default map

[mj84]

I implemented a quick way to allow the use of "global-overview" as the default map.
While mod=Overview does not respect header_menu=0 it is still nice to have a simple way to include the NagVis default overview.

[yhabteab]

Hi, thanks for contributing! Though, I'm not quite sure yet what this is supposed to fix. There's already an option, as far as I can see, that lets you define your own default map.

I implemented a quick way to allow the use of "global-overview" as the default map.

Where does the hard-coded global-overview map name come from?

[mj84]

Hi there,
sorry it took me a while to come back to you.
Currently, it is not possible to use NagVis' own overview (which is called via URL parameters mod=Overview) when using this module.
Since we are running quite a lot of NagVis maps, which also change semi-frequently, it is not feasible to manually create a map which links to other maps and use that "map" as an overview.
There is a comment in the source code at

icingaweb2-module-nagvis/application/controllers/ShowController.php

Line 54 in 140347a

// TODO: I'd prefer to have mod=Overview as a default, that would also

which states, that mod=Overview is currently not being used in this module because it doesn't respect the header_menu=0 parameter.
My PR restores this functionality by explicitly setting the default map to "global-overview" and the overview menu of NagVis returns:

[mj84]

Hi,
any thoughts about this?
For us, the native NagVis overview is really useful, I bet others would benefit from this as well.

That being said, I guess the "proper" solution would be a native overview implementation within this module, but this exceeds my abilities in Icingaweb2 module development by far :)

So this could be used as a temporary solution to provide an overview of accessible maps until a better one is available.

Best regards