Any way to modify charts to only return a subset of data?

I'm attempting to create a user activity chart and presently have users organized into smaller sub-sets. I wondered if there was a way to only show chart results for users within the same user's sub-set list? For example

... // the widget
[$users, $months] = $this->getPerMonth();
... // rest of widget


private function getPerMonth(): array
    {
        $activity = UserActivity::get()->where('organizations_id', auth()->user()->organizations_id); // not in use currently
        $data = Trend::model(UserActivity::class)
            ->between(
                start: now()->startOfYear(),
                end: now()->endOfYear(),
            )
            ->perMonth()
            ->count();

        return [
            $data->map(fn (TrendValue $value) => $value->aggregate),
            $data->map(fn (TrendValue $value) => now()->rawParse($value->date)->format('M')),
        ];
    }

... // the widget
[$users, $months] = $this->getPerMonth();
... // rest of widget


private function getPerMonth(): array
    {
        $activity = UserActivity::get()->where('organizations_id', auth()->user()->organizations_id); // not in use currently
        $data = Trend::model(UserActivity::class)
            ->between(
                start: now()->startOfYear(),
                end: now()->endOfYear(),
            )
            ->perMonth()
            ->count();

        return [
            $data->map(fn (TrendValue $value) => $value->aggregate),
            $data->map(fn (TrendValue $value) => now()->rawParse($value->date)->format('M')),
        ];
    }

I'm wondering if there is a way to use the

$activity

$activity

variable somehow instead of the class? Or can I filter it after the values are returned from getPerMonth?
Thanks in advance!

Solution

You just used the get() in the wrong order.

Jump to solution

Steve_OHOP•5/23/24, 7:26 PM

Bump

awcodes•5/23/24, 9:06 PM

You should be able to use instead of the model class.

Steve_OHOP•5/24/24, 1:54 AM

Thank you for responding! Attempted a few different variations with Trend::query(), but seem to get one of two errors.

For the first query, which attempts to place $activity into the query

The error is (ignore the line number, I have each attempt commented/uncommented on different lines):

and when my query is:

The error returns as:

Dennis Koch•5/24/24, 6:17 AM

It doesn’t work because it needs a Builder instance (before you call .

Dennis Koch•5/24/24, 6:18 AM

BTW you code loads all activities and filters them in PHP instead of on a DB level.

DDennis Koch BTW you code loads *all* activities and filters them in PHP instead of on a DB l...

Steve_OHOP•5/24/24, 1:54 PM

I wanted to understand what was possible programmatically

DDennis Koch BTW you code loads *all* activities and filters them in PHP instead of on a DB l...

Steve_OHOP•5/24/24, 6:55 PM

If you were to simplify it on a DB level, how would you? Using a DB:: call?

SSteve_OH If you were to simplify it on a DB level, how would you? Using a DB:: call?

Dennis Koch•5/24/24, 6:58 PM

What are you trying to simplify? Can't you just apply a where condition?

Steve_OHOP•5/24/24, 7:08 PM

You will have to forgive me, I only recently picked up and fell in love with laravel/filament so my understanding comes from my years of PHP knowhow and from other languages I know. I guess the issue is that I'm trying to learn the type of data that should be returned in order to structure the query correctly.

What I'm trying to do is to filter the Trend::model(UserActivity::class) to only output when a certain field in that class matches, eg: where('organizations_id', auth()->user()->organizations_id). The where condition seems to return the errors as shown above, whether called directly or as a variable passed in. Is it possible to utilize something like Trend(DB::table('user_activity')->where('value', $value)->get())?

awcodes•5/24/24, 8:12 PM

Solution

awcodes•5/24/24, 8:12 PM

You just used the get() in the wrong order.

Aawcodes ```php Trend::query(UserActivity::where(‘organizations_id’, auth()->user()->orga...

Dennis Koch•5/24/24, 8:48 PM

I think you shouldn't have a get at all. It expects a Builder

DDennis Koch I think you shouldn't have a get at all. It expects a Builder

awcodes•5/24/24, 8:54 PM

Possibly, I’m just quoting my understanding from the packages readme.

awcodes•5/24/24, 8:55 PM

But yea with a Builder the get() is implied.

awcodes•5/24/24, 8:55 PM

So, you’re right.

Steve_OHOP•5/29/24, 8:13 PM

Solved, thank you very much!