[Digikam-devel] Reg Kipi-interface in Digikam

[Vardhman Jain]

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 ?

Please let me know ur viewpoints...

Vardhman
On 5/8/06, Caulier Gilles <caulier.gilles at kdemail.net> wrote:
>
> Le Samedi 06 Mai 2006 11:46 AM, Vardhman Jain a écrit:
> > Hi,
> >        I am trying to hack in to the kipiinterface.cpp/.h files for
> adding
> > some functionality to the attributes/setAttributes etc functions. I have
>
> > some doubts whose clarfification will help me a great deal.
> >
>
> You want said these methods :
>
> QMap<QString,QVariant> DigikamImageInfo::attributes()
> void DigikamImageInfo::clearAttributes()
> void DigikamImageInfo::addAttributes( const QMap<QString,QVariant>& )
>
> right ?
>
> > The function Albumdb->getItemTagNames( ImageId id)   takes an image id
> as
> > an argument, while almost all other functions mostly use a combination
> of
> > Album id and photo Url.
> > There are some functions which work on ImageId but they are not at all
> used
> > in implementation.
> >
> > As far as I can see there is no easy way to obtain an ImageId given a
> > DigikamImageInfo object. Therefore I would like to ask if it would be
> okay
> > to add another method in albumDb interface to allow seeking the tag info
> by
> > combination of AlbumId and imageURL or a function should be added to
> return
> > the imageId of a image given its AlbumId and imageURL in the albumDB
> file
> > again ?
>
> Yes, you can do it. Just overload the current methods and don't touch the
> current implementation. Like this you don't unbreak the tests released on
> this methods for 0.9.0 release.
>
> Please, add comments in method headers, like others current methods using
> Doxygen syntax.
>
> When you commit, please CCMAIL on digikam-devel at kde.org . Thanks in
> advance...
>
> Gilles
> _______________________________________________
> Digikam-devel mailing list
> Digikam-devel at kde.org
> https://mail.kde.org/mailman/listinfo/digikam-devel
>



--
Blogs: http://vardhman.blogspot.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20060509/7bd16bc6/attachment.html>