fix: avoid NPE and expose modifier

Signed-off-by: Andreas Reichel <[email protected]>

Author: manticore-projects

src/main/java/net/sf/jsqlparser/statement/select/SetOperation.java

@@ -13,25 +13,29 @@
 import net.sf.jsqlparser.statement.select.SetOperationList.SetOperationType;
 
 public abstract class SetOperation extends ASTNodeAccessImpl {
-    String modifier = "";
+    String modifier;
+
+    public String getModifier() {
+        return modifier != null ? modifier : "";
+    }
 
     public boolean isAll() {
-        return modifier.contains("ALL");
+        return modifier != null && modifier.contains("ALL");
     }
 
     public void setAll(boolean all) {
         this.modifier = "ALL";
     }
 
     public boolean isDistinct() {
-        return modifier.contains("DISTINCT");
+        return modifier != null && modifier.contains("DISTINCT");
     }
 
     public void setDistinct(boolean distinct) {
         this.modifier = "DISTINCT";
     }
 
-    private SetOperationType type;
+    private final SetOperationType type;
 
     public SetOperation(SetOperationType type) {
         this.type = type;