Import/Export Parameter-Fehler und update-sichere Erweiterung

[noRiddle]

In der Funktion insertCategory() ist ein Parameter an verkehrter Stelle eingesetzt

function insertCategory(& $dataArray, $mode = 'insert', $pID) {

Der vordefinierte Parameter muß am Ende stehen, also so

function insertCategory(& $dataArray, $pID, $mode = 'insert') {

Außerdem gibt es bislang nur für den Import die Möglichkeit update-sicher Felder hinzuzufügen, nicht jedoch für den Export.

Gruß,
noRiddle

[Gerhard Waldemair]

ich kann das nicht nachvollziehen. Die Methode findet nur an 1 Stelle Verwendung und dort werden die Parameter korrekt übergeben und verarbeitet.

[noRiddle]

Dann habe ich mich wohl nicht eindeutig ausgedrückt.
Parameter einer Funktion mit einer Vorgabe, hier also $mode, müssen nach Parametern ohne Vorgabe stehen. Hier also $mode = 'insert' als letztes.
Ansonsten gibt es ein Warning (noch nur ein Warning).

Zitat ​PHP-Doku:

[...]
Zu beachten ist, dass alle Parameter mit Vorgabewert rechts von den Parametern ohne Vorgabewert stehen müssen - sonst wird es nicht funktionieren. [...]

Gruß,
noRiddle

[Gerhard Waldemair]

das würde nur zu einem Problem werden, wenn die Funktion so deklariert wird:

function insertCategory(& $dataArray, $mode = '', $pID) {}

damit ist der 2te Parameter optional und der 3te ist Pflicht.

Ich ändere das dennoch, weil es nur in der Importklasse zur Verwendung kommt.

[Gerhard Waldemair]

In 14126:

fix #1402 & #2175 - fix function parameter order and add auto_include for import/export

[noRiddle]

Deinen letzten Begründungssatz verstehe ich nicht.
Du änderst das weil es nur in der Importklasse zur Verwendung kommt?
Ääähmm, öööh, häää ?

"zu einem Problem werden" ist außerdem ein Frage der Betrachtung.
Fest steht, daß PHP ein WARNING ausgeben würde wenn es so geblieben wäre wie vorher und das kann irgendwann auch mal Probleme machen, indem es zu einem ERROR wird.

Meinen Zusatz im Ticket
"Außerdem gibt es bislang nur für den Import die Möglichkeit update-sicher Felder hinzuzufügen, nicht jedoch für den Export."
hast du nicht bemerkt ?
Das ist doch bislang unvollständig gemacht und folglich nicht so sinnvoll.

Gruß,
noRiddle

[Torsten Riemer]

Replying to noRiddle:

[...]
Meinen Zusatz im Ticket
"Außerdem gibt es bislang nur für den Import die Möglichkeit update-sicher Felder hinzuzufügen, nicht jedoch für den Export."
hast du nicht bemerkt ?
Das ist doch bislang unvollständig gemacht und folglich nicht so sinnvoll.
[...]

Du hast doch Leseberechtigung im SVN. Schau dir doch mal das Changeset r14126 an. Fehlt da deiner Meinung noch etwas?

[noRiddle]

Okay, hab's mir im SVN angeschaut. Sieht okay aus. Danke.

Gruß,
noRiddle