From adb1c2a1f8c686403a3e5c7de5bedd5dee866dcf Mon Sep 17 00:00:00 2001
From: Chris Jones <cmsj@tenshu.net>
Date: Sun, 9 Aug 2009 23:54:14 +0100
Subject: [PATCH] raise the correct exception if methods are called from the
 base class that must be overridden by subclasses. Remove unnecessary
 dependencies and achieve 10/10 from pylint

---
 terminatorlib/container.py | 45 +++++++++++++++-----------------------
 1 file changed, 18 insertions(+), 27 deletions(-)
 mode change 100644 => 100755 terminatorlib/container.py

diff --git a/terminatorlib/container.py b/terminatorlib/container.py
old mode 100644
new mode 100755
index d8343ba9..234529d9
--- a/terminatorlib/container.py
+++ b/terminatorlib/container.py
@@ -4,11 +4,9 @@
 """container.py - classes necessary to contain Terminal widgets"""
 
 import gobject
-import gtk
 
-from util import debug, err
-
-class Container():
+# pylint: disable-msg=R0921
+class Container(object):
     """Base class for Terminator Containers"""
 
     immutable = None
@@ -32,15 +30,19 @@ class Container():
         self.children = []
         self.config = configobject
 
-    def register_signals(self, object):
+    def register_signals(self, widget):
         """Register gobject signals in a way that avoids multiple inheritance"""
         for signal in self.signals:
             gobject.signal_new(signal['name'],
-                               object,
+                               widget,
                                signal['flags'],
                                signal['return_type'],
                                signal['param_types'])
 
+    def emit(self, signal):
+        """Emit a gobject signal"""
+        raise NotImplementedError('emit')
+
     def get_offspring(self):
         """Return a list of child widgets, if any"""
         return(self.children)
@@ -55,17 +57,15 @@ class Container():
 
     def split_axis(self, widget, vertical=True):
         """Default axis splitter. This should be implemented by subclasses"""
-        err('split_axis called from base class. This is a bug')
-        return(False)
+        raise NotImplementedError('split_axis')
 
     def unsplit(self, widget, keep=False):
         """Default unsplitter. This should be implemented by subclasses"""
-        err('unsplit called from base class. This is a bug')
-        return(False)
+        raise NotImplementedError('unsplit')
 
     def remove(self, widget):
         """Remove a widget from the container"""
-        err('remove called from base class. This is a bug')
+        raise NotImplementedError('remove')
 
     def closeterm(self, widget):
         """Handle the closure of a terminal"""
@@ -78,22 +78,9 @@ class Container():
         self.emit('need_group_hoover')
         return(True)
 
-    def closegroupterms(self, widget):
-        """Handle the closure of a group of terminals"""
-        if self.state_zoomed != self.states_zoom['normal']:
-            self.unzoom(widget)
-
-        all_closed = True
-        for term in self.term_list[:]:
-            if term._group == widget._group and not self.remove(term):
-                all_closed = False
-
-        self.emit('need_group_hoover')
-        return(all_closed)
-
     def resizeterm(self, widget, keyname):
         """Handle a keyboard event requesting a terminal resize"""
-        err('resizeterm called from base class. This is a bug')
+        raise NotImplementedError('resizeterm')
 
     def toggle_zoom(self, widget, fontscale = False):
         """Toggle the existing zoom state"""
@@ -104,10 +91,14 @@ class Container():
 
     def zoom(self, widget, fontscale = False):
         """Zoom a terminal"""
-        err('zoom called from base class. This is a bug')
+        raise NotImplementedError('zoom')
 
     def unzoom(self, widget):
         """Unzoom a terminal"""
-        err('unzoom called from base class. This is a bug')
+        raise NotImplementedError('unzoom')
+
+if __name__ == '__main__':
+    CONTAINER = Container()
+    CONTAINER.zoom()
 
 # vim: set expandtab ts=4 sw=4: