Bug 125234

Summary: Flickr tool should be able to access image tags
Product: [Applications] digikam Reporter: Jesús Jiménez <jesjimenez>
Component: Plugin-WebService-FlickrAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist CC: caulier.gilles
Priority: NOR    
Version: 5.6.0   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Jesús Jiménez 2006-04-09 17:13:59 UTC
Version:            (using KDE KDE 3.4.3)
Installed from:    Ubuntu Packages

I've recently discovered the flickrexport kipi plugin, and I use it a lot. Despite it rocks, I've found that it exports image comments, date and such, but it doesn't export tag info, which seems silly since flickr tags and digikam tags share the same concept, and tags are a key feature for cataloguing images. 

After messing a little bit with flickrexport sources, I've noticed that the problem comes from KIPI not exposing tag info to plugins. I think that it would fit quite well in KIPI::ImageInfo class, as a getTags() method or something similar. Since a lot of image programs use tags nowadays, I think that being able to access them using kipi would lead to more and better plugins.

Thanks for your attention
Comment 1 Vardhman 2006-04-09 19:44:43 UTC
On 9 Apr 2006 15:13:59 -0000, Jes
Comment 2 Jesper Pedersen 2006-04-11 15:24:10 UTC
May I draw your attention to ImageInfo::attributes(), this was designed with 
tagging in mind, so I believe this is what you need.

Kind Regards
Jesper.

On Sunday 09 April 2006 19:44, Vardhman Jain wrote:
| On 9 Apr 2006 15:13:59 -0000, Jes
Comment 3 caulier.gilles 2006-04-11 15:35:11 UTC
Vardhman, 

i second Jesper about this point. ImageInfo::attributes() is the better way to use Image tags.

This is want mean that :

- plugins need to check this properties.
- kipi Hosts interface need to do something about this method, if host support Tags (KPhotoAlbum, and digiKam for example).

Like you working with digiKam, and if you want do some tests, i recommend you to check the digikam kipi-interface :

http://websvn.kde.org/trunk/extragear/graphics/digikam/digikam/kipiinterface.cpp?rev=526233&view=auto

... look like we have this code :

QMap<QString,QVariant> DigikamImageInfo::attributes()
{
    QMap<QString,QVariant> res;

    // TODO ! This will used for the futures tags Digikam features.

    return res;
}

void DigikamImageInfo::clearAttributes()
{
    // TODO ! This will used for the futures tags Digikam features.
}

void DigikamImageInfo::addAttributes( const QMap<QString,QVariant>& )
{
    // TODO ! This will used for the futures tags Digikam features.
}

This is want mean that something must be done here (:=))). If you is interressed to implement this feature, we can trying to implement this method together and after you just need to support ImageInfo::attributes() in your Flicker plugin.

My 10 cts...

Gilles
Comment 4 Vardhman 2006-04-29 11:46:49 UTC
>
> ------- Additional Comments From caulier.gilles free fr  2006-04-11 15:35
> -------
> Vardhman,
>
> i second Jesper about this point. ImageInfo::attributes() is the better
> way to use Image tags.
>
> This is want mean that :
>
> - plugins need to check this properties.
> - kipi Hosts interface need to do something about this method, if host
> support Tags (KPhotoAlbum, and digiKam for example).
>
> Like you working with digiKam, and if you want do some tests, i recommend
> you to check the digikam kipi-interface :
>
>
> http://websvn.kde.org/trunk/extragear/graphics/digikam/digikam/kipiinterface.cpp?rev=526233&view=auto
>
> ... look like we have this code :
>
> QMap<QString,QVariant> DigikamImageInfo::attributes()
> {
>     QMap<QString,QVariant> res;
>
>     // TODO ! This will used for the futures tags Digikam features.
>
>     return res;
> }
>
> void DigikamImageInfo::clearAttributes()
> {
>     // TODO ! This will used for the futures tags Digikam features.
> }
>
> void DigikamImageInfo::addAttributes( const QMap<QString,QVariant>& )
> {
>     // TODO ! This will used for the futures tags Digikam features.
> }
>
> This is want mean that something must be done here (:=))). If you is
> interressed to implement this feature, we can trying to implement this
> method together and after you just need to support ImageInfo::attributes()
> in your Flicker plugin.
>
> Gilles,

           Should we start working on this? Or we have to wait for some
release etc ?
--
Blogs: http://vardhman.blogspot.com
<div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">------- Additional Comments From caulier.gilles free fr&nbsp;&nbsp;2006-04-11 15:35 -------<br>Vardhman,
<br><br>i second Jesper about this point. ImageInfo::attributes() is the better way to use Image tags.<br><br>This is want mean that :<br><br>- plugins need to check this properties.<br>- kipi Hosts interface need to do something about this method, if host support Tags (KPhotoAlbum, and digiKam for example).
<br><br>Like you working with digiKam, and if you want do some tests, i recommend you to check the digikam kipi-interface :<br><br><a href="http://websvn.kde.org/trunk/extragear/graphics/digikam/digikam/kipiinterface.cpp?rev=526233&amp;view=auto">
http://websvn.kde.org/trunk/extragear/graphics/digikam/digikam/kipiinterface.cpp?rev=526233&amp;view=auto</a><br><br>... look like we have this code :<br><br>QMap&lt;QString,QVariant&gt; DigikamImageInfo::attributes()<br>
{<br>&nbsp;&nbsp;&nbsp;&nbsp;QMap&lt;QString,QVariant&gt; res;<br><br>&nbsp;&nbsp;&nbsp;&nbsp;// TODO ! This will used for the futures tags Digikam features.<br><br>&nbsp;&nbsp;&nbsp;&nbsp;return res;<br>}<br><br>void DigikamImageInfo::clearAttributes()<br>{<br>&nbsp;&nbsp;&nbsp;&nbsp;// TODO ! This will used for the futures tags Digikam features.
<br>}<br><br>void DigikamImageInfo::addAttributes( const QMap&lt;QString,QVariant&gt;&amp; )<br>{<br>&nbsp;&nbsp;&nbsp;&nbsp;// TODO ! This will used for the futures tags Digikam features.<br>}<br><br>This is want mean that something must be done here (:=))). If you is interressed to implement this feature, we can trying to implement this method together and after you just need to support ImageInfo::attributes() in your Flicker plugin.
<br><br></blockquote></div>Gilles,<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Should we start working on this? Or we have to wait for some release etc ?<br>-- <br>Blogs: <a href="http://vardhman.blogspot.com">http://vardhman.blogspot.com</a><br>
Comment 5 caulier.gilles 2006-04-29 12:22:21 UTC
yes we can implement this feature in digiKam kipi interface from trunk. It's not very complicated, normally the implementation is just limited to the empty digiKam kipi interface methods.

Gilles
Comment 6 caulier.gilles 2006-09-26 07:49:29 UTC
Implementation is done in kipi-interface of digikam from trunk. Tests are require using flickr export plugin.

Gilles Caulier
Comment 7 caulier.gilles 2008-03-16 20:25:36 UTC
This entry is implemented since a long time. I close it now...

Gilles Caulier
Comment 8 caulier.gilles 2008-03-16 20:29:00 UTC
Closed