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

Proposed by Pietro Albini
Status: Needs review
Proposed branch: lp:~pietroalbini/ubuntu-it-ask/fix-lp1197300
Merge into: lp:ubuntu-it-ask
Diff against target: 19 lines (+2/-2)
1 file modified
forum/models/meta.py (+2/-2)
To merge this branch: bzr merge lp:~pietroalbini/ubuntu-it-ask/fix-lp1197300
Reviewer Review Type Date Requested Status
Milo Casagrande (community) Needs Information
Review via email: mp+172778@code.launchpad.net

Description of the change

Removed lenght limit for flags

To post a comment you must log in.
Revision history for this message
Milo Casagrande (milo) wrote :

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
Revision history for this message
Pietro Albini (pietroalbini) wrote :

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

Revision history for this message
Milo Casagrande (milo) wrote :

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.

Revision history for this message
Pietro Albini (pietroalbini) wrote :

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

Revision history for this message
Milo Casagrande (milo) wrote :

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

Unmerged revisions

81. By Pietro Albini

Removed lenght limit for flags

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'forum/models/meta.py'
2--- forum/models/meta.py 2012-06-22 20:37:14 +0000
3+++ forum/models/meta.py 2013-07-03 10:08:23 +0000
4@@ -16,7 +16,7 @@
5 class Flag(models.Model):
6 user = models.ForeignKey(User, related_name="flags")
7 node = models.ForeignKey(Node, related_name="flags")
8- reason = models.CharField(max_length=300)
9+ reason = models.CharField()
10 action = models.OneToOneField(Action, related_name="flag")
11 flagged_at = models.DateTimeField(default=datetime.datetime.now)
12
13@@ -79,4 +79,4 @@
14
15 class Meta:
16 unique_together = ('user', 'badge', 'node')
17- app_label = 'forum'
18\ No newline at end of file
19+ app_label = 'forum'

Subscribers

People subscribed via source and target branches