Fix "warning: operation on 'y' may be undefined".

[wxWidgets.git] / wxPython / wx / lib / dialogs.py

diff --git a/wxPython/wx/lib/dialogs.py b/wxPython/wx/lib/dialogs.py
index 3012e18cdf1e60985796cb91c98ae79434280c35..788ad8d45d936a3131d0fd86d8013737652d1e2e 100644 (file)
--- a/wxPython/wx/lib/dialogs.py
+++ b/wxPython/wx/lib/dialogs.py
@@ -1,6 +1,6 @@
 #----------------------------------------------------------------------
 #----------------------------------------------------------------------

-# Name:        wxPython.lib.dialogs
-# Purpose:     wxScrolledMessageDialog, wxMultipleChoiceDialog and
+# Name:        wx.lib.dialogs
+# Purpose:     ScrolledMessageDialog, MultipleChoiceDialog and

 #              function wrappers for the common dialogs by Kevin Altis.
 #
 # Author:      Various
 #              function wrappers for the common dialogs by Kevin Altis.
 #
 # Author:      Various


@@ -14,34 +14,41 @@
 #
 # o Updated for 2.5 compatability.
 #
 #
 # o Updated for 2.5 compatability.
 #

+# 12/18/2003 - Jeff Grimmett (grimmtooth@softhome.net)
+#
+# o wxScrolledMessageDialog -> ScrolledMessageDialog
+# o wxMultipleChoiceDialog -> MultipleChoiceDialog
+#

 
 import  wx
 import  layoutf
 
 #----------------------------------------------------------------------
 
 
 import  wx
 import  layoutf
 
 #----------------------------------------------------------------------
 

-class wxScrolledMessageDialog(wx.Dialog):
-    def __init__(self, parent, msg, caption, pos = wx.DefaultPosition, 
-                 size = (500,300)):
-        wx.Dialog.__init__(self, parent, -1, caption, pos, size)
+class ScrolledMessageDialog(wx.Dialog):
+    def __init__(self, parent, msg, caption,
+                 pos=wx.DefaultPosition, size=(500,300),
+                 style=wx.DEFAULT_DIALOG_STYLE):
+        wx.Dialog.__init__(self, parent, -1, caption, pos, size, style)

         x, y = pos
         if x == -1 and y == -1:
             self.CenterOnScreen(wx.BOTH)
 
         x, y = pos
         if x == -1 and y == -1:
             self.CenterOnScreen(wx.BOTH)
 

-        text = wx.TextCtrl(self, -1, msg, wx.DefaultPosition, wx.DefaultSize,
-                           wx.TE_MULTILINE | wx.TE_READONLY)
+        text = wx.TextCtrl(self, -1, msg, 
+                           style=wx.TE_MULTILINE | wx.TE_READONLY)

 
         ok = wx.Button(self, wx.ID_OK, "OK")
 
         ok = wx.Button(self, wx.ID_OK, "OK")

+        ok.SetDefault()

         lc = layoutf.Layoutf('t=t5#1;b=t5#2;l=l5#1;r=r5#1', (self,ok)) 
         text.SetConstraints(lc)
 
         lc = layoutf.Layoutf('t=t5#1;b=t5#2;l=l5#1;r=r5#1', (self,ok)) 
         text.SetConstraints(lc)
 

-        lc = layoutf.Layoutf('b=b5#1;x%w50#1;w!80;h!25', (self,))
+        lc = layoutf.Layoutf('b=b5#1;x%w50#1;w!80;h*', (self,))

         ok.SetConstraints(lc)
         self.SetAutoLayout(1)
         self.Layout()
 
 
         ok.SetConstraints(lc)
         self.SetAutoLayout(1)
         self.Layout()
 
 

-class wxMultipleChoiceDialog(wx.Dialog):
+class MultipleChoiceDialog(wx.Dialog):

     def __init__(self, parent, msg, title, lst, pos = wx.DefaultPosition,
                  size = (200,200), style = wx.DEFAULT_DIALOG_STYLE):
         wx.Dialog.__init__(self, parent, -1, title, pos, size, style)
     def __init__(self, parent, msg, title, lst, pos = wx.DefaultPosition,
                  size = (200,200), style = wx.DEFAULT_DIALOG_STYLE):
         wx.Dialog.__init__(self, parent, -1, title, pos, size, style)


@@ -50,30 +57,27 @@ class wxMultipleChoiceDialog(wx.Dialog):
         if x == -1 and y == -1:
             self.CenterOnScreen(wx.BOTH)
 
         if x == -1 and y == -1:
             self.CenterOnScreen(wx.BOTH)
 

-        dc = wx.ClientDC(self)
-        height = 0
-        for line in msg.splitlines():
-            height = height + dc.GetTextExtent(line)[1] + 2
-

         stat = wx.StaticText(self, -1, msg)
         self.lbox = wx.ListBox(self, 100, wx.DefaultPosition, wx.DefaultSize, 
                                lst, wx.LB_MULTIPLE)
 
         ok = wx.Button(self, wx.ID_OK, "OK")
         stat = wx.StaticText(self, -1, msg)
         self.lbox = wx.ListBox(self, 100, wx.DefaultPosition, wx.DefaultSize, 
                                lst, wx.LB_MULTIPLE)
 
         ok = wx.Button(self, wx.ID_OK, "OK")

+        ok.SetDefault()

         cancel = wx.Button(self, wx.ID_CANCEL, "Cancel")
         cancel = wx.Button(self, wx.ID_CANCEL, "Cancel")

-        lc = layoutf.Layoutf('t=t10#1;l=l5#1;r=r5#1;h!%d' % (height,), (self,)) 
-        stat.SetConstraints(lc)
-
-        lc = layoutf.Layoutf('t=b10#2;l=l5#1;r=r5#1;b=t5#3', (self, stat, ok)) 
-        self.lbox.SetConstraints(lc)
-
-        lc = layoutf.Layoutf('b=b5#1;x%w25#1;w!80;h!25', (self,))
-        ok.SetConstraints(lc)
-
-        lc = layoutf.Layoutf('b=b5#1;x%w75#1;w!80;h!25', (self,))
-        cancel.SetConstraints(lc)

         
         

-        self.SetAutoLayout(1)
+        dlgsizer = wx.BoxSizer(wx.VERTICAL)
+        dlgsizer.Add(stat, 0, wx.ALL, 4)
+        dlgsizer.Add(self.lbox, 1, wx.EXPAND | wx.ALL, 4)
+        
+        btnsizer = wx.StdDialogButtonSizer()
+        btnsizer.AddButton(ok)
+        btnsizer.AddButton(cancel)
+        btnsizer.Realize()
+        
+        dlgsizer.Add(btnsizer, 0, wx.ALL | wx.ALIGN_RIGHT, 4)
+        
+        self.SetSizer(dlgsizer)
+        

         self.lst = lst
         self.Layout()
 
         self.lst = lst
         self.Layout()
 


@@ -82,11 +86,7 @@ class wxMultipleChoiceDialog(wx.Dialog):
 
     def GetValueString(self):
         sel = self.lbox.GetSelections()
 
     def GetValueString(self):
         sel = self.lbox.GetSelections()

-        val = []
-
-        for i in sel:
-            val.append(self.lst[i])
-
+        val = [ self.lst[i] for i in sel ]

         return tuple(val)
 
 
         return tuple(val)
 
 


@@ -145,13 +145,14 @@ def returnedString(ret):
 ## this dialog is always modal, while wxFindReplaceDialog is
 ## modeless and so doesn't lend itself to a function wrapper
 def findDialog(parent=None, searchText='', wholeWordsOnly=0, caseSensitive=0):
 ## this dialog is always modal, while wxFindReplaceDialog is
 ## modeless and so doesn't lend itself to a function wrapper
 def findDialog(parent=None, searchText='', wholeWordsOnly=0, caseSensitive=0):

-    dlg = wx.Dialog(parent, -1, "Find", wx.DefaultPosition, (370, 120))
+    dlg = wx.Dialog(parent, -1, "Find", wx.DefaultPosition, (380, 120))

 
     wx.StaticText(dlg, -1, 'Find what:', (7, 10))
 
     wx.StaticText(dlg, -1, 'Find what:', (7, 10))

-    wSearchText = wx.TextCtrl(dlg, -1, searchText, (70, 7), (195, -1))
+    wSearchText = wx.TextCtrl(dlg, -1, searchText, (80, 7), (195, -1))

     wSearchText.SetValue(searchText)
     wSearchText.SetValue(searchText)

-    wx.wxButton(dlg, wx.ID_OK, "Find Next", (280, 5), wx.DefaultSize).SetDefault()
-    wx.wxButton(dlg, wx.ID_CANCEL, "Cancel", (280, 35), wx.DefaultSize)
+    wx.Button(dlg, wx.ID_OK, "Find Next", (285, 5), wx.DefaultSize).SetDefault()
+    wx.Button(dlg, wx.ID_CANCEL, "Cancel", (285, 35), wx.DefaultSize)
+

     wWholeWord = wx.CheckBox(dlg, -1, 'Match whole word only',
                             (7, 35), wx.DefaultSize, wx.NO_BORDER)
 
     wWholeWord = wx.CheckBox(dlg, -1, 'Match whole word only',
                             (7, 35), wx.DefaultSize, wx.NO_BORDER)
 


@@ -167,9 +168,9 @@ def findDialog(parent=None, searchText='', wholeWordsOnly=0, caseSensitive=0):
     wSearchText.SetFocus()
 
     result = DialogResults(dlg.ShowModal())
     wSearchText.SetFocus()
 
     result = DialogResults(dlg.ShowModal())

-    result.text = wSearchText.GetValue()
-    result.wholeword = wWholeWord.GetValue()
-    result.casesensitive = wCase.GetValue()
+    result.searchText = wSearchText.GetValue()
+    result.wholeWordsOnly = wWholeWord.GetValue()
+    result.caseSensitive = wCase.GetValue()

     dlg.Destroy()
     return result
 
     dlg.Destroy()
     return result
 


@@ -213,9 +214,11 @@ def colourDialog(parent=None, colourData=None, colour=None):
 def fontDialog(parent=None, fontData=None, font=None):
     if fontData is None:
         fontData = wx.FontData()
 def fontDialog(parent=None, fontData=None, font=None):
     if fontData is None:
         fontData = wx.FontData()

+        fontData.SetColour(wx.BLACK)
+        fontData.SetInitialFont(wx.SystemSettings.GetFont(wx.SYS_DEFAULT_GUI_FONT))

 
     if font is not None:
 
     if font is not None:

-        aFontData.SetInitialFont(font)
+        fontData.SetInitialFont(font)

 
     dialog = wx.FontDialog(parent, fontData)
     result = DialogResults(dialog.ShowModal())
 
     dialog = wx.FontDialog(parent, fontData)
     result = DialogResults(dialog.ShowModal())


@@ -263,7 +266,7 @@ def alertDialog(parent=None, message='', title='Alert', pos=wx.DefaultPosition):
 def scrolledMessageDialog(parent=None, message='', title='', pos=wx.DefaultPosition,
                           size=(500,300)):
 
 def scrolledMessageDialog(parent=None, message='', title='', pos=wx.DefaultPosition,
                           size=(500,300)):
 

-    dialog = wxScrolledMessageDialog(parent, message, title, pos, size)
+    dialog = ScrolledMessageDialog(parent, message, title, pos, size)

     result = DialogResults(dialog.ShowModal())
     dialog.Destroy()
     return result
     result = DialogResults(dialog.ShowModal())
     dialog.Destroy()
     return result


@@ -314,32 +317,35 @@ directoryDialog = dirDialog
 
 def singleChoiceDialog(parent=None, message='', title='', lst=[], 
                        style=wx.OK | wx.CANCEL | wx.CENTRE):
 
 def singleChoiceDialog(parent=None, message='', title='', lst=[], 
                        style=wx.OK | wx.CANCEL | wx.CENTRE):

-    dialog = wx.SingleChoiceDialog(parent, message, title, lst, style)
+    dialog = wx.SingleChoiceDialog(parent, message, title, list(lst), style | wx.DEFAULT_DIALOG_STYLE)

     result = DialogResults(dialog.ShowModal())
     result.selection = dialog.GetStringSelection()
     dialog.Destroy()
     return result
 
 
     result = DialogResults(dialog.ShowModal())
     result.selection = dialog.GetStringSelection()
     dialog.Destroy()
     return result
 
 

-def multipleChoiceDialog(parent=None, message='', title='', lst=[], pos=wx.DefaultPosition, 
-                         size=(200,200)):
+def multipleChoiceDialog(parent=None, message='', title='', lst=[],
+                         pos=wx.DefaultPosition, size=wx.DefaultSize):

 
 

-    dialog = wxMultipleChoiceDialog(parent, message, title, lst, pos, size)
+    dialog = wx.MultiChoiceDialog(parent, message, title, lst,
+                                  wx.CHOICEDLG_STYLE, pos)

     result = DialogResults(dialog.ShowModal())
     result = DialogResults(dialog.ShowModal())

-    result.selection = dialog.GetValueString()
+    result.selection = tuple([lst[i] for i in dialog.GetSelections()])

     dialog.Destroy()
     return result
 
 
 if __name__ == '__main__':
     dialog.Destroy()
     return result
 
 
 if __name__ == '__main__':

+    #import os
+    #print os.getpid()
+    

     class MyApp(wx.App):
 
         def OnInit(self):
     class MyApp(wx.App):
 
         def OnInit(self):

-            frame = wx.Frame(None, -1, "Dialogs", size=(400, 200))
+            self.frame = frame = wx.Frame(None, -1, "Dialogs", size=(400, 240))

             panel = wx.Panel(frame, -1)
             self.panel = panel
 
             panel = wx.Panel(frame, -1)
             self.panel = panel
 

-            frame.Show(1)

 
             dialogNames = [
                 'alertDialog',
 
             dialogNames = [
                 'alertDialog',


@@ -356,16 +362,27 @@ if __name__ == '__main__':
                 'singleChoiceDialog',
                 'textEntryDialog',
             ]
                 'singleChoiceDialog',
                 'textEntryDialog',
             ]

-            self.nameList = wx.ListBox(panel, -1, (0, 0), (130, 180), dialogNames, style=wx.LB_SINGLE)
-            self.Bind(wx.EVT_LISTBOX, self.OnNameListSelected, id=self.nameList.GetId())
+
+            self.nameList = wx.ListBox(panel, -1,
+                                       size=(130, 180),
+                                       choices=dialogNames,
+                                       style=wx.LB_SINGLE)
+            self.Bind(wx.EVT_LISTBOX, self.OnNameListSelected, self.nameList)

 
             tstyle = wx.TE_RICH2 | wx.TE_PROCESS_TAB | wx.TE_MULTILINE
 
             tstyle = wx.TE_RICH2 | wx.TE_PROCESS_TAB | wx.TE_MULTILINE

-            self.text1 = wx.TextCtrl(panel, -1, pos=(150, 0), size=(200, 180), style=tstyle)
+            self.text1 = wx.TextCtrl(panel, -1, size=(200, 180), style=tstyle)

 
 

-            self.SetTopWindow(frame)
+            sizer = wx.BoxSizer(wx.HORIZONTAL)
+            sizer.Add(self.nameList, 0, wx.EXPAND|wx.ALL, 20)
+            sizer.Add(self.text1, 1,  wx.EXPAND|wx.ALL, 20)
+
+            panel.SetSizer(sizer)

 
 

+            self.SetTopWindow(frame)
+            frame.Show(1)

             return 1
 
             return 1
 

+

         def OnNameListSelected(self, evt):
             import pprint
             sel = evt.GetString()
         def OnNameListSelected(self, evt):
             import pprint
             sel = evt.GetString()


@@ -385,8 +402,8 @@ if __name__ == '__main__':
                 result = fontDialog()
             elif sel == 'messageDialog':
                 result = messageDialog(None, 'Hello from Python and wxPython!',
                 result = fontDialog()
             elif sel == 'messageDialog':
                 result = messageDialog(None, 'Hello from Python and wxPython!',

-                          'A Message Box', wx.wxOK | wx.wxICON_INFORMATION)
-                          #wx.wxYES_NO | wx.wxNO_DEFAULT | wx.wxCANCEL | wx.wxICON_INFORMATION)
+                          'A Message Box', wx.OK | wx.ICON_INFORMATION)
+                          #wx.YES_NO | wx.NO_DEFAULT | wx.CANCEL | wx.ICON_INFORMATION)

                 #result = messageDialog(None, 'message', 'title')
             elif sel == 'multipleChoiceDialog':
                 result = multipleChoiceDialog(None, "message", "title", ['one', 'two', 'three'])
                 #result = messageDialog(None, 'message', 'title')
             elif sel == 'multipleChoiceDialog':
                 result = multipleChoiceDialog(None, "message", "title", ['one', 'two', 'three'])