Details
diff --git a/payment/src/main/java/org/killbill/billing/payment/core/sm/PurchaseOperation.java b/payment/src/main/java/org/killbill/billing/payment/core/sm/PurchaseOperation.java
index 2fe96fc..8f68686 100644
--- a/payment/src/main/java/org/killbill/billing/payment/core/sm/PurchaseOperation.java
+++ b/payment/src/main/java/org/killbill/billing/payment/core/sm/PurchaseOperation.java
@@ -37,7 +37,7 @@ public class PurchaseOperation extends DirectPaymentOperation {
}
@Override
- protected PaymentTransactionInfoPlugin doPluginOperation() throws PaymentPluginApiException {
+ protected PaymentTransactionInfoPlugin doCallSpecificOperationCallback() throws PaymentPluginApiException {
logger.debug("Starting PURCHASE for payment {} ({} {})", directPaymentStateContext.getDirectPaymentId(), directPaymentStateContext.getAmount(), directPaymentStateContext.getCurrency());
return plugin.purchasePayment(directPaymentStateContext.getAccount().getId(),
directPaymentStateContext.getDirectPaymentId(),
diff --git a/payment/src/main/java/org/killbill/billing/payment/dao/PaymentAttemptSqlDao.java b/payment/src/main/java/org/killbill/billing/payment/dao/PaymentAttemptSqlDao.java
index cb406d6..1e58d1b 100644
--- a/payment/src/main/java/org/killbill/billing/payment/dao/PaymentAttemptSqlDao.java
+++ b/payment/src/main/java/org/killbill/billing/payment/dao/PaymentAttemptSqlDao.java
@@ -17,11 +17,9 @@
package org.killbill.billing.payment.dao;
import java.util.List;
-import java.util.UUID;
import org.killbill.billing.callcontext.InternalCallContext;
import org.killbill.billing.callcontext.InternalTenantContext;
-import org.killbill.billing.payment.api.PaymentAttempt;
import org.killbill.billing.util.audit.ChangeType;
import org.killbill.billing.util.entity.Entity;
import org.killbill.billing.util.entity.dao.Audited;
diff --git a/payment/src/main/java/org/killbill/billing/payment/dao/PaymentModelDao.java b/payment/src/main/java/org/killbill/billing/payment/dao/PaymentModelDao.java
index b5fda86..358dc84 100644
--- a/payment/src/main/java/org/killbill/billing/payment/dao/PaymentModelDao.java
+++ b/payment/src/main/java/org/killbill/billing/payment/dao/PaymentModelDao.java
@@ -42,8 +42,7 @@ public class PaymentModelDao extends EntityBase implements EntityModelDao<Direct
public PaymentModelDao() { /* For the DAO mapper */ }
public PaymentModelDao(final UUID id, @Nullable final DateTime createdDate, @Nullable final DateTime updatedDate, final UUID accountId,
- final UUID paymentMethodId, final Integer paymentNumber, @Nullable final String externalKey,
- @Nullable final String extFirstPaymentRefId, @Nullable final String extSecondPaymentRefId) {
+ final UUID paymentMethodId, final Integer paymentNumber, @Nullable final String externalKey) {
super(id, createdDate, updatedDate);
this.accountId = accountId;
this.paymentMethodId = paymentMethodId;
@@ -53,7 +52,7 @@ public class PaymentModelDao extends EntityBase implements EntityModelDao<Direct
public PaymentModelDao(@Nullable final DateTime createdDate, @Nullable final DateTime updatedDate, final UUID accountId,
final UUID paymentMethodId, @Nullable final String externalKey) {
- this(UUID.randomUUID(), createdDate, updatedDate, accountId, paymentMethodId, INVALID_PAYMENT_NUMBER, externalKey, null, null);
+ this(UUID.randomUUID(), createdDate, updatedDate, accountId, paymentMethodId, INVALID_PAYMENT_NUMBER, externalKey);
}
public UUID getAccountId() { return accountId; }
diff --git a/payment/src/main/resources/org/killbill/billing/payment/ddl.sql b/payment/src/main/resources/org/killbill/billing/payment/ddl.sql
index 1bcb82b..f2d7e35 100644
--- a/payment/src/main/resources/org/killbill/billing/payment/ddl.sql
+++ b/payment/src/main/resources/org/killbill/billing/payment/ddl.sql
@@ -96,7 +96,7 @@ CREATE TABLE payments (
account_id char(36) NOT NULL,
payment_method_id char(36) NOT NULL,
external_key varchar(255) NOT NULL,
- state_name varchar(64) NOT NULL,
+ state_name varchar(64) DEFAULT NULL,
created_by varchar(50) NOT NULL,
created_date datetime NOT NULL,
updated_by varchar(50) NOT NULL,
@@ -119,7 +119,7 @@ CREATE TABLE payment_history (
account_id char(36) NOT NULL,
payment_method_id char(36) NOT NULL,
external_key varchar(255) NOT NULL,
- state_name varchar(64) NOT NULL,
+ state_name varchar(64) DEFAULT NULL,
change_type char(6) NOT NULL,
created_by varchar(50) NOT NULL,
created_date datetime NOT NULL,
@@ -157,7 +157,7 @@ CREATE TABLE transactions (
PRIMARY KEY (record_id)
) /*! CHARACTER SET utf8 COLLATE utf8_bin */;
CREATE UNIQUE INDEX transactions_id ON transactions(id);
-CREATE INDEX transactions_id ON transactions(payment_id);
+CREATE INDEX transactions_payment_id ON transactions(payment_id);
CREATE INDEX transactions_key ON transactions(transaction_external_key);
CREATE INDEX transactions_tenant_account_record_id ON transactions(tenant_record_id, account_record_id);
diff --git a/payment/src/test/java/org/killbill/billing/payment/core/sm/TestRetryableDirectPayment.java b/payment/src/test/java/org/killbill/billing/payment/core/sm/TestRetryableDirectPayment.java
index 188f899..b10934d 100644
--- a/payment/src/test/java/org/killbill/billing/payment/core/sm/TestRetryableDirectPayment.java
+++ b/payment/src/test/java/org/killbill/billing/payment/core/sm/TestRetryableDirectPayment.java
@@ -39,11 +39,11 @@ import org.killbill.billing.payment.api.TransactionStatus;
import org.killbill.billing.payment.api.TransactionType;
import org.killbill.billing.payment.core.DirectPaymentProcessor;
import org.killbill.billing.payment.core.PluginControlledPaymentProcessor;
-import org.killbill.billing.payment.dao.PaymentModelDao;
-import org.killbill.billing.payment.dao.PaymentTransactionModelDao;
import org.killbill.billing.payment.dao.MockPaymentDao;
import org.killbill.billing.payment.dao.PaymentAttemptModelDao;
import org.killbill.billing.payment.dao.PaymentDao;
+import org.killbill.billing.payment.dao.PaymentModelDao;
+import org.killbill.billing.payment.dao.PaymentTransactionModelDao;
import org.killbill.billing.payment.dao.PluginPropertyModelDao;
import org.killbill.billing.payment.glue.PaymentModule;
import org.killbill.billing.payment.plugin.api.PaymentPluginApi;
diff --git a/util/src/main/java/org/killbill/billing/util/audit/DefaultAccountAuditLogs.java b/util/src/main/java/org/killbill/billing/util/audit/DefaultAccountAuditLogs.java
index 6995911..b1d74bc 100644
--- a/util/src/main/java/org/killbill/billing/util/audit/DefaultAccountAuditLogs.java
+++ b/util/src/main/java/org/killbill/billing/util/audit/DefaultAccountAuditLogs.java
@@ -92,12 +92,12 @@ public class DefaultAccountAuditLogs implements AccountAuditLogs {
@Override
public List<AuditLog> getAuditLogsForDirectPayment(final UUID directPaymentId) {
- return getAuditLogs(ObjectType.DIRECT_PAYMENT).getAuditLogs(directPaymentId);
+ return getAuditLogs(ObjectType.PAYMENT).getAuditLogs(directPaymentId);
}
@Override
public List<AuditLog> getAuditLogsForDirectPaymentTransaction(final UUID directPaymentTransactionId) {
- return getAuditLogs(ObjectType.DIRECT_TRANSACTION).getAuditLogs(directPaymentTransactionId);
+ return getAuditLogs(ObjectType.TRANSACTION).getAuditLogs(directPaymentTransactionId);
}
@Override