202411 feat print part labels for stock #406
Conversation
für z.B. Etiketten
Kein Neuladen der Seite nach dem Speichern notwendig.
Setze die von Template erwarteten Variablen.
Übersetze den Templatenamen.
Suche nach Template der Sprache.
Tabelle zum Merken ob Etiketten für eine Preisänderung gedruckt
wurden.
z4m1n0
force-pushed
the
202411-feat-print_part_labels_for_stock
branch
from
8ab5366 to
d13114c
Compare
| Artikelklassifizierung: & $( KiviLatex.filter(LxERP.t8(part.classification.description)) )$ \\ | ||
| %$(END)$ | ||
| %$(IF part.customm_tariff_number )$ | ||
| Zolltarifnummer: & $( KiviLatex.filter(LxERP.t8(part.customm_tariff_number)) )$ \\ |
There was a problem hiding this comment.
customm_
Das sieht nach einem Typo aus.
There was a problem hiding this comment.
Moin Tamino, erster Block Review von diesem Monster.
Vorweg: Ich hab mir hier nur die Sachen angeschaut, die nach dem 202410-feat-part_printing reingekommen sind, und selbst das ist recht schwer zu lesen. Ich würde auf jeden Fall zusehen, dass der vorher noch gemergt wird.
Das was ich gefunden habe bisher sieht zumindest kaputt aus und muss noch gefixt werden, ich habe aber auch das filter query im Part Manager noch nicht verstanden, da kommt nochmal eine zweite Runde.
|
|
||
| $self->models->finalize; # for filter laundering | ||
|
|
||
| $::form->{filter}->{price_change_printed} = ${$::form->{filter}->{price_change_printed}}; |
There was a problem hiding this comment.
Das hier verstehe ich nicht ganz. Soll das price_change_printed vor dem laundering retten?
| return | ||
| [\qq{( | ||
| SELECT DISTINCT CASE WHEN count(*) $comp 1 THEN FALSE ELSE TRUE END | ||
| FROM ( |
There was a problem hiding this comment.
Das geht zumindest auf meiner postgres 12 kaputt, weil subqueries in FROM clauses einen alias brauchen. Ich hab nachgeschaut, das ist erst ab postgres 16 optional. Die UPGRADE nennt immernoch Postgres 12 als minimum.
There was a problem hiding this comment.
Die beiden templates fehlen in den alten templates, oder?
There was a problem hiding this comment.
Die beiden templates fehlen in den alten templates, oder?
| @@ -43,7 +43,7 @@ __PACKAGE__->run_before('set_layout'); | |||
| sub action_stock_in { | |||
| my ($self) = @_; | |||
|
|
|||
| $::form->{title} = t8('Stock'); | |||
| $::form->{title} = t8('Store'); | |||
There was a problem hiding this comment.
Hmm, ich bin kein Freund von der Umbenennung. Üblicherweise haben wir für "Einlagern" dann "Stock In" benutzt. Auch nicht schön, aber zumindest eindeutig. "Store" kann ausserdem noch Geschäft, Filiale oder Speicher heißen, finde ich hier sehr verwirrend.
Basiert auf 202410-feat_part_printing