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

Query Control: filter by Author caused php warning. #9864

Closed
2 tasks done
steve231293 opened this issue Dec 13, 2019 · 1 comment
Closed
2 tasks done

Query Control: filter by Author caused php warning. #9864

steve231293 opened this issue Dec 13, 2019 · 1 comment
Labels
status/merged Indicates when a Pull Request has been merged to a Release.
Milestone

Comments

@steve231293
Copy link

Prerequisites

  • I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
  • The issue still exists against the latest stable version of Elementor.

Description

Hello great team,

I've just found 2 issues with Query Control of Posts widget.

download (1)

Whenever i filter posts by author then debug.log will show php warning:

PHP Notice: Undefined index: display in /Volumes/DATA/steve/test/wp-content/plugins/elementor-pro/modules/query-control/module.php on line 290

The other issue, I can't search author by Nick Name.
It will be better when user can search by Nick Name instead of Username only, right?

Could you check & improve it? please

Environment

Elementor Pro 2.7.3
Hello Elementor theme

@elementorbot elementorbot added the status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. label Dec 16, 2019
@elementorbot elementorbot added this to the Pro 2.8.0 milestone Dec 16, 2019
@shilo-ey shilo-ey added status/merged Indicates when a Pull Request has been merged to a Release. and removed status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. labels Dec 17, 2019
@shilo-ey
Copy link
Contributor

This issue has been resolved in Elementor Pro v2.8.1

Feel free to update

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/merged Indicates when a Pull Request has been merged to a Release.
Projects
None yet
Development

No branches or pull requests

3 participants