Skip to content

variationen Endpunkt korrektur #238

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.

Sign up for GitHub

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

Closed
Avatarsia wants to merge 2 commits into from
Closed

variationen Endpunkt korrektur #238

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
59bdd7d
fix: use variation endpoint for category sync in WooCommerce product …
Mar 24, 2026
54b6fb6
fix: handle stock sync for variable products by updating each variation
Mar 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
53 changes: 40 additions & 13 deletions www/pages/shopimporter_woocommerce.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -549,17 +549,43 @@ public function ImportSendListLager()
// WooCommerce doesnt have a standard property for the other values, we're ignoring them
];
if ($remoteIdInformation['isvariant']) {
// This is a variation — update via parent/variations endpoint
$result = $this->client->put('products/' . $remoteIdInformation['parent'] . '/variations/' . $remoteIdInformation['id'], $updateProductParams);

$this->logger->error(
"WooCommerce Lagerzahlenübertragung für Variante: $nummer / $remoteIdInformation[id] (Parent: $remoteIdInformation[parent]) - Anzahl: $lageranzahl",
['result' => $result]
);
} elseif ($remoteIdInformation['type'] === 'variable') {
// This is a variable parent product — stock must be set on each variation individually
$variations = $this->client->get('products/' . $remoteIdInformation['id'] . '/variations', ['per_page' => 100]);

if (!empty($variations)) {
foreach ($variations as $variation) {
$result = $this->client->put(
'products/' . $remoteIdInformation['id'] . '/variations/' . $variation->id,
$updateProductParams
);

$this->logger->error(
"WooCommerce Lagerzahlenübertragung für Variante von variablem Produkt: $nummer / Variation: $variation->id (Parent: $remoteIdInformation[id]) - Anzahl: $lageranzahl",
['result' => $result]
);
}
} else {
$this->logger->error(
"WooCommerce variables Produkt $nummer / $remoteIdInformation[id] hat keine Variationen — Lager-Sync übersprungen"
);
}
} else {
// Simple product — direct update
$result = $this->client->put('products/' . $remoteIdInformation['id'], $updateProductParams);
}

$this->logger->error(
"WooCommerce Lagerzahlenübertragung für Artikel: $nummer / $remoteIdInformation[id] - Anzahl: $lageranzahl",
[
'result' => $result
]
);
$this->logger->error(
"WooCommerce Lagerzahlenübertragung für Artikel: $nummer / $remoteIdInformation[id] - Anzahl: $lageranzahl",
['result' => $result]
);
}
$anzahl++;
}
return $anzahl;
Expand Down Expand Up @@ -726,9 +752,7 @@ public function ImportSendList()
$this->logger->info("WooCommerce Variante geändert für Artikel: $nummer / Variation: $product_id (Parent: $parent_id)");
} else {
// This is a regular product
$this->client->put('products/' . $product_id, array_merge([

], $commonProductAtts));
$this->client->put('products/' . $product_id, $commonProductAtts);

$this->logger->info("WooCommerce Artikel geändert für Artikel: $nummer / $product_id");
}
Expand All @@ -753,6 +777,8 @@ public function ImportSendList()
// }

// Update the associated product categories
// For variations, categories belong to the parent product
$category_product_id = ($remoteIdInformation['isvariant']) ? $remoteIdInformation['parent'] : $product_id;

$chosenCats = array();
if (isset($tmp[$i]['kategorien']) || isset($tmp[$i]['kategoriename'])) {
Expand Down Expand Up @@ -795,7 +821,7 @@ public function ImportSendList()

if ($wcCatId) {
// update category. We first retrieve the product and append the new product category, not replace the entire category array.
$alreadyAssignedWCCats = $this->client->get('products/' . $product_id, [
$alreadyAssignedWCCats = $this->client->get('products/' . $category_product_id, [
'per_page' => 1,
])->categories;

Expand All @@ -814,7 +840,7 @@ public function ImportSendList()
}

// Update category assignment
$this->client->put('products/' . $product_id, [
$this->client->put('products/' . $category_product_id, [
'categories' => $allCatIdsWCAPIRep,
]);

Expand Down Expand Up @@ -1010,7 +1036,8 @@ private function getShopIdBySKU($sku)
return [
'id' => $product[0]->id,
'parent' => $product[0]->parent_id,
'isvariant' => !empty($product[0]->parent_id)
'isvariant' => !empty($product[0]->parent_id),
'type' => $product[0]->type ?? 'simple'
];
}

Expand Down