Well discussed this on the mailing list with Abhilash Raj and have no more queries regarding this.
On Thu, Mar 12, 2015 at 11:43 AM, Ankush Sharma < ankush.sharma.ece12@iitbhu.ac.in> wrote:
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