D12156: implement reading of rating tag

Alexander Stippich noreply at phabricator.kde.org
Thu Apr 26 06:56:48 UTC 2018


astippich added a comment.


  In D12156#252575 <https://phabricator.kde.org/D12156#252575>, @mgallien wrote:
  
  > If I have correctly understood the ideas behind the conception of Baloo, we should probably prefer to store the rating with a "native" solution instead of the xattr one that is not portable outside of software using KFileMetaData.
  >
  > We have to stay compatible with older versions of KFileMetaData. So we should push the same rating to both properties and try to read it from both properties in the cases where only one exists.
  >
  > Could you try to modify your patch to do that ?
  
  
  I don't understand. Which patch to you want me to modify? This one here only adds the ability to read the rating embedded in the tags in addition to the xattr rating. It's up to the application to decide what to do with the information. 
  The patch for baloo-widgets just hides this new rating tag to avoid duplicate entries in e.g. dolphin, and actually preserves the current behavior this way. Baloo caches the embedded rating anyways. 
  When KFileMetaData gains the ability to write the tags, one should write to both properties, but that is currently not possible.

REPOSITORY
  R286 KFileMetaData

REVISION DETAIL
  https://phabricator.kde.org/D12156

To: astippich, mgallien, michaelh
Cc: bruns, #frameworks, ashaposhnikov, michaelh, astippich, spoorun
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180426/bdac9540/attachment.html>


More information about the Kde-frameworks-devel mailing list