Répare le compte de caractères des contenus #4171
Conversation
vhf
added
C-Back
| """ | ||
|
|
||
| def handle(self, *args, **options): | ||
| for content in PublishedContent.objects.filter(nb_letter=None): | ||
| content.nb_letter = content.get_nb_letters() | ||
| for content in PublishedContent.objects.filter(char_count=None, must_redirect=False): |
There was a problem hiding this comment.
Le problème était là : les contenus redirigés plantaient la commande.
| @@ -126,7 +126,7 @@ def publish_content(db_object, versioned, is_major_update=True): | |||
| public_version.content = db_object | |||
| public_version.must_reindex = True | |||
| public_version.save() | |||
| public_version.nb_letter = public_version.get_nb_letters(md_file_path) | |||
| public_version.char_count = public_version.get_char_count(md_file_path) | |||
There was a problem hiding this comment.
Il ne manquerais pas une migration ? J'ai checkout ta branche, il me met « no such colomn tutorialv2_publishedcontent.char_count »
There was a problem hiding this comment.
Ça doit être mon installation qui à foirer.
|
QA: Ok sauf cette histoire de fichier de migration. Mon fichier de migration généré : # -*- coding: utf-8 -*-
from __future__ import unicode_literals
from django.db import migrations, models
class Migration(migrations.Migration):
dependencies = [
('tutorialv2', '0015_publishedcontent_nb_letter'),
]
operations = [
migrations.RemoveField(
model_name='publishedcontent',
name='nb_letter',
),
migrations.AddField(
model_name='publishedcontent',
name='char_count',
field=models.IntegerField(default=None, null=True, verbose_name=b'Nombre de lettres du contenu', blank=True),
),
] |
|
Il y a peut-être un bug quelque part. J'ai comparé la migration que tu proposes avec celle de cette PR, artragis aussi, personne n'a trouvé de différence. Est-ce qu'elle n'est pas détectée par Django ? Tu peux essayer de la renommer ? |
|
Excuse moi, je sais pas ce qui s'est passé, quand j'ai fetch la branche et j'ai fait un migrate, j'ai pas vu le fichier de migration et il me l'a réclamé. Peut-être qu'à force de tester les PR, j'ai fait merder mes migrations ou la base de données. Autant pour moi. |
|
Haha pas grave ! Du coup si c'est bon je te laisse merger ça. Merci ! |
vhf
added
S-BUG
Hope, une 2e PR pour corriger #4093
QA
None(mettre à NULL si besoin)adjust_char_count