Hi,        There is something extra which I guess needs to be discussed. Most of the methods in KipiInterface just use albumDB interface to fetch/set some data about the photo in the database. This construction doesn't basically allow some temporary data storage which might be required for the session purposes only, let me explain.

If the user says he wants to upload the photos to flickr and wants to add some extra tags to the photos when they are uploaded to flickr and doesn't want to effect his original tags in digikam database. We cannot use AlbumDB object here to store the temporary information.

The solution in my opinion is to allow a DigikamImageInfo::QMap<QString,QVariant> attributes; member in the DigikamImageInfo to allow saving of data temporarily in the object using addAttributes() by the host application, while using attributes() from the plugin code to obtain the data ?

<br><br>Please let me know ur viewpoints...<br><br>Vardhman<br><div><span class="gmail_quote">On 5/8/06, <b class="gmail_sendername">Caulier Gilles</b> <<a href="mailto:caulier.gilles@kdemail.net" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

caulier.gilles@kdemail.net</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Le Samedi 06 Mai 2006 11:46 AM, Vardhman Jain a écrit:<br>> Hi,<br>>        I am trying to hack in to the kipiinterface.cpp/.h files for adding<br>> some functionality to the attributes/setAttributes etc functions. I have

<br>> some doubts whose clarfification will help me a great deal.<br>><br><br>You want said these methods :<br><br>QMap<QString,QVariant> DigikamImageInfo::attributes()<br>void DigikamImageInfo::clearAttributes()

<br>void DigikamImageInfo::addAttributes( const QMap<QString,QVariant>& )<br><br>right ?<br><br>> The function Albumdb->getItemTagNames( ImageId id)   takes an image id as<br>> an argument, while almost all other functions mostly use a combination of

<br>> Album id and photo Url.<br>> There are some functions which work on ImageId but they are not at all used<br>> in implementation.<br>><br>> As far as I can see there is no easy way to obtain an ImageId given a

<br>> DigikamImageInfo object. Therefore I would like to ask if it would be okay<br>> to add another method in albumDb interface to allow seeking the tag info by<br>> combination of AlbumId and imageURL or a function should be added to return

<br>> the imageId of a image given its AlbumId and imageURL in the albumDB file<br>> again ?<br><br>Yes, you can do it. Just overload the current methods and don't touch the<br>current implementation. Like this you don't unbreak the tests released on

<br>this methods for 0.9.0 release.<br><br>Please, add comments in method headers, like others current methods using<br>Doxygen syntax.<br><br>When you commit, please CCMAIL on <a href="mailto:digikam-devel@kde.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

digikam-devel@kde.org

</a>. Thanks in advance...<br><br>Gilles<br>_______________________________________________<br>Digikam-devel mailing list<br><a href="mailto:Digikam-devel@kde.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

Digikam-devel@kde.org</a><br><a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

https://mail.kde.org/mailman/listinfo/digikam-devel</a><br></blockquote></div><br><br clear="all"><br>-- <br>Blogs: <a href="http://vardhman.blogspot.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

http://vardhman.blogspot.com</a><br>