current community

your communities

more stack exchange communities

Stack Exchange
tour help
careers 2.0
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite

Python 2.7

Ubuntu 12.04

My code is available here.

I'm trying to put together a base for myself to make creating wxPython apps easier, I'm using the set up of my last program as that base but it's not working properly.

The problem arises when I try to bind a button to a function, this is the same layout that has worked for me before but now when I try to start the program with the button and the bind in place I get this error:

AttributeError: 'my_panel' object has no attribute 'on_quit'

If I remove the bind the program launches.

What's going on?

I've looked at other questions regarding this and it seems that most of the time it's a typing error, I've looked but I cannot find one.

share|improve this question

1 Answer 1

up vote 1 down vote accepted

The on_quit method in your code is indented too far. It needs to be at the same indention level as the init function. This is what you have:

def __init__(self, parent):
    """docstring for __"""
    wx.Panel.__init__(self, parent)
    self.frame = parent

    self.main_sizer = wx.BoxSizer(wx.VERTICAL)
    self.widget_sizer = wx.BoxSizer(wx.VERTICAL)

    self.text_object = wx.StaticText(self, -1, 'Example')
    self.button_object = wx.Button(self, -1, 'QUIT')
    self.button_object.Bind(wx.EVT_BUTTON, self.on_quit)

    self.widget_sizer.Add(self.text_object, 0)
    self.widget_sizer.Add(self.button_object, 0)

    self.main_sizer.Add(self.widget_sizer, 0)


    def on_quit(self, event):
        """docstring for on"""
        self.Close()

This is what it should look like:

def __init__(self, parent):
    """docstring for __"""
    wx.Panel.__init__(self, parent)
    self.frame = parent

    self.main_sizer = wx.BoxSizer(wx.VERTICAL)
    self.widget_sizer = wx.BoxSizer(wx.VERTICAL)

    self.text_object = wx.StaticText(self, -1, 'Example')
    self.button_object = wx.Button(self, -1, 'QUIT')
    self.button_object.Bind(wx.EVT_BUTTON, self.on_quit)

    self.widget_sizer.Add(self.text_object, 0)
    self.widget_sizer.Add(self.button_object, 0)

    self.main_sizer.Add(self.widget_sizer, 0)


def on_quit(self, event):
    """docstring for on"""
    self.Close()

Also note that "self.Close()" will not work. It should be self.frame.Close()

share|improve this answer
    
That did it, not sure why I totally missed that. –  Joseph Adams Nov 15 '13 at 16:41

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.