Fix for opening of multiple document templates not working on OSX

[SnowLeopard]

Say that you have a program with more than one document template (e.g., files with the DOC1 and DOC2 extensions). When a wxID_OPEN event is fired, the OSX file open dialog will appear and you will be able to select any file that matches the extensions for your templates (i.e., you can select either DOC1 or DOC2). Note how this is different from Windows where you have a file filter and you need to change that filter to select different document types--this is because the OSX file open doesn't have a file filter combobox like Windows. It just clumps all allowable file types into one filter.

Here is the problem--if you select any document type other than the first one (e.g., select DOC2), then the program will always load it as a DOC1 file. The culprit is "wxDocManager::SelectDocumentPath" in docview.cpp:

if ( FilterIndex != -1 )
            theTemplate = templates[FilterIndex];

On OSX, no matter what doc type you select, "FilterIndex" will always be zero. If you wrap this logic in a "#ifndef WXMAC" block, then that fixes it because then FindTemplateForPath is called and selects the proper template (see attached patch).

[SnowLeopard]

Fix

[vadz]

Thanks for finding and reporting the problem! Unfortunately I don't think the patch provides the correct solution for it. The code in docview.cpp already has provision for not using the filter index if the current platform doesn't support it, it's just supposed to remain -1 then. It's wrong that it is set to 0 by default by wxFileSelectorEx().

Could you please test if the following patch (also) fixes the problem:

src/common/fldlgcmn.cpp

@@ -36 +36 @@
 
 void wxFileDialogBase::Init()
 {
-    m_filterIndex = 0;
+    m_filterIndex = -1; // unknown
     m_windowStyle = 0;
     m_extraControl = NULL;
     m_extraControlCreator = NULL;
@@ -59 +59 @@
 
     m_parent = parent;
     m_windowStyle = style;
-    m_filterIndex = 0;
 
     if (!HasFdFlag(wxFD_OPEN) && !HasFdFlag(wxFD_SAVE))
         m_windowStyle |= wxFD_OPEN;     // wxFD_OPEN is the default

TIA!

[vadz]

Oops, a further grep showed that maybe changing this is not as simple as I thought... Carbon and generic implementation might be not happy with filter index being -1 by default. So maybe the best would be to just make wxFileDialog::GetFilterIndex() always return -1 in wxOSX/Cocoa? At least until Stefan ports Carbon code (which, IIUC, implements filters on its own) to Cocoa if this is possible at all?

[csomor]

was fixed in r63178, closing manually