Thread

I’ve noticed when using the AJ Penninga image replacement method (with Matrix, ImageSizer and NSM Transplant) that when a file field is left blank, Safecracker file still sends a ‘.’ through to ImageSizer and thus ImageSizr chokes looking for a file and spitting out errors about “this format or extension not found/supported.”

So basically when a page is created without an image it has all these PHP errors at the top of the page because ImageSizer is choking on it. I know there are a lot of other Addon’s involved here, but this worked fine with the EE file field and only after switching to Safecracker did this problem start happening. I’d like to see if this is reproducible before filing a bug.

EE 2.1.3, latest versions of Safecracker, Matrix, NSM Transplant, Imgasizer)

Jonathan,

I tested this myself and using a combination of Safecracker File and Matrix produces this ‘.’
Using Safecracker File on it’s own does not.

The cell in the database is also empty

I couldn’t file anything initially while looking through the code for Safecracker File.

Could you go back and ask Brandon on this one as it sees, to only be the combination of both that causes this issue

Absolutely, I’ll check with him and let you know.

Let us know what you find out.. We’ll be here.

I don’t think this problem has anything to do with Matrix. I just ran into the same issue and tracked the period down to line 284 in ft.file.php

$full_path = $file_info['path'].$file_info['filename'].'.'.$file_info['extension'];

The problem is the file_info array is blank in our case so all that’s being outputted there is the period before the file extension.

Hey Jamie,

So you’re saying this happens for you with the regular file field as well?

Hey Jon.

No it doesn’t happen with the regular file. Only with safecracker file being used as a celltype. But the problem (I think) is with Safecracker File, not matrix.

I’m no PHP developer but I amended the code slightly to get things working. I’m sure there’s a better way of doing it but at least it might point EllisLab support to the problem and help you in the short term.

File: ft.safecracker_file.php
Method: replace_tag()

The problem seems to occur when the $file_info array exists but has empty values.

Existing code:

if ($file_info)
{
    if ( ! is_array($file_info))
    {
        $file_info = parent::pre_process($file_info);
    }
    
    return parent::replace_tag($file_info, $params, $tagdata);
}

Change it to:

if ($file_info)
{
    if ( ! is_array($file_info))
    {
        $file_info = parent::pre_process($file_info);
    }
            
    $imploded_file_info = implode('', $file_info); 
            
    if (empty($imploded_file_info)) { 
        return '';
    } else { 
        return parent::replace_tag($file_info, $params, $tagdata);
    }

}

I went ahead and created a bug report here.

Hey Jamie,

Yeah that seems to take care of it for me. Thanks, man!

And thanks for filing the bug John Henry 😉

Hi, Jonathan,

Glad you have a workaround. Since there’s a bug report for this, I’m going to go ahead and close the thread. Jonathan, if you comment on the bug report, you’ll be notified when the bug changes status.