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.ec...@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:/src/mailmanclient/_client.py#L254 > 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:/src/mailmanclient/_client.py#L549. > 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: > > 1. 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. > 2. 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 > _______________________________________________ Mailman-Developers mailing list Mailman-Developers@python.org https://mail.python.org/mailman/listinfo/mailman-developers Mailman FAQ: http://wiki.list.org/x/AgA3 Searchable Archives: http://www.mail-archive.com/mailman-developers%40python.org/ Unsubscribe: https://mail.python.org/mailman/options/mailman-developers/archive%40jab.org Security Policy: http://wiki.list.org/x/QIA9
