Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CRM-19325 - enable search for contributions that are not in any batch. #9001

Closed

Conversation

johanv
Copy link
Contributor

@johanv johanv commented Sep 8, 2016

I added 'none' to the 'contribution name' drop down on the contribution search form. This allows searching for contributions not in any batch.


@@ -568,6 +568,9 @@ public static function whereClauseSingle(&$values, &$query) {

case 'contribution_batch_id':
$batches = CRM_Contribute_PseudoConstant::batch();
// The key 'NULL' indicates that the user is looking for contributions
// that are not contained in a batch, see CRM-19325.
$batches['NULL'] = ts('(none)');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@johanv Johan why not set here $op = 'IS NULL' and $value = '' if $batches = 'NULL' ? It would be similar to L345 of this file and then i think you wouldn't need the changes in the CRM_Contact_BAO_Query i don't think

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for pointing this out. I will look at this, but probably not this week.

@monishdeb
Copy link
Member

@johanv are you working on the changes mentioned above ?

@johanv
Copy link
Contributor Author

johanv commented Oct 6, 2016

No I am not, sorry. I have little time to work on core CiviCRM for the moment.

@monishdeb
Copy link
Member

No problem, am marking it with [WIP] so you can continue later.

@monishdeb monishdeb added the needs-work-not-review-ready Submitter required to take action, will be closed after 1-2 weeks in this state label Oct 6, 2016
@monishdeb
Copy link
Member

Closing in favor of #9492

@monishdeb monishdeb closed this Dec 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master needs-work-not-review-ready Submitter required to take action, will be closed after 1-2 weeks in this state
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants