Border corrections

[wxWidgets.git] / src / generic / filectrlg.cpp

diff --git a/src/generic/filectrlg.cpp b/src/generic/filectrlg.cpp
index cfecb2217de3e250aafe3fa06a53412fb4e5dfa4..66ee552da4dba9ae498823c05250d57ff6f6c2f0 100644 (file)
--- a/src/generic/filectrlg.cpp
+++ b/src/generic/filectrlg.cpp
@@ -28,7 +28,6 @@
     #include "wx/filedlg.h"
 #endif
 
     #include "wx/filedlg.h"
 #endif
 

-#include "wx/filename.h"

 #include "wx/clntdata.h"
 #include "wx/file.h"        // for wxS_IXXX constants only
 #include "wx/generic/dirctrlg.h" // for wxFileIconsTable
 #include "wx/clntdata.h"
 #include "wx/file.h"        // for wxS_IXXX constants only
 #include "wx/generic/dirctrlg.h" // for wxFileIconsTable


@@ -224,7 +223,7 @@ void wxFileData::ReadData()
 
 #if defined(__UNIX__) && (!defined( __OS2__ ) && !defined(__VMS))
     lstat( m_filePath.fn_str(), &buff );
 
 #if defined(__UNIX__) && (!defined( __OS2__ ) && !defined(__VMS))
     lstat( m_filePath.fn_str(), &buff );

-    m_type |= S_ISLNK( buff.st_mode ) != 0 ? is_link : 0;
+    m_type |= S_ISLNK(buff.st_mode) ? is_link : 0;

 #else // no lstat()
     // only translate to file charset if we don't go by our
     // wxStat implementation
 #else // no lstat()
     // only translate to file charset if we don't go by our
     // wxStat implementation


@@ -393,8 +392,6 @@ void wxFileData::MakeItem( wxListItem &item )
 //  wxFileListCtrl
 //-----------------------------------------------------------------------------
 
 //  wxFileListCtrl
 //-----------------------------------------------------------------------------
 

-static bool ignoreChanges = false;
-

 IMPLEMENT_DYNAMIC_CLASS(wxFileListCtrl,wxListCtrl)
 
 BEGIN_EVENT_TABLE(wxFileListCtrl,wxListCtrl)
 IMPLEMENT_DYNAMIC_CLASS(wxFileListCtrl,wxListCtrl)
 
 BEGIN_EVENT_TABLE(wxFileListCtrl,wxListCtrl)


@@ -408,7 +405,7 @@ END_EVENT_TABLE()
 wxFileListCtrl::wxFileListCtrl()
 {
     m_showHidden = false;
 wxFileListCtrl::wxFileListCtrl()
 {
     m_showHidden = false;

-    m_sort_foward = 1;
+    m_sort_forward = true;

     m_sort_field = wxFileData::FileList_Name;
 }
 
     m_sort_field = wxFileData::FileList_Name;
 }
 


@@ -430,7 +427,7 @@ wxFileListCtrl::wxFileListCtrl(wxWindow *win,
 
     m_showHidden = showHidden;
 
 
     m_showHidden = showHidden;
 

-    m_sort_foward = 1;
+    m_sort_forward = true;

     m_sort_field = wxFileData::FileList_Name;
 
     m_dirName = wxT("*");
     m_sort_field = wxFileData::FileList_Name;
 
     m_dirName = wxT("*");


@@ -541,11 +538,23 @@ void wxFileListCtrl::UpdateFiles()
     {
         wxArrayString names, paths;
         wxArrayInt icons;
     {
         wxArrayString names, paths;
         wxArrayInt icons;

-        size_t n, count = wxGetAvailableDrives(paths, names, icons);
+        const size_t count = wxGetAvailableDrives(paths, names, icons);

 
 

-        for (n=0; n<count; n++)
+        for ( size_t n = 0; n < count; n++ )

         {
         {

-            wxFileData *fd = new wxFileData(paths[n], names[n], wxFileData::is_drive, icons[n]);
+            // use paths[n] as the drive name too as our HandleAction() can't
+            // deal with the drive names (of the form "System (C:)") currently
+            // as it mistakenly treats them as file names
+            //
+            // it would be preferable to show names, and not paths, in the
+            // dialog just as the native dialog does but for this we must:
+            //  a) store the item type as item data and modify HandleAction()
+            //     to use it instead of wxDirExists() to check whether the item
+            //     is a directory
+            //  b) store the drives by their drive letters and not their
+            //     descriptions as otherwise it's pretty confusing to the user
+            wxFileData *fd = new wxFileData(paths[n], paths[n],
+                                            wxFileData::is_drive, icons[n]);

             if (Add(fd, item) != -1)
                 item.m_itemId++;
             else
             if (Add(fd, item) != -1)
                 item.m_itemId++;
             else


@@ -626,7 +635,7 @@ void wxFileListCtrl::UpdateFiles()
         }
     }
 
         }
     }
 

-    SortItems(m_sort_field, m_sort_foward);
+    SortItems(m_sort_field, m_sort_forward);

 }
 
 void wxFileListCtrl::SetWild( const wxString &wild )
 }
 
 void wxFileListCtrl::SetWild( const wxString &wild )


@@ -677,7 +686,7 @@ void wxFileListCtrl::MakeDir()
 
     if (id != -1)
     {
 
     if (id != -1)
     {

-        SortItems(m_sort_field, m_sort_foward);
+        SortItems(m_sort_field, m_sort_forward);

         id = FindItem( 0, wxPtrToUInt(fd) );
         EnsureVisible( id );
         EditLabel( id );
         id = FindItem( 0, wxPtrToUInt(fd) );
         EnsureVisible( id );
         EditLabel( id );


@@ -709,10 +718,8 @@ void wxFileListCtrl::GoToParentDir()
         long id = FindItem( 0, fname );
         if (id != wxNOT_FOUND)
         {
         long id = FindItem( 0, fname );
         if (id != wxNOT_FOUND)
         {

-            ignoreChanges = true;

             SetItemState( id, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED );
             EnsureVisible( id );
             SetItemState( id, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED );
             EnsureVisible( id );

-            ignoreChanges = false;

         }
     }
 }
         }
     }
 }


@@ -730,9 +737,7 @@ void wxFileListCtrl::GoToDir( const wxString &dir )
     m_dirName = dir;
     UpdateFiles();
 
     m_dirName = dir;
     UpdateFiles();
 

-    ignoreChanges = true;

     SetItemState( 0, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED );
     SetItemState( 0, wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED );

-    ignoreChanges = false;

 
     EnsureVisible( 0 );
 }
 
     EnsureVisible( 0 );
 }


@@ -805,9 +810,7 @@ void wxFileListCtrl::OnListEndLabelEdit( wxListEvent &event )
     {
         fd->SetNewName( new_name, event.GetLabel() );
 
     {
         fd->SetNewName( new_name, event.GetLabel() );
 

-        ignoreChanges = true;

         SetItemState( event.GetItem(), wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED );
         SetItemState( event.GetItem(), wxLIST_STATE_SELECTED, wxLIST_STATE_SELECTED );

-        ignoreChanges = false;

 
         UpdateItem( event.GetItem() );
         EnsureVisible( event.GetItem() );
 
         UpdateItem( event.GetItem() );
         EnsureVisible( event.GetItem() );


@@ -834,17 +837,17 @@ void wxFileListCtrl::OnListColClick( wxListEvent &event )
     }
 
     if ((wxFileData::fileListFieldType)col == m_sort_field)
     }
 
     if ((wxFileData::fileListFieldType)col == m_sort_field)

-        m_sort_foward = !m_sort_foward;
+        m_sort_forward = !m_sort_forward;

     else
         m_sort_field = (wxFileData::fileListFieldType)col;
 
     else
         m_sort_field = (wxFileData::fileListFieldType)col;
 

-    SortItems(m_sort_field, m_sort_foward);
+    SortItems(m_sort_field, m_sort_forward);

 }
 
 void wxFileListCtrl::SortItems(wxFileData::fileListFieldType field, bool forward)
 {
     m_sort_field = field;
 }
 
 void wxFileListCtrl::SortItems(wxFileData::fileListFieldType field, bool forward)
 {
     m_sort_field = field;

-    m_sort_foward = forward;
+    m_sort_forward = forward;

     const long sort_dir = forward ? 1 : -1;
 
     switch (m_sort_field)
     const long sort_dir = forward ? 1 : -1;
 
     switch (m_sort_field)


@@ -1027,67 +1030,102 @@ bool wxGenericFileCtrl::Create( wxWindow *parent,
     return true;
 }
 
     return true;
 }
 

+// NB: there is an unfortunate mismatch between wxFileName and wxFileDialog
+//     method names but our GetDirectory() does correspond to wxFileName::
+//     GetPath() while our GetPath() is wxFileName::GetFullPath()

 wxString wxGenericFileCtrl::GetPath() const
 {
 wxString wxGenericFileCtrl::GetPath() const
 {

-    return DoGetFilename( true );
+    wxASSERT_MSG ( !(m_style & wxFC_MULTIPLE), "use GetPaths() instead" );
+
+    return DoGetFileName().GetFullPath();

 }
 
 wxString wxGenericFileCtrl::GetFilename() const
 {
 }
 
 wxString wxGenericFileCtrl::GetFilename() const
 {

-    return DoGetFilename( false );
+    wxASSERT_MSG ( !(m_style & wxFC_MULTIPLE), "use GetFilenames() instead" );
+
+    return DoGetFileName().GetFullName();
+}
+
+wxString wxGenericFileCtrl::GetDirectory() const
+{
+    // don't check for wxFC_MULTIPLE here, this one is probably safe to call in
+    // any case as it can be always taken to mean "current directory"
+    return DoGetFileName().GetPath();

 }
 
 }
 

-wxString wxGenericFileCtrl::DoGetFilename( const bool fullPath ) const
+wxFileName wxGenericFileCtrl::DoGetFileName() const

 {
 {

-    wxASSERT_MSG( ( m_style & wxFC_MULTIPLE ) == 0,
-                  wxT( "With controls that has wxFC_MULTIPLE style " )
-                  wxT( "use GetFilenames/GetPaths to get all filenames/paths selected" ) );
+    wxFileName fn;

 
 

-    const wxString value = m_text->GetValue();
+    wxString value = m_text->GetValue();
+    if ( value.empty() )
+    {
+        // nothing in the text control, get the selected file from the list
+        wxListItem item;
+        item.m_itemId = m_list->GetNextItem(-1, wxLIST_NEXT_ALL,
+                                            wxLIST_STATE_SELECTED);
+        m_list->GetItem(item);

 
 

-    if ( !value.empty() )
-        return value;
-    return fullPath ? ( GetProperFileListDir() + value ) : value;
+        fn.Assign(m_list->GetDir(), item.m_text);
+    }
+    else // user entered the value
+    {
+        // the path can be either absolute or relative
+        fn.Assign(value);
+        if ( fn.IsRelative() )
+            fn.MakeAbsolute(m_list->GetDir());
+    }
+
+    return fn;
+}
+
+// helper used in DoGetFilenames() and needed because Borland can't compile
+// operator?: inline
+static inline wxString GetFileNameOrPath(const wxFileName& fn, bool fullPath)
+{
+    return fullPath ? fn.GetFullPath() : fn.GetFullName();

 }
 
 }
 

-void wxGenericFileCtrl::DoGetFilenames( wxArrayString& filenames, const bool fullPath ) const
+void
+wxGenericFileCtrl::DoGetFilenames(wxArrayString& filenames, bool fullPath) const

 {
 {

-    filenames.Empty();
+    filenames.clear();

 
 

-    const wxString dir = GetProperFileListDir();
-    const wxString value = m_text->GetValue();
+    const wxString dir = m_list->GetDir();

 
 

+    const wxString value = m_text->GetValue();

     if ( !value.empty() )
     {
     if ( !value.empty() )
     {

-        if ( fullPath )
-            filenames.Add( dir + value );
-        else
-            filenames.Add( value );
+        wxFileName fn(value);
+        if ( fn.IsRelative() )
+            fn.MakeAbsolute(dir);
+
+        filenames.push_back(GetFileNameOrPath(fn, fullPath));

         return;
     }
 
         return;
     }
 

-    if ( m_list->GetSelectedItemCount() == 0 )
-    {
+    const int numSel = m_list->GetSelectedItemCount();
+    if ( !numSel )

         return;
         return;

-    }

 
 

-    filenames.Alloc( m_list->GetSelectedItemCount() );
+    filenames.reserve(numSel);

 
     wxListItem item;
     item.m_mask = wxLIST_MASK_TEXT;
 
     wxListItem item;
     item.m_mask = wxLIST_MASK_TEXT;

-
-    item.m_itemId = m_list->GetNextItem( -1, wxLIST_NEXT_ALL, wxLIST_STATE_SELECTED );
-    while ( item.m_itemId != -1 )
+    item.m_itemId = -1;
+    for ( ;; )

     {
     {

-        m_list->GetItem( item );
+        item.m_itemId = m_list->GetNextItem(item.m_itemId, wxLIST_NEXT_ALL,
+                                            wxLIST_STATE_SELECTED);

 
 

-        if ( fullPath )
-            filenames.Add( dir + item.m_text );
-        else
-            filenames.Add( item.m_text );
+        if ( item.m_itemId == -1 )
+            break;
+
+        m_list->GetItem(item);

 
 

-        item.m_itemId = m_list->GetNextItem( item.m_itemId,
-                                             wxLIST_NEXT_ALL, wxLIST_STATE_SELECTED );
+        const wxFileName fn(dir, item.m_text);
+        filenames.push_back(GetFileNameOrPath(fn, fullPath));

     }
 }
 
     }
 }
 


@@ -1101,11 +1139,6 @@ bool wxGenericFileCtrl::SetDirectory( const wxString& dir )
     return wxFileName( dir ).SameAs( m_list->GetDir() );
 }
 
     return wxFileName( dir ).SameAs( m_list->GetDir() );
 }
 

-wxString wxGenericFileCtrl::GetDirectory() const
-{
-    return m_list->GetDir();
-}
-

 bool wxGenericFileCtrl::SetFilename( const wxString& name )
 {
     const long item = m_list->FindItem( -1, name );
 bool wxGenericFileCtrl::SetFilename( const wxString& name )
 {
     const long item = m_list->FindItem( -1, name );


@@ -1440,17 +1473,4 @@ void wxGenericFileCtrl::GoToHomeDir()
     UpdateControls();
 }
 
     UpdateControls();
 }
 

-wxString wxGenericFileCtrl::GetProperFileListDir() const
-{
-    wxString dir = m_list->GetDir();
-#ifdef __UNIX__
-    if ( dir != wxT( "/" ) )
-#elif defined(__WXWINCE__)
-    if ( dir != wxT( "/" ) && dir != wxT( "\\" ) )
-#endif
-        dir += wxFILE_SEP_PATH;
-
-    return dir;
-}
-

 #endif // wxUSE_FILECTRL
 #endif // wxUSE_FILECTRL