Ticket #106 (closed difetto: sistemata)
Classi `Meta` duplicate nel modello BlockConfiguration
| Reported by: | lfranc | Owned by: | |
|---|---|---|---|
| Priority: | importante | Milestone: | ACQUISTIAMO - L'utente può ordinare |
| Component: | sviluppo | Keywords: | |
| Cc: |
Description
Nel modello gasistafelice.rest.models.blocks.BlockConfiguration la classe interna Meta compare due volte; ciò, ovviamente, invalida il modello.
Come procedere ? Accorpare gli attributi delle 2 classi ? Eliminarne una delle 2 ?
Change History
comment:1 follow-up: ↓ 2 Changed 22 months ago by fero
- Status changed from new to closed
- Resolution set to sistemata
comment:2 in reply to: ↑ 1 Changed 22 months ago by lfranc
Buone le segnalazioni peccato che avvengano oggi e non avevo capito se ti davano problemi anche precedentemente.
Mi sono limitato a segnalare i problemi rilevati da Eclipse (quelli che mi sembravano potenzialmente nocivi per la dimostrazione di stasera). Alcuni errori negli import mi hanno dato problemi quando andavo a invocare ./manage.py shell et simili, altri invece no (come hai detto tu, c'è parecchio codice "morto"). A volte questi errori li ho fixati direttamente, a volte ho sorvolato, visto che venivano introdotti con una certa frequenza; d'altra parte, mi pare fisiologico, per un codice di sviluppo che manca di un apparato di test "esaustivo".
comment:3 follow-up: ↓ 4 Changed 22 months ago by fero
forte eclipse :)
devo trovare qualcosa di analogo in vim (anche se non è proprio fondamentale). Come si chiama questa funzionalità?
comment:4 in reply to: ↑ 3 Changed 22 months ago by lfranc
Replying to fero:
Come si chiama questa funzionalità?
È una funzione built-in nell'editor del modulo PyDev di Eclipse; non so se abbia un nome preciso, forse qualcosa del tipo dynamic code analisys. In sostanza, è una funzionalità simile a quella di syntax checker come PyLint, solo in tempo reale.

In questo caso le ho accorpate. Capisco le difficoltà nell'allinearsi sul codice. Tutti i problemi che hai segnalato riguardano codice non usato da gasista felice. Codice importato da SANET, codice che deve essere rivisto e integrato, o in alcuni casi buttato.
Ho evitato di integrare in blocco tutto quanto fatto in SANET, ma ho cercato di limitarmi alle parti che potevano farci comodo. Importare codice aggiuntivo è stato comunque un compromesso necessario.
Buone le segnalazioni peccato che avvengano oggi e non avevo capito se ti davano problemi anche precedentemente.