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

Empty Bindings and Backtrace #1684

Closed
rolland97 opened this issue Oct 5, 2024 · 8 comments · Fixed by #1686
Closed

Empty Bindings and Backtrace #1684

rolland97 opened this issue Oct 5, 2024 · 8 comments · Fixed by #1686

Comments

@rolland97
Copy link

rolland97 commented Oct 5, 2024

image

Environment:

  • Windows 11
  • php 8.2.23
  • Laravel 10.48.22
  • Filament 3.2.115
@rolland97
Copy link
Author

Laravel Debugbar v3.14.2 functions correctly, but the latest version (v3.14.3) is not working as expected.

@Johnefsh
Copy link

Johnefsh commented Oct 8, 2024

Same here any help please.

@tpetry
Copy link
Contributor

tpetry commented Oct 9, 2024

Laravel Debugbar v3.14.2 functions correctly, but the latest version (v3.14.3) is not working as expected.

Thanks for finding the exact version that broke this.

@mccahan This seems to be because of your values.values() change. I don't understand why it should be a problem, but its the only change. Are there any other possible implementations that would fix the edge case you had by extending the Array prototype?

@Johnefsh
Copy link

Johnefsh commented Oct 9, 2024

For me noting changed I just upgraded to version v3.14.3, should I return to v3.14.2 for now?

@mccahan
Copy link
Contributor

mccahan commented Oct 9, 2024

@tpetry Whoops, this does in fact seem to be mine. I've got a PR in #1686

@Johnefsh
Copy link

I have uninstalled the laravel-debugbar and install it and I cleared my config but still having the same exact problem.

@barryvdh
Copy link
Owner

Can you try with v3.14.4 ?

@Johnefsh
Copy link

Can you try with v3.14.4 ?

@barryvdh Thanks, this solved the issue.

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

Successfully merging a pull request may close this issue.

5 participants