Bug 279545 - Picasa import fetches scaled-down photos [patch]
Summary: Picasa import fetches scaled-down photos [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-WebService-Google (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-06 18:28 UTC by Havard Skinnemoen
Modified: 2018-01-30 21:26 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 2.2.0
Sentry Crash Report:


Attachments
Patch to add support for the picasaweb 'imgmax' query parameter (5.83 KB, patch)
2011-08-07 01:53 UTC, Havard Skinnemoen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Havard Skinnemoen 2011-08-06 18:28:15 UTC
Version:           1.9.0 (using KDE 4.6.5) 
OS:                Linux

I use the PicasaWeb import tool to fetch one of my albums into Digikam. The process finished very quickly, but the images were much smaller than the originals and all the EXIF data was missing. For example, a photo which is 4288×2848 originally, was scaled down to 1600x1063.

I know the PicasaWeb API allows you to choose between various image sizes when downloading. It would be nice if Digikam could pick the original size (or even better, let the user choose).

Reproducible: Always

Steps to Reproduce:
Import -> Import from Picasaweb
Log in
Select an album and a destination
Start Download
Compare image size reported by Digikam to image size reported by picasaweb

Actual Results:  
Image size in Digikam is much smaller than the original size as reported by picasaweb.

Expected Results:  
Image size should be the same.
Comment 1 Havard Skinnemoen 2011-08-07 01:53:39 UTC
Created attachment 62625 [details]
Patch to add support for the picasaweb 'imgmax' query parameter

This patch works for me -- I can now download images in any size, and EXIF data is preserved if I choose 'Original Size'.

The combo box initialization could probably be done more elegantly. If people are interested, I'll be happy to work a bit more on this.
Comment 2 caulier.gilles 2011-08-07 09:24:44 UTC
Thanks for patch

Do you created it against git master source code (future 2.1.0 release) ?

Gilles Caulier
Comment 3 Havard Skinnemoen 2011-08-07 17:25:22 UTC
Yes, it's against master.

git describe HEAD^
v2.0.0-18-gf42b5c4
Comment 4 Petri Damstén 2011-09-13 07:31:42 UTC
Git commit 912a66f18d91f6d225e1733125a6f5f6a9156c8b by Petri Damstén.
Committed on 12/09/2011 at 11:11.
Pushed by pdamsten into branch 'master'.

Picasa import fetches scaled-down photos
BUG: 279545

M  +4    -1    picasawebexport/picasawebtalker.cpp
M  +2    -1    picasawebexport/picasawebtalker.h
M  +20   -0    picasawebexport/picasawebwidget.cpp
M  +1    -0    picasawebexport/picasawebwidget.h
M  +2    -1    picasawebexport/picasawebwindow.cpp

http://commits.kde.org/kipi-plugins/912a66f18d91f6d225e1733125a6f5f6a9156c8b