Bug 148084 - Cannot limit smart playlists to more than 1000 tracks
Summary: Cannot limit smart playlists to more than 1000 tracks
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: Collections/Media Devices (show other bugs)
Version: 1.4.6
Platform: NetBSD pkgsrc NetBSD
: NOR wishlist
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-21 15:08 UTC by shattered
Modified: 2007-07-21 15:35 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
proposed fix (522 bytes, patch)
2007-07-21 15:09 UTC, shattered 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description shattered 2007-07-21 15:08:17 UTC 
Version:           1.4.6 (using KDE KDE 3.5.7)
Installed from:    NetBSD pkgsrc
OS:                NetBSD

To work around the bug 144041, I tried to limit smart playlist size to 1500 tracks (more or less fits on 8GB ipod).  This isn't possible in the UI -- the upper limit is 1000.
Comment 1 shattered 2007-07-21 15:09:18 UTC 
Created attachment 21215 [details]
proposed fix
Comment 2 Seb Ruiz 2007-07-21 15:35:49 UTC 
SVN commit 690583 by seb:

Maximum limit for smart playlists has been bumped from 1000 to 100000 tracks
BUG: 148084


 M  +1 -1      smartplaylisteditor.cpp  


--- branches/stable/extragear/multimedia/amarok/src/smartplaylisteditor.cpp #690582:690583
@@ -221,7 +221,7 @@
     limitBox->setSpacing( 5 );
     m_limitSpin = new KIntSpinBox( limitBox );
     m_limitSpin->setMinValue( 1 );
-    m_limitSpin->setMaxValue( 1000 );
+    m_limitSpin->setMaxValue( 100000 );
     m_limitSpin->setValue( 15 );
     new QLabel( i18n("tracks"), limitBox );
     hbox1->setStretchFactor( new QWidget( hbox1 ), 1 );