merdan
/
sarga
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

test and fix skipping of attributes and relations (product copy)

This commit is contained in:
Herbert Maschke 2020-08-10 09:51:15 +02:00
parent 3670a6210b
commit 6a248c6236
2 changed files with 37 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
packages/Webkul/Product/src/Repositories/ProductRepository.php
View File

@ -660,43 +660,43 @@ class ProductRepository extends Repository
{
$attributesToSkip = config('products.skipAttributesOnCopy') ?? [];
if (! isset($attributesToSkip['categories'])) {
if (! in_array('categories', $attributesToSkip)) {
foreach ($originalProduct->categories as $category) {
$copiedProduct->categories()->save($category->replicate());
}
}
if (! isset($attributesToSkip['inventories'])) {
if (! in_array('inventories', $attributesToSkip)) {
foreach ($originalProduct->inventories as $inventory) {
$copiedProduct->inventories()->save($inventory->replicate());
}
}
if (! isset($attributesToSkip['customer_group_prices'])) {
if (! in_array('customer_group_pricces', $attributesToSkip)) {
foreach ($originalProduct->customer_group_prices as $customer_group_price) {
$copiedProduct->customer_group_prices()->save($customer_group_price->replicate());
}
}
if (! isset($attributesToSkip['images'])) {
if (! in_array('images', $attributesToSkip)) {
foreach ($originalProduct->images as $image) {
$copiedProduct->images()->save($image->replicate());
}
}
if (! isset($attributesToSkip['super_attributes'])) {
if (! in_array('super_attributes', $attributesToSkip)) {
foreach ($originalProduct->super_attributes as $super_attribute) {
$copiedProduct->super_attributes()->save($super_attribute->replicate());
}
}
if (! isset($attributesToSkip['bundle_options'])) {
if (! in_array('bundle_options', $attributesToSkip)) {
foreach ($originalProduct->bundle_options as $bundle_option) {
$copiedProduct->bundle_options()->save($bundle_option->replicate());
}
}
if (! isset($attributesToSkip['variants'])) {
if (! in_array('variants', $attributesToSkip)) {
foreach ($originalProduct->variants as $variant) {
$variant = $this->copy($variant);
$variant->parent_id = $copiedProduct->id;

30
tests/functional/Product/ProductCopyCest.php
View File

@ -12,6 +12,36 @@ use Webkul\Product\Models\ProductAttributeValue;
class ProductCopyCest
{
public function testSkipAttributes(FunctionalTester $I)
{
$I->loginAsAdmin();
config(['products.skipAttributesOnCopy' => ['name', 'inventories']]);
$original = $I->haveProduct(Laravel5Helper::SIMPLE_PRODUCT, [
'productInventory' => [
'qty' => 10,
],
'attributeValues' => [
'name' => 'Original',
],
]);
$I->amOnAdminRoute('admin.catalog.products.copy', ['id' => $original->id], false);
// test attribute is skipped:
$attr = $I->dontSeeRecord(ProductAttributeValue::class, [
'attribute_id' => 2, // name
'product_id' => $original->id + 1,
]);
// test relation is skipped:
$I->dontSeeRecord(ProductInventory::class, [
'product_id' => $original->id + 1,
'qty' => 10,
]);
}
public function testProductCopy(FunctionalTester $I)
{
$I->loginAsAdmin();