correct syntax "IS NOT NULL" to get pledge payment count

[sunilpawar]

Overview

Get Count for Pledge Payment.
Once Pledge have payment (status is In-Progress), we can not update the pledge amount and number of instalment.
but if Pledge have payment and some payment status is overdue then Pledge status change to 'overdue' and this allow to update pledge amount and number of instalment.

Before

API return 0 count

After

API return actual payment count. Edit screen screen get freezed.

Technical Details

correct syntax for is not null

[eileenmcnaughton]

Change looks correct - should be easy to write a test for

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[prondubuisi]

Hello @sunilpawar @eileenmcnaughton can I please get a little background story on this issue?
what does the PR fix?

[JoeMurray]

To test, create a pledge. Do a get count on contributions against the pledge. Apparently it should fail prior to this PR and succeed after this PR.

@sunilpawar I don't see relevance of much of the description to the code change. Are those areas where the api bug manifests in the UI?

[seamuslee001]

I'm going to merge this as its correct and we have a test in #14350

[eileenmcnaughton]

Thanks @seamuslee001

@sunilpawar can you add a test next time please