Programatically add file/image to page/field

plagasul · March 18, 2022, 5:23pm

I am creating pages from CSV files. I also have a folder with images that are referenced by filename in the CSVs. Both under ‘/assets/’

So looping the CSV I can get the file with something like this in a loop:

asset('assets/mediaimport/'.$mainImage)

Now I’d like to bring one or several of those into the page on creation time, or aftewards with update. Specifically into two files fields, one multiple, one single.

Can I get an example or a page from the docs etc, please?

Thank you

texnixe · March 18, 2022, 6:13pm

You cannot link a file in the assets folder to a files field, for that you would need a file object. A file object, however, is tied to a page. So you could either put those files into the site or a specific mediafolder page, or create a page folder for the virtual page where those files could live (like in this example: Merging content sources | Kirby CMS).

plagasul · March 18, 2022, 7:01pm

Thanks @texnixe

Sorry, my question was actually about ‘uploading’ the file to the page folder programatically and then use it in the files field.

I found the “Uploading files from frontend” cookbook, and was trying to adapt part of the code, something like this:

    $img = asset('assets/mediaimport/'.$mainImage);
    try {               
        $file = page('myPage')->createFile([
            'source'   => $img,
            'filename' => $img->filename(),
        ]);

        page('myPage')->update([
            'filesfield' => Data::encode([$img->filename()], 'yaml')
        ])

    } catch (Exception $e) {
        $e->getMessage();
    }

Should this work? edit: it does not

Can it be adapted to run on page creation? This is not critical, tho.

Thank you

texnixe · March 18, 2022, 8:19pm

Sorry for the misunderstanding, I thought you were using virtual pages, not actually creating them from a csv file.

As source, you need the path to the file instead of an asset object:

$img = 'assets/mediaimport/' . $mainImage;

//...
 $file = page('myPage')->createFile([
            'source'   => $img,
            'filename' => $mainImage,
        ]);

plagasul · March 18, 2022, 8:22pm

Thank you,

Hmm… but then , when I update the files field, should I also use merely the filename? so $mainImage?

Thanks

texnixe · March 18, 2022, 8:24pm

Oh, yes, sorry, left that part out.

plagasul · March 18, 2022, 8:34pm

Thanks again,

I can’t make it work. I must be doing something wrong. Maybe related to this being a multilang website?

This is the code I am using:

$p = page('fairs')->findPageOrDraft(str::slug($title . formatDate($f['Date Start'], 'd/m/Y', 'Y')));
echo $p->slug() . 'exists' . '<br />';
$img = 'assets/mediaimport/'.$mainImage;
try {               
    $file = $p->createFile([
        'source'   => $img,
        'filename' => $mainImage,
    ]);

    $p->update([
        'cover' => Data::encode([$mainImage], 'yaml')
    ]);

} catch (Exception $e) {
    $e->getMessage();
}

texnixe · March 18, 2022, 8:38pm

Do you get an error? You are authenticating, aren’t you ($kirby->impersonate())?

plagasul · March 18, 2022, 8:41pm

I don’t seem to get an error.

Yes, I am impersonating as kirby. Here is more complete code (not all of it)

// Fairs
foreach ($fairs as $f) {
    $title = $f['Title'];
    $artists = $f['Artists'];
    $otherArtists = $f['Other Artists'];
    $mainImage = $f['Main Image'];
    $location = $f['Location'];
    $dateStart = formatDate($f['Date Start']);
    $dateEnd = formatDate($f['Date End']);
    $installationViews = $f['Installation views'];
    $logo = $f['Logo'];
    
    if (!page('fairs')->findPageOrDraft(str::slug($title . formatDate($f['Date Start'], 'd/m/Y', 'Y')))) {
   
        page('fairs')->createChild([
            'slug' => str::slug($title . formatDate($f['Date Start'], 'd/m/Y', 'Y')),
            'template' => 'fair',
            'content' => [
                'title' => $title,
                'artists' => parseArtists($artists, $otherArtists),
                'venue' => $location,
                'startdate' => $dateStart,
                'enddate' => $dateEnd
            ]                    
        ]);

    } else {
        $p = page('fairs')->findPageOrDraft(str::slug($title . formatDate($f['Date Start'], 'd/m/Y', 'Y')));
        echo $p->slug() . ' already exists' . '<br />';
        $img = 'assets/mediaimport/'.$mainImage;
        try {               
            $file = $p->createFile([
                'source'   => $img,
                'filename' => $mainImage,
            ]);
   
            $p->update([
                'cover' => Data::encode([$mainImage], 'yaml')
            ]);

        } catch (Exception $e) {
            $e->getMessage();
        }
    }
    
}

…is $img a correct accessible path here ?

texnixe · March 18, 2022, 8:47pm

This code works for me. For the test, I put the file attention-sharks.jpg into assets/images (testing this in the about.php template in a starterkit):

<?php
$kirby->impersonate('kirby');
$path = 'assets/images/attention-sharks.jpg';
try {
    $file = $page->createFile([
        'source'   => $path,
        'filename' => 'attention-sharks.jpg',
        'template' => 'image',
    ]);

    $page->update([
        'cover' => Data::encode([$file->filename()], 'yaml')
    ]);

} catch (Exception $e) {
    $e->getMessage();
}

plagasul · March 18, 2022, 8:59pm

Yes, ok

I AM getting an exception, I dumped $e, this:

…folders seem writable, permissions look good, pages were created using this method and file, and I can update them via the panel

texnixe · March 18, 2022, 9:10pm

The error is thrown when either the source does not exist or the target file already exists or if the file cannot be copied to the target location (probably due to permissions).

Are you familiar with xDebug? Then I would do a step debug to see where exactly it fails

plagasul · March 18, 2022, 9:14pm

Content folder and subfolders are -rw-rw-r-- and belong to www-data and/or have this as group, which is the usual in my setup.

The images are certainly not in the folders.

Could this be related to the fact that all this pages are still _drafts ? edit> no, does not seem so…

Thanks

texnixe · March 18, 2022, 9:26pm

Could you set a file template and create that with an accept function that allows certain file types.

Are the files particularly large?

plagasul · March 18, 2022, 9:42pm

Ok, it is solved. The path to files was not correct, small typo.

I am confused as the asset() function didn’t seem to have a problem with the provided path, which threw me off track.

Probably human error in any case.

Thank you again

Topic		Replies	Views
Can't immediately reference a file that's been programmatically uploaded Solved ✅	4	269	August 6, 2022
Programatically upload files with template, and update `files` field with these files Solved ✅	3	325	March 22, 2023
Updating files field programmatically Solved ✅	4	930	March 11, 2019
File::create problem Questions 🤔	22	1218	June 24, 2019
Add file to a files collection Solved ✅	5	262	January 9, 2021

Programatically add file/image to page/field

Related Topics