treating the error generated when the user tries to use

[cascardo/movie.git] / gzv.py

diff --git a/gzv.py b/gzv.py
index 2149bd0..6820f83 100644 (file)
--- a/gzv.py
+++ b/gzv.py
@@ -15,6 +15,7 @@
 import os
 import gtk
 import gtk.glade
+import gobject
 import math
 import cairo
 from ConfigParser import ConfigParser
@@ -26,9 +27,9 @@ class Ball(object):
 
     def __init__(self, x, y, r, name='', position=0):
         self.position = position
-        self.x = x
-        self.y = y
-        self.radius = r
+        self.x = int(x)
+        self.y = int(y)
+        self.radius = int(r)
         self.name = name
 
 class BallManager(list):
@@ -170,8 +171,7 @@ class Gzv(GladeLoader):
         print 
 
     def on_cell_edited(self, renderer, path, value):
-        ball = Ball(self.start_x, self.start_y, self.radius, value, int(path))
-        self.balls.append(ball)
+        self.balls[int(path)].name = value
         self.load_balls_to_treeview()
         self.draw.queue_draw()
 
@@ -199,10 +199,16 @@ class Gzv(GladeLoader):
         self.load_balls_to_treeview()
         self.draw.queue_draw()
 
+    def unload_project(self):
+        self.project = None
+        self.image = None
+        self.balls = BallManager()
+        self.draw.queue_draw()
+
     def load_balls_to_treeview(self):
-        model = self.treeview.get_model()
+        self.model.clear()
         for i in self.balls:
-            model.append([i.position, i.name])
+            self.model.append([i.position, i.name])
 
     def load_balls_from_file(self, fname):
         balls = BallManager()
@@ -212,9 +218,9 @@ class Gzv(GladeLoader):
         for index, line in enumerate(file(fname)):
             if not line:
                 continue
-            pos, radius, name = line.split()
+            pos, radius, name = line.split(None, 2)
             x, y = pos.split(',')
-            balls.append(Ball(int(x), int(y), int(radius), name, index))
+            balls.append(Ball(x, y, radius, name.strip(), index))
         return balls
 
     def remove_fp(self, *args):
@@ -255,7 +261,21 @@ class Gzv(GladeLoader):
 
         # loading the picture image and getting some useful
         # information to draw it in the widget's background
-        img = gtk.gdk.pixbuf_new_from_file(self.image)
+        try:
+            img = gtk.gdk.pixbuf_new_from_file(self.image)
+        except gobject.GError:
+            msg = _("Couldn't recognize the image file format.")
+            dialog = gtk.MessageDialog(self.window,
+                                       gtk.DIALOG_MODAL,
+                                       gtk.MESSAGE_ERROR,
+                                       gtk.BUTTONS_CLOSE)
+            dialog.set_markup(msg)
+            dialog.run()
+            dialog.destroy()
+
+            self.draw.stop_emission('expose-event')
+            return self.unload_project()
+
         pixels = img.get_pixels()
         rowstride = img.get_rowstride()
         width = img.get_width()
@@ -280,20 +300,18 @@ class Gzv(GladeLoader):
         ctx.fill()
 
         ctx.set_line_width(10.0)
-        ctx.set_source_rgba (0.5, 0.0, 0.0, 0.4)
+        ctx.set_source_rgba(0.5, 0.0, 0.0, 0.4)
 
         for i in self.balls:
             ctx.arc(i.x, i.y, i.radius, 0, 64*math.pi)
-
-        ctx.fill()
-        ctx.stroke()
+            ctx.fill()
 
     def draw_current_ball(self):
         if self.start_x < 0:
             return
         ctx = self.draw.window.cairo_create()
         ctx.arc(self.start_x, self.start_y, self.radius, 0, 64*math.pi)
-        ctx.set_source_rgba (0.5, 0.0, 0.0, 0.4)
+        ctx.set_source_rgba(0.5, 0.0, 0.0, 0.4)
         ctx.fill()
 
     def button_press(self, widget, event):
@@ -323,6 +341,8 @@ class Gzv(GladeLoader):
         self.ball_width = Ball.DEFAULT_WIDTH
 
         position = len(self.balls)
+        ball = Ball(self.start_x, self.start_y, self.radius, '', position)
+        self.balls.append(ball)
         self.model.append([position, ''])
         self.treeview.set_cursor(str(position), self.fpcolumn, True)