From e8977dfee14858f5cb8a19f2207a576d4e1d25e7 Mon Sep 17 00:00:00 2001
From: Lukas Eder <lukas.eder@gmail.com>
Date: Mon, 10 Aug 2020 12:11:04 +0200
Subject: [PATCH] [jOOQ/jOOQ#10480] Avoid generating empty header comments in
 Keys.java

---
 .../java/org/jooq/codegen/JavaGenerator.java   | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/jOOQ-codegen/src/main/java/org/jooq/codegen/JavaGenerator.java b/jOOQ-codegen/src/main/java/org/jooq/codegen/JavaGenerator.java
index 919a624ca6..a5d7082c95 100644
--- a/jOOQ-codegen/src/main/java/org/jooq/codegen/JavaGenerator.java
+++ b/jOOQ-codegen/src/main/java/org/jooq/codegen/JavaGenerator.java
@@ -757,9 +757,6 @@ public class JavaGenerator extends AbstractGenerator {
         List<ForeignKeyDefinition> allForeignKeys = new ArrayList<>();
 
         // Unique keys
-        out.header("UNIQUE and PRIMARY KEY definitions");
-        out.println();
-
         for (TableDefinition table : database.getTables(schema)) {
             try {
                 List<UniqueKeyDefinition> uniqueKeys = table.getUniqueKeys();
@@ -771,6 +768,12 @@ public class JavaGenerator extends AbstractGenerator {
                     final String keyId = getStrategy().getJavaIdentifier(uniqueKey);
                     final int block = allUniqueKeys.size() / INITIALISER_SIZE;
 
+                    // [#10480] Print header before first key
+                    if (allUniqueKeys.isEmpty()) {
+                        out.header("UNIQUE and PRIMARY KEY definitions");
+                        out.println();
+                    }
+
                     if (scala || kotlin)
                         out.println("val %s = UniqueKeys%s.%s", keyId, block, keyId);
                     else
@@ -785,9 +788,6 @@ public class JavaGenerator extends AbstractGenerator {
         }
 
         // Foreign keys
-        out.header("FOREIGN KEY definitions");
-        out.println();
-
         for (TableDefinition table : database.getTables(schema)) {
             try {
                 List<ForeignKeyDefinition> foreignKeys = table.getForeignKeys();
@@ -800,6 +800,12 @@ public class JavaGenerator extends AbstractGenerator {
                     final String keyId = getStrategy().getJavaIdentifier(foreignKey);
                     final int block = allForeignKeys.size() / INITIALISER_SIZE;
 
+                    // [#10480] Print header before first key
+                    if (allForeignKeys.isEmpty()) {
+                        out.header("FOREIGN KEY definitions");
+                        out.println();
+                    }
+
                     if (scala || kotlin)
                         out.println("val %s = ForeignKeys%s.%s", keyId, block, keyId);
                     else