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

"Invalid supplied argument" warning for hidden repeater fields #3886

Closed
tudormnt opened this issue Mar 25, 2018 · 3 comments
Closed

"Invalid supplied argument" warning for hidden repeater fields #3886

tudormnt opened this issue Mar 25, 2018 · 3 comments

Comments

@tudormnt
Copy link

Hi,

Please consider adding an additional check before this loop:

https://github.com/pojome/elementor/blob/59850ebed85c2e1c57cb6f914576677851e7c67d/includes/base/controls-stack.php#L1055

to make sure that $settings[ $control_name ] is not null. This is the case when the repeater field is displayed conditionally in the editor panel. Currently it fires off an "invalid argument supplied" warning in such cases.

Thanks!

@bainternet
Copy link
Contributor

@tudormnt

Can you provide steps to replicate?

@bainternet bainternet added the status/needs-feedback Indicates when an Issue, Pull Request, or Discussion awaits Feedback from the poster. label Mar 26, 2018
@tudormnt
Copy link
Author

@bainternet none of the Elementor widgets have a repeater field under a condition, so this isn't a bug that appears for Elementor or Elementor Pro. In one of our Extras plugin widgets we show a repeater field conditionally. If for the same widget we activate a dynamic tag for a control, the above error appears. I don't honestly know what more I can say to replicate this :)

@KingYes
Copy link
Member

KingYes commented Mar 27, 2018

Fixed by this PR: #3916

@KingYes KingYes closed this as completed Mar 27, 2018
@KingYes KingYes removed the status/needs-feedback Indicates when an Issue, Pull Request, or Discussion awaits Feedback from the poster. label Mar 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants