← Back to branch summary

~gary-lasker/software-center/tos-simple-fix

« back to all changes in this revision

Viewing changes to softwarecenter/backend/aptd.py

  • Committer: Michael Vogt
  • Date: 2010-08-31 19:06:37 UTC
  • Revision ID: michael.vogt@ubuntu.com-20100831190637-6hbe935y7uof5yg1
* softwarecenter/backend/aptd.py:
  - make defaults for addons_{install,remove} empty lists instead
    of None to prevent crash when called with a empty argument

Show diffs side-by-side

added added

removed removed

Lines of Context:
softwarecenter/backend/aptd.py
128
128
 
129
129
    # FIXME: upgrade add-ons here
130
130
    @inline_callbacks
131
 
    def upgrade(self, pkgname, appname, iconname, addons_install=None, addons_remove=None, metadata=None):
 
131
    def upgrade(self, pkgname, appname, iconname, addons_install=[], addons_remove=[], metadata=None):
132
132
        """ upgrade a single package """
133
133
        self.emit("transaction-started")
134
134
        try:
139
139
            self._on_trans_error(error, pkgname)
140
140
 
141
141
    @inline_callbacks
142
 
    def remove(self, pkgname, appname, iconname, addons_install=None, addons_remove=None, metadata=None):
 
142
    def remove(self, pkgname, appname, iconname, addons_install=[], addons_remove=[], metadata=None):
143
143
        """ remove a single package """
144
144
        self.emit("transaction-started")
145
145
        try:
150
150
            self._on_trans_error(error, pkgname)
151
151
 
152
152
    @inline_callbacks
153
 
    def remove_multiple(self, pkgnames, appnames, iconnames, addons_install=None, addons_remove=None, metadatas=None):
 
153
    def remove_multiple(self, pkgnames, appnames, iconnames, addons_install=[], addons_remove=[], metadatas=None):
154
154
        """ queue a list of packages for removal  """
155
155
        if metadatas == None:
156
156
            metadatas = []
160
160
            yield self.remove(pkgname, appname, iconname, metadata)
161
161
 
162
162
    @inline_callbacks
163
 
    def install(self, pkgname, appname, iconname, filename=None, addons_install=None, addons_remove=None, metadata=None):
 
163
    def install(self, pkgname, appname, iconname, filename=None, addons_install=[], addons_remove=[], metadata=None):
164
164
        """Install a single package from the archive
165
165
           If filename is given a local deb package is installed instead.
166
166
        """
176
176
            self._on_trans_error(error, pkgname)
177
177
 
178
178
    @inline_callbacks
179
 
    def install_multiple(self, pkgnames, appnames, iconnames, addons_install=None, addons_remove=None, metadatas=None):
 
179
    def install_multiple(self, pkgnames, appnames, iconnames, addons_install=[], addons_remove=[], metadatas=None):
180
180
        """ queue a list of packages for install  """
181
181
        if metadatas == None:
182
182
            metadatas = []
186
186
            yield self.install(pkgname, appname, iconname, metadata=metadata)
187
187
            
188
188
    @inline_callbacks
189
 
    def apply_changes(self, pkgname, appname, iconname, addons_install=None, addons_remove=None, metadata=None):
 
189
    def apply_changes(self, pkgname, appname, iconname, addons_install=[], addons_remove=[], metadata=None):
190
190
        """ install and remove add-ons """
191
191
        self.emit("transaction-started")
192
192
        try: