Cannot add photos into wallpaper list

[cathyli]

Description:
------------------------
Cannot add photos into wallpaper list. Found the issue in moblin-applet 0.52, and player 0.1.0.8 in QA 20080407 RC1 image .

Reproduced steps:
---------------------------
1. Launch photos from Sound&Video to generate media folder under /home/ume/media
2. Put some image files under /home/ume/media/photo/, it's preparation step.
3. Launch photo from home screen, you will see thumbnail files of photos.
4. Launch background applet from Preferences to check if all photos added to wallpaper list.

Current results:
-------------------------
In step4, wallpaper list is empty, cannot add photos into wallpaper list.

Expected results:
-------------------------
In step4, all photos should be added into the wallpaper list.

Root Cause analysis:
-------------------------
It seems photo viewer doesn't manager wallpaper by new player.

[cathyli (cathy-li)]

Todd, the issue should be fixed in m10. If you are not the proper owner, pls, help to re-assign. Also subscribe to Rusty and Bob.

[ToddBrandt (todd-e-brandt)]

Please post here exactly what the image names were you added. It could be a problem of unrecognized file type, case, or a number of things cause by the specific files you're copying. Could you also maybe create a zip file with those images and post them to this bug?

[cathyli (cathy-li)]

Todd, I attached the image files. All these image files are .bmp format which is supported by player. And Can view thumbnails of these files from Photo viewer. I used these files in previous moblin-applets version, they can be added to wallpaper list.

[ToddBrandt (todd-e-brandt)]

ok, the fix will be committed in moblin-applets GASTON 0.53. The .realplayer directory produces the exact same thumbnail directories and files, but to be safe, I do a recursive directory search in /home/ume for all instances of "thumbnails/largest". For example if the directory /home/ume/.realwinampvlcintervideo/junk/stuff/thumbnails/largest exists it will be scanned for image files that match the thumbnail naming format. If there are multiple thumbnail/largest directories, all are scanned, and if there are any thumbnail duplicates only the first one found is added to the wallpaper list. I tried this on the 20080407_q1u_installusb.img target and it worked well.

[ToddBrandt (todd-e-brandt)]

Ok, the fix has changed a little. The package now installs a /desktop/moblin/background/thumbpaths folder in gconf and adds keys for every thumbnail cache folder the backgrounds applet should traverse looking for images. It installs two now, one for .moblin-media/thumbnails/largest and one for .realplayer/thumbnails/largest. It looks for them in /home/ume and pulls in any images named in the expected format. If there are duplicates it only takes the first found.

[cathyli (cathy-li)]

Verified in applets 0.54, the issue is fixed, close it.