Merge pull request #634 from sircpl/sircpl/revert_triggers_cbp

Revert changes to getTriggersWithParameters due to CBP bug in API

Author: PierreBtz

src/main/java/org/zendesk/client/v2/Zendesk.java

@@ -954,12 +954,11 @@ public Iterable<Trigger> getTriggers() {
   public Iterable<Trigger> getTriggers(
       String categoryId, boolean active, String sortBy, SortOrder sortOrder) {
     return new PagedIterable<>(
-        tmpl("/triggers.json{?category_id,active,sort_by,sort_order}&page[size]={pageSize}")
+        tmpl("/triggers.json{?category_id,active,sort_by,sort_order}")
             .set("category_id", categoryId)
             .set("active", active)
             .set("sort_by", sortBy)
-            .set("sort_order", sortOrder.getQueryParameter())
-            .set("pageSize", cbpPageSize),
+            .set("sort_order", sortOrder.getQueryParameter()),
         handleList(Trigger.class, "triggers"));
   }
 

src/test/java/org/zendesk/client/v2/RealSmokeTest.java

@@ -315,9 +315,9 @@ public void getTriggersWithParameters() throws Exception {
     createClientWithTokenOrPassword(2);
     int count = 0;
     String title = null;
-    for (Trigger t : instance.getTriggers(null, true, "title", SortOrder.ASCENDING)) {
+    for (Trigger t : instance.getTriggers(null, true, "alphabetical", SortOrder.ASCENDING)) {
       if (title != null) {
-        assertTrue(title.compareTo(t.getTitle()) < 0);
+        assertTrue(title.compareToIgnoreCase(t.getTitle()) < 0);
       }
       title = t.getTitle();
       if (++count > 10) {