Merge pull request #2566 from alphagov/PP-7226-codeql-changes-to-conn…

…ector

PP-7226 CodeQL Changes

src/main/java/uk/gov/pay/connector/charge/model/domain/Charge.java

@@ -159,6 +159,13 @@ public boolean equals(Object obj) {
                 Objects.equals(email, charge.email);
     }
 
+    @Override
+    public int hashCode() {
+        return Objects.hash(externalId, amount, status, externalStatus, gatewayTransactionId, corporateSurcharge,
+                refundAvailabilityStatus, reference, description, createdDate, email, gatewayAccountId,
+                paymentGatewayName, historic);
+    }
+
     public String getExternalStatus() {
         return externalStatus;
     }

src/main/java/uk/gov/pay/connector/gateway/stripe/json/StripePayoutStatus.java

@@ -36,7 +36,7 @@ public static StripePayoutStatus fromString(String status) {
                 return stat;
             }
         }
-        throw new IllegalArgumentException(format("Stripe payout status not recognized: [{}]", status));
+        throw new IllegalArgumentException(format("Stripe payout status not recognized: [%s]", status));
     }
 
     public Optional<Class<? extends PayoutEvent>> getEventClass() {

src/main/java/uk/gov/pay/connector/refund/model/domain/Refund.java

@@ -107,4 +107,10 @@ public boolean equals(Object obj) {
                 Objects.equals(chargeExternalId, refund.chargeExternalId) &&
                 Objects.equals(gatewayTransactionId, refund.gatewayTransactionId);
     }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(externalId, amount, userExternalId, userEmail, gatewayTransactionId, chargeExternalId,
+                externalStatus, historic);
+    }
 }