#12446: Add GetUtilityPageIdentifiers for Manage Custom Pages to be excluded …

[osrecio]

Added Interface and class to get collection of pages identifiers to be excluded from sitemap

Description

Get default pages from config to be excluded in Sitemap.

Fixed Issues (if relevant)

1. Remove /home from the sitemap.xml #12446: Remove /home from the sitemap.xml

Manual testing scenarios

1. Create Sitemap
2. the pages like {domain}/home {domain}/no-route no longer exists for sitemap

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[osrecio]

@vkublytskyi I should add @SuppressWarnings(PHPMD.LongVariable) below PhpDocBlock? Or I can omit it?

/**
 * Sitemap cms page collection model
 *
 * @api
 * @since 100.0.2
 * @SuppressWarnings(PHPMD.LongVariable)
 */

[vkublytskyi]

@osrecio you may ignore Codacy failure if Magento static tests are green. Codacy is a recommendation, not a requirement.

[osrecio]

Some update @vkublytskyi ? Can I help you with something in this PR ?

[vkublytskyi]

No need to mark class as an @api only interface is and API. Class as implementation is module internal details.

[vkublytskyi]

@osrecio I think we should remove @api annotation from a class as its interface already declared as module service contract and we can merge this PR.

[osrecio]

@vkublytskyi Deleted @apiannotation :)