<?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>324295</bug_id>
          
          <creation_ts>2013-08-30 21:34:34 +0000</creation_ts>
          <short_desc>Maintain setting for &quot;show only matches&quot; option</short_desc>
          <delta_ts>2014-05-21 10:46:23 +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>Playlist</component>
          <version>unspecified</version>
          <rep_platform>unspecified</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>2.9</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="hvm">hvm2hvm</reporter>
          <assigned_to name="Amarok Bugs">amarok-bugs-null</assigned_to>
          <cc>teo</cc>
          
          <cf_commitlink>http://commits.kde.org/amarok/43f66b35d4e389044970565deacef13fa6470842</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1393853</commentid>
    <comment_count>0</comment_count>
    <who name="hvm">hvm2hvm</who>
    <bug_when>2013-08-30 21:34:34 +0000</bug_when>
    <thetext>The option to only show matches on playlist search is reset at each amarok startup. I think it should be saved (if you close amarok with the option on it stays on and if off it stays off).

Reproducible: Always</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1393973</commentid>
    <comment_count>1</comment_count>
    <who name="hvm">hvm2hvm</who>
    <bug_when>2013-08-31 07:30:21 +0000</bug_when>
    <thetext>OK, so the option does get saved but not everytime. AFAICT when I quit amarok manually it gets saved, when I just shutdown the PC it doesn&apos;t get saved.

So... maybe it&apos;s a distribution problem (i.e. depends on the way shutdown is handled) - I&apos;m using Kubuntu 13.04</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1393981</commentid>
    <comment_count>2</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2013-08-31 08:28:03 +0000</bug_when>
    <thetext>And which exact Amarok version do you use? This might well be solved in 2.8 already</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1393985</commentid>
    <comment_count>3</comment_count>
    <who name="hvm">hvm2hvm</who>
    <bug_when>2013-08-31 08:35:40 +0000</bug_when>
    <thetext>It&apos;s Amarok 2.7 but like I said, I just started amarok and the setting was saved. I don&apos;t know exactly why it dind&apos;t earlier but if it happens again I will come back with details.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1394094</commentid>
    <comment_count>4</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2013-08-31 18:45:13 +0000</bug_when>
    <thetext>You should upgrade, there have been improvements on the session saving in 2.8</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1394168</commentid>
    <comment_count>5</comment_count>
    <who name="Mark Kretschmann">kretschmann</who>
    <bug_when>2013-09-01 08:19:48 +0000</bug_when>
    <thetext>Can you point me to the relevant commits or discussion please? I haven&apos;t heard of any such changes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1394361</commentid>
    <comment_count>6</comment_count>
    <who name="Mark Kretschmann">kretschmann</who>
    <bug_when>2013-09-02 09:46:27 +0000</bug_when>
    <thetext>Git commit 744b516c9c47643ab6fb2592371c0016b3fae629 by Mark Kretschmann.
Committed on 02/09/2013 at 09:45.
Pushed by markey into branch &apos;master&apos;.

Sync playlist search option right after setting.

M  +3    -1    src/playlist/ProgressiveSearchWidget.cpp

http://commits.kde.org/amarok/744b516c9c47643ab6fb2592371c0016b3fae629</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1394364</commentid>
    <comment_count>7</comment_count>
    <who name="Mark Kretschmann">kretschmann</who>
    <bug_when>2013-09-02 09:50:08 +0000</bug_when>
    <thetext>Git commit cbae195a5e784833644cf73290f93ad44cc3adcd by Mark Kretschmann.
Committed on 02/09/2013 at 09:48.
Pushed by markey into branch &apos;master&apos;.

Revert &quot;Sync playlist search option right after setting.&quot;

This reverts commit 744b516c9c47643ab6fb2592371c0016b3fae629.

The patch was complete nonsense, sorry.

M  +1    -3    src/playlist/ProgressiveSearchWidget.cpp

http://commits.kde.org/amarok/cbae195a5e784833644cf73290f93ad44cc3adcd</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1394370</commentid>
    <comment_count>8</comment_count>
    <who name="Mark Kretschmann">kretschmann</who>
    <bug_when>2013-09-02 10:56:23 +0000</bug_when>
    <thetext>Git commit e44eedd6c42526d4e4cdc14d1bdf18e6660a46ac by Mark Kretschmann.
Committed on 02/09/2013 at 10:55.
Pushed by markey into branch &apos;master&apos;.

Sync playlist search config instantly.

M  +5    -6    src/playlist/ProgressiveSearchWidget.cpp
M  +0    -5    src/playlist/ProgressiveSearchWidget.h

http://commits.kde.org/amarok/e44eedd6c42526d4e4cdc14d1bdf18e6660a46ac</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450131</commentid>
    <comment_count>9</comment_count>
    <who name="vedant agarwala">vedant.kota</who>
    <bug_when>2014-05-21 10:41:38 +0000</bug_when>
    <thetext>Git commit fcf3dbb95158951045b35f4534cfccf3a0e91506 by Vedant Agarwala, on behalf of Mark Kretschmann.
Committed on 02/09/2013 at 09:45.
Pushed by vedanta into branch &apos;tagguessing&apos;.

Sync playlist search option right after setting.

M  +3    -1    src/playlist/ProgressiveSearchWidget.cpp

http://commits.kde.org/amarok/fcf3dbb95158951045b35f4534cfccf3a0e91506</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450137</commentid>
    <comment_count>10</comment_count>
    <who name="vedant agarwala">vedant.kota</who>
    <bug_when>2014-05-21 10:45:21 +0000</bug_when>
    <thetext>Git commit 43f66b35d4e389044970565deacef13fa6470842 by Vedant Agarwala, on behalf of Mark Kretschmann.
Committed on 02/09/2013 at 10:55.
Pushed by vedanta into branch &apos;tagguessing&apos;.

Sync playlist search config instantly.

M  +5    -6    src/playlist/ProgressiveSearchWidget.cpp
M  +0    -5    src/playlist/ProgressiveSearchWidget.h

http://commits.kde.org/amarok/43f66b35d4e389044970565deacef13fa6470842</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450141</commentid>
    <comment_count>11</comment_count>
    <who name="vedant agarwala">vedant.kota</who>
    <bug_when>2014-05-21 10:46:23 +0000</bug_when>
    <thetext>Git commit c8df38f04f9d73ab9ebe4a3a51d8e29edf1d467a by Vedant Agarwala, on behalf of Mark Kretschmann.
Committed on 02/09/2013 at 09:48.
Pushed by vedanta into branch &apos;tagguessing&apos;.

Revert &quot;Sync playlist search option right after setting.&quot;

This reverts commit 744b516c9c47643ab6fb2592371c0016b3fae629.

The patch was complete nonsense, sorry.

M  +1    -3    src/playlist/ProgressiveSearchWidget.cpp

http://commits.kde.org/amarok/c8df38f04f9d73ab9ebe4a3a51d8e29edf1d467a</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>