victornicolet
commented
9 years ago On peut merger, voir commentaires en haut.
Closed victornicolet closed 9 years ago
victornicolet
commented
9 years ago On peut merger, voir commentaires en haut.
aymericbouzy
commented
9 years ago C'est vraiment mieux sur eclipse ? En soi ce n'est pas trop gênant si tu arrives à le configurer comme atom : il faut qu'une tabulation soit convertie en 2 espaces, pas plus. sinon on va vraiment se faire chier avec ça à chaque fois, ça nous fera des diffs de merde et on perdra plein de temps. Faudrait être sur aussi qu'il enlève bien les espaces inutiles à la fin des lignes. Mais je suis sur que c'est configurable, et dans ce cas pas de problème. A part ça, c'est cool que tu utilises le grid system de bootstrap :) Tu comptes l'utiliser de plus en plus partout ?
victornicolet
commented
9 years ago Éclipse apporte le confort d'un IDE, je ai plus a aller chercher dans quel ordre sont les arguments d'une fonction, l'auto complétion est efficace, tu paix aller directement a la décoration d'une fonction, les erreurs de syntaxe sont surlignées ( mieux que dans atom qui le fait un peu ), etc. La tabulation c'est bon je l'ai réglée, je vais voir pour les espaces en fin de ligne.
Oui je vais l'utiliser des que possible. Il y a des endroits ou ça va être un peu chiant par exemple les show avec les boutons ronds, mais sinon c'est juste un changement de div encapsulants
victornicolet
commented
9 years ago Une autre remaque générale sur la forme du code , je pense qu'il faudrait essayer de respecter la "longueur de ligne maximum", c'est assez moche ces lignes interminables...
victornicolet
commented
9 years ago C'est bon pour les espaces en fin de ligne : à chaque sauvegarde eclipse les élimine.
aymericbouzy
commented
9 years ago Ok cool ! Pour la longueur de ligne maximum, c'est vrai que c'est un peu un problème ... En même temps, c'est pire de faire un retour à la ligne qui ne veut rien dire. Il faudrait que l'éditeur de texte mette la ligne sur plusieurs lignes de lui-même, sans que ça n'affecte le code.
aymericbouzy
commented
9 years ago du coup si tu as bien configuré eclipse, tu peux essayer de corriger le fichier review_form ? après je merge.
victornicolet
commented
9 years ago Pour les saut de lignes, on peut par exemple couper entre chaque argument d'une fonction, couper les chaines de caractères trop longues, instancier les arguments au lieu d'écrire une fonction avec des arguments trop longs.
victornicolet
commented
9 years ago On peut merger du coup ?
On peut merger, si vous êtes d'accord.
J'ai fait une nouvelle branche pendant que je travaillais dans le train.
Quelques petites corrections + j'ai mis le projet dans eclipse comme ça je m'y retrouve mieux dans les fonctions.
J'ai rajouté des infos sur les subventions dans le formulaire d'attribution des requêtes request/review_form . Il n'y a pas de vérification des droits, étant donné qu'un binet accédant à un formulaire d'attribution pour une requête a les droits de voir ce que le binet fait ! Je ne sais pas si il faut re-checker les droits...