Revision 02990794 snf-cyclades-app/synnefo/userdata/rest.py

b/snf-cyclades-app/synnefo/userdata/rest.py
32 32
# interpreted as representing official policies, either expressed
33 33
# or implied, of GRNET S.A.
34 34

  
35
import logging
36

  
35 37
from django import http
36 38
from django.utils import simplejson as json
37 39
from django.core.urlresolvers import reverse
......
56 58
        Constructor. Called in the URLconf; can contain helpful extra
57 59
        keyword arguments, and other things.
58 60
        """
61
        self.logger = logging.getLogger(self.model._meta.db_table)
59 62
        # Go through keyword arguments, and either save their values to our
60 63
        # instance, or raise an error.
61 64
        for key, value in kwargs.items():
......
194 197
        self.update_instance(instance, data, self.exclude_fields)
195 198
        instance.full_clean()
196 199
        instance.save()
200
        self.logger.info("Instance [%d] updated", instance.pk)
197 201
        return self.GET(request, data, *args, **kwargs)
198 202

  
199 203
    def DELETE(self, request, data, *args, **kwargs):
200
        self.instance().delete()
204
        instance = self.instance()
205
        pk = instance.pk
206
        instance.delete()
207
        self.logger.info("Instance [%d] removed", pk)
201 208
        return self.json_response("")
202 209

  
203 210

  
......
219 226
        self.update_instance(instance, data, self.exclude_fields)
220 227
        instance.full_clean()
221 228
        instance.save()
229
        self.logger.info("Instance [%d] created", instance.pk)
222 230
        return self.json_response(
223 231
            self.instance_to_dict(instance, self.exclude_fields))
224 232

  
......
246 254
        instance.user = request.user_uniq
247 255
        instance.full_clean()
248 256
        instance.save()
257
        self.logger.info("Instance [%d] created", instance.pk)
249 258
        return self.json_response(
250 259
            self.instance_to_dict(instance, self.exclude_fields))

Also available in: Unified diff