<?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>300557</bug_id>
          
          <creation_ts>2012-05-24 11:22:11 +0000</creation_ts>
          <short_desc>When a collection is updated, Amarok expands collapsed entries in the Collection Browser</short_desc>
          <delta_ts>2016-10-17 08:33:52 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>amarok</product>
          <component>Collection Browser</component>
          <version>2.8.0</version>
          <rep_platform>Compiled Sources</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.kde.org/show_bug.cgi?id=335217</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>188074</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Piotr Keplicz">keplicz</reporter>
          <assigned_to name="Amarok Bugs">amarok-bugs-null</assigned_to>
          <cc>bart.cerneels</cc>
    
    <cc>e.m.a.t.i.r.o.v</cc>
    
    <cc>maddiemadan</cc>
    
    <cc>matej</cc>
    
    <cc>robert</cc>
          
          <cf_commitlink>http://commits.kde.org/amarok/3b5c6b2039ad943ae209341b5920fee929da9320</cf_commitlink>
          <cf_versionfixedin>2.9</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1258075</commentid>
    <comment_count>0</comment_count>
    <who name="Piotr Keplicz">keplicz</who>
    <bug_when>2012-05-24 11:22:11 +0000</bug_when>
    <thetext>When I plug in an iPod (a 6G), Amarok starts counting the tracks. Now, when I fold the iPod&apos;s listing, it automatically unfolds a moment later, when the tracks count is updated. That disrupts browsing the tracks in other collections.

Reproducible: Always

Steps to Reproduce:
1. Plug in an iPod.
2. Fold the iPod listing while tracks are being counted.
Actual Results:  
The iPod&apos;s listing is unfolded when the tracks count is updated.

Expected Results:  
The listing should remain folded until the user unfolds it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1258414</commentid>
    <comment_count>1</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2012-05-25 20:24:04 +0000</bug_when>
    <thetext>Piotr could you please not forget to set the Amarok version in your reports.. I assume this is from git as well, right?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1258438</commentid>
    <comment_count>2</comment_count>
    <who name="Piotr Keplicz">keplicz</who>
    <bug_when>2012-05-25 21:27:15 +0000</bug_when>
    <thetext>Yes, it&apos;s from an up-to-date git.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1260117</commentid>
    <comment_count>3</comment_count>
    <who name="Bart Cerneels">bart.cerneels</who>
    <bug_when>2012-05-29 14:51:40 +0000</bug_when>
    <thetext>This needs to be fixed in the view by keeping state of the opened sub-trees. It&apos;s non trivial and not possible to do for the next release. But I&apos;ll put it one the list of items needed for 2.7.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1272101</commentid>
    <comment_count>4</comment_count>
    <who name="Matěj Laitl">matej</who>
    <bug_when>2012-07-03 13:49:43 +0000</bug_when>
    <thetext>See also bug 188074 that complements this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1319940</commentid>
    <comment_count>5</comment_count>
    <who name="Mayank Madan">maddiemadan</who>
    <bug_when>2012-12-01 08:00:22 +0000</bug_when>
    <thetext>Reproducible in v2.6.0-421-gf66a306</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1320017</commentid>
    <comment_count>6</comment_count>
    <who name="Matěj Laitl">matej</who>
    <bug_when>2012-12-01 12:53:22 +0000</bug_when>
    <thetext>This is not specific to iPods and is a visual bug, nothing to do with iPod Collection. iPod collection is just one way to trigger this, other is UMS Collection, local collection...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1370621</commentid>
    <comment_count>7</comment_count>
    <who name="Matěj Laitl">matej</who>
    <bug_when>2013-05-23 13:47:33 +0000</bug_when>
    <thetext>*** Bug 319508 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1415509</commentid>
    <comment_count>8</comment_count>
    <who name="Mikhail Ivchenko">e.m.a.t.i.r.o.v</who>
    <bug_when>2013-12-01 12:06:00 +0000</bug_when>
    <thetext>Reproducible in v2.8.0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1447930</commentid>
    <comment_count>9</comment_count>
    <who name="Matěj Laitl">matej</who>
    <bug_when>2014-05-12 21:05:58 +0000</bug_when>
    <thetext>Git commit 3b5c6b2039ad943ae209341b5920fee929da9320 by Matěj Laitl.
Committed on 12/05/2014 at 21:01.
Pushed by laitl into branch &apos;master&apos;.

CollectionBrowser: do not expand tree needlessly

The reasons were 2:
 * we were adding every collection to m_expandedCollections all the
   time, which then resulted in their expansion in each slotFilter()
 * we were doing auto-expansion even in cases where collection was
   updated, not just when user typed something

(fixed in is still 2.9 for consistency with earlier commits)

BACKPORT
FIXED-IN: 2.9

M  +1    -0    ChangeLog
M  +21   -16   src/browsers/CollectionTreeItemModelBase.cpp
M  +8    -3    src/browsers/CollectionTreeItemModelBase.h
M  +5    -5    src/browsers/CollectionTreeView.cpp
M  +1    -1    src/browsers/CollectionTreeView.h
M  +1    -1    tests/browsers/TestSingleCollectionTreeItemModel.cpp

http://commits.kde.org/amarok/3b5c6b2039ad943ae209341b5920fee929da9320</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1632810</commentid>
    <comment_count>10</comment_count>
    <who name="Matěj Laitl">matej</who>
    <bug_when>2016-10-16 22:09:10 +0000</bug_when>
    <thetext>Git commit 9fb00e5a97574af3f0cb22d740da7e0cd0761d78 by Matěj Laitl.
Committed on 16/10/2016 at 22:03.
Pushed by laitl into branch &apos;master&apos;.

Fix Collection Browser auto-expand after search expanding too little

Underlying model is asynchronous, we need to cope with that. Do it
using a RAII self-destructing object AutoExpander.

As a side-effect, Amarok no longer expands collection after start in
split collection mode. If this is desired, we may expand them
explicitly upon start.
Related: bug 335217
FIXED-IN: 2.9

M  +1    -0    ChangeLog
M  +68   -20   src/browsers/CollectionTreeView.cpp

http://commits.kde.org/amarok/9fb00e5a97574af3f0cb22d740da7e0cd0761d78</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1632902</commentid>
    <comment_count>11</comment_count>
    <who name="Piotr Keplicz">keplicz</who>
    <bug_when>2016-10-17 08:33:52 +0000</bug_when>
    <thetext>Thanks :)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>