Merge lp:~jml/gtg/deepcopy-398360 into lp:~gtg/gtg/old-trunk

[Jonathan Lange]

[Jonathan Lange (jml)]

This branch deletes the 'listes' module from GTG.tools and replaces all calls to returnlist with calls to copy.deepcopy.

=== modified file 'GTG/core/requester.py'
--- GTG/core/requester.py	2009-03-29 19:32:59 +0000
+++ GTG/core/requester.py	2009-07-12 07:14:42 +0000
@@ -18,7 +18,8 @@
 # -----------------------------------------------------------------------------
 
 
-from GTG.tools.listes import *
+from copy import deepcopy
+
 #Requester is a pure View object. It will not do anything but it will
 #be used by any Interface to handle the requests to the datastore
 
@@ -179,7 +180,7 @@
     
     #Not used currently because it returns every tag that was ever used
     def get_all_tags(self):
-        return returnlist(self.ds.get_tagstore().get_all_tags())
+        return deepcopy(self.ds.get_tagstore().get_all_tags())
         
     def get_notag_tag(self) :
         return self.ds.get_tagstore().get_notag_tag()

=== modified file 'GTG/core/task.py'
--- GTG/core/task.py	2009-04-15 13:20:00 +0000
+++ GTG/core/task.py	2009-07-12 07:14:42 +0000
@@ -17,10 +17,10 @@
 # this program.  If not, see <http://www.gnu.org/licenses/>.
 # -----------------------------------------------------------------------------
 
+from copy import deepcopy
 from datetime import date
 import xml.dom.minidom
 
-from GTG.tools.listes import *
 from GTG.tools.dates import *
 
 #This class represent a task in GTG.
@@ -318,7 +318,7 @@
         return zelist
     
     def get_subtasks_tid(self) :
-        return returnlist(self.children)
+        return deepcopy(self.children)
         
         
     #add and remove parents are private
@@ -345,7 +345,7 @@
                 parent.sync()
     
     def get_parents(self):
-        return returnlist(self.parents)
+        return deepcopy(self.parents)
  
     #Return true is the task has parent
     #If tag is provided, return True only
@@ -398,7 +398,7 @@
         
     #return a copy of the list of tag objects
     def get_tags(self) :
-        return returnlist(self.tags)
+        return deepcopy(self.tags)
 
     #This function add tag by name
     def add_tag(self, tagname):

=== removed file 'GTG/tools/listes.py'
--- GTG/tools/listes.py	2009-03-01 14:09:12 +0000
+++ GTG/tools/listes.py	1970-01-01 00:00:00 +0000
@@ -1,30 +0,0 @@
-# -*- coding: utf-8 -*-
-# -----------------------------------------------------------------------------
-# Gettings Things Gnome! - a personnal organizer for the GNOME desktop
-# Copyright (c) 2008-2009 - Lionel Dricot & Bertrand Rousseau
-#
-# This program is free software: you can redistribute it and/or modify it under
-# the terms of the GNU General Public License as published by the Free Software
-# Foundation, either version 3 of the License, or (at your option) any later
-# version.
-#
-# This program is distributed in the hope that it will be useful, but WITHOUT
-# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
-# FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
-# details.
-#
-# You should have received a copy of the GNU General Public License along with
-# this program.  If not, see <http://www.gnu.org/licenses/>.
-# -----------------------------------------------------------------------------
-
-#Return a copy of the list and avoid a to keep the same object
-#It handles multilple dimension matrix through recursion
-def returnlist(liste) :
-    if liste.__class__ is list :
-        to_return = []
-        for i in liste :
-            elem = returnlist(i)
-            to_return.append(elem)
-    else :
-        to_return = liste
-    return to_return