LocalFiles image path contains 'public'

Hi,

First of all I’d just like to say thanks for an amazing cms : )

After implementing the localFiles field type within my model the images do not correctly show within the admin ui.

The path to the images contains ‘public’ which obviously can’t be accessed frontend.

I’ve worked about this by adding this line above live 15 in keystone/templates/localfiles/form.jade

- value.path = value.path.replace('public/', '')

Is there a nicer way to solve this?

Thanks,

[JedWatson]

Sorry to take so long to get to this! it was a real headache, there were a lot of things to fix in the LocalFiles field type when I started digging into it, so that this wasn’t just a hacky fix.

I’ve brought the field implementation into line with the LocalFile type, and it now supports the same options of dest, prefix and format. This allows you to upload files to locations other than the public folder (which I personally would recommend) and still have them work correctly with public paths.

In my test project I have it working, including image previews, and it looks like this:

In keystone.js

Set add whatever path you’re uploading files to as one of your static paths, using an array:

keystone.set('static', ['public', 'files']);

In your Model

Specify the dest as the folder you want the uploads to go into, the same one you exposed with your static option. Set the prefix option to the public path of that directory, and if you’re using images, you can also set a format option to display thumbnails.

    localFile: {
        type: Types.LocalFile,
        dest: './files/single',
        prefix: '/single',
        format: function(item, file){
            return '<img src="' + file.href + '" style="max-width: 300px">'
        }
    },
    localFiles: {
        type: Types.LocalFiles,
        dest: './files/multiple',
        prefix: '/multiple',
        format: function(item, file){
            return '<img src="' + file.href + '" style="max-width: 300px">'
        }
    }

As you can see in the example, there’s also a new href property on the file inside the format funcition that gives access to the public href, it’s available on the List too.

For more details, check out the commit I did all this in.

If anything else isn’t working, please let me know, and I’ll sort it out.

[JedWatson]

Thanks for pointing that out @tylkomat, I’ve fixed it and will push a new version soon.