202405 feat part printing #332
Conversation
There was a problem hiding this comment.
Sieht ziemlich gut aus.
Noch ein changelog-Eintrag wäre schön.
Und evtl. einen Hinweis darauf, dass xetex oder luatex und das LaTeX-Paket pst-barcode benötigt wird für den Label-Druck mit den marai-Druckvorlagen. Oder mache ich was falsch?
Kleinigkeiten habe ich an entsprechenden den Stellen kommentiert.
SL/Controller/Part.pm
Outdated
| $self->send_file( | ||
| $pdf, | ||
| type => 'application/pdf', | ||
| name => $file_name, |
There was a problem hiding this comment.
An anderen Stellen haben wir eine Überstzung der Dateinamen. Geht das hier evtl. auch?
There was a problem hiding this comment.
Wird jetzt gemacht. Auch die Artikelnummer wird angehangen.
SL/Controller/Part.pm
Outdated
| && $::form->{part}{partnumber} | ||
| && SL::DB::Manager::Part->find_by(partnumber => $::form->{part}{partnumber}) | ||
| ) { | ||
| return $self->js->error(t8('The partnumber is already being used'))->render; |
There was a problem hiding this comment.
Das sieht irgendwie falsch aus, da save ja eigentlich ein array ref zurück liefern soll. Funktioniert, da die benutzte partnumber schon vorher abgefangen wird (in check_form). War vermutlich schon vorher so,, finde ich aber nicht richtig.
There was a problem hiding this comment.
Habe ich entfernt, war überflüssig.
| t8('Save and print'), | ||
| call => [ 'kivi.Part.show_print_options' ], | ||
| disabled => !$may_edit ? t8('You do not have the permissions to access this function.') : undef, | ||
| checks => [ 'kivi.validate_form' ], | ||
| ], |
There was a problem hiding this comment.
Das "Speichern und Drucken" sollte nur angeboten werden, wenn man nicht inline einen neuen Artikel anlegt.
only_if => !$::form->{inline_create}
Kein Neuladen der Seite nach dem Speichern notwendig.
Setze die von Template erwarteten Variablen.
Übersetze den Templatenamen.
Suche nach Template der Sprache.
No description provided.