Fix debug logging in MergingPersistenceUnitManager.

missingdays · gregturn · commit 0896c231a316 · 2022-05-10T16:51:25.000-05:00
A String.format token was missing, so `mappingFileName` was printed instead of a `persistenceUnitName`, and `persistenceUnitName` wasn't actually printed at all. See #2526.
diff --git a/src/main/java/org/springframework/data/jpa/support/MergingPersistenceUnitManager.java b/src/main/java/org/springframework/data/jpa/support/MergingPersistenceUnitManager.java
@@ -90,7 +90,7 @@ void postProcessPersistenceUnitInfo(MutablePersistenceUnitInfo pui, PersistenceU
 			if (!pui.getMappingFileNames().contains(mappingFileName)) {
 
 				if (LOG.isDebugEnabled()) {
-					LOG.debug(String.format("Adding mapping file to persistence unit %s.", mappingFileName, persistenceUnitName));
+					LOG.debug(String.format("Adding mapping file %s to persistence unit %s.", mappingFileName, persistenceUnitName));
 				}
 				pui.addMappingFileName(mappingFileName);
 			}

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ void postProcessPersistenceUnitInfo(MutablePersistenceUnitInfo pui, PersistenceU`
`90`	`90`	`if (!pui.getMappingFileNames().contains(mappingFileName)) {`
`91`	`91`
`92`	`92`	`if (LOG.isDebugEnabled()) {`
`93`		`- LOG.debug(String.format("Adding mapping file to persistence unit %s.", mappingFileName, persistenceUnitName));`
	`93`	`+ LOG.debug(String.format("Adding mapping file %s to persistence unit %s.", mappingFileName, persistenceUnitName));`
`94`	`94`	`}`
`95`	`95`	`pui.addMappingFileName(mappingFileName);`
`96`	`96`	`}`