<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>322348</bug_id>
          
          <creation_ts>2013-07-14 13:40:59 +0000</creation_ts>
          <short_desc>Removed labels still shows up in column view</short_desc>
          <delta_ts>2013-07-18 15:10:29 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>dolphin</product>
          <component>view-engine: columns mode</component>
          <version>2.2</version>
          <rep_platform>Ubuntu</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>damien.porquet</reporter>
          <assigned_to name="Dolphin Bug Assignee">dolphin-bugs-null</assigned_to>
          <cc>emmanuelpescosta099</cc>
          
          <cf_commitlink>http://commits.kde.org/kde-baseapps/5b81abea5be6d6d57da7a1ca953f8371b23b6528</cf_commitlink>
          <cf_versionfixedin>4.11.0</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1383272</commentid>
    <comment_count>0</comment_count>
    <who name="">damien.porquet</who>
    <bug_when>2013-07-14 13:40:59 +0000</bug_when>
    <thetext>I need to refresh the view to make them disappear.
No problem if I add a label.

Reproducible: Always

Steps to Reproduce:
1. setup dolphin to column view mode
2. add a label to a file
3. remove it
Actual Results:  
removed label still shown 

Expected Results:  
removed label should disappear</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383331</commentid>
    <comment_count>1</comment_count>
    <who name="Emmanuel Pescosta">emmanuelpescosta099</who>
    <bug_when>2013-07-14 18:18:17 +0000</bug_when>
    <thetext>Thanks for the bug report!

I can reproduce this problem in Dolphin 4.10.90</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383339</commentid>
    <comment_count>2</comment_count>
    <who name="Emmanuel Pescosta">emmanuelpescosta099</who>
    <bug_when>2013-07-14 19:05:59 +0000</bug_when>
    <thetext>Review Request: https://git.reviewboard.kde.org/r/111505/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383376</commentid>
    <comment_count>3</comment_count>
    <who name="Frank Reininghaus">frank78ac</who>
    <bug_when>2013-07-14 22:40:57 +0000</bug_when>
    <thetext>Just to be sure that we understand what the problem is, could you try to describe the problem more clearly or provide a screenshot of the bug? The combination of &quot;column view mode&quot;, &quot;label&quot; and the component &quot;Information Panel&quot; doesn&apos;t make much sense to me.

Emmanuel&apos;s patch looks like it makes sense (I&apos;ll try to have a closer look tomorrow or on Tuesday), but it doesn&apos;t have anything to do with the Information Panel, so I&apos;d prefer to make sure that we don&apos;t misunderstand the problem that you are seeing. Thanks for your help!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1383453</commentid>
    <comment_count>4</comment_count>
    <who name="">damien.porquet</who>
    <bug_when>2013-07-15 08:33:24 +0000</bug_when>
    <thetext>Thank you for the quick reply (and fix!).
I attached it to the component &quot;panel&quot; since I change tags through this panel, but it should probably go to component &quot;view engine: column mode&quot; instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1384291</commentid>
    <comment_count>5</comment_count>
    <who name="Emmanuel Pescosta">emmanuelpescosta099</who>
    <bug_when>2013-07-18 15:10:29 +0000</bug_when>
    <thetext>Git commit 5b81abea5be6d6d57da7a1ca953f8371b23b6528 by Emmanuel Pescosta.
Committed on 18/07/2013 at 15:07.
Pushed by emmanuelp into branch &apos;KDE/4.11&apos;.

Overwrite the changed role value with an empty QVariant,
because the nepomuk roles provider doesn&apos;t overwrite it when the property
value list is empty.
REVIEW: 111505
FIXED-IN: 4.11.0

M  +10   -2    dolphin/src/kitemviews/kfileitemmodelrolesupdater.cpp
M  +5    -1    dolphin/src/kitemviews/kfileitemmodelrolesupdater.h
M  +6    -1    dolphin/src/kitemviews/private/knepomukrolesprovider.cpp
M  +2    -0    dolphin/src/kitemviews/private/knepomukrolesprovider.h

http://commits.kde.org/kde-baseapps/5b81abea5be6d6d57da7a1ca953f8371b23b6528</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>