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

Improved filter : none and not empty are available on custom fields a… #1670

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

l2m83
Copy link
Contributor

@l2m83 l2m83 commented May 18, 2020

…nd user fields

This pull request aims to provide a patch for these issues. It adds a new option [not empty] to user field filters and custom filters (except for text area).

0004864: Unable to filter on empty custom field value
0008149: New filter option: [Someone] for Assigned to and other filter fields
0012798: Add a filter option [Something] as opposite of [None] i.e. field not null
(and other closed/duplicate issues that are attached)

I have tried to make the smallest possible changes to the existing code except for one function in core/filter_form_api.php which has been factorized. More improvements could have been made to this class but i didn't want to risk having unwanted behaviours.

I hope the patch will be accepted into the official code and i'm expecting your remarks if additional code improvements are needed.

@vboctor vboctor requested a review from cproensa July 21, 2020 02:34
@stevecharon
Copy link

@cproensa Can you review this or let another reviewer know about this in order to move forward?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants