Add config setting to allow default filename for paste uploads

[Steve James]

Per your source code for paste uploads (upload repository):

If not set and the second parameter is a Base64 data string, then the 
file name will be created based on the MIME type by replacing '/' with 
'.', for example for `image/png` the file name will be `image.png`.

I just discovered this in via my upload handler... What if a user uploads a file actually named image.png? (not that far fetched) How am I supposed to discriminate between that case and the case where CKEditor used the default name?

It would be better to allow establishing the default name via a config setting. Then, I could set something like ~_noname_~ + extension that I could easily detect.

[Piotrek Koszuliński]

Makes sense.

[Szymon Kupś]

Pushed ​branch:t/13501.

[Steve James]

I looked at the code for this...

I was expecting to see logic along the lines of fetch the default from the config or set it as 'image' as a fallback. e.g., something like

var mimeParts,defaultFileName = editor.config.fileTools_defaultFileName || 'image';

unless 'image' is already the fallback in config.js. (I did not see that, though.). However, the code as written does not have the

|| 'image'

or other fallback in it, so you have a potential issue if the config value is not set.

Thus, the value from the config is an override, since you cannot reliably depend on developers always setting this value. A reasonable approach might be to just put fileTools_defaultFileName setting in your default config.js with a value of 'image'. That way, it is documented and you know you have a value, but it can be overridden with a better value.

[Szymon Kupś]

This code checks if defaultFileName is present. If not - it leaves creating file name from MIME type as it was before introducing this fix. So for image/png MIME type file name will be image.png and so on.

mimeParts = this.file.type.split( '/' );

if ( defaultFileName ) {
	mimeParts[ 0 ] = defaultFileName;
}

this.fileName = mimeParts.join( '.' );

[Steve James]

Ok, sorry, I missed that. Thx.

[Piotrek Koszuliński]

Merged to master with ​git:9c406a0.

I needed to update one test file though. There's a bit more tests to run together with the filetools plugin (uploadimage, uploadwidget, etc). Always make sure to find related plugins or simply run all plugins tests.

[Steve James]

Is there also a related doc update needed?

[Piotrek Koszuliński]

There are API docs for the new feature and they will be available on ​http://docs.ckeditor.com before the release. But we may also need to mention something in the guides, although I'm not sure if have a good place for this yet.

[Piotr Jasiun]

I have added comment to the guild about this option: ​https://github.com/ckeditor/ckeditor-docs/commit/6825577.

[Marcin K.]

Not working in Firefox, but it could be Firefox fault. Firefox set own name - in polish language is "obraz.png"