Ontology importer fixes (including db v.2)#247
Conversation
|
Ho aggiunto delle tabelle "temporanee" (non sono effettivamente temporary tables perché altrimenti si perdevano nei cambi di connessione al db) per importation e drawn, così che in una stessa sessione si possano distinguere gli inserimenti nel db che vanno salvati e quelli che vanno rimossi. Il problema del salvataggio invece era legato al salvataggio da pulsante invece che alla chiusura del progetto, perché in quel caso veniva lanciato comunque il segnale per la rimozione delle importazioni dal db. |
|
Al momento ottengo il seguente errore quando provo ad importare un'ontologia: Manca la parentesi di chiusura nello statement menzionato. |
|
Ok ora sembra che il salvataggio funzioni insieme all'upgrade di versione del db. |
mnamici
left a comment
There was a problem hiding this comment.
Ricorda che il database è condiviso tra le varie sessioni (progetti aperti). Da quello che vedo le tabelle vengono cancellate al salvataggio, quindi due progetti fanno interferenza tra loro
(salvando uno si cancellano le cose temporanee dell'altro).
mnamici
left a comment
There was a problem hiding this comment.
Un ultimo paio di osservazioni. In sostanza secondo me session non dovrebbe uscire modificato da questa PR, ci sta già tutto quello che serve mi sembra.
… on session_id on temp tables
Previous to this, the signal was emitted even when the project was closed from a clean state, i.e. without changes triggering the confirmation dialog, and resulted in the unintended removal of any importation process started during the current session.
b42bfae to
b252f06
Compare
|
Visto che c'erano alcune modifiche che si annullano ho fatto squash di alcuni degli ultimi commit, fai force pull. |
No description provided.