Commit 1f590ffc authored by Maxime Bombar's avatar Maxime Bombar

[forms.py] On utilise .format() au lieu de %s pour formatter les strings

parent b8a1b7e0
......@@ -244,14 +244,14 @@ class SearchHistoriquePseudoForm(BootstrapForm):
# Les formats de timestamps acceptés
# Les dates
### Les %s sont là parce qu'on a besoin de placer les groupes à deux endroits
### Les {} sont là parce qu'on a besoin de placer les groupes à deux endroits
### dans la regexp, et ça nécessite qu'ils aient deux noms différents
year_match = ur'(?P<y%s>[0-9]{2}|[0-9]{4})'
ym1, ym2 = year_match % (1), year_match % (2)
month_match = ur'(?P<m%s>[0-9]{1,2})'
mm1, mm2 = month_match % (1), month_match % (2)
day_match = ur'(?P<d%s>[0-9]{1,2})'
dm1, dm2 = day_match % (1), day_match % (2)
year_match = ur'(?P<y{}>[0-9]{{2}}|[0-9]{{4}})'
ym1, ym2 = year_match.format(1), year_match.format(2)
month_match = ur'(?P<m{}>[0-9]{{1,2}})'
mm1, mm2 = month_match.format(1), month_match.format(2)
day_match = ur'(?P<d{}>[0-9]{{1,2}})'
dm1, dm2 = day_match.format(1), day_match.format(2)
date_match = ur'^(?:' + ym1 + '-' + mm1 + '-' + dm1 + '|' + dm2 + '/' + mm2 + '(?:/' + ym2 + ')?)$'
date_matcher = re.compile(date_match)
......@@ -320,7 +320,7 @@ class MyDateField(forms.CharField):
raise forms.ValidationError(_(u'"{}" ne peut pas être transformé en date : "{}"'.format(raw_value, exc)))
return data
else:
raise forms.ValidationError(u'"%s" ne peut pas être transformé en date (essaye le format JJ/MM/AAAA)' % (raw_value))
raise forms.ValidationError(u'"{}" ne peut pas être transformé en date (essaye le format JJ/MM/AAAA)'.format(raw_value))
class MyTimeField(forms.CharField):
"""Un champ personnalisé de détection d'heure.
......@@ -340,9 +340,9 @@ class MyTimeField(forms.CharField):
try:
return datetime.time(data["H"], data["M"], data["S"])
except Exception as exc:
raise forms.ValidationError(u'"%s" ne peut pas être transformé en heure : "%s"' % (raw_value, exc))
raise forms.ValidationError(u'"{}" ne peut pas être transformé en heure : "{}"'.format(raw_value, exc))
else:
raise forms.ValidationError(u'"%s" ne peut pas être transformé en heure (essaye le format HH:MM:SS)' % (raw_value))
raise forms.ValidationError(u'"{}" ne peut pas être transformé en heure (essaye le format HH:MM:SS)'.format(raw_value))
class ActiviteForm(BootstrapForm):
"""Formulaire pour créer ou modifer une activité"""
......@@ -359,7 +359,7 @@ class ActiviteForm(BootstrapForm):
listeimprimee = forms.BooleanField(label=_(u"Liste imprimée"), required=False)
def clean(self):
"""Récupère les données et fusionne les dates et les heures
pour finalement donner une chaîne %Y-%m-%d %H:%M:%S"""
pour finalement donner une chaîne %Y-%m-%d %H:%M:{}"""
out = super(ActiviteForm, self).clean()
# à ce stade, out ne contient pas forcément tous les champs si certains ont été fournis vides
if not set(["debut_date", "debut_time", "fin_date", "fin_time"]).issubset(out.keys()):
......@@ -370,7 +370,7 @@ class ActiviteForm(BootstrapForm):
raise forms.ValidationError(_(u"La méthode DeLorean_TimeTravel() n'est pas encore implémentée dans la Note Kfet 2015, merci de réessayer plus tard ou bien de te résoudre à faire commencer ton activité avant qu'elle ne soit finie."))
else:
del out["debut_date"], out["debut_time"], out["fin_date"], out["fin_time"]
out["debut"], out["fin"] = debut.strftime("%Y-%m-%d %H:%M:%S"), fin.strftime("%Y-%m-%d %H:%M:%S")
out["debut"], out["fin"] = debut.strftime("%Y-%m-%d %H:%M:{}"), fin.strftime("%Y-%m-%d %H:%M:{}")
return out
def __init__(self, *args, **kwargs):
......@@ -383,9 +383,9 @@ class ActiviteForm(BootstrapForm):
champs = kwargs['initial'].keys()
for champ_date in ["debut", "fin"]:
if champ_date in champs:
date = time.strptime(kwargs['initial'][champ_date],"%Y-%m-%d %H:%M:%S")
date = time.strptime(kwargs['initial'][champ_date],"%Y-%m-%d %H:%M:{}")
# On décompose le timestamp en date/heure
kwargs['initial'][champ_date + "_date"], kwargs['initial'][champ_date + '_time'] = time.strftime("%d/%m/%Y %H:%M:%S", date).split(" ")
kwargs['initial'][champ_date + "_date"], kwargs['initial'][champ_date + '_time'] = time.strftime("%d/%m/%Y %H:%M:{}", date).split(" ")
del kwargs['initial'][champ_date]
# Il faut aussi faire en sorte que le champ "liste imprimée" ne soit pas toujours disponible
if 'listeimprimee' in kwargs.keys():
......@@ -437,7 +437,7 @@ class PhotoForm(forms.Form):
if out.has_key("photo"):
photo = out["photo"]
if photo != None and photo.size > settings.MAX_PHOTO_SIZE:
raise forms.ValidationError(u"Photo trop volumineuse (%s octets), maximum %s" % (photo.size, settings.MAX_PHOTO_SIZE))
raise forms.ValidationError(u"Photo trop volumineuse ({} octets), maximum {}".format(photo.size, settings.MAX_PHOTO_SIZE))
return out
class MoneyForm(BootstrapForm):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment