browse should use a better name when uploading an entry from the journal

[tomeu]

Report from Tony Forster:

download a file from the net
saves in journal as:
File inventory.gmd from http://rupert.id.au/schoolgamemaker/samples1/inventory.gmd.
attach the file in webmail
file attached is titled :
tmpTjCrkn.bin
full path is:
/home/olpc/isolation/1/uid_to_home_dir/10011/tmp/tmpTjCrkn.bin

[tomeu]

We may be able to split it in separate issues:

• The extension should be stored and preserved. Possible problem: what if the user opens the entry with some activity that changes the format in which the file is saved?

• The base file name should be stored and preserved. Possible problem: what should we do if the user changes the title? Should we discard the old filename and generate a new one from the entry title?

I think this ticket could be dealt solely from the Browse activity, but depending on the solution we may be able to implement it in the datastore and benefit other use cases.

[tonyforster]

See also ticket 899, preservation of filename on downloading

[erikos]

Do we have a chance to get this in?

[alsroot]

I guess the simplest way would be just using title in all cases
and initial value for title could be original filename(with extension).

[alsroot]

..and while uploading file from ds, sugar can detect(somehow) should it add mime_type extension or use title as is

[alsroot]

..and while downloading file to ds, sugar can remove mime_type extension

[alsroot]

After discussing this issue on irc,
Looks like coding this feature in shell could bring undesirable issues(e.g. in case of creating tmp directory instead of tmp file cou, developper should remove directory and it breaks existed API).
At least for 0.86 this feature could be implemented only in Browse.

[alsroot]

As was mentioned above, one of issues is extension.

So, it could be:

• (already implemented) while downloading to Journal, keep full name(w/ extension) as a 'title'
• while uploading from Journal try to guess

The major point of guessing

• instead of just using 'title' as is, jobject could be generated by sugar(so, w/o any extensions)
• instead of using mime_type extension, original(downloaded) file can have different extension e.g. .conf and not .txt)

Guessing could be

• if 'title' ends by mime_type extension, leave it as is
• if 'title' doesn't end by '\.\S$', add mime_type extension

[erikos]

Thanks for the patch!

​http://git.sugarlabs.org/projects/browse/repos/mainline/commits/142ff4aaeeff8d9e75d944533cdece54593c52d5

[erikos]

|TestCase|
- upload a journal entry using browse
- check that the file name is constructed of the entry title

[dnarvaez]

Milestone 0.86 deleted