[Bug + Query] Does delete_list ever used in the mailman REST client ?
![](https://secure.gravatar.com/avatar/9bcb1f3534340bd175e280f1aa14b4f3.jpg?s=120&d=mm&r=g)
12 Mar
2015
12 Mar
'15
12:13 p.m.
Hey everyone,
I want to know whether *delete_list *method in the Mailman REST client at http://bazaar.launchpad.net/~mailman-coders/mailman.client/trunk/view/head:/... ever used ? As per deleting the list is considered the List object returned from the models API which calls the REST API on the back has delete method of its own at http://bazaar.launchpad.net/~mailman-coders/mailman.client/trunk/view/head:/.... Therefore, we do not require this *delete_list *at all and this is simply unused code.
In case it is unused, I would like to file a bug and fix it. And the possible *fixes *would be:
- Use this *delete_list* method of the Client
<http://bazaar.launchpad.net/~mailman-coders/mailman.client/trunk/view/head:/src/mailmanclient/_client.py#L121>
class
and call the other by doing something like
_List(self._connection,fqdn_listname, None).delete()
in it and that would require changing MailmanRestManager class in postorius models also. - Remove the unused code.
In case it is used, I want to know the parts of the code using it.
Thanks, Ankush Sharma github.com/black-perl
3454
Age (days ago)
3454
Last active (days ago)
0 comments
1 participants
participants (1)
-
Ankush Sharma