Project

General

Profile

Development #65191

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

Added by Thomas Noël 17 days ago. Updated 12 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
12 May 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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)


Files

Associated revisions

Revision b6b9b181 (diff)
Added by Benjamin Dauvergne 16 days ago

arcgis: fix migration detection failure (#65191)

History

#1

Updated by Benjamin Dauvergne 17 days ago

  • Assignee set to Benjamin Dauvergne
#2

Updated by Benjamin Dauvergne 17 days ago

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

Updated by Thomas Noël 16 days ago

Œ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

Updated by Benjamin Dauvergne 16 days ago

C'est vert maintenant.

#5

Updated by Thomas Noël 16 days ago

  • Status changed from Solution proposée to Solution validée
#6

Updated by Benjamin Dauvergne 16 days ago

  • Status changed from Solution validée to 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

Updated by Transition automatique 12 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF