Changeset 2913

Timestamp:

Aug 23, 2007, 7:02:33 PM (15 years ago)

Author:

charles

Message:

• simplify the filters a bit.
• fix a bug in filtering complete/incomplete.
• when adding torrents, save a copy in transmission's torrents directory.

Location:

trunk/wx

Files:

• Makefile.am (modified) (1 diff)
• filter.cc (moved) (moved from trunk/wx/torrent-filter.cc) (6 diffs)
• filter.h (moved) (moved from trunk/wx/torrent-filter.h) (1 diff)
• xmission.cc (modified) (11 diffs)

trunk/wx/Makefile.am

@@ -12 +12 @@
 Xmission_SOURCES = \
   speed-stats.cc \
-  torrent-filter.cc \
+  filter.cc \
   torrent-list.cc \
   torrent-stats.cc \

trunk/wx/filter.cc

@@ -20 +20 @@
 
 #include "foreach.h"
-#include "torrent-filter.h"
+#include "filter.h"
 
 int
@@ -30 +30 @@
     switch( s->status )
     {
-        case TR_STATUS_STOPPING:
-        case TR_STATUS_STOPPED:
-        case TR_STATUS_CHECK:
-        case TR_STATUS_CHECK_WAIT:
-            flags |= FLAG_STOPPED;
-            break;
-
         case TR_STATUS_DOWNLOAD:
             flags |= FLAG_LEECHING;
@@ -45 +38 @@
             flags |= FLAG_SEEDING;
             break;
+
+        case TR_STATUS_STOPPING:
+        case TR_STATUS_STOPPED:
+        case TR_STATUS_CHECK:
+        case TR_STATUS_CHECK_WAIT:
+            break;
     }
 
@@ -52 +51 @@
 
     flags |= s->left
-        ? FLAG_DONE
-        : FLAG_NOT_DONE;
+        ? FLAG_INCOMPLETE
+        : FLAG_COMPLETE;
+
+    flags |= FLAG_ALL;
 
     return flags;
@@ -65 +66 @@
     foreach_const( torrents_v, torrents, it ) {
         const int flags = GetFlags( *it );
-        if( flags & FLAG_STOPPED )  ++counts[STOPPED];
-        if( flags & FLAG_LEECHING ) ++counts[LEECHING];
-        if( flags & FLAG_SEEDING )  ++counts[SEEDING];
-        if( flags & FLAG_ACTIVE )   ++counts[ACTIVE];
-        if( flags & FLAG_IDLE )     ++counts[IDLE];
-        if( flags & FLAG_DONE )     ++counts[DONE];
-        if( flags & FLAG_NOT_DONE ) ++counts[NOT_DONE];
+        if( flags & FLAG_ALL )        ++counts[ALL];
+        if( flags & FLAG_LEECHING )   ++counts[LEECHING];
+        if( flags & FLAG_SEEDING )    ++counts[SEEDING];
+        if( flags & FLAG_ACTIVE )     ++counts[ACTIVE];
+        if( flags & FLAG_IDLE )       ++counts[IDLE];
+        if( flags & FLAG_COMPLETE )   ++counts[COMPLETE];
+        if( flags & FLAG_INCOMPLETE ) ++counts[INCOMPLETE];
     }
 }
@@ -78 +79 @@
 TorrentFilter :: GetName( int show, int count )
 {
-    wxString xstr;
+    static const wxString names[N_FILTERS] = {
+        _("&All"),
+        _("&Complete"),
+        _("&Incomplete"),
+        _("&Seeding"),
+        _("&Leeching"),
+        _("Acti&ve"),
+        _("I&dle")
+    };
 
-    switch( show )
-    {
-        case SEEDING:  xstr = _("&Seeds");  break;
-        case LEECHING: xstr = _("&Leeches"); break;
-        case STOPPED:  xstr = _("Sto&pped");  break;
-        case ACTIVE:   xstr = _("&Active");   break;
-        case IDLE:     xstr = _("&Idle");     break;
-        case DONE:     xstr = _("&Done");     break;
-        case NOT_DONE: xstr = _("&Not Done"); break;
-        default: assert(0);
-    }
+    assert( 0<=show && show<N_FILTERS );
 
-    xstr += wxString::Format(_T(" (%d)"), count );
-
+    wxString xstr = names[show];
+    if( count )
+        xstr += wxString::Format(_T(" (%d)"), count );
     return xstr;
 }
 
-
 void
-TorrentFilter :: RemoveFailures( int flags, torrents_v& torrents )
+TorrentFilter :: RemoveFailures( int            show,
+                                 torrents_v  &  torrents )
 {
     torrents_v tmp;
 
-    for( torrents_v::iterator it(torrents.begin()), end(torrents.end()); it!=end; ++it )
-        if( flags & GetFlags ( *it ) )
+    foreach_const( torrents_v, torrents, it ) {
+        const int flags = GetFlags( *it );
+        if(   ( ( show == ALL )        && ( flags & FLAG_ALL ) )
+           || ( ( show == LEECHING )   && ( flags & FLAG_LEECHING ) )
+           || ( ( show == SEEDING )    && ( flags & FLAG_SEEDING ) )
+           || ( ( show == ACTIVE )     && ( flags & FLAG_ACTIVE ) )
+           || ( ( show == IDLE )       && ( flags & FLAG_IDLE ) )
+           || ( ( show == COMPLETE )   && ( flags & FLAG_COMPLETE ) )
+           || ( ( show == INCOMPLETE ) && ( flags & FLAG_INCOMPLETE ) ) )
             tmp.push_back( *it );
+    }
 
     torrents.swap( tmp );

trunk/wx/filter.h

@@ -28 +28 @@
 class TorrentFilter
 {
+    private:
+
+        enum ShowFlags
+        {
+            FLAG_ALL        = (1<<0),
+            FLAG_COMPLETE   = (1<<1),
+            FLAG_INCOMPLETE = (1<<2),
+            FLAG_SEEDING    = (1<<3),
+            FLAG_LEECHING   = (1<<4),
+            FLAG_ACTIVE     = (1<<5),
+            FLAG_IDLE       = (1<<6)
+        };
+
     public:
 
         typedef std::vector<tr_torrent_t*> torrents_v;
 
-        enum ShowFlags
-        {
-            FLAG_SEEDING    = (1<<0),
-            FLAG_LEECHING   = (1<<1),
-            FLAG_STOPPED    = (1<<2),
-
-            FLAG_ACTIVE     = (1<<3),
-            FLAG_IDLE       = (1<<4),
-
-            FLAG_DONE       = (1<<5),
-            FLAG_NOT_DONE   = (1<<6)
-        };
-
         enum Show {
-            SEEDING, LEECHING, STOPPED,
+            ALL,
+            COMPLETE, INCOMPLETE,
+            SEEDING, LEECHING,
             ACTIVE, IDLE,
-            DONE, NOT_DONE,
             N_FILTERS
         };

trunk/wx/xmission.cc

@@ -65 +65 @@
 #include "foreach.h"
 #include "speed-stats.h"
-#include "torrent-filter.h"
+#include "filter.h"
 #include "torrent-list.h"
 #include "torrent-stats.h"
@@ -169 +169 @@
     void OnDeselectAllUpdate( wxUpdateUIEvent& );
 
-    void OnFilterToggled( wxCommandEvent& );
+    void OnFilterChanged( wxCommandEvent& );
 
     void OnPulse( wxTimerEvent& );
@@ -193 +193 @@
     torrents_v myTorrents;
     torrents_v mySelectedTorrents;
-    int myFilterFlags;
+    int myFilter;
     std::string mySavePath;
     time_t myExitTime;
-    wxToggleButton * myFilterToggles[TorrentFilter::N_FILTERS];
+    wxToggleButton* myFilterButtons[TorrentFilter::N_FILTERS];
 
 private:
@@ -213 +213 @@
 
 BEGIN_EVENT_TABLE(MyFrame, wxFrame)
-    EVT_COMMAND_RANGE( ID_Filter, ID_Filter+TorrentFilter::N_FILTERS, wxEVT_COMMAND_TOGGLEBUTTON_CLICKED, MyFrame::OnFilterToggled )
+    EVT_COMMAND_RANGE( ID_Filter, ID_Filter+TorrentFilter::N_FILTERS, wxEVT_COMMAND_TOGGLEBUTTON_CLICKED, MyFrame::OnFilterChanged )
     EVT_MENU     ( wxID_ABOUT, MyFrame::OnAbout )
     EVT_TIMER    ( ID_Pulse, MyFrame::OnPulse )
@@ -236 +236 @@
 
 void
-MyFrame :: OnFilterToggled( wxCommandEvent& e )
-{
-    const int index = e.GetId() - ID_Filter;
-    int flags = 1<<index;
-    const bool active = myFilterToggles[index]->GetValue ( );
-    if( active )
-        myFilterFlags |= flags;
-    else
-        myFilterFlags &= ~flags;
-    ApplyCurrentFilter ( );
+MyFrame :: OnFilterChanged( wxCommandEvent& e )
+{
+    static bool ignore = false;
+
+    if( !ignore )
+    {
+        myFilter = e.GetId() - ID_Filter;
+        std::cerr << " OnFilterChanged, myFilter is now " << myFilter << std::endl;
+        ApplyCurrentFilter ( );
+
+        ignore = true;
+        for( int i=0; i<TorrentFilter::N_FILTERS; ++i )
+            myFilterButtons[i]->SetValue( i==myFilter );
+        ignore = false;
+    }
 }
 
@@ -367 +372 @@
                                                  filename.c_str(),
                                                  mySavePath.c_str(),
-                                                 0, NULL );
+                                                 TR_FLAG_SAVE, NULL );
             if( tor )
                 myTorrents.push_back( tor );
@@ -422 +427 @@
     TorrentFilter::CountHits( myTorrents, hits );
     for( int i=0; i<TorrentFilter::N_FILTERS; ++i )
-        myFilterToggles[i]->SetLabel( TorrentFilter::GetName( i, hits[i] ) );
+        myFilterButtons[i]->SetLabel( TorrentFilter::GetName( i, hits[i] ) );
 }
 
@@ -429 +434 @@
 {
     torrents_v tmp( myTorrents );
-    TorrentFilter :: RemoveFailures( myFilterFlags, tmp );
+    TorrentFilter :: RemoveFailures( myFilter, tmp );
     myTorrentList->Assign( tmp );
 }
@@ -495 +500 @@
     myTrayIconIcon( systray_xpm ),
     mySpeedStats( 0 ),
-    myFilterFlags( ~0 ),
+    myFilter( TorrentFilter::ALL ),
     myExitTime( 0 )
 {
@@ -612 +617 @@
     wxPanel * panel_1 = new wxPanel( hsplit, wxID_ANY );
 
-    wxBoxSizer * buttonSizer = new wxBoxSizer( wxHORIZONTAL );
-
-    wxStaticText * text = new wxStaticText( panel_1, wxID_ANY, _("Show:") );
-    buttonSizer->Add( text, 0, wxALIGN_CENTER|wxLEFT|wxRIGHT, 3 );
-
-    int rightButtonSpacing[TorrentFilter::N_FILTERS] = { 0, 0, 10, 0, 10, 0, 0 };
+    wxBoxSizer * buttonSizer = new wxBoxSizer( wxVERTICAL );
+
     for( int i=0; i<TorrentFilter::N_FILTERS; ++i ) {
-        wxToggleButton * tb = new wxToggleButton( panel_1, ID_Filter+i, TorrentFilter::GetName(i) );
-        tb->SetValue( true );
-        myFilterToggles[i] = tb;
-        //buttonSizer->Add( tb, 0, wxRIGHT, rightButtonSpacing[i] );
-        buttonSizer->Add( tb, 1, wxEXPAND|wxRIGHT, rightButtonSpacing[i] );
+        wxToggleButton * b = new wxToggleButton( panel_1, ID_Filter+i, TorrentFilter::GetName(i), wxDefaultPosition, wxDefaultSize, (i==0)?wxRB_GROUP:0 );
+        b->SetValue( i==0 );
+        myFilterButtons[i] = b;
+        buttonSizer->Add( b, 1, wxGROW, 0 );
     }
 
@@ -629 +629 @@
     myTorrentList->AddListener( this );
 
-    wxBoxSizer * panelSizer = new wxBoxSizer( wxVERTICAL );
-    panelSizer->Add( new wxStaticLine( panel_1 ), 0, wxEXPAND, 0 );
-    panelSizer->Add( buttonSizer, 0, 0, 0 );
-    panelSizer->Add( myTorrentList, 1, wxEXPAND, 0 );
+    wxBoxSizer * panelSizer = new wxBoxSizer( wxHORIZONTAL );
+    panelSizer->Add( buttonSizer, 0, wxALL, 5 );
+    panelSizer->Add( myTorrentList, 1, wxGROW|wxALL, 5 );
 
     panel_1->SetSizer( panelSizer );