Skip to content

Fix: nell'elenco dei seriali manca il tipo di documento - #1929

Open
Andrea-Bruno wants to merge 1 commit into
devcode-it:masterfrom
Andrea-Bruno:fix/etichetta-seriali-1922
Open

Andrea-Bruno wants to merge 1 commit into
devcode-it:masterfrom
Andrea-Bruno:fix/etichetta-seriali-1922

Conversation

@Andrea-Bruno

Copy link
Copy Markdown

Ciao e grazie per il progetto.

Ho letto la issue #1922 e ho preparato una correzione su una parte del problema segnalato, quella delle diciture sbagliate nell'elenco dei seriali.

Il difetto che ho sistemato e' questo: quando un seriale e' collegato a un ordine, il programma va a prendere il nome del tipo di documento da una tabella aggiuntiva delle traduzioni. Se quella riga di traduzione non c'e', il nome arriva vuoto, e la scritta che si legge nell'elenco perde la parola che indica il documento: resta solo il numero e la data, senza il tipo. E' esattamente il caso che segnali quando dici che manca la parola "Ordine".

La correzione aggiunge un controllo: se il tipo di documento arriva vuoto, al suo posto viene usato il nome del modulo di provenienza, che c'e' sempre. Cosi' la scritta mostra sempre il tipo di documento, e non resta mai monca. Ho messo questo controllo sia nell'elenco delle vendite sia in quello degli acquisti, perche' entrambi avevano la stessa fragilita'.

Per onesta' segnalo che questa proposta copre la parte delle diciture. La parte piu' delicata, cioe' il fatto che un seriale inserito in un documento resti ancora disponibile per altri documenti, tocca la logica di disponibilita' dei seriali e merita un lavoro a se', per non rischiare di rompere i movimenti di magazzino.

…s empty

The serials list read the document type from a LEFT JOIN on the localized order-type table, so when that language row was missing the label lost its document word and showed only the number and the date. Both the purchase and the sale side now fall back to the source module name when the localized type is empty, so the label always names the document.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant