ConoscereLinux / DeskApp

Utility per gestire, rinominare e indicizzare file
GNU General Public License v3.0
5 stars 23 forks source link

Digger manager better and improved! #39

Closed pg-88 closed 8 months ago

pg-88 commented 9 months ago

Ciao! Sto facendo questa pull request oggi (lunedì 22) perché domani purtroppo non riuscirò a partecipare alla serata del laboratorio Python né in presenza né da remoto.

Nel mio modulo rispetto alla scorsa versione troverete meno metodi, infatti, ho eliminato "update_indexer" e "already_indexed" perché (se non ricordo male) allo scorso incontro si era detto che quel tipo di funzionalità dovrebbero spettare a "indexer_manager". Inoltre ho eliminato tutti i print e al loro posto ho usato il logger. Ho anche inserito un log di errore nel caso il percorso passato dalla configurazione non punti a una cartella del sistema (questo check si potrebbe forse implementare meglio). Nello "scan" ora viene gestito in un try-except l'errore "PermissionError" nel caso non sia possibile aprire il file. Anche questa gestione dell'errore forse è da rivedere, nel senso che forse sarebbe da inserire nel modulo "MetaManager" nel momento in cui va effettivamente ad aprire il file.

Fatemi sapere se trovate qualche cosa da sistemare e nel caso come procedere per il prossimo incontro

Happy Coding A presto Pietro