key-value and json arrays: {closure}(): Argument #1 ($value) must be of type ?string, array given

Hey all,
I'm working on an admin section for my portfolio page.
The data is from a bunch of Markdown files with some frontmatter; this frontmatter has been converted and stored as json in a sqlite database. When editing the Project, I chose to display the frontmatter with a KeyValue form component. After trying to save an instance without changing the values (meaning they should be valid), I get the following error:

Filament\Forms\Components\KeyValue::Filament\Forms\Components\{closure}(): Argument #1 ($value) must be of type ?string, array given

Filament\Forms\Components\KeyValue::Filament\Forms\Components\{closure}(): Argument #1 ($value) must be of type ?string, array given

Could it be that an array value is incompatible with the KeyValue component? Or am I doing something wrong ?

Solution

Looking at it, you are storing as an array but we load that magically and it renders ok. But it should always be stored as a string in a keyValue.... you could look to cast it to a string with attributes.

If you are insistent on storing it as an Array you can use this model class for your project using the get/set attributes

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Project extends Model
{
    use HasFactory;

    protected $fillable = ["fm", "content"];
    protected $casts = [
        "fm" => "array"
    ];


    public function getFmAttribute($value)
    {
        $fm = json_decode($value, true);

        if($tags = data_get($fm, 'tags')) {
            $fm['tags'] = implode(',', $tags);
        }

        return $fm;
    }

    public function setFmAttribute($value)
    {
        if($tags = data_get($value, 'tags')) {
            $value['tags'] = explode(',', $tags);
        }
        // dd($value);
        return $value;
    }
}

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Project extends Model
{
    use HasFactory;

    protected $fillable = ["fm", "content"];
    protected $casts = [
        "fm" => "array"
    ];


    public function getFmAttribute($value)
    {
        $fm = json_decode($value, true);

        if($tags = data_get($fm, 'tags')) {
            $fm['tags'] = implode(',', $tags);
        }

        return $fm;
    }

    public function setFmAttribute($value)
    {
        if($tags = data_get($value, 'tags')) {
            $value['tags'] = explode(',', $tags);
        }
        // dd($value);
        return $value;
    }
}

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Project extends Model
{
    use HasFactory;

    protected $fillable = ["fm", "content"];
    protected $casts = [
        "fm" => "array"
    ];


    public function getFmAttribute($value)
    {
        $fm = json_decode($value, true);

        if($tags = data_get($fm, 'tags')) {
            $fm['tags'] = implode(',', $tags);
        }

        return $fm;
    }

    public function setFmAttribute($value)
    {
        if($tags = data_get($value, 'tags')) {
            $value['tags'] = explode(',', $tags);
        }
        // dd($value);
        return $value;
    }
}

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Project extends Model
{
    use HasFactory;

    protected $fillable = ["fm", "content"];
    protected $casts = [
        "fm" => "array"
    ];


    public function getFmAttribute($value)
    {
        $fm = json_decode($value, true);

        if($tags = data_get($fm, 'tags')) {
            $fm['tags'] = implode(',', $tags);
        }

        return $fm;
    }

    public function setFmAttribute($value)
    {
        if($tags = data_get($value, 'tags')) {
            $value['tags'] = explode(',', $tags);
        }
        // dd($value);
        return $value;
    }
}

Jump to solution

toeknee•8/21/24, 12:28 PM

Yes 'KeyValue' needs to be a string as it's the value

Matteo GOP•8/21/24, 12:35 PM

ah, and is there no way to convert it to and from an array on save ? I've been trying to use mutateFormDataBeforeSave, but it doesn't look to be triggering ?

MMatteo G ah, and is there no way to convert it to and from an array on save ? I've been t...

toeknee•8/21/24, 12:35 PM

Is it the key value pair field or the keyValue?

Matteo GOP•8/21/24, 12:36 PM

the value can be an array (I know which ones are arrayis in advance)

toeknee•8/21/24, 12:38 PM

So the field should be cast as an Array on the model

Matteo GOP•8/21/24, 12:38 PM

done

toeknee•8/21/24, 12:39 PM

If that's done it should be ok then...

Matteo GOP•8/21/24, 12:40 PM

an example of data is this:

{
"title": "example title"
"stack":["typescript","another tech"]
}

{
"title": "example title"
"stack":["typescript","another tech"]
}

Shows up fine, with the stack element separated by , but then the error happens when I try to save

Matteo GOP•8/21/24, 12:52 PM

so could it be that it gets interpreted as an array when it gets sent back to be saved ? maybe because of the cast ?

toeknee•8/21/24, 1:00 PM

It will be failing at "stack":["typescript","another tech"] surely? because stack is an array

Matteo GOP•8/21/24, 1:18 PM

yes, but only on save I think (it displays fine). Is there a way to handle saving this field manually ? I thought maybe mutateFormDataBeforeSaving would work, but it does not even trigger, it seems

toeknee•8/21/24, 1:20 PM

Can you provide the field code

Matteo GOP•8/21/24, 1:21 PM

Forms\Components\KeyValue::make('fm'),

Forms\Components\KeyValue::make('fm'),

Matteo GOP•8/24/24, 3:02 PM

upping this since I'm still trying to make this work; I tried using mutateFormDataBeforeSave and handleRecordUpdate, but neither gets called before the error is thrown

toeknee•8/25/24, 9:39 AM

Can you provide an example repo and we can test it

Matteo GOP•8/25/24, 1:14 PM

https://github.com/matfire/filament-kv-repro
here it is

GitHub

GitHub - matfire/filament-kv-repro

Contribute to matfire/filament-kv-repro development by creating an account on GitHub.

MMatteo G https://github.com/matfire/filament-kv-repro here it is

toeknee•8/27/24, 9:08 AM

Thanks, why are you saving it as an array to the DB?

Solution

toeknee•8/27/24, 9:34 AM

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Project extends Model
{
    use HasFactory;

    protected $fillable = ["fm", "content"];
    protected $casts = [
        "fm" => "array"
    ];


    public function getFmAttribute($value)
    {
        $fm = json_decode($value, true);

        if($tags = data_get($fm, 'tags')) {
            $fm['tags'] = implode(',', $tags);
        }

        return $fm;
    }

    public function setFmAttribute($value)
    {
        if($tags = data_get($value, 'tags')) {
            $value['tags'] = explode(',', $tags);
        }
        // dd($value);
        return $value;
    }
}

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Project extends Model
{
    use HasFactory;

    protected $fillable = ["fm", "content"];
    protected $casts = [
        "fm" => "array"
    ];


    public function getFmAttribute($value)
    {
        $fm = json_decode($value, true);

        if($tags = data_get($fm, 'tags')) {
            $fm['tags'] = implode(',', $tags);
        }

        return $fm;
    }

    public function setFmAttribute($value)
    {
        if($tags = data_get($value, 'tags')) {
            $value['tags'] = explode(',', $tags);
        }
        // dd($value);
        return $value;
    }
}

toeknee•8/27/24, 9:35 AM

It will work for tags only

Matteo GOP•8/27/24, 10:01 AM

hey, thank you so much! Works like a charm on the filament admin!
but now it doesn't load properly on the rest of the website because it gets converted to a string (so looping over it does not work anymore :(). But I could change that section to explode the string it receives, though it seems a bit weird and inefficient (imploding to explode it right after...) ?
They are stored as an array because of the way Eloquent does casts automatically, so i don't know how hard it would be to customize that process...

toeknee•8/27/24, 10:03 AM

Right I think I get you, why not just store it as strings and then explode on the frontend?

toeknee•8/27/24, 10:04 AM

It is a string by default, the only difference seems that you have some values stored as an array some how for tags

Matteo GOP•8/27/24, 10:06 AM

I was also thinking of that, but I don't know how hard it would be to change how Laravel handles converting to an array. This is the import script I'm using on the actual website.

$fmParser = new FrontMatterParser(new SymfonyYamlFrontMatterParser);
$files = Storage::files('markdown/projects');
        foreach ($files as $file) {
            $parsed = $fmParser->parse(Storage::get($file));
            $project = Project::query()->updateOrCreate(['slug' => MarkdownUtils::getSlug($file)], [
                'path' => $file,
                'published_at' => $parsed->getFrontMatter()['date'],
                'content' => $parsed->getContent(),
                'fm' => $parsed->getFrontMatter(),
            ]);
            $project->save();
        }

$fmParser = new FrontMatterParser(new SymfonyYamlFrontMatterParser);
$files = Storage::files('markdown/projects');
        foreach ($files as $file) {
            $parsed = $fmParser->parse(Storage::get($file));
            $project = Project::query()->updateOrCreate(['slug' => MarkdownUtils::getSlug($file)], [
                'path' => $file,
                'published_at' => $parsed->getFrontMatter()['date'],
                'content' => $parsed->getContent(),
                'fm' => $parsed->getFrontMatter(),
            ]);
            $project->save();
        }

getFrontMatter

getFrontMatter

returns an array representing the frontmatter on a markdown file. It is stored in the database as a JSON column

toeknee•8/27/24, 10:26 AM

Dude remove my getter/setter and just change:

'fm' => $parsed->getFrontMatter(),

'fm' => $parsed->getFrontMatter(),

'fm' => implode(',', $parsed->getFrontMatter()),

'fm' => implode(',', $parsed->getFrontMatter()),

and re-run the import and everything will be fine.

Matteo GOP•8/27/24, 12:14 PM

just tried it, getting:

ErrorException Array to string conversion

ErrorException Array to string conversion

on the changed line

MMatteo G just tried it, getting: ```php ErrorException Array to string conversion ``` on ...

toeknee•8/27/24, 12:38 PM

ahh sorry!!! can you show the getFrontMatter() function

Matteo GOP•8/27/24, 12:38 PM

I didn't make it, it comes from the commonmark packages:
League\CommonMark\Extension\FrontMatter\Data\SymfonyYamlFrontMatterParser
League\CommonMark\Extension\FrontMatter\FrontMatterParser
I'll see if I can find some readable source code

toeknee•8/27/24, 12:40 PM

Ok so it's ok you can do this instead

toeknee•8/27/24, 12:42 PM

$fmParser = new FrontMatterParser(new SymfonyYamlFrontMatterParser);
$files = Storage::files('markdown/projects');
        foreach ($files as $file) {
            $parsed = $fmParser->parse(Storage::get($file));
            $fm = $parsed->getFrontMatter();
            foreach($fm as $key => $value) {
              $fm[$key] = $key === 'tags' ? implode(',', $value) : $value;
            }  

            $project = Project::query()->updateOrCreate(['slug' => MarkdownUtils::getSlug($file)], [
                'path' => $file,
                'published_at' => $parsed->getFrontMatter()['date'],
                'content' => $parsed->getContent(),
                'fm' => $fm,
            ]);
            $project->save();
        }

$fmParser = new FrontMatterParser(new SymfonyYamlFrontMatterParser);
$files = Storage::files('markdown/projects');
        foreach ($files as $file) {
            $parsed = $fmParser->parse(Storage::get($file));
            $fm = $parsed->getFrontMatter();
            foreach($fm as $key => $value) {
              $fm[$key] = $key === 'tags' ? implode(',', $value) : $value;
            }  

            $project = Project::query()->updateOrCreate(['slug' => MarkdownUtils::getSlug($file)], [
                'path' => $file,
                'published_at' => $parsed->getFrontMatter()['date'],
                'content' => $parsed->getContent(),
                'fm' => $fm,
            ]);
            $project->save();
        }

toeknee•8/27/24, 12:42 PM

something like that should work

Matteo GOP•8/27/24, 1:15 PM

the import worked! thanks!
just generalized the code a bit to convert all arrays into a comma-separated string to ease the transition of all other models.

Matteo GOP•8/27/24, 1:15 PM

            foreach($fm as $key => $value) {
                $fm[$key] = is_array($value) ? implode(',', $value) : $value;
            }

            foreach($fm as $key => $value) {
                $fm[$key] = is_array($value) ? implode(',', $value) : $value;
            }

key-value and json arrays: {closure}(): Argument #1 ($value) must be of type ?string, array given

Similar Threads

Similar Threads

Similar Threads