8000 Bun on activating properties column · Issue #50 · rmsramos/activitylog · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Bun on activating properties column #50

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.

Sign up for GitHub

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

Jump to bottom
Open
quasiperfect opened this issue Sep 21, 2024 · 3 comments
Open

Bun on activating properties column #50

quasiperfect opened this issue Sep 21, 2024 · 3 comments
Labels
bug Something isn't working unconfirmed

Comments

@quasiperfect
Copy link

Package Version

v1.0.0

Laravel Version

Laravel 11.23.5

PHP Version

PHP 8.2.20

Problem description

i have activated the properties column in the listing view and i got an exception foreach() argument must be of type array|object, null given

Expected behavior

to see the list of events

Steps to reproduce

install the plugin, enable events on a model, go to the listing page and enable the properties column

this is the content of the logs table
image

Reproduction repository (issue will be closed if this is not valid)

https://github.com/

Relevant log output

No response
@quasiperfect quasiperfect added bug Something isn't working unconfirmed labels Sep 21, 2024
@WillianDamasceno
Copy link

I'm facing the same problem.

This is not a solution, just an observation:

If you open the file vendor\rmsramos\activitylog\resources\views\filament\tables\columns\activity-logs-properties.blade.php you will find a @foreach($getState() as $key => $value), if you change it to @foreach($getState() ?? [] as $key => $value).

Maybe you could publish the view somehow and apply the change, but it would be way better if the package author fixed it.

@RyanPaiva56
Copy link

I'm able to reproduce this on the latest version. The error output is:

foreach() argument must be of type array|object, null given

@RyanPaiva56
Copy link

This is resolved in the latest version. I recommend this issue be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working unconfirmed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@quasiperfect @RyanPaiva56 @WillianDamasceno
0