Bug 291449 - wish: to check the file size and report to the user if the file size exceeds limits set by flickr
Summary: wish: to check the file size and report to the user if the file size exceeds ...
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-WebService-Flickr (show other bugs)
Version: 5.6.0
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-13 14:29 UTC by Martin Senftleben
Modified: 2018-01-30 21:29 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.11.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Senftleben 2012-01-13 14:29:30 UTC
Version:           2.5.0 (using KDE 4.7.3) 
OS:                Linux

I just observed that flickrexport reports an "unknown error" when it attempts to upload images that exceed the maximum allowed filesize. The major problem is: it first attempts to upload the file, which takes a long time depending on the internet connection, and then reports the unspecified error, which leaves the user at first helpless.
I suggest to make flickrexport to check for the file size and report right away that those images won't be accepted. Since pro users can store bigger files on flickr, and I'm not sure if the flickr API reports back if the user is a pro user, a second suggestion:
to have two fields for max file size and max video size. Then flickr export checks only against those entries. A short info in the interface can explain to the user what the current limits are, and suggest maybe the non-pro user limits at first.
flickr has currently these limits: 15 MB per image (and, I believe, 100 or 200 images) or 2 movies (each <= 150 MB)/month for non-pro users, and for pro users unlimited amount with 20 MB per image and videos up to 500 MB each.

It would be a question if the quantity limits for non-pro users can also be checked, but that would be more a question to the API than to flickrexport, even though this would be nice to have (not for me since I am a pro user, but maybe for others who are not). I am not sure if a total of 200 images is allowed throughout, or a total of uploads per month.

Reproducible: Didn't try

Steps to Reproduce:
None, this is a wish.

Actual Results:  
None, this is a wish.

Expected Results:  
No guessing what could be wrong
Comment 1 Shourya Singh Gupta 2015-05-15 21:06:40 UTC
This feature has been added. Commit can be seen in http://commits.kde.org/kipi-plugins/07a7a9ba00ca25f6216cc123098237a87facffae. Awaiting reviews :)
Comment 2 caulier.gilles 2015-05-15 21:11:55 UTC
Git commit d8330e063292a00801274a57d790dd1a7fe7b29c by Gilles Caulier.
Committed on 15/05/2015 at 21:11.
Pushed by cgilles into branch 'frameworks'.

sync with master

M  +1    -0    PORT.KF5/FACTORIZE/flickrexport/flickritem.h
M  +191  -82   PORT.KF5/FACTORIZE/flickrexport/flickrtalker.cpp
M  +8    -3    PORT.KF5/FACTORIZE/flickrexport/flickrtalker.h
M  +53   -41   PORT.KF5/FACTORIZE/flickrexport/flickrwindow.cpp

http://commits.kde.org/kipi-plugins/d8330e063292a00801274a57d790dd1a7fe7b29c
Comment 3 caulier.gilles 2015-05-15 21:15:27 UTC
Shourya,

You can use these macros at commit time to close bug, CC comment to files, mail somebody, etc:

https://techbase.kde.org/Policies/Commit_Policy#Special_keywords_in_GIT_and_SVN_log_messages

Gilles Caulier
Comment 4 Shourya Singh Gupta 2015-05-16 17:43:39 UTC
Sure. I will close the bug that way from next time :)