Project

General

Profile

Segnalazione #61

Modifica "subject taught" a campo foreign

Added by Marco Marinello about 5 years ago. Updated over 4 years ago.

Status:
Commenti
Priority:
Bassa
Target version:
-
Start date:
07/02/2016
Due date:
% Done:

0%

Resolution:

Description

Modificare nei teachers la subject taught a riferimento ad un'altra tabella (subjects)

Associated revisions

Revision 875749a9 (diff)
Added by Marco Marinello about 5 years ago

Aggiunta tabella materie
refs #61

Revision a4ca567e (diff)
Added by Marco Marinello about 5 years ago

Aggiunta materie a modelli amministrati
refs #61

Revision 7f71987b (diff)
Added by Marco Marinello about 5 years ago

Ricreazione migrazioni
refs #61

Revision a67e30c2 (diff)
Added by Marco Marinello about 5 years ago

Aggiornamento locale
refs #61

History

#1 Updated by Marco Marinello about 5 years ago

  • Status changed from Nuovo to Risolto
  • Resolution set to fixed

Creata tabella, collegata e amministrata

#2 Updated by Marco Marinello about 5 years ago

  • Status changed from Risolto to Chiuso

#3 Updated by Christopher R. Gabriel about 5 years ago

  • Status changed from Chiuso to In elaborazione
  • Resolution deleted (fixed)

Da revertare e spostare in branch separato, e documentare le motivazioni della scelta.

#4 Updated by Christopher R. Gabriel about 5 years ago

  • Target version deleted (django1.9)

#5 Updated by Marco Marinello about 5 years ago

  • Status changed from In elaborazione to Commenti
  • Assignee deleted (Marco Marinello)
  • Priority changed from Normale to Bassa

Revertate tutte le modifiche in attesa di future decisioni

#6 Updated by Christopher R. Gabriel about 5 years ago

  • Assignee set to Marco Marinello

Intanto, puoi iniziare a presentare le motivazioni alla base della modifica.

#7 Updated by Marco Marinello about 5 years ago

Il campo correlato all'altra tabella consentirebbe di dare uno `standard` alle materie di insegnamento. Il campo di testo è facilmente `aggirabile`

#8 Updated by Christopher R. Gabriel about 5 years ago

E chi compila l'elenco delle materie e decide quali sono? Quindi e' prevista una Many2Many in questa relazione?

#9 Updated by Marco Marinello about 5 years ago

Questo bisogna chiederlo agli utenti finali

#10 Updated by Christopher R. Gabriel about 5 years ago

E quindi l'implementazione precedente su cosa era basata? :)

#11 Updated by Christopher R. Gabriel about 5 years ago

(in ogni caso, agli utenti finali non chiedi che tipo di relazione va messa sul database, quello lo decidono i tecnici.. agli utenti finali bisogna chiedere il caso d'uso)

#12 Updated by Marco Marinello about 5 years ago

[...]
class Subjects(models.Model):
    name = models.CharField(max_length=1024, unique=True, verbose_name=_("name"))
[...]
class Teacher(models.Model):
    [...]
    subject_taught = models.ForeignKey("hearings.Subjects",
                                      verbose_name=_("subject taught"))

#13 Updated by Christopher R. Gabriel about 5 years ago

Grazie, ho visto i diff associati al ticket, il commento e' un po' superfluo. Proprio da quel diff la mia domanda: non sono tanto sicuro che una 1-to-many sia la soluzione migliore.

Pero' mancando il caso d'uso specifico, si puo' decidere poco a riguardo.

#14 Updated by Marco Marinello about 5 years ago

La situazione dei consigli di classe non varia. Potresti spiegarti un po meglio? Non vedo il problema

#15 Updated by Christopher R. Gabriel about 5 years ago

Marco Marinello ha scritto:

La situazione dei consigli di classe non varia.

Cosa c'entrano i consigli di classe?

Potresti spiegarti un po meglio? Non vedo il problema

Il problema: ti ho chiesto se pensavi che una ForeignKey fosse corretta, a me sembra piu' adatta una many2many. Mi rispondi dicendo che lo sanno gli utenti finali, ma non si parla di quale sia il caso d'uso. Mi rispondi con un diff (inutile).

Ora, per dipanare la questione, e' necessario il caso d'uso, altrimenti non e' dato sapere se e' corretto utilizzata una FK o una Many2Many. Il campo attuale di testo e' stato fatto proprio per garantire flessibilità e minor sforzo possibile di configurazione, come dai casi d'uso espressi quando fu implementato. Se il caso d'uso cambia, e' necessario sapere quale sia.

#16 Updated by Marco Marinello almost 5 years ago

  • Assignee changed from Marco Marinello to Christopher R. Gabriel

Se non sbaglio ogni docente ha un record per ogni classe in cui insegna quindi la Many2Many sarebbe inutile.
In una tabella "Subjects" con elementi tipo "Italiano" "Italiano e storia" "Matematica" ecc... si collega con una ForeignKey la SubjectTaught alla tabella ed è risolto. Ciò donerebbe più ordine ed uniformità nel tutto.
Poi magari sarebbe utile sentire l'opinione degli utilizzatori.

#17 Updated by Christopher R. Gabriel over 4 years ago

  • Assignee deleted (Christopher R. Gabriel)

#18 Updated by Marco Marinello over 4 years ago

  • Assignee set to Piergiorgio Cemin

Also available in: Atom PDF