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

Fix php errors with filters #525

Closed
wants to merge 1 commit into from

Conversation

vboctor
Copy link
Member

@vboctor vboctor commented Oct 25, 2014

This fixes the php errors that occur when clicking on View State, Match Type, and Highlight Changed.

The change includes:

  • When a filter is retrieved for a user, if the user doesn't have a filter or something is wrong with it, return default fitler.
  • Move access to $g_filter into the filter_api and introduce filter_init() API. Default $g_filter in filter_api as well.
  • View State initialization should use correct type to avoid type mismatch in check_selected().

Fixes #17654

This fixes the php errors that occur when clicking on View State, Match Type, and Highlight Changed.

The change includes:
- When a filter is retrieved for a user, if the user doesn't have a filter or something is wrong with it, return default fitler.
- Move access to $g_filter into the filter_api and introduce filter_init() API.  Default $g_filter in filter_api as well.
- View State initialization should use correct type to avoid type mismatch in check_selected().

Fixes #17654
@dregad
Copy link
Member

dregad commented Oct 27, 2014

Is this a replacement / alternate fix for #319 ?

@vboctor
Copy link
Member Author

vboctor commented Oct 28, 2014

@dregad yes, it is a replacement for #319.

@vboctor
Copy link
Member Author

vboctor commented Oct 31, 2014

Merged into master via cb23fb2

@vboctor vboctor closed this Oct 31, 2014
@vboctor vboctor deleted the Issue17654_filter_error branch April 25, 2015 18:10
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