Projet

Général

Profil

Development #65191

curiosité : makemigrations arcgis ajoute sans arrêt une migration

Ajouté par Thomas Noël il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 mai 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Peut-être un bogue dans le Django 2.26 que nous utilisons :

$ passerelle-manage makemigrations arcgis
Migrations for 'arcgis':
  migrations/0008_auto_20220512_1705.py
    - Alter field resource on query

$ passerelle-manage makemigrations arcgis
Migrations for 'arcgis':
  migrations/0009_auto_20220512_1705.py
    - Alter field resource on query

et c'est sans fin, avec toujours la même migration :

    operations = [
        migrations.AlterField(
            model_name='query',
            name='resource',
            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='queries', to='arcgis.ArcGIS', verbose_name='Resource'),
        ),

qui est une migration qui existe déjà dans 0006_auto_20200401_1025.py (et qui n'était déjà peut-être pas nécessaire après la création dans 0005)


Fichiers

Révisions associées

Révision b6b9b181 (diff)
Ajouté par Benjamin Dauvergne il y a presque 2 ans

arcgis: fix migration detection failure (#65191)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

Un souci de sensibilité à la casse, il semblerait que la vérification soit sensible à la casse lorsqu'il compare ForeignKey.to mais pas entre la migration de création du modèle et le nom de la classe.

(Pdb) p old_field_dec
('django.db.models.ForeignKey', [], {'verbose_name': 'Resource', 'related_name': 'queries', 'on_delete': <function CASCADE at 0x7fd51085bf40>, 'to': 'arcgis.Arcgis'})
(Pdb) p new_field_dec
('django.db.models.ForeignKey', [], {'verbose_name': 'Resource', 'related_name': 'queries', 'on_delete': <function CASCADE at 0x7fd51085bf40>, 'to': 'arcgis.ArcGIS'})
(Pdb)

#3

Mis à jour par Thomas Noël il y a presque 2 ans

Œil de lynx, bravo !

(a priori c'est rouge parce que pylint « passerelle/apps/arcgis/migrations/0006_auto_20200401_1025.py:5: [W0611, ] Unused import django.db.models.deletion »)

#4

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

C'est vert maintenant.

#5

Mis à jour par Thomas Noël il y a presque 2 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 91aeea79d99329a6d1527552f1fb48ccdd56b319
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu May 12 21:08:19 2022 +0200

    arcgis: fix migration detection failure (#65191)
#7

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#8

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF