Bug #5014
Ajouter un dialog de recherche du lieu de socialisation
90%
Description
Pour l'ajout d'un lieu de socialisation ajouter une première fenêtre de recherche du lieu dans laquelle on choisi le type, si public/privé, la ville ou le code postal, puis bouton rechercher. Cela affiche une liste de lieux. Après sélection du lieux est affiché la fenêtre pour les autres champs d'ajout de la période de socialisation.
Files
History
Updated by Jérôme Schneider over 10 years ago
- Patch proposed set to No
Petite proposition un peu différente mais qui me semble plus pratique :
- élargir le champ lieu de socialisation
- rajouter une case public / privée
- gérer la recherche dans le champ à la google (actuellement on recherche l'expression exact "collège toto"et il serait plus utile de rechercher collège et toto)
- mettre en gras les informations recherchées
Updated by Jérôme Schneider over 10 years ago
- File 0001_improved_socialisation_places_search.patch added
- Patch proposed changed from No to Yes
Un patch qui devrait corriger ce ticket. Je n'ai finalement pas rajouter la case public / privée vu que ça partie de la recherche.
Updated by Jérôme Schneider over 10 years ago
- Status changed from Nouveau to Solution déployée
- Assignee set to Jérôme Schneider
Updated by Frédéric Péters over 10 years ago
Je ne sais pas ce qui est entendu par "Solution deployed" dans le cadre de Calebasse, mais je ne vois pas le commit dans le dépôt (http://repos.entrouvert.org/calebasse.git/). Je regardais parce qu'il manque au patch attaché à ce ticket la migration qui ajoute le display_name et l'alimente comme il l'est dans le save().
Updated by Jérôme Schneider over 10 years ago
- File deleted (
0001_improved_socialisation_places_search.patch)
Updated by Jérôme Schneider over 10 years ago
- File 0001-ressources-improved-socialisation-place-search.patch 0001-ressources-improved-socialisation-place-search.patch added
- Status changed from Solution déployée to En cours
- % Done changed from 0 to 80
git diff n'ajoute pas les nouveaux fichiers. J'ai finalement utilisé un git format-patch.
Updated by Serghei Mihai over 10 years ago
Je mettrais toutes les chaînes concatenées dans display_name en unicode: par exemple u" (Public)"
Updated by Frédéric Péters over 10 years ago
Running migrations for ressources: - Migrating forwards to 0009_auto__add_field_school_display_name. > ressources:0009_auto__add_field_school_display_name FailedDryRun: ! Error found during dry run of '0009_auto__add_field_school_display_name'! Aborting. [...]
Dans ma version de django/south, on ne peut pas accéder ainsi à l'ORM depuis une migration. On peut y accéder via self.orm.School, mais ça donne la situation d'avant la migration. Pour ce qu'il faut faire ici, il faut passer par une migration supplémentaire, dont le rôle serait de passer sur tous les éléments pour en faire un save().
http://south.readthedocs.org/en/latest/tutorial/part3.html#data-migrations
Updated by Frédéric Péters over 10 years ago
Allo? Ce patch qui d'après moi ne marche pas se trouve dans le dépôt, et il y a déjà par dessus un autre patch, qui laisse penser que le commentaire de Serghei a été lu, mais pas la suite.
Updated by Jérôme Schneider over 10 years ago
On Fri, Jul 11, 2014 at 01:47:36PM +0200, redmine@entrouvert.com wrote:
Allo? Ce patch qui d'après moi ne marche pas se trouve dans le dépôt, et il y a déjà par dessus un autre patch, qui laisse penser que le commentaire de Serghei a été lu, mais pas la suite.
Désolé j'avais bien lu ton commentaire mais je suis repassé sur Vincennes.
La migration fonctionne pour Serghei et pour moi.
C'est un problème de version de South.
J'ai tout de même préparer un patch pour ça et pour forcer la version de
south en >= 0.8.4 (je n'ai juste pas eu le temps de finaliser).
Updated by Frédéric Péters over 10 years ago
Pour info ma version de south est également la 0.8.4.
Et si ça fonctionne, très bien, mais je ne vois alors pas de nécessité de patch (sauf modifier une ligne dans le requirements.txt, mais ça m'étonnerait que ça soit ça qui prenne du temps à être finalisé).
Updated by Benjamin Dauvergne about 10 years ago
Ce qu'il manque ici c'est l'option no_dry_run = True dans la migration. On peut aussi faire:
if not db.dry_rune:
....
autour du code qui modifie la base. Voir http://south.readthedocs.org/en/latest/commands.html pour --db-dry-run.
Updated by Frédéric Péters about 10 years ago
Il y a eu vendredi ce commit de Jérôme : ressources: test db.dry_run on migration 0009, http://repos.entrouvert.org/calebasse.git/commit/?id=26fb289560a20c268419d448031fc9a82a22675f
Ce ticket peut-il ainsi être fermé ?
Updated by Jérôme Schneider about 10 years ago
- Status changed from En cours to Résolu (à déployer)
- % Done changed from 80 to 90
Updated by Mikaël Ates about 10 years ago
- Status changed from Résolu (à déployer) to En cours
Lors de l'ajout d'une périodie de socialisation, le retour se fait sur l'onglet général du dossier et pas sur l'onglet socialisation.
Updated by Jérôme Schneider about 10 years ago
- Status changed from En cours to Information nécessaire
Je viens de tester sur Firefox et Chromium. J'ai ajouté des périodes de socialisations et je retourne bien dans l'onglet Socialisation et non Général. J'ai également testé avec une erreur dans le formulaire sans plus de succès.