fix(topology) Correct URL options for service pages

[UrBnW]

Hi,

Description

This PR closes #6162, where the default "Display details" filter set to Problems is not enforced in Moniotring Services views.

Type of change

• Patch fixing an issue (non-breaking change)

Target serie

Please delete series that are not relevant.

• 20.04.x (master)

How this pull request can be tested ?

Go to Monitoring / Status Detail / Services grid
Go to Monitoring / Status Detail / Services by Hostgroups
Go to Monitoring / Status Detail / Services by Servicegroups

And verify that the default "Display details" filter set to Problems is correctly active.

Edit : issue still present in 19.10.3.

[adr-mo]

Hi @cpbn,

Thanks for the contribution 💯

We'll try to make your work happen in Centreon as soon as possible 👍

Regards,

[UrBnW]

Rebased for 19.04.1.
@lpinsivy Laurent could we then think about adding this tiny PR to the 19.04.1 project please, so that I don't have to change SQL filename on every release ? :)
Many thanks 👍

[adr-mo]

Hi @cpbn

I'll request some reviews for this PR so that we can start the validation on it :)

@lpinsivy I am adding this to the 19.04.1

Regards,

[UrBnW]

Hi @adr-mo, and thank you very much !

[UrBnW]

@adr-mo, sounds like this has been moved to DONE status without having been implemented ? :-|
Thank you !

[UrBnW]

For info this fix was already pushed in Centreon 2.7.1 : Update-DB-2.7.1.sql.
Unfortunately, insertTopology.sql was not updated at the same time.
This is what this PR does.

[sc979]

@adr-mo , forking, rebasing and launching a job on the CI for you to be able to test TNR ;)
check branch :
CPbN-fix-topology-url-for-service-pages

[sc979]

I resolved conflict and updated the SQL upgrade script to 18.10.x and 19.04 branches

[UrBnW]

Rebased for master / 20.04.0-beta.1.

[UrBnW]

Just rebased again.
@kduret could you please re-approve this so that we can move on ?
Thank you !

[adr-mo]

Hi @cpbn
I'll create a branch from Centreon repository with your work for building / validation matters.
In fact the build of your work fails on our acceptance tests but it is not related to your work ;)

This will avoid to ask you to rebase your work everytime some changes are pushed on master.

I'll let you know when the validation is done, and your work merged :)

Thanks again for the contribution 💯

[adr-mo]

Branch created MON-4140-topology-services-issue and currently being built :)

[adr-mo]

@cpbn PR created above yours so that we can easily validate it :)

[UrBnW]

Good news, many thanks @adr-mo 👍
Glad to see it almost landed 😉