Return Value From Wxpython Main Frame

i am facing some issues to get return value from wxpython main fram. my script: import wx import wx.xrc class Email_template( wx.Frame ): def __init__( self, parent, to, cc,

Solution 1:

First, I have changed your sizers. I have replaced them with one GridBagSizer which is very powerful. I have also deleted most of the widget size parameters as sizer is now going to handle widget sizes based on window size. I have also removed a lot of parameters from your wx calls, as those are default. By removing default optional parameters, the code is more readable. You mostly do not need wx.ID_ANY, wx.DefaultSize, wx.DefaultPosition or wx.EmptyString etc. in wxPython as you would in wxWidgets.

Second, the way you have programmed your app is weird. I think you want either of the two:

1) Create an GUI application which has a window, allows you to edit, does something as you press Send. It also closes itself when you press Send in your case, so maybe that is not what you are after...

import wx

classEmailTemplate(wx.Frame):
    def__init__(self, to, cc, subject, message_body):
        wx.Frame.__init__(self, None, size=(650, 500))

        self.panel = wx.Panel(self)       
        self.panel.SetBackgroundColour(wx.SystemSettings.GetColour(wx.SYS_COLOUR_INFOBK))

        self.sizer = wx.GridBagSizer(5, 5)

        self.sizer.AddGrowableCol(3)
        self.sizer.AddGrowableRow(5)

        # Add spacers. Not the best, but avoids another sizer for border
        self.sizer.Add((0, 0), pos=(0, 4))
        self.sizer.Add((0, 0), pos=(6, 0))

        self.mail_send_button = wx.Button(self.panel, label="Send")
        self.mail_send_button.Bind(wx.EVT_BUTTON, self.OnClick_send)
        self.sizer.Add(self.mail_send_button, pos=(1, 1), span=(2, 1), flag=wx.EXPAND)

        self.mail_edit_button = wx.Button(self.panel, label="Edit")
        self.mail_edit_button.Bind(wx.EVT_BUTTON, self.OnClick_edit)
        self.sizer.Add(self.mail_edit_button, pos=(3, 1), span=(2, 1), flag=wx.EXPAND)

        self.from_staticText = wx.StaticText(self.panel, label="From:")
        self.sizer.Add(self.from_staticText, pos=(1, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.from_textCtrl = wx.TextCtrl(self.panel, value="from_mail@mymail.com")
        self.sizer.Add(self.from_textCtrl, pos=(1, 3), flag=wx.EXPAND)

        self.to_staticText = wx.StaticText(self.panel, label="To:")
        self.sizer.Add(self.to_staticText, pos=(2, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.to_textCtrl = wx.TextCtrl(self.panel, value=to)
        self.sizer.Add(self.to_textCtrl, pos=(2, 3), flag=wx.EXPAND)

        self.cc_staticText = wx.StaticText(self.panel, label="CC:")
        self.sizer.Add(self.cc_staticText, pos=(3, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.cc_textCtrl = wx.TextCtrl(self.panel, value=cc)
        self.sizer.Add(self.cc_textCtrl, pos=(3, 3), flag=wx.EXPAND)

        self.subject_staticText = wx.StaticText(self.panel, label="Subject:")
        self.sizer.Add(self.subject_staticText, pos=(4, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.subject_textCtrl = wx.TextCtrl(self.panel, value=subject)
        self.sizer.Add(self.subject_textCtrl, pos=(4, 3), flag=wx.EXPAND)

        self.mail_textCtrl = wx.TextCtrl(self.panel, value=message_body, style=wx.TE_MULTILINE)
        self.sizer.Add(self.mail_textCtrl, pos=(5, 1), span=(1, 3), flag=wx.EXPAND)

        self.panel.SetSizerAndFit(self.sizer)

        self.to_textCtrl.Disable()
        self.cc_textCtrl.Disable()
        self.subject_textCtrl.Disable()
        self.mail_textCtrl.Disable()

        self.Show()

    defOnClick_send(self, event):
        self.DoSomethingWithEmail(self.to_textCtrl.GetValue(),
                                  self.cc_textCtrl.GetValue(),
                                  self.subject_textCtrl.GetValue(),
                                  self.mail_textCtrl.GetValue())
        self.Close()

    defOnClick_edit(self, event):
        self.to_textCtrl.Enable(True)
        self.cc_textCtrl.Enable(True)
        self.subject_textCtrl.Enable(True)
        self.mail_textCtrl.Enable(True)

    defDoSomethingWithEmail(self, to, cc, subject, text):
        # Maybe send here?print to, cc, subject, text

app = wx.App(False)
window = EmailTemplate("tomail@mymail.com", "cc_copy@mymail.com", "message subject", "mail")
app.MainLoop()

2) Create just a Dialog, show the dialog, have it closed by the user (using Send) and follow with your Python script without having a real GUI application.

import wx

classEmailTemplate(wx.Dialog):
    def__init__(self, to, cc, subject, message_body):
        wx.Dialog.__init__(self, None, size=(650, 500))

        self.panel = wx.Panel(self)       
        self.panel.SetBackgroundColour(wx.SystemSettings.GetColour(wx.SYS_COLOUR_INFOBK))

        self.main_sizer = wx.BoxSizer()

        self.sizer = wx.GridBagSizer(5, 5)

        self.sizer.AddGrowableCol(3)
        self.sizer.AddGrowableRow(5)

        # Add spacers. Not the best, but avoids another sizer for border
        self.sizer.Add((0, 0), pos=(0, 4))
        self.sizer.Add((0, 0), pos=(6, 0))

        self.mail_send_button = wx.Button(self.panel, label="Send")
        self.mail_send_button.Bind(wx.EVT_BUTTON, self.OnClick_send)
        self.sizer.Add(self.mail_send_button, pos=(1, 1), span=(2, 1), flag=wx.EXPAND)

        self.mail_edit_button = wx.Button(self.panel, label="Edit")
        self.mail_edit_button.Bind(wx.EVT_BUTTON, self.OnClick_edit)
        self.sizer.Add(self.mail_edit_button, pos=(3, 1), span=(2, 1), flag=wx.EXPAND)

        self.from_staticText = wx.StaticText(self.panel, label="From:")
        self.sizer.Add(self.from_staticText, pos=(1, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.from_textCtrl = wx.TextCtrl(self.panel, value="from_mail@mymail.com")
        self.sizer.Add(self.from_textCtrl, pos=(1, 3), flag=wx.EXPAND)

        self.to_staticText = wx.StaticText(self.panel, label="To:")
        self.sizer.Add(self.to_staticText, pos=(2, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.to_textCtrl = wx.TextCtrl(self.panel, value=to)
        self.sizer.Add(self.to_textCtrl, pos=(2, 3), flag=wx.EXPAND)

        self.cc_staticText = wx.StaticText(self.panel, label="CC:")
        self.sizer.Add(self.cc_staticText, pos=(3, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.cc_textCtrl = wx.TextCtrl(self.panel, value=cc)
        self.sizer.Add(self.cc_textCtrl, pos=(3, 3), flag=wx.EXPAND)

        self.subject_staticText = wx.StaticText(self.panel, label="Subject:")
        self.sizer.Add(self.subject_staticText, pos=(4, 2), flag=wx.ALIGN_CENTER_VERTICAL)
        self.subject_textCtrl = wx.TextCtrl(self.panel, value=subject)
        self.sizer.Add(self.subject_textCtrl, pos=(4, 3), flag=wx.EXPAND)

        self.mail_textCtrl = wx.TextCtrl(self.panel, value=message_body, style=wx.TE_MULTILINE)
        self.sizer.Add(self.mail_textCtrl, pos=(5, 1), span=(1, 3), flag=wx.EXPAND)

        self.panel.SetSizer(self.sizer)
        self.main_sizer.Add(self.panel, 1, flag=wx.EXPAND)
        self.SetSizer(self.main_sizer)

        self.to_textCtrl.Disable()
        self.cc_textCtrl.Disable()
        self.subject_textCtrl.Disable()
        self.mail_textCtrl.Disable()

    defOnClick_send(self, event):
        self.to = self.to_textCtrl.GetValue()
        self.cc = self.cc_textCtrl.GetValue()
        self.subject = self.subject_textCtrl.GetValue()
        self.body = self.mail_textCtrl.GetValue()
        self.EndModal(wx.ID_OK)

    defOnClick_edit(self, event):
        self.to_textCtrl.Enable(True)
        self.cc_textCtrl.Enable(True)
        self.subject_textCtrl.Enable(True)
        self.mail_textCtrl.Enable(True)

    defGetValuesAsDict(self):
        d = {"to": self.to,
             "cc": self.cc,
             "subject": self.subject,
             "body": self.body}
        return d

app = wx.App(False)
dialog = EmailTemplate("tomail@mymail.com", "cc_copy@mymail.com", "message subject", "mail")
result = dialog.ShowModal()
if result == wx.ID_OK:
    # Maybe send here instead of print?print dialog.GetValuesAsDict()

Note the differences of how the code is composed. The difference may be subtle in your case, but will be more significant if you decide to build something bigger. It is also idiomatic way to use wxPython, so other programmers will find it easier to understand your code.