Ich verstehe die invoicepdfoxorder.php nicht

nbert · January 25, 2023, 10:45am

Guten Tag liebe Freunde des OXID-Systems.
Ich betreue für einen Kunden einen Shop der Cpommunity Edition 6.14
Dieser Kunde kam mit dem Wunsch auf mich zu anstelle der Kundennummer auf der Rechnung unter “Bei Zahlungen bitte immer angeben” (oben rechts) die Bestellnummer einzutragen.
“Das sollte kein Problem sein” verkündete ich.
Ha! Ich Großmaul!

Ich nahm mir also die invoicepdfoxorder.php vor und änderte die zeilen, in denen ich diese Angaben vermutete:


424        // customer number
425      $sCustNr = $this->translate('ORDER_OVERVIEW_PDF_CUSTNR') . ' ' . $oUser->oxuser__oxcustnr->value;
426        $oPdf->setFont($oPdfBlock->getFont(), '', 7);
427        $oPdf->text(195 - $oPdf->getStringWidth($sCustNr), 59, $sCustNr);

Was soll ich sagen?
Nix.

Ich spare euch jetzt einmal die Beschreibung sämtlicher Varianten die ich inzwischen probiert habe, nur soviel: ich habe sichergestellt, dass ich an der richtigen invoicepdfoxorder.php im richtigen Ordner arbeite (in dem ich an anderer Stelle Änderungen vorgenommen habe, die auch erschienen).

Ich finde keine Antwort auf die Frage "wo kommen die beiden Zeilen

Bei Zahlungen bitte immer angeben.
Kundennummer XXXX

her.

leofonic · January 25, 2023, 11:41am

Naja schon genau daher würde ich sagen
Wenn ich z.B. folgendes ändere

    // customer number
    $sCustNr = "test";
    $oPdf->setFont($oPdfBlock->getFont(), '', 7);
    $oPdf->text(195 - $oPdf->getStringWidth($sCustNr), 59, $sCustNr);

Dannn steht das auch in der Rechnung.

nbert · January 25, 2023, 12:05pm

Bei mir nicht!
Ich mache offenbar irgendetwas falsch.

Was allerdings passiert:
Wenn ich die Zeilen einfach lösche verschwindet die Kundennummer auf dem Lieferschein!

patchwork.de · January 25, 2023, 12:17pm

Lieferschein - nicht Rechnung?
→ ist function exportDeliveryNote() ab Zeile 580 in Modul-Version 2.2.1!

By the way: schau Dir mal unser kostenloses WaWi an:

nbert · January 25, 2023, 1:53pm

Das ist mir schon klar.

Deshalb wundere ich mich ja auch.

Und ich würde ungern ein komplettes Warenwirtschaftssystem implementieren, wenn ich doch nur eine einzige Zeile in der Rechnung ändern will.

nbert · January 25, 2023, 4:49pm

Ich versuche die Problematik noch einmal exakter und strukturierter darzulegen:

So sieht es aus:

Wenn ich in dem Bereich, in dem der Lieferschein aufgebaut wird, den Code von


568   // customer number
569   $sCustNr = $this->translate('ORDER_OVERVIEW_PDF_CUSTNR') . ' ' . 
570   $oUser->oxuser__oxcustnr->value; $oPdf->setFont($oPdfBlock->getFont(), '', 7);
571   $oPdf->text(195 - $oPdf->getStringWidth($sCustNr), 59, $sCustNr);

zu

568        // customer number -> order number
569        $sOrdnNr = $this->translate('ORDER_OVERVIEW_PDF_DELIVNOTE') . ' ' .$this->oxorder__oxordernr->value;
570        $oPdf->setFont($oPdfBlock->getFont(), '', 8);
571        $oPdf->text(195 - $oPdf->getStringWidth($sOrdnNr), 73, $sOrdnNr);

ändere, passiert das, was ich mir für die Rechnung wünsche:

Mache ich jetzt das gleiche im oberen Abschnitt, in dem die Rechnung generiert wird:

424        // customer number
425        $sCustNr = $this->translate('ORDER_OVERVIEW_PDF_CUSTNR') . ' ' . $oUser->oxuser__oxcustnr->value;
426        $oPdf->setFont($oPdfBlock->getFont(), '', 7);
427        $oPdf->text(195 - $oPdf->getStringWidth($sCustNr), 59, $sCustNr);

zu:

        // customer number -> order number
        $sOrdnNr = $this->translate('ORDER_OVERVIEW_PDF_DELIVNOTE') . ' ' . $this->oxorder__oxordernr->value;
        $oPdf->setFont($oPdfBlock->getFont(), '', 8);
        $oPdf->text(195 - $oPdf->getStringWidth($sOrdnNr), 73, $sOrdndNr);

ändert sich nichts:

Ich weiß nicht weiter.

rubbercut · January 25, 2023, 6:03pm

Wenn garnichts passiert, wird vielleicht die Methode exportStandart durch ein anderes Modul überschrieben. Einfach in den Modulen nach invoicepdfoxorder suchen.

Oder die Methode exportStandart mal umbennen. Wenn kein Fehler auftritt, gibt es die Methode nochmal.

nbert · January 25, 2023, 8:13pm

Zu 1:
Gibt es nicht noch mal
zu 2:
Wenn ich exportStandart umbenenne verschwindet die Zeile “Kundennummer XXX”

rubbercut · January 25, 2023, 9:25pm

Das Umbenennen sollte im Normalfall zu einem Fehler führen und das PDF nicht ausgegeben werden. Zum letzten Test kannst den Inhalt von exportStandart entfernen, d.h. nur die äußeren Klammern bleiben bestehen. Sollte in der Rechnung mehr als header und footer erscheinen, hast nicht die normale Version oder übersiehst etwas.

Hol Dir Hilfe!

public function exportStandart(){#leer#}

nbert · January 25, 2023, 9:58pm

OK
Das hatte ich falsch verstanden.
Ich hatte sowohl die Definition, als auch den Aufruf umbenannt.

Wenn ich es so mache wie von dir vorgeschlagen, passiert das was du als “sollte” bezeichnet, es wird keine PDF generiert.

patchwork.de · January 26, 2023, 10:14am

welche Version hat das Modul? Aktuell ist 2.2.1
Mach mal ein Update auf die aktuelle Version.

nbert · January 26, 2023, 11:38am

Wenn ich bei OXID das Wort Update höre krieg ich schon Stresspickel.
Gibt es einen einfachen Weg das Modul zu aktualisieren?
Ich habe Version 2.1.1

patchwork.de · January 26, 2023, 11:45am

deine Zeilennummern stimmen nicht mit den Nummern des Originals überein
→ in deiner Version hat jemand etwas geändert ?!
Original Version 2.1.1:

kannst Du auch per zip-Datei updaten

nbert · January 26, 2023, 3:19pm

Dass die Zeilennummern nicht übereinstimmen hängt damit zusammen, das oben der Footer definiert wird, und ich dort Änderungen vorgenommen habe.

Um jetzt einmal grundsätzlicher vorzugehen habe ich folgendes getan:
Ich habe die Original invoicepdfoxorder.php genommen.
In dieser Datei habe ich nur die Zeilen

424        // customer number -> order number
425        $sOrdnNr = $this->translate('ORDER_OVERVIEW_PDF_DELIVNOTE') . ' ' . $this->oxorder__oxordernr->value;
426        $oPdf->setFont($oPdfBlock->getFont(), '', 8);
427        $oPdf->text(195 - $oPdf->getStringWidth($sOrdnNr), 73, $sOrdndNr);

entsprechend dem Kundenwunsch geändert.

Diese habe ich temporär hochgeladen, Rechnung und Lieferschein erzeugt, überprüft ob das tatsächlich auf der Urversion aufbaut (also dass meine Änderungen im Layout verschwunden sind), und geschaut ob nun anstelle der Kundennummer die Ordernummer erscheint.
Was soll ich sagen?
Nein.

Es ist also irgendwo der Wurm drin.
Und ich habe nicht die geringste Ahnung wo der Kollege die Kundenummer herzaubert.
Der oxuser__oxcustnr taucht im gesamten Modul außer in der invoicepdfoxorder.php nirgends auf.

Stellt sich die Frage ob ich ein Update auf 2.2.1 machen soll (bedeutet alle Modifikationen für die PDF-Dateien neu einzubauen), ich befürchte eigentlich, dass das nichts ändern wird.
Und funktioniert diese Version auch mit meiner CE 6.14 (laut read.me ja)
Und könnte ich theoretisch auch downgraden?
Und wenn ja wie?

rubbercut · January 26, 2023, 3:38pm

Bestimmt. Die Zeilenangabe 424-427 stimmt nicht. Das ist nicht original.

$oUser->oxuser__oxcustnr->value kommt über $oUser->load($this->oxorder__oxuserid->value). Damit ist der “ganze” User vorhanden.

Ja, funktioniert.

Weil es funktioniert, brauchst es nicht.

patchwork.de · January 26, 2023, 3:45pm

wird beim Erstellen der Rechnung diese Methode verwendet?
$this->exportStandart($oPdf);
(im Original Zeile 268)

nbert · January 26, 2023, 3:45pm

Was bringt dich zu dieser Annahme?
Ich habe die Original-Dateien der Version 2.1.1 noch einmal aus der pdf-invoice-module-master.zip extrahiert, und da steht dieser Aufruf genau an dieser Stelle.

rubbercut · January 26, 2023, 3:48pm

github.com

OXIDprojects/pdf-invoice-module/blob/master/models/invoicepdfoxorder.php

<?php
/**
 * This file is part of OXID eSales Invoice PDF module.
 *
 * OXID eSales Invoice PDF module is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * OXID eSales Invoice PDF module is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with OXID eSales Invoice PDF module.  If not, see <http://www.gnu.org/licenses/>.
 *
 * @category      module
 * @package       oeinvoicepdf
 * @author        OXID eSales AG

This file has been truncated. show original

nbert:

424        // customer number
425      $sCustNr = $this->translate('ORDER_OVERVIEW_PDF_CUSTNR') . ' ' . $oUser->oxuser__oxcustnr->value;
426        $oPdf->setFont($oPdfBlock->getFont(), '', 7);
427        $oPdf->text(195 - $oPdf->getStringWidth($sCustNr), 59, $sCustNr);

Findest Du bei 448

rubbercut · January 26, 2023, 3:49pm

Will ich hoffen. Das hatten wir ja oben bereits besprochen.

Am besten lade das gesamte Modul zum Download irgendwo hin. Jemand, der Zeit hat, wird den Fehler finden.

nbert · January 26, 2023, 3:54pm

Das ist ja auch eine aktuelle Version.
Kein Wunder, dass da die Zeilennummern nicht stimmen.
Ich habe auf der Version 2.1.1 augebaut.
Das schien mir sinnvoller, als die Versionen zu mischen.