Adding a query param class for each operation instead of sharing a single class for all operations

.github/workflows/pr-review.yml

@@ -22,6 +22,6 @@ jobs:
       - name: Check Labels
         uses: docker://agilepathway/pull-request-label-checker:latest
         with:
-          one_of: V5
+          one_of: V6
           repo_token: ${{ secrets.GITHUB_TOKEN }}
 

openapi/api.yaml

@@ -511,7 +511,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListSitesParams params = new ListSitesParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Site> sites = client.listSites(params);
 
@@ -733,7 +733,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountsParams params = new ListAccountsParams();
           params.setLimit(200); // Pull 200 records at a time
           Pager<Account> accounts = client.listAccounts(params);
 
@@ -2956,7 +2956,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountCouponRedemptionsParams params = new ListAccountCouponRedemptionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<CouponRedemption> redemptions = client.listAccountCouponRedemptions(accountId, params);
 
@@ -3455,7 +3455,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountCreditPaymentsParams params = new ListAccountCreditPaymentsParams();
           params.setLimit(200); // Pull 200 records at a time
           Pager<CreditPayment> payments = client.listAccountCreditPayments(accountId, params);
 
@@ -3797,7 +3797,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountInvoicesParams params = new ListAccountInvoicesParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Invoice> invoices = client.listAccountInvoices(accountId, params);
 
@@ -4241,7 +4241,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountLineItemsParams params = new ListAccountLineItemsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<LineItem> lineItems = client.listAccountLineItems(accountId, params);
 
@@ -4500,7 +4500,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountNotesParams params = new ListAccountNotesParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<AccountNote> notes = client.listAccountNotes(accountId, params);
 
@@ -4785,7 +4785,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListShippingAddressesParams params = new ListShippingAddressesParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<ShippingAddress> addresses = client.listShippingAddresses(accountId, params);
 
@@ -5474,7 +5474,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountSubscriptionsParams params = new ListAccountSubscriptionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Subscription> subscriptions = client.listAccountSubscriptions(accountId, params);
 
@@ -5580,7 +5580,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountTransactionsParams params = new ListAccountTransactionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Transaction> transactions = client.listAccountTransactions(accountId, params);
 
@@ -5763,7 +5763,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAccountAcquisitionParams params = new ListAccountAcquisitionParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<AccountAcquisition> acquisitions = client.listAccountAcquisition(params);
 
@@ -5866,7 +5866,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListCouponsParams params = new ListCouponsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Coupon> coupons = client.listCoupons(params);
 
@@ -6665,7 +6665,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListCreditPaymentsParams params = new ListCreditPaymentsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<CreditPayment> payments = client.listCreditPayments(params);
 
@@ -6801,7 +6801,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListCustomFieldDefinitionsParams params = new ListCustomFieldDefinitionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<CustomFieldDefinition> fields = client.listCustomFieldDefinitions(params);
 
@@ -7211,7 +7211,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListItemsParams params = new ListItemsParams();
           params.setLimit(200); // Pull 200 records at a time
           Pager<Item> items = client.listItems(params);
 
@@ -8718,7 +8718,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListInvoicesParams params = new ListInvoicesParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Invoice> invoices = client.listInvoices(params);
 
@@ -10037,7 +10037,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListInvoiceLineItemsParams params = new ListInvoiceLineItemsParams();
           params.setLimit(200);
           Pager<LineItem> lineItems = client.listInvoiceLineItems(invoiceId, params);
 
@@ -10131,7 +10131,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListInvoiceCouponRedemptionsParams params = new ListInvoiceCouponRedemptionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<CouponRedemption> redemptions = client.listInvoiceCouponRedemptions(invoiceId, params);
 
@@ -10499,7 +10499,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListLineItemsParams params = new ListLineItemsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<LineItem> lineItems = client.listLineItems(params);
 
@@ -10836,7 +10836,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListPlansParams params = new ListPlansParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Plan> plans = client.listPlans(params);
 
@@ -11505,7 +11505,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListPlanAddOnsParams params = new ListPlanAddOnsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<AddOn> addOns = client.listPlanAddOns(planId, params);
 
@@ -12235,7 +12235,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListAddOnsParams params = new ListAddOnsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<AddOn> addOns = client.listAddOns(params);
 
@@ -12452,7 +12452,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListShippingMethodsParams params = new ListShippingMethodsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<ShippingMethod> shippingMethods = client.listShippingMethods(params);
 
@@ -12701,7 +12701,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListSubscriptionsParams params = new ListSubscriptionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Subscription> subscriptions = client.listSubscriptions(params);
 
@@ -13294,7 +13294,7 @@ paths:
       - lang: Java
         source: |
           try {
-              QueryParams queryParams = new QueryParams();
+              TerminateSubscriptionParams queryParams = new TerminateSubscriptionParams();
               queryParams.setRefund(Constants.RefundType.NONE); // "full" for a full refund, "partial" for prorated refund
               client.terminateSubscription(subscriptionId, queryParams);
               System.out.println("Terminated Subscription: " + subscriptionId);
@@ -14582,7 +14582,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListSubscriptionInvoicesParams params = new ListSubscriptionInvoicesParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Invoice> invoices = client.listSubscriptionInvoices(subscriptionId, params);
 
@@ -14688,7 +14688,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListSubscriptionLineItemsParams params = new ListSubscriptionLineItemsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<LineItem> lineItems = client.listSubscriptionLineItems(subscriptionId, params);
 
@@ -14786,7 +14786,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListSubscriptionCouponRedemptionsParams params = new ListSubscriptionCouponRedemptionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<CouponRedemption> redemptions = client.listSubscriptionCouponRedemptions(subscriptionId, params);
 
@@ -15157,7 +15157,7 @@ paths:
           end
       - lang: Java
         source: |
-          QueryParams params = new QueryParams();
+          ListTransactionsParams params = new ListTransactionsParams();
           params.setLimit(200); // Pull 200 records at a time
           final Pager<Transaction> transactions = client.listTransactions(params);
 

src/main/java/com/recurly/v3/AbstractQueryParams.java

@@ -0,0 +1,22 @@
+package com.recurly.v3;
+
+import java.util.HashMap;
+import java.util.List;
+
+public abstract class AbstractQueryParams
+ {
+
+  private HashMap<String, Object> params;
+
+  public AbstractQueryParams() {
+    this.params = new HashMap<String, Object>();
+  }
+
+  public HashMap<String, Object> getParams() {
+    return this.params;
+  }
+
+  protected void add(final String key, final Object value) {
+    this.params.put(key, value);
+  }
+}