Fix bom generation

[codeconsole]

The following introduces a lot of confusion and adds an unnecessary check

grails-core/grails-bom/build.gradle

Lines 95 to 99 in 7db9a62

	if(artifactId == 'grails-gradle-plugin') {
	mkp.groupId 'org.grails'
	} else {
	mkp.groupId 'org.grails.plugins'
	}

grails-gradle-plugin should not be part of plugin.properties in the first place

grails-core/grails-bom/plugins.properties

Line 4 in 7db9a62

grails-gradle-plugin=7.0.0-SNAPSHOT

Also, it is bad practice to have to add exclusions this way:

grails-core/grails-bom/build.gradle

Lines 49 to 68 in 7db9a62

	if (sub.name == 'grails-bootstrap') {
	mkp.exclusions {
	mkp.exclusion {
	mkp.groupId 'jline'
	mkp.artifactId 'jline'
	}
	mkp.exclusion {
	mkp.groupId 'org.fusesource.jansi'
	mkp.artifactId 'jansi'
	}
	mkp.exclusion {
	mkp.groupId 'net.java.dev.jna'
	mkp.artifactId 'jna'
	}
	mkp.exclusion {
	mkp.groupId 'org.apache.groovy'
	mkp.artifactId 'groovy-ant'
	}
	}
	}

The reason for this should be addressed.

Current bom

[matrei]

grails-gradle-plugin should not be part of plugin.properties in the first place

That is another issue. #13737 sets the correct groupId.

[codeconsole]

@matrei why does it have the wrong groupId in the first place?

[matrei]

why does it have the wrong groupId in the first place?

@codeconsole Looks like an oversight, as it's the only artifact in plugins.xml not using org.grails.plugins.

[codeconsole]

@codeconsole Looks like an oversight, as it's the only artifact in plugins.xml not using org.grails.plugins.

@matrei what is the best solution for this? It isn't a Grails plugin so it should stay org.grails, right?
Is this just a matter of moving the version from plugins.properties?

Please see my PR. Will that fix it?

#13755