ConfiguratorExtension should only return factory configurations if checking for existing factory config

• Type: Bug
• Status: Closed
• Resolution: Won't Fix
• Affects Version/s: 7.0.X, 7.1.X, Master
• Fix Version/s: None
• Component/s: Portal Configuration
• Labels: None
• Fix Priority: 3

_process method in ConfiguratorExtension.java could return without creating a factory config because a single config with "configurator.url" property could be found by the query.

(modules/apps/portal-configuration/portal-configuration-extender/src/main/java/com/liferay/portal/configuration/extender/internal/ConfiguratorExtension.java)

	private void _process(ConfigurationDescription configurationDescription)
...
		if (configurationDescription.getFactoryPid() == null) {
...
		else {
			configuratorURL =
				_namespace + "#" + configurationDescription.getPid();

			if (_configurationExists(
					"(configurator.url=" + configuratorURL + ")")) {

				return;
			}

			configuration = _configurationAdmin.createFactoryConfiguration(
				configurationDescription.getFactoryPid(), StringPool.QUESTION);
		}

It is suggested that we also query for factory pid

"(&(factory.pid=" + configurationDescription.getFactoryPid() + ")(configurator.url=" + configuratorURL + "))"