<br><br><div class="gmail_quote">On Wed, Nov 7, 2012 at 5:36 AM, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
El Dimarts, 6 de novembre de 2012, a les 18:30:27, Albert Astals Cid va<br>
escriure:<br>
<div class="im">> El Dimarts, 6 de novembre de 2012, a les 18:17:29, Vishesh Handa va<br>
escriure:<br>
> > Ping?<br>
> ><br>
> > I'll be merging this on Thursday. I would appreciate it if somone could<br>
> > review the code.<br>
><br>
> I'll have a look later today (based on my 0 knowledge of nepomuk & friends).<br>
<br>
</div>Had a look to the code and to my knowledge it seems that it does not do any<br>
ultra silly thing, also it's a kioslave so people won't use it unless they<br>
want so in case it is horribly broken the "bad stuff" it can do it is pretty<br>
limited (just don't use the kioslave and done!)<br>
<br>
So i'd say go ahead.<br>
<br>
I'm curious with what you mean with "loosing"  in "You can move files into a<br>
tag folder thereby effectively loosing them" though. Could you give some more<br>
info?<br></blockquote><div><br>The file will be deleted. From what I understand KIO does moves by copy + delete. The copy operation results in the corresponding tags being applied, and the delete operation then deletes the file. The correct behaviour in our case would be to either only allow copy operations or convert move operations into copy operations.<br>
<br>We'll need to introduce some new kio flags to fix this. At least that's what I understood from my discussion with David.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Cheers,<br>
  Albert<br>
<div><div class="h5"><br>
><br>
> Cheers,<br>
>   Albert<br>
><br>
> P.S: You sent the email to the wrong list, we do reviews in kde-core-devel<br>
> not kde-devel<br>
><br>
> > On Wed, Oct 31, 2012 at 3:12 AM, Vishesh Handa <<a href="mailto:me@vhanda.in">me@vhanda.in</a>> wrote:<br>
> > > Hey everyone<br>
> > ><br>
> > > I recently wrote a 'tags' kio slave, which allows one to browse nepomuk<br>
> > > tags, and the files containing those tags. It is currently in<br>
> > > kde-runtime/nepomuk/kioslaves/tags/, in the nepomuk/tagsKioSlave branch.<br>
> > > Could someone please review it? I would like to merge it before the hard<br>
> > > freeze.<br>
> > ><br>
> > > It has a custom url naming scheme -<br>
> > ><br>
> > > *tags:/tag1* => All the files tagged with 'tag1' and additional tags<br>
> > > which can be used filter the results.<br>
> > ><br>
> > > *tags:/tag1/file_url_of_the_file* => This is the url of the file.<br>
> > > Ideally<br>
> > > I would just use the normal url of the file, but we need to have the<br>
> > > 'tags'<br>
> > > protocol, so that we can handle delete events. We don't want to delete<br>
> > > the<br>
> > > file, rather just remove that tag.<br>
> > ><br>
> > > This file_url_of_the_file is added as the UDS_NAME, and results in the<br>
> > > search results looking slightly ugly if the file properties are checked.<br>
> > > Is<br>
> > > there a better way of handling this?<br>
> > ><br>
> > > *tags:/tag1/tag2 *=> Files tagged with both tag1 and tag2<br>
> > ><br>
> > > Known problems -<br>
> > > * You can move files into a tag folder thereby effectively loosing them<br>
> > > * No automatic updates<br>
> > > * It shows the tags with the nepomuk icon. We need a better 'tags' icon.<br>
> > ><br>
> > ><br>
> > > --<br>
> > > Vishesh Handa<br>
> >><br>
</div></div>> >> Visit <a href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub" target="_blank">http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to unsubscribe<br>
> >> <<<br>
<br>
>> Visit <a href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub" target="_blank">http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to unsubscribe <<<br>
</blockquote></div><br><br clear="all"><br>-- <br><span style="color:rgb(192,192,192)">Vishesh Handa</span><br><br>