Skip to content
This repository was archived by the owner on Sep 14, 2020. It is now read-only.

Modify attributes only from product attribute set#350

Open
ArtsiomBruneuski wants to merge 1 commit into
blmage:1.0.0-wip-edgefrom
ArtsiomBruneuski:patch-1
Open

Modify attributes only from product attribute set#350
ArtsiomBruneuski wants to merge 1 commit into
blmage:1.0.0-wip-edgefrom
ArtsiomBruneuski:patch-1

Conversation

@ArtsiomBruneuski
Copy link
Copy Markdown

@ArtsiomBruneuski ArtsiomBruneuski commented Sep 5, 2017

Used version: 1.0.0.1

Description:
There are two products:

  • product_a with attribute_set_x
  • product_b with attribute_set_y

As a store administrator tried to change product_a from product grid.

Expected: only attribute_set_x attributes should be changed
Actual: attribute_set_x attributes changed but also attribute_set_y attributes changed to default values

Problem:
All available attributes loaded in catalog product resource model (it is singleton) in app/code/community/BL/CustomGrid/Model/Grid/Type/Product.php:72
Then during saving product all attributes values change to default values even if they are not in product attribute set

Solution:
Reset previously loaded attributes cache before product initialization

Used version: 1.0.0.1

Description:
There are two product:
- product_a with attribute_set_x
- product_b with attribute_set_y

As a store administrator tried to change product_a from product grid.

Expected: only attribute_set_x attributes should be changed
Actual: attribute_set_x attributes changed but also attribute_set_y attributes changed to default values

Problem:
All available attributes loaded in catalog product resource model (it is singleton) in app/code/community/BL/CustomGrid/Model/Grid/Type/Product.php:72
Then during saving product all attributes values change to default values even if they are not in product attribute set

Solution:
Reset previously loaded attributes cache before product initialization
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant