]> git.saurik.com Git - wxWidgets.git/blobdiff - src/osx/cocoa/filedlg.mm
Fix bug with out of range date in wxMSW wxDatePickerCtrl under XP too.
[wxWidgets.git] / src / osx / cocoa / filedlg.mm
index b4decc5dcd16a48ec2d467141ec57f5c90067b73..a1c79b7fc51150881b913791457c7765d4123ae2 100644 (file)
@@ -4,7 +4,7 @@
 // Author:      Ryan Norton
 // Modified by:
 // Created:     2004-10-02
-// RCS-ID:      $Id: filedlg.mm 40007 2006-07-05 13:10:46Z SC $
+// RCS-ID:      $Id$
 // Copyright:   (c) Ryan Norton
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 #ifndef WX_PRECOMP
     #include "wx/msgdlg.h"
     #include "wx/app.h"
+    #include "wx/sizer.h"
+    #include "wx/stattext.h"
+    #include "wx/choice.h"
 #endif
 
 #include "wx/filename.h"
 #include "wx/tokenzr.h"
 
 #include "wx/osx/private.h"
+#include "wx/sysopt.h"
 
 // ============================================================================
 // implementation
 // ============================================================================
 
 // Open Items:
-// - support for old style MacOS creator / type combos
 // - parameter support for descending into packages as directories (setTreatsFilePackagesAsDirectories)
+// - as setAllowedFileTypes is only functional for NSOpenPanel on 10.6+, on earlier systems, the file
+// type choice will not be shown, but all possible file items will be shown, if a popup must be working
+// then the delegate method - (BOOL)panel:(id)sender shouldShowFilename:(NSString *)filename will have to
+// be implemented
 
-IMPLEMENT_CLASS(wxFileDialog, wxFileDialogBase)
+@interface wxOpenPanelDelegate : NSObject wxOSX_10_6_AND_LATER(<NSOpenSavePanelDelegate>)
+{
+    wxFileDialog* _dialog;
+}
 
-wxFileDialog::wxFileDialog(
-    wxWindow *parent, const wxString& message,
-    const wxString& defaultDir, const wxString& defaultFileName, const wxString& wildCard,
-    long style, const wxPoint& pos, const wxSize& sz, const wxString& name)
-    : wxFileDialogBase(parent, message, defaultDir, defaultFileName, wildCard, style, pos, sz, name)
+- (wxFileDialog*) fileDialog;
+- (void) setFileDialog:(wxFileDialog*) dialog;
+
+- (BOOL)panel:(id)sender shouldShowFilename:(NSString *)filename;
+
+@end
+
+@implementation wxOpenPanelDelegate
+
+- (id) init
 {
+    self = [super init];
+    _dialog = NULL;
+    return self;
 }
 
-NSArray* GetTypesFromFilter( const wxString filter )
+- (wxFileDialog*) fileDialog
 {
-    NSMutableArray* types = nil;
-    if ( !filter.empty() )
-    {
-        wxArrayString names ;
-        wxArrayString extensions;
+    return _dialog;
+}
 
-        wxString filter2(filter) ;
-        int filterIndex = 0;
-        bool isName = true ;
-        wxString current ;
+- (void) setFileDialog:(wxFileDialog*) dialog
+{
+    _dialog = dialog;
+}
 
-        for ( unsigned int i = 0; i < filter2.length() ; i++ )
+- (BOOL)panel:(id)sender shouldShowFilename:(NSString *)filename
+{
+    BOOL showObject = YES;
+    
+    NSString* resolvedLink = [[NSFileManager defaultManager] pathContentOfSymbolicLinkAtPath:filename];
+    if ( resolvedLink != nil )
+        filename = resolvedLink;
+    
+    NSDictionary* fileAttribs = [[NSFileManager defaultManager]
+                                 fileAttributesAtPath:filename traverseLink:YES];
+    if (fileAttribs)
+    {
+        // check for packages
+        if ([NSFileTypeDirectory isEqualTo:[fileAttribs objectForKey:NSFileType]])
         {
-            if ( filter2.GetChar(i) == wxT('|') )
+            if ([[NSWorkspace sharedWorkspace] isFilePackageAtPath:filename] == NO)
+                showObject = YES;    // it's a folder, OK to show
+            else
             {
-                if ( isName )
-                {
-                    names.Add( current ) ;
-                }
-                else
+                // it's a packaged directory, apply check
+                wxCFStringRef filecf([filename retain]);
+                showObject = _dialog->CheckFile(filecf.AsString());  
+            }
+        }
+        else
+        {
+            // the code above only solves links, not aliases, do this here:
+            
+            NSString* resolvedAlias = nil;
+            
+            CFURLRef url = CFURLCreateWithFileSystemPath (kCFAllocatorDefault, 
+                                                          (CFStringRef)filename, 
+                                                          kCFURLPOSIXPathStyle,
+                                                          NO); 
+            if (url != NULL) 
+            {
+                FSRef fsRef; 
+                if (CFURLGetFSRef(url, &fsRef)) 
                 {
-                    extensions.Add( current ) ;
-                    ++filterIndex ;
+                    Boolean targetIsFolder, wasAliased;
+                    OSErr err = FSResolveAliasFile (&fsRef, true, &targetIsFolder, &wasAliased);
+                    
+                    if ((err == noErr) && wasAliased) 
+                    {
+                        CFURLRef resolvedUrl = CFURLCreateFromFSRef(kCFAllocatorDefault,  &fsRef);
+                        if (resolvedUrl != NULL) 
+                        {
+                            resolvedAlias = (NSString*) CFURLCopyFileSystemPath(resolvedUrl,
+                                                                               kCFURLPOSIXPathStyle); 
+                            CFRelease(resolvedUrl);
+                        }
+                    } 
                 }
+                CFRelease(url);
+            }
 
-                isName = !isName ;
-                current = wxEmptyString ;
+            if (resolvedAlias != nil) 
+            {
+                // recursive call
+                [resolvedAlias autorelease];
+                showObject = [self panel:sender shouldShowFilename:resolvedAlias];
             }
             else
             {
-                current += filter2.GetChar(i) ;
+                wxCFStringRef filecf([filename retain]);
+                showObject = _dialog->CheckFile(filecf.AsString());  
             }
         }
-        // we allow for compatibility reason to have a single filter expression (like *.*) without
-        // an explanatory text, in that case the first part is name and extension at the same time
+    }
 
-        wxASSERT_MSG( filterIndex == 0 || !isName , wxT("incorrect format of format string") ) ;
-        if ( current.empty() )
-            extensions.Add( names[filterIndex] ) ;
-        else
-            extensions.Add( current ) ;
-        if ( filterIndex == 0 || isName )
-            names.Add( current ) ;
+    return showObject;    
+}
 
-        ++filterIndex ;
+@end
 
-        const size_t extCount = extensions.GetCount();
-        for ( size_t i = 0 ; i < extCount; i++ )
-        {
-            wxString extensiongroup = extensions[i];
-            wxStringTokenizer tokenizer( extensiongroup , wxT(";") ) ;
-            while ( tokenizer.HasMoreTokens() )
-            {
-                wxString extension = tokenizer.GetNextToken() ;
-                // Remove leading '*'
-                if (extension.length() && (extension.GetChar(0) == '*'))
-                    extension = extension.Mid( 1 );
+IMPLEMENT_CLASS(wxFileDialog, wxFileDialogBase)
 
-                // Remove leading '.'
-                if (extension.length() && (extension.GetChar(0) == '.'))
-                    extension = extension.Mid( 1 );
+wxFileDialog::wxFileDialog(
+    wxWindow *parent, const wxString& message,
+    const wxString& defaultDir, const wxString& defaultFileName, const wxString& wildCard,
+    long style, const wxPoint& pos, const wxSize& sz, const wxString& name)
+    : wxFileDialogBase(parent, message, defaultDir, defaultFileName, wildCard, style, pos, sz, name)
+{
+    m_filterIndex = -1;
+    m_sheetDelegate = [[ModalDialogDelegate alloc] init];
+    [(ModalDialogDelegate*)m_sheetDelegate setImplementation: this];
+}
 
-                if ( extension.IsEmpty() )
-                {
-                    if ( types != nil )
-                        [types release];
-                    return nil;
-                }
+wxFileDialog::~wxFileDialog()
+{
+    [m_sheetDelegate release];
+}
 
-                if ( types == nil )
-                    types = [[NSMutableArray alloc] init];
+bool wxFileDialog::SupportsExtraControl() const
+{
+    return true;
+}
 
-                wxCFStringRef cfext(extension);
-                [types addObject: (NSString*)cfext.AsNSString()  ];
+NSArray* GetTypesFromExtension( const wxString extensiongroup, wxArrayString& extensions )
+{
+    NSMutableArray* types = nil;
+    extensions.Clear();
+
+    wxStringTokenizer tokenizer( extensiongroup, wxT(";") ) ;
+    while ( tokenizer.HasMoreTokens() )
+    {
+        wxString extension = tokenizer.GetNextToken() ;
+        // Remove leading '*'
+        if ( extension.length() && (extension.GetChar(0) == '*') )
+            extension = extension.Mid( 1 );
+
+        // Remove leading '.'
+        if ( extension.length() && (extension.GetChar(0) == '.') )
+            extension = extension.Mid( 1 );
+
+        // Remove leading '*', this is for handling *.*
+        if ( extension.length() && (extension.GetChar(0) == '*') )
+            extension = extension.Mid( 1 );
+
+        if ( extension.IsEmpty() )
+        {
+            extensions.Clear();
+            [types release];
+            types = nil;
+            return nil;
+        }
+
+        if ( types == nil )
+            types = [[NSMutableArray alloc] init];
+
+        extensions.Add(extension.Lower());
+        wxCFStringRef cfext(extension);
+        [types addObject: (NSString*)cfext.AsNSString()  ];
 #if 0
-                // add support for classic fileType / creator here
-                wxUint32 fileType, creator;
-                // extension -> mactypes
+        // add support for classic fileType / creator here
+        wxUint32 fileType, creator;
+        // extension -> mactypes
 #endif
+    }
+    [types autorelease];
+    return types;
+}
+
+NSArray* GetTypesFromFilter( const wxString& filter, wxArrayString& names, wxArrayString& extensiongroups )
+{
+    NSMutableArray* types = nil;
+    bool allowAll = false;
+
+    names.Clear();
+    extensiongroups.Clear();
+
+    if ( !filter.empty() )
+    {
+        wxStringTokenizer tokenizer( filter, wxT("|") );
+        int numtokens = (int)tokenizer.CountTokens();
+        if(numtokens == 1)
+        {
+            // we allow for compatibility reason to have a single filter expression (like *.*) without
+            // an explanatory text, in that case the first part is name and extension at the same time
+            wxString extension = tokenizer.GetNextToken();
+            names.Add( extension );
+            extensiongroups.Add( extension );
+        }
+        else
+        {
+            int numextensions = numtokens / 2;
+            for(int i = 0; i < numextensions; i++)
+            {
+                wxString name = tokenizer.GetNextToken();
+                wxString extension = tokenizer.GetNextToken();
+                names.Add( name );
+                extensiongroups.Add( extension );
             }
+        }
+
+        const size_t extCount = extensiongroups.GetCount();
+        wxArrayString extensions;
+        for ( size_t i = 0 ; i < extCount; i++ )
+        {
+            NSArray* exttypes = GetTypesFromExtension(extensiongroups[i], extensions);
+            if ( exttypes != nil )
+            {
+                if ( allowAll == false )
+                {
+                    if ( types == nil )
+                        types = [[NSMutableArray alloc] init];
 
+                    [types addObjectsFromArray:exttypes];
+                }
+            }
+            else
+            {
+                allowAll = true;
+                [types release];
+                types = nil;
+            }
         }
     }
+    [types autorelease];
     return types;
 }
 
@@ -154,10 +296,14 @@ void wxFileDialog::ShowWindowModal()
         parentWindow = dynamic_cast<wxNonOwnedWindow*>(wxGetTopLevelParent(GetParent()));
 
     wxASSERT_MSG(parentWindow, "Window modal display requires parent.");
-    
-    if (HasFlag(wxFD_SAVE))
+
+    NSArray* types = GetTypesFromFilter( m_wildCard, m_filterNames, m_filterExtensions ) ;
+    if ( HasFlag(wxFD_SAVE) )
     {
         NSSavePanel* sPanel = [NSSavePanel savePanel];
+
+        SetupExtraControls(sPanel);
+
         // makes things more convenient:
         [sPanel setCanCreateDirectories:YES];
         [sPanel setMessage:cf.AsNSString()];
@@ -165,40 +311,163 @@ void wxFileDialog::ShowWindowModal()
         // be able to pass this in
         [sPanel setTreatsFilePackagesAsDirectories:NO];
         [sPanel setCanSelectHiddenExtension:YES];
+        [sPanel setAllowedFileTypes:types];
+        [sPanel setAllowsOtherFileTypes:NO];
         
         NSWindow* nativeParent = parentWindow->GetWXWindow();
-        ModalDialogDelegate* sheetDelegate = [[ModalDialogDelegate alloc] init];
-        [sheetDelegate setImplementation: this];
         [sPanel beginSheetForDirectory:dir.AsNSString() file:file.AsNSString()
-            modalForWindow: nativeParent modalDelegate: sheetDelegate
+            modalForWindow: nativeParent modalDelegate: m_sheetDelegate
             didEndSelector: @selector(sheetDidEnd:returnCode:contextInfo:)
             contextInfo: nil];
     }
     else 
     {
-        NSArray* types = GetTypesFromFilter( m_wildCard ) ;
         NSOpenPanel* oPanel = [NSOpenPanel openPanel];
+        
+        SetupExtraControls(oPanel);
+
         [oPanel setTreatsFilePackagesAsDirectories:NO];
         [oPanel setCanChooseDirectories:NO];
         [oPanel setResolvesAliases:YES];
         [oPanel setCanChooseFiles:YES];
         [oPanel setMessage:cf.AsNSString()];
-    
+        [oPanel setAllowsMultipleSelection: (HasFlag(wxFD_MULTIPLE) ? YES : NO )];
+        
         NSWindow* nativeParent = parentWindow->GetWXWindow();
-        ModalDialogDelegate* sheetDelegate = [[ModalDialogDelegate alloc] init];
-        [sheetDelegate setImplementation: this];
         [oPanel beginSheetForDirectory:dir.AsNSString() file:file.AsNSString()
             types: types modalForWindow: nativeParent
-            modalDelegate: sheetDelegate
+            modalDelegate: m_sheetDelegate
             didEndSelector: @selector(sheetDidEnd:returnCode:contextInfo:)
             contextInfo: nil];
     }
 }
 
-int wxFileDialog::ShowModal()
+// Create a panel with the file type drop down list
+// If extra controls need to be added (see wxFileDialog::SetExtraControlCreator), add
+// them to the panel as well
+// Returns the newly created wxPanel
+
+wxWindow* wxFileDialog::CreateFilterPanel(wxWindow *extracontrol)
 {
-    NSSavePanel *panel = nil;
+    wxPanel *extrapanel = new wxPanel(this, wxID_ANY, wxDefaultPosition, wxDefaultSize);
+    wxBoxSizer *verticalSizer = new wxBoxSizer(wxVERTICAL);
+    extrapanel->SetSizer(verticalSizer);
+    
+    // the file type control
+    {
+        wxBoxSizer *horizontalSizer = new wxBoxSizer(wxHORIZONTAL);
+        verticalSizer->Add(horizontalSizer, 0, wxEXPAND, 0);
+        wxStaticText *stattext = new wxStaticText( extrapanel, wxID_ANY, _("File type:") );
+        horizontalSizer->Add(stattext, 0, wxALIGN_CENTER_VERTICAL|wxALL, 5);
+        m_filterChoice = new wxChoice(extrapanel, wxID_ANY);
+        horizontalSizer->Add(m_filterChoice, 1, wxALIGN_CENTER_VERTICAL|wxALL, 5);
+        m_filterChoice->Append(m_filterNames);
+        if( m_filterNames.GetCount() > 0)
+        {
+            if ( m_firstFileTypeFilter >= 0 )
+                m_filterChoice->SetSelection(m_firstFileTypeFilter);
+        }
+        m_filterChoice->Connect(wxEVT_COMMAND_CHOICE_SELECTED, wxCommandEventHandler(wxFileDialog::OnFilterSelected), NULL, this);
+    }
+        
+    if(extracontrol)
+    {
+        wxBoxSizer *horizontalSizer = new wxBoxSizer(wxHORIZONTAL);
+        verticalSizer->Add(horizontalSizer, 0, wxEXPAND, 0);
+
+        extracontrol->Reparent(extrapanel);
+        horizontalSizer->Add(extracontrol);
+    }
+
+    verticalSizer->Layout();
+    verticalSizer->SetSizeHints(extrapanel);
+    return extrapanel;
+}
+
+void wxFileDialog::DoOnFilterSelected(int index)
+{
+    NSArray* types = GetTypesFromExtension(m_filterExtensions[index],m_currentExtensions);
+    NSSavePanel* panel = (NSSavePanel*) GetWXWindow();
+    if ( m_delegate )
+        [panel validateVisibleColumns];
+    else
+        [panel setAllowedFileTypes:types];
+}
+
+// An item has been selected in the file filter wxChoice:
+void wxFileDialog::OnFilterSelected( wxCommandEvent &WXUNUSED(event) )
+{
+    DoOnFilterSelected( m_filterChoice->GetSelection() );
+}
+
+bool wxFileDialog::CheckFile( const wxString& filename )
+{
+    if ( m_currentExtensions.GetCount() == 0 )
+        return true;
+    
+    wxString ext = filename.AfterLast('.').Lower();
+    
+    for ( size_t i = 0; i < m_currentExtensions.GetCount(); ++i )
+    {
+        if ( ext == m_currentExtensions[i] )
+            return true;
+    }
+    return false;
+}
+
+void wxFileDialog::SetupExtraControls(WXWindow nativeWindow)
+{
+    NSSavePanel* panel = (NSSavePanel*) nativeWindow;
+    
+    wxNonOwnedWindow::Create( GetParent(), nativeWindow );
+    wxWindow* extracontrol = NULL;
+    if ( HasExtraControlCreator() )
+    {
+        CreateExtraControl();
+        extracontrol = GetExtraControl();
+    }
+
+    NSView* accView = nil;
+    m_delegate = nil;
+
+    if ( m_useFileTypeFilter )
+    {
+        m_filterPanel = CreateFilterPanel(extracontrol);
+        accView = m_filterPanel->GetHandle();
+        if( HasFlag(wxFD_OPEN) )
+        {
+            if ( UMAGetSystemVersion() < 0x1060 )
+            {
+                wxOpenPanelDelegate* del = [[wxOpenPanelDelegate alloc]init];
+                [del setFileDialog:this];
+                [panel setDelegate:del];
+                m_delegate = del;
+            }
+        }
+    }
+    else
+    {
+        m_filterPanel = NULL;
+        m_filterChoice = NULL;
+        if ( extracontrol != nil )
+            accView = extracontrol->GetHandle();
+    }
+
+    if ( accView != nil )
+    {
+        [accView removeFromSuperview];
+        [panel setAccessoryView:accView];
+    }
+    else
+    {
+        [panel setAccessoryView:nil];
+    }
+}
 
+int wxFileDialog::ShowModal()
+{
+    wxMacAutoreleasePool autoreleasepool;
+    
     wxCFStringRef cf( m_message );
 
     wxCFStringRef dir( m_dir );
@@ -207,8 +476,6 @@ int wxFileDialog::ShowModal()
     m_path = wxEmptyString;
     m_fileNames.Clear();
     m_paths.Clear();
-    // since we don't support retrieving the matching filter
-    m_filterIndex = -1;
 
     wxNonOwnedWindow* parentWindow = NULL;
     int returnCode = -1;
@@ -218,9 +485,63 @@ int wxFileDialog::ShowModal()
         parentWindow = dynamic_cast<wxNonOwnedWindow*>(wxGetTopLevelParent(GetParent()));
     }
 
-    if (HasFlag(wxFD_SAVE))
+
+    NSArray* types = GetTypesFromFilter( m_wildCard, m_filterNames, m_filterExtensions ) ;
+
+    m_useFileTypeFilter = m_filterExtensions.GetCount() > 1;
+
+    if( HasFlag(wxFD_OPEN) )
+    {
+        if ( !(wxSystemOptions::HasOption( wxOSX_FILEDIALOG_ALWAYS_SHOW_TYPES ) && (wxSystemOptions::GetOptionInt( wxOSX_FILEDIALOG_ALWAYS_SHOW_TYPES ) == 1)) )
+            m_useFileTypeFilter = false;            
+    }
+
+    m_firstFileTypeFilter = -1;
+
+    if ( m_useFileTypeFilter
+        && m_filterIndex >= 0 && m_filterIndex < m_filterExtensions.GetCount() )
+    {
+        m_firstFileTypeFilter = m_filterIndex;
+    }
+    else if ( m_useFileTypeFilter )
+    {
+        types = nil;
+        bool useDefault = true;
+        for ( size_t i = 0; i < m_filterExtensions.GetCount(); ++i )
+        {
+            types = GetTypesFromExtension(m_filterExtensions[i], m_currentExtensions);
+            if ( m_currentExtensions.GetCount() == 0 )
+            {
+                useDefault = false;
+                m_firstFileTypeFilter = i;
+                break;
+            }
+            
+            for ( size_t j = 0; j < m_currentExtensions.GetCount(); ++j )
+            {
+                if ( m_fileName.EndsWith(m_currentExtensions[j]) )
+                {
+                    m_firstFileTypeFilter = i;
+                    useDefault = false;
+                    break;
+                }
+            }
+            if ( !useDefault )
+                break;
+        }
+        if ( useDefault )
+        {
+            types = GetTypesFromExtension(m_filterExtensions[0], m_currentExtensions);
+            m_firstFileTypeFilter = 0;
+        }
+    }
+
+    if ( HasFlag(wxFD_SAVE) )
     {
         NSSavePanel* sPanel = [NSSavePanel savePanel];
+
+        SetupExtraControls(sPanel);
+
         // makes things more convenient:
         [sPanel setCanCreateDirectories:YES];
         [sPanel setMessage:cf.AsNSString()];
@@ -228,31 +549,57 @@ int wxFileDialog::ShowModal()
         // be able to pass this in
         [sPanel setTreatsFilePackagesAsDirectories:NO];
         [sPanel setCanSelectHiddenExtension:YES];
+        [sPanel setAllowedFileTypes:types];
+        [sPanel setAllowsOtherFileTypes:NO];
 
         if ( HasFlag(wxFD_OVERWRITE_PROMPT) )
         {
         }
 
-        returnCode = [sPanel runModalForDirectory:dir.AsNSString() file:file.AsNSString() ];
+        /*
+        Let the file dialog know what file type should be used initially.
+        If this is not done then when setting the filter index
+        programmatically to 1 the file will still have the extension
+        of the first file type instead of the second one. E.g. when file
+        types are foo and bar, a filename "myletter" with SetDialogIndex(1)
+        would result in saving as myletter.foo, while we want myletter.bar.
+        */
+        if(m_firstFileTypeFilter > 0)
+        {
+            DoOnFilterSelected(m_firstFileTypeFilter);
+        }
+
+        returnCode = [sPanel runModalForDirectory: m_dir.IsEmpty() ? nil : dir.AsNSString() file:file.AsNSString() ];
         ModalFinishedCallback(sPanel, returnCode);
     }
     else
     {
-        NSArray* types = GetTypesFromFilter( m_wildCard ) ;
         NSOpenPanel* oPanel = [NSOpenPanel openPanel];
+        
+        SetupExtraControls(oPanel);
+                
         [oPanel setTreatsFilePackagesAsDirectories:NO];
         [oPanel setCanChooseDirectories:NO];
         [oPanel setResolvesAliases:YES];
         [oPanel setCanChooseFiles:YES];
         [oPanel setMessage:cf.AsNSString()];
+        [oPanel setAllowsMultipleSelection: (HasFlag(wxFD_MULTIPLE) ? YES : NO )];
 
-        returnCode = [oPanel runModalForDirectory:dir.AsNSString()
-                        file:file.AsNSString() types:types];
+        if ( UMAGetSystemVersion() < 0x1060 )
+        {
+            returnCode = [oPanel runModalForDirectory:m_dir.IsEmpty() ? nil : dir.AsNSString()
+                                                 file:file.AsNSString() types:(m_delegate == nil ? types : nil)];
+        }
+        else 
+        {
+            [oPanel setAllowedFileTypes: (m_delegate == nil ? types : nil)];
+            if ( !m_dir.IsEmpty() )
+                [oPanel setDirectoryURL:[NSURL fileURLWithPath:dir.AsNSString() 
+                                               isDirectory:YES]];
+            returnCode = [oPanel runModal];
+        }
 
         ModalFinishedCallback(oPanel, returnCode);
-        
-        if ( types != nil )
-            [types release];
     }
 
     return GetReturnCode();
@@ -271,6 +618,10 @@ void wxFileDialog::ModalFinishedCallback(void* panel, int returnCode)
             m_path = wxCFStringRef::AsString([sPanel filename]);
             m_fileName = wxFileNameFromPath(m_path);
             m_dir = wxPathOnly( m_path );
+            if (m_filterChoice)
+            {
+                m_filterIndex = m_filterChoice->GetSelection();
+            }
         }
     }
     else
@@ -294,11 +645,20 @@ void wxFileDialog::ModalFinishedCallback(void* panel, int returnCode)
                 }
             }
         }
+        if ( m_delegate )
+        {
+            [oPanel setDelegate:nil];
+            [m_delegate release];
+            m_delegate = nil;
+        }
     }
     SetReturnCode(result);
     
     if (GetModality() == wxDIALOG_MODALITY_WINDOW_MODAL)
         SendWindowModalDialogEvent ( wxEVT_WINDOW_MODAL_DIALOG_CLOSED  );
+    
+    UnsubclassWin();
+    [(NSSavePanel*) panel setAccessoryView:nil];
 }
 
 #endif // wxUSE_FILEDLG