More helpful error messages
authorGu1 <gu1@cafai.fr>
Sat, 27 Apr 2013 14:31:43 +0000 (16:31 +0200)
committerGu1 <gu1@cafai.fr>
Sat, 27 Apr 2013 14:31:43 +0000 (16:31 +0200)
If the eligibility server return an error message, print it instead of
just saying that an error occurred.

fdneligibility/locale/fr/LC_MESSAGES/django.mo
fdneligibility/locale/fr/LC_MESSAGES/django.po
fdneligibility/templates/fdneligibility/step2.html
fdneligibility/views.py

index 0e8b1ec..08028d8 100644 (file)
Binary files a/fdneligibility/locale/fr/LC_MESSAGES/django.mo and b/fdneligibility/locale/fr/LC_MESSAGES/django.mo differ
index 6a6e4e7..a4f2dce 100644 (file)
@@ -3,8 +3,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: django-fdn-eligibility\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2013-04-27 15:30+0200\n"
-"PO-Revision-Date: 2013-04-27 15:35+0200\n"
+"POT-Creation-Date: 2013-04-27 16:28+0200\n"
+"PO-Revision-Date: 2013-04-27 16:30+0200\n"
 "Last-Translator: gu1 <gu1@cafai.fr>\n"
 "Language: fr\n"
 "MIME-Version: 1.0\n"
@@ -40,27 +40,27 @@ msgstr ""
 "Le serveur de test d'éligibilité a retourné une erreur, veuillez réessayez "
 "ultérieurement. Si le problème persiste, contactez nous."
 
-#: views.py:102
+#: views.py:103
 msgid "partially unbundled by Nerim"
 msgstr "dégroupé partiellement par Nerim"
 
-#: views.py:104
+#: views.py:105
 msgid "partially unbundled"
 msgstr "dégroupé partiellement"
 
-#: views.py:106
+#: views.py:107
 msgid "not unbundled (ACA)"
 msgstr "non dégroupé (ACA)"
 
-#: views.py:108
+#: views.py:109
 msgid "not unbundled (IP/ADSL)"
 msgstr "non dégroupé (IP/ADSL)"
 
-#: views.py:110
+#: views.py:111
 msgid "unknown"
 msgstr "dégroupage inconnu"
 
-#: views.py:118
+#: views.py:119
 msgid "Could not test your eligibility"
 msgstr "Impossible de vérifier votre éligibilité auprès du serveur"
 
@@ -122,25 +122,45 @@ msgstr "mètres"
 msgid "Unfortunately, you are not eligible to our ADSL service :("
 msgstr "Malheureusement, vous ne semblez pas éligible à notre service ADSL :("
 
-#: templates/fdneligibility/step2.html:62
+#: templates/fdneligibility/step2.html:63
+msgid "The eligibility test server returned an error:"
+msgstr "Le serveur de test d'éligibilité a retourné une erreur:"
+
+#: templates/fdneligibility/step2.html:65
+msgid ""
+"We remind you that in order to test your eligibility, you must have an "
+"active France Télécom phone line. The test <strong>won't work</strong> if "
+"your phone line is already unbundled by another operator (Free, SFR...). You "
+"can always test the phone number of a neighbor if you only want to check "
+"your eligibility and don't plan on subscribing yet."
+msgstr ""
+"Nous vous rappelons que pour tester votre éligibilité, vous devez avoir une "
+"ligne téléphone France Télécom active. Le test <strong>ne fonctionnera pas</"
+"strong> si votre ligne est dégroupée par un opérateur alternatif (Free, "
+"SFR...). Vous pouvez toujours tester le numéro d'un voisin si vous souhaitez "
+"simplement tester votre éligibilité mais ne comptez pas vous abonner dans "
+"l'immédiat."
+
+#: templates/fdneligibility/step2.html:67
 msgid "Possible reasons for this are:"
 msgstr "Les raisons les plus courantes sont:"
 
-#: templates/fdneligibility/step2.html:63
+#: templates/fdneligibility/step2.html:68
 msgid ""
 "Your phone line could already be unbundled by another ISP, in which case you "
-"would have to cancel your subscription with them in order to test your "
-"eligibility and subscribe. An easier solution would be to test the phone "
-"number of a neighbor if you only want to check your eligibility and don't "
-"plan on subscribing."
+"would have to cancel your subscription with them and open a France Télécom "
+"phone line in order to test your eligibility and subscribe. An easier "
+"solution would be to test the phone number of a neighbor if you only want to "
+"check your eligibility and don't plan on subscribing yet."
 msgstr ""
 "Votre ligne téléphonique est peut-être déjà dégroupée par un autre "
 "opérateur, auquel cas vous devrez libérer la ligne en résiliant l'abonnement "
-"existant pour pouvoir tester votre éligibilité et vous abonner. Une solution "
-"moins contraignante si vous souhaitez simplement tester votre éligibilité "
-"mais ne comptez pas vous abonner, serait de tester le numéro d'un voisin."
+"existant et souscrire à un abonnement France Télécom pour pouvoir tester "
+"votre éligibilité et vous abonner. Une solution moins contraignante si vous "
+"souhaitez simplement tester votre éligibilité mais ne comptez pas vous "
+"abonner, serait de tester le numéro d'un voisin."
 
-#: templates/fdneligibility/step2.html:64
+#: templates/fdneligibility/step2.html:69
 msgid ""
 "You could also be in a \"white area\" with no ADSL, in which case we can't "
 "help you."
@@ -148,17 +168,14 @@ msgstr ""
 "Peut-être habitez-vous dans une zone blanche non couverte par l'ADSL, auquel "
 "cas, nous ne pouvons rien pour vous."
 
-#: templates/fdneligibility/step2.html:65
+#: templates/fdneligibility/step2.html:71
 msgid ""
 "You should try to <a href=\"/contact/\">get in touch</a> with us directly to "
 "see if we can help."
 msgstr ""
-"Merci de nous <a href=\"/contact/\">contacter</a> directement pour voir si "
-"nous pouvons aider."
+"Si vous avez besoin d'aide ou de conseils supplémentaires, n'hésitez pas à "
+"nous <a href=\"/contact/\">contacter</a> directement."
 
-#: templates/fdneligibility/step2.html:71
+#: templates/fdneligibility/step2.html:77
 msgid "Offers you are not eligible to"
 msgstr "Offres auquelles vous n'êtes pas éligibles"
-
-#~ msgid "FUCK YOU"
-#~ msgstr "LOL"
index 2196613..56de35c 100644 (file)
@@ -40,7 +40,7 @@ var line_attenuation = {{line_info.attenuation}};
 </script>
 {% endblock %}
 {% block formeligibility %}
-<h3>Informations sur la ligne</h3>
+{% if si_res.erreur == 0 %}<h3>Informations sur la ligne</h3>
 <div id="adsl-speed-chart-container">
     <div id="adsl-speed-chart"></div>
 </div>
@@ -50,7 +50,7 @@ var line_attenuation = {{line_info.attenuation}};
 <li><p><strong>{% trans 'Length' %}</strong>: {{ line_info.length }} {% trans 'meters' %}</p></li>
 </ul>
 <div class="line-separator" style="margin: 15px 0;"></div>
-{{ block.super }}
+{% endif %}{{ block.super }}
 {% endblock %}
 
 {% block form_fields %}
@@ -59,9 +59,15 @@ var line_attenuation = {{line_info.attenuation}};
 {% else %}
 <div class="alert alert-error">
     <h4>{% trans "Unfortunately, you are not eligible to our ADSL service :(" %}</h4>
+    {% if si_res.erreur %}
+    <p style="margin-top: 15px;">{% trans "The eligibility test server returned an error:"%}</p>
+    <p>"<b>{{ si_res.message.strip }}</b>"</p>
+    <p>{% blocktrans %}We remind you that in order to test your eligibility, you must have an active France Télécom phone line. The test <strong>won't work</strong> if your phone line is already unbundled by another operator (Free, SFR...). You can always test the phone number of a neighbor if you only want to check your eligibility and don't plan on subscribing yet.{% endblocktrans %}</p>
+    {% else %}
     <p style="margin-top: 15px;">{% trans "Possible reasons for this are:"%}</p>
-    <p>{% blocktrans %}Your phone line could already be unbundled by another ISP, in which case you would have to cancel your subscription with them in order to test your eligibility and subscribe. An easier solution would be to test the phone number of a neighbor if you only want to check your eligibility and don't plan on subscribing.{% endblocktrans %}</p>
+    <p>{% blocktrans %}Your phone line could already be unbundled by another ISP, in which case you would have to cancel your subscription with them and open a France Télécom phone line in order to test your eligibility and subscribe. An easier solution would be to test the phone number of a neighbor if you only want to check your eligibility and don't plan on subscribing yet.{% endblocktrans %}</p>
     <p>{% blocktrans %}You could also be in a "white area" with no ADSL, in which case we can't help you.{% endblocktrans %}</p>
+    {% endif %}
     <p>{% blocktrans %}You should try to <a href="/contact/">get in touch</a> with us directly to see if we can help.{% endblocktrans %}</p>
 </div>
 {% endif %}
index a5a4bff..1cc17fc 100644 (file)
@@ -86,10 +86,11 @@ class EligibilityWizard(SessionWizardView):
             zip_code     = form.cleaned_data.get('zip_code')
             try:
                 si_res=contact_si_helper(phone_number, zip_code)
-                if si_res['erreur'] != 0 or 'info_ligne' not in si_res or 'offres' not in si_res:
+                if 'info_ligne' not in si_res or 'offres' not in si_res:
                     form._errors[NON_FIELD_ERRORS] = form.error_class([_(u"The eligibility test server returned an error, please try again "
                                                                           "at a later time. If the problem persist, contact us.")])
                     return self.render(form)
+                si_res.setdefault('erreur', 0)
                 for o in si_res['offres']:
                     o['fas']=Decimal(o['fas'].replace('E TTC', u''))
                     o['abo']=Decimal(o['abo'].replace('E TTC', u''))