Use specific exception for MaillingList sync failure

For more fine exception catching

maillists/models.py

@@ -12,6 +12,9 @@ from django.dispatch import receiver
 from coin.members.models import Member
 
 
+class SyncCommandError(SystemError):
+    pass
+
 class MaillingList(models.Model):
     short_name = models.CharField(
         'identifiant technique', max_length=50,
@@ -60,9 +63,10 @@ class MaillingList(models.Model):
                 text_listing = ''
             else:
                 text_listing = self.as_text_listing()
+
             out_stdout, out_stderr = p.communicate(text_listing)
             if p.returncode != 0:
-                raise SystemError(
+                raise SyncCommandError(
                     "Erreur à l'appel de la commande : \"{}\"".format(
                         out_stderr.decode('utf-8')))
 

maillists/views.py

@@ -7,11 +7,9 @@ from django import forms
 from django.forms import formset_factory
 from django.shortcuts import render, redirect
 
-from .models import MaillingList
+from .models import MaillingList, SyncCommandError
 from coin.members.models import Member
 
-#MaillingListFormSet = modelformset_factory(MaillingList, fields=)
-
 
 class SubscriptionForm(forms.Form):
     subscribed = forms.BooleanField(widget=forms.Select(choices=(
@@ -58,7 +56,7 @@ def lists_list(request):
 
                 for mail_list in old_subscriptions - new_subscriptions:
                     request.user.subscribed_maillinglists.remove(mail_list)
-            except SystemError as e:
+            except SyncCommandError as e:
                 messages.error(
                     request,
                     "Impossible de sauvegarder tes abonnements. "