Skip to content

Commit b561868

Browse files
committed
Disable parameter names for test compilation.
We now no longer use compiled parameter names for tests to avoid interference with Eclipselink's mechanism of using these improperly for stored procedure calls. Especially the fact that a parameter is named "in" lets the database consider using an IN clause instead of the parameter name. When configuring the compile plugin, we also need to ensure that the AspectJ plugin post-processes the compiled classes and doesn't run before the Maven compiler plugin. Closes: #2298 See also spring-projects/spring-data-build#1513
1 parent ccf6039 commit b561868

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

pom.xml

+17
Original file line numberDiff line numberDiff line change
@@ -475,6 +475,22 @@
475475
</executions>
476476
</plugin>
477477

478+
<plugin>
479+
<artifactId>maven-compiler-plugin</artifactId>
480+
<executions>
481+
<execution>
482+
<id>java-test-compile</id>
483+
<phase>test-compile</phase>
484+
<goals>
485+
<goal>testCompile</goal>
486+
</goals>
487+
<configuration>
488+
<parameters>false</parameters>
489+
</configuration>
490+
</execution>
491+
</executions>
492+
</plugin>
493+
478494
<plugin>
479495
<groupId>io.starter</groupId>
480496
<artifactId>aspectj-maven-plugin</artifactId>
@@ -496,6 +512,7 @@
496512
<goals>
497513
<goal>compile</goal>
498514
</goals>
515+
<phase>process-classes</phase>
499516
</execution>
500517
</executions>
501518
<configuration>

0 commit comments

Comments
 (0)