Powerpoint is slow to load

Bug #1677037 reported by Tomas Groth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenLP
Confirmed
Wishlist
Tomas Groth

Bug Description

Originally reported in forum: https://forums.openlp.org/discussion/3412/power-point-takes-too-long-to-load

The problem is that when we load a PPT we generate thumbnails for the preview and the slidecontroller, and extract notes and titles for showing on the stage view. This data is stored in a folder which name is a md5-hash of the full path of the ppt-file.
When a presentation is placed in the servicemanager, the file is copied to the servicemanager folder, and when going live OpenLP creates a hash of the new full path of the file (now in the servicemanager folder) which of course is different from the old hash, and therefore OpenLP generates the thumbnails etc again.

A few solutions could be made:
 * Make the hash based on the file content instead of the file path (might be a bit slow on big files).
 * When adding a ppt-file to the servicemanager, copy the old folder to the new hash-folder.
 * Add the thumbnails etc to the service file.

Related branches

Revision history for this message
Steven Phillips (sasid) wrote :

Alternatively base the hash on:

    <file name without the hash>+<file size in bytes>+<last modification date>

And then make sure the last modification date is not changed when it is copied into the servicemanager folder.

If currently only the full file name is used in the hash what happens when a new version of the file is copied over the original? Do the thumbnails get regenerated? Adding the size and mod-time to the hash would address this too, but if there already is some mechanism to spot and cope with this it could cause a conflict.

Revision history for this message
Steven Phillips (sasid) wrote :

Sorry, hash was meant to be <file name without the path>, so given the ppt: c:\temp\test.ppt is 23456 bytes in size and last edited on the first second of 1 Jan 2018 the hash would be:

   'test.ppt:23456:20180101000001'

Or some binary equivalent for the size and date.

Tomas Groth (tomasgroth)
Changed in openlp:
assignee: nobody → Tomas Groth (tomasgroth)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.