Commit b63fc512 authored by Nicolas Joyard's avatar Nicolas Joyard

Nettoyage flake8

parent 10a60764
......@@ -14,7 +14,6 @@ from .models import db
manager = Manager(app)
migrate = Migrate(app, db)
manager.add_command('db', MigrateCommand)
......@@ -30,12 +29,14 @@ def import_etapes():
app.config.update(SQLALCHEMY_ECHO=False)
EtapesImporter(app).run()
@manager.command
def import_nd():
"""Importe les députés depuis NosDéputés.fr"""
app.config.update(SQLALCHEMY_ECHO=False)
NosDeputesImporter(app).run()
@manager.command
def import_adresses():
"""Importe adresses postales des parlementaires"""
......
......@@ -9,7 +9,7 @@ from ..models import db, Parlementaire
class AdressesImporter(BaseImporter):
def import_adresse(self, parl):
if not 'assemblee-nationale' in parl.url_off:
if 'assemblee-nationale' not in parl.url_off:
return
soup = BeautifulSoup(requests.get(parl.url_off).text, 'html5lib')
......@@ -31,7 +31,7 @@ class AdressesImporter(BaseImporter):
self.error('Erreur sur %s: %s' % (parl.url_off, e))
return
parl.adresse = '\n'.join(lines);
parl.adresse = '\n'.join(lines)
db.session.commit()
def run(self):
......
......@@ -32,7 +32,6 @@ class EtapesImporter(BaseImporter):
return created, updated
def run(self):
self.info('Début import étapes')
......
......@@ -116,7 +116,8 @@ class NosDeputesImporter(BaseImporter):
try:
data = requests.get(self.URL_DEPUTES).json()
except Exception as e:
self.error('Téléchargement %s impossible: %s' % (URL_DEPUTES, e))
self.error('Téléchargement %s impossible: %s' % (self.URL_DEPUTES,
e))
return
self.info('%s députés trouvés' % len(data['deputes']))
......@@ -173,7 +174,8 @@ class NosDeputesImporter(BaseImporter):
try:
data = requests.get(self.URL_GROUPES).json()
except Exception as e:
self.error('Téléchargement %s impossible: %s' % (URL_DEPUTES, e))
self.error('Téléchargement %s impossible: %s' % (self.URL_GROUPES,
e))
return
self.info('%s organismes trouvés' % len(data['organismes']))
......
# -*- coding: utf-8 -*-
from .database import db
from .parlementaire import Groupe, Parlementaire
from .procedure import Action, Etape
from .database import db # noqa
from .parlementaire import Groupe, Parlementaire # noqa
from .procedure import Action, Etape # noqa
......@@ -63,7 +63,8 @@ ETAPES = [
'ordre': 30,
'label': 'AR reçu',
'description': """
Le parlementaire a reçu la demande d'accès à ses relevés de comptes.
Le parlementaire a reçu la demande d'accès à ses relevés de
comptes.
""",
'couleur': '#8888dd',
},
......
# -*- coding: utf-8 -*-
from .constants import ETAPES
from .database import db
......@@ -31,5 +30,6 @@ class Action(db.Model):
etape_id = db.Column(db.Integer, db.ForeignKey('etapes.id'))
etape = db.relationship('Etape')
parlementaire_id = db.Column(db.Integer, db.ForeignKey('parlementaires.id'))
parlementaire_id = db.Column(db.Integer,
db.ForeignKey('parlementaires.id'))
parlementaire = db.relationship('Parlementaire', back_populates='actions')
......@@ -17,7 +17,7 @@ def setup(app):
'id': app.config['PIWIK_ID']
}
return {'piwik':piwik}
return {'piwik': piwik}
@app.context_processor
def inject_menu():
......
......@@ -11,10 +11,10 @@ def setup(app):
def titre_parlementaire(parl):
if parl.sexe == 'F':
return 'Madame la %s' % ('Sénatrice' if parl.chambre == 'SEN'
else 'Députée')
else 'Députée')
else:
return 'Monsieur le %s' % ('Sénateur' if parl.chambre == 'SEN'
else 'Député')
else 'Député')
@app.template_filter('fonc_parlementaire')
def fonc_parlementaire(parl):
......@@ -42,8 +42,8 @@ def setup(app):
@app.template_filter('nl2br')
@evalcontextfilter
def nl2br(eval_ctx, value):
result = u'\n\n'.join(u'%s' % p.replace('\n', '<br>\n') \
for p in _paragraph_re.split(escape(value)))
result = u'\n\n'.join(u'%s' % p.replace('\n', '<br>\n')
for p in _paragraph_re.split(escape(value)))
if eval_ctx.autoescape:
result = Markup(result)
return result
......@@ -74,7 +74,7 @@ def setup_routes(app):
def envoi(id):
try:
# SELECT FOR UPDATE sur le parlementaire pour éviter une race
# SELECT FOR UPDATE sur le parlementaire pour éviter une race
# condition sur son étape courante
parl = Parlementaire.query \
.filter_by(id=id) \
......
......@@ -2,14 +2,13 @@
from io import BytesIO
from flask import abort, make_response, render_template
from flask import make_response, render_template
from xhtml2pdf import pisa
from .util import not_found, slugify
from ..models import Parlementaire
def setup_routes(app):
@app.route('/parlementaire/<id>/demande/<mode>', endpoint='demande_pdf')
......@@ -22,8 +21,7 @@ def setup_routes(app):
slug = slugify(parl.nom_complet)
html = render_template('demande.html.j2',
parlementaire=parl
)
parlementaire=parl)
pdf = BytesIO()
pisa.CreatePDF(html, pdf)
......
# -*- coding: utf-8 -*-
from flask import abort, flash, redirect, request, session, url_for
from flask import request, session
from .util import check_email, redirect_back, sanitize
......
......@@ -4,8 +4,7 @@ import re
import unicodedata
from urllib.parse import urlparse, urljoin
from flask import (abort, flash, make_response, redirect, render_template,
request, session, url_for)
from flask import flash, redirect, request, session, url_for
SLUG_STRIP_RE = re.compile(r'[^\w\s-]')
......@@ -26,7 +25,7 @@ def is_safe_url(target):
ref_url = urlparse(request.host_url)
test_url = urlparse(urljoin(request.host_url, target))
return test_url.scheme in ('http', 'https') and \
ref_url.netloc == test_url.netloc
ref_url.netloc == test_url.netloc
def redirect_back(fallback=None, error=None):
......
......@@ -10,7 +10,6 @@ from .config import get_secret_key
from .routes import setup_routes
def setup_app(name):
# Create app
app = Flask(name)
......
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