Project

General

Profile

Bug #5014

Ajouter un dialog de recherche du lieu de socialisation

Added by Mikaël Ates almost 10 years ago. Updated almost 10 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Jérôme Schneider
Start date:
20 June 2014
Due date:
% Done:

90%

Estimated time:
Patch proposed:
Yes
Planning:

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

#1

Updated by Jérôme Schneider almost 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
#2

Updated by Jérôme Schneider almost 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.

#3

Updated by Jérôme Schneider almost 10 years ago

  • Status changed from Nouveau to Solution déployée
  • Assignee set to Jérôme Schneider
#4

Updated by Frédéric Péters almost 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().

#5

Updated by Jérôme Schneider almost 10 years ago

  • File deleted (0001_improved_socialisation_places_search.patch)
#6

Updated by Jérôme Schneider almost 10 years ago

git diff n'ajoute pas les nouveaux fichiers. J'ai finalement utilisé un git format-patch.

#7

Updated by Serghei Mihai almost 10 years ago

Je mettrais toutes les chaînes concatenées dans display_name en unicode: par exemple u" (Public)"

#8

Updated by Frédéric Péters almost 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

#9

Updated by Frédéric Péters almost 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.

#10

Updated by Jérôme Schneider almost 10 years ago

On Fri, Jul 11, 2014 at 01:47:36PM +0200, 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).

#11

Updated by Frédéric Péters almost 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é).

#12

Updated by Benjamin Dauvergne almost 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.

#13

Updated by Frédéric Péters almost 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é ?

#14

Updated by Jérôme Schneider almost 10 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 80 to 90
#15

Updated by Mikaël Ates almost 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.

#16

Updated by Jérôme Schneider almost 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.

#17

Updated by Mikaël Ates almost 10 years ago

  • Status changed from Information nécessaire to Fermé

Ok

Also available in: Atom PDF