[Kde-imaging] [Bug 180055] Can't create new photoset: Invalid primary photo id (n)

Pieter Edelman p.edelman at gmx.net
Sat May 16 10:03:44 CEST 2009


https://bugs.kde.org/show_bug.cgi?id=180055





--- Comment #4 from Pieter Edelman <p edelman gmx net>  2009-05-16 10:03:42 ---
Created an attachment (id=33708)
 --> (http://bugs.kde.org/attachment.cgi?id=33708)
Delay photo set creation until first picture is uploaded

Flickr doesn't know the concept of empty photosets, is expects that sets are
created from an already present set of photos, one of which should be the main
photo. I think the most sensible thing to do is to delay the actual creation of
the photo set until the first photo for it is uploaded. The following patch
implements this principle.

After the photo set creation dialog is executed, a new FPhotoSet object is
created with a temporary ID in the form of "UNIDENTIFIED_" followed by a unique
number, instead of the Flickr ID. When the addToPhotoSet method encounters
this, it invokes the createPhotoSet method. The resulting ID is consequently
set in the FPhotoSet object and the relevant entry in the m_photoSetList.

To facilitate this, I modified the FlickrTalker object so that it doesn’t store
the ID of the selected photo (m_selectedPhotoSetId), but instead the complete
FPhotoSet (m_selectedPhotoSet).

This patch removes the 'primary photo id' option altogether from the photo set
creation dialogue. I also removed the 'photoset name' option – as far as I can
tell it isn't used for anything.

This patch implies the patch in bug 192541.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the Kde-imaging mailing list