Merge lp:~pietroalbini/ubuntu-it-ask/fix-lp1197300 into lp:ubuntu-it-ask

[Pietro Albini]

Removed lenght limit for flags

[Milo Casagrande (milo)]

Questa modifica è stata provata in locale o su un'istanza di test?

Quando si tratta di modificare una classe in models/ è sempre meglio controllare due volte. Qui stai modificando la lunghezza del campo nel database. È effettivamente quello che vuoi fare o la soluzione al problema originale?

Potrebbe non essere un problema fare il deploy con questa modifica, sopratutto se l'applicazione usa South per gestire le migrazioni del DB, ma va assolutamente verificato.

review: Needs Information

[Pietro Albini (pietroalbini)]

Ho verificato in locale e andava, adesso pensavo di inviarlo sul server di
test di Riccardo (appena può) prima di caricarlo in produzione.

[Milo Casagrande (milo)]

Andava nel senso che il bug originale non si verifica più?
C'è modo di riprodurlo ora sul server di test?

Il valore predefinito per max_length quando non è specificato, se non erro, dovrebbe essere 255. Nel caso si debbano inserire più caratteri forse conviene trasformare quel CharField in un TextField.

[Pietro Albini (pietroalbini)]

Ho provato a fare una segnalazione lunghissima e andava.
Comunque il server di test sarà disponibile solo quando riccardo tornerà.

[Milo Casagrande (milo)]

È stata più fatta una prova sul server di test?

Unmerged revisions

81. By Pietro Albini on 2013-07-03

Removed lenght limit for flags

=== modified file 'forum/models/meta.py'
--- forum/models/meta.py	2012-06-22 20:37:14 +0000
+++ forum/models/meta.py	2013-07-03 10:08:23 +0000
@@ -16,7 +16,7 @@
 class Flag(models.Model):
     user = models.ForeignKey(User, related_name="flags")
     node = models.ForeignKey(Node, related_name="flags")
-    reason = models.CharField(max_length=300)
+    reason = models.CharField()
     action = models.OneToOneField(Action, related_name="flag")
     flagged_at = models.DateTimeField(default=datetime.datetime.now)
 
@@ -79,4 +79,4 @@
 
     class Meta:
         unique_together = ('user', 'badge', 'node')
-        app_label = 'forum'
\ No newline at end of file
+        app_label = 'forum'