Project

General

Profile

Bug #7432

Java Client does not support idFilter for v2 release

Added by Robert Waltz over 8 years ago. Updated almost 5 years ago.

Status:
In Progress
Priority:
Normal
Assignee:
Category:
d1_libclient_java
Start date:
2015-10-15
Due date:
% Done:

30%

Story Points:
Sprint:

Description

d1-libclient-java in V1 implemented 'pidFilter' in the getLogRecords method. However, in v2, 'pidFilter' has been renamed 'idFilter' for both v1 and v2 log endpoints.

Associated revisions

Revision 19248
Added by Rob Nahf almost 6 years ago

refs #7432. Reverted v1.getLogRecords parameter name to 'pidFilter' to align with documentation (and python implementation).

Revision 19248
Added by Rob Nahf almost 6 years ago

refs #7432. Reverted v1.getLogRecords parameter name to 'pidFilter' to align with documentation (and python implementation).

History

#1 Updated by Andrei Buium over 8 years ago

  • % Done changed from 0 to 30
  • Status changed from New to In Progress

#2 Updated by Andrei Buium over 8 years ago

  • Status changed from In Progress to Testing
  • % Done changed from 30 to 50

Committed change to libclient

#3 Updated by Andrei Buium over 8 years ago

  • % Done changed from 50 to 100
  • Status changed from Testing to Closed

URLs created through libclient tested and look as expected

#4 Updated by Rob Nahf almost 6 years ago

  • % Done changed from 100 to 30
  • Status changed from Closed to In Progress
  • Target version changed from CCI-2.0.0 to CCI-2.3.10

the fix mistakenly changed the parameter name for v1 as well as v2. (That's to say that the ticket shouldn't have asked to change the parameter name in the v1 implementation.)

The new fix that reverts the parameter to 'pidFilter' for v1 implementations should be applied to trunk, the v2.3 branch, and maybe the other v2.x branches as well.

#5 Updated by Rob Nahf almost 5 years ago

  • Assignee changed from Andrei Buium to Rob Nahf

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 14.8 MB)