Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDF-Rechnung im Benutzerkonto ermöglichen #2

Closed
therouv opened this issue Aug 14, 2012 · 3 comments
Closed

PDF-Rechnung im Benutzerkonto ermöglichen #2

therouv opened this issue Aug 14, 2012 · 3 comments
Assignees
Labels

Comments

@therouv
Copy link
Member

therouv commented Aug 14, 2012

Analog wie hier exemplarisch geschrieben und entwickelt: http://www.satollo.net/magento-pdf-invoices-customer-dashboard

Macht mehr Sinn im Benutzerkonto ebenfalls die PDF-Rechnung anzubieten, als diese "dämliche" HTML-Ansicht.. Ist zugleich auch für den Kunden besser.

@ghost ghost assigned gerritpechmann Sep 26, 2012
gerritpechmann pushed a commit that referenced this issue Sep 26, 2012
@gerritpechmann
Copy link
Member

Ich habe das Feature etwas anders gelöst, als im Artikel beschrieben.

Die HTML-Ansicht beim "Print Invoice" und "Print All Invoices" wurde durch ein Rewrite des Controllers ersetzt und stattdessen Invoice PDFs zum Download angeboten.

Bitte mal reviewen.

@ghost ghost assigned therouv Sep 26, 2012
@mklooss
Copy link
Contributor

mklooss commented Oct 17, 2012

Wäre es hier dann nicht besser, wenn man die Möglichkeit des Abspeicherns der PDF Dateien mit als Option anbietet.
Damit meinte ich auf dem Server, sodass die nur einmal generiert wird. Weil sonst frist sowas ja nur unnötig Serverperformance.

@therouv
Copy link
Member Author

therouv commented Oct 17, 2012

Rechnungen auf dem Server abspeichern würde ich aus Sicherheitsgründen eher nicht. Falls jemand so clever ist, den Pfad zur Datei zu rekonstruieren wäre das ein Datenleck mit unvorstellbarem Ausmaß. Würde es so belassen, da ich nicht glaube, dass die Server-Performance von ein paar Rechnungen zu drucken merklich belastet wird.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants