On 10/16/07, David Larlet <[EMAIL PROTECTED]> wrote: > 2007/10/15, Joseph Heck <[EMAIL PROTECTED]>: > > > > Hi David, > > > > If you'd be willing to indulge me, why doesn't DQS doesn't fit your > > requirements? I'd love to have some explicit feedback on where it > > could be improved to match your needs. And yes, I know there's a LOT > > of places where it could be improved... :-) > > > > Hi Joe, > > Here are a couple of suggestions of what I'd like to do if I need to > use this project: > > * Turns it to a RESTful one which mean at least use HTTP verbs instead > of /delete/ and so one in URLs, I will probably use > django-rest-interface to do that. > * Move doctests to the tests.py file and add more tests before the next point. > * Review some parts of the code, for example use queryset.count() > instead of len(queryset), or change this part: > > q = None > try: > q = Queue.objects.get(name=queue_name) > except Queue.DoesNotExist: > pass > if q is None: > return None > > To this one: > > try: > q = Queue.objects.get(name=queue_name) > except Queue.DoesNotExist: > return None > > I don't know if it works but it seems more natural to me.
Thanks for the feedback! We (well Rajesh and Marcus) have been adding a huge number of tests in a development branch, so you'll be seeing that fairly shortly. I appreciate the request for RESTful verbs in the environment, we'll definitely look into that. If I'm reading it correctly, the lack of RESTful verbs was your primary consideration for what the API to the queue service failed to provide. Is that correct? --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django users" group. To post to this group, send email to django-users@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-users?hl=en -~----------~----~----~----~------~----~------~--~---
