Closed php4umagento closed 6 years ago
Hey @php4umagento it overall looks good. Now that it is working I would see this as a good opportunity to refactor and tidy the code up a bit. For example the current static tests complain about The method getAttributeFilterType() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10.
https://travis-ci.org/magento-engcom/import-export-improvements/jobs/330107223 I know this is not your fault but it would be great if we could some how sort this issue.
Other than that it looks great.
@php4umagento thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository.
Code wise this is looking good to me @php4umagento I will run this through our internal tools and the QA team and see if anything else comes up.
@dmanners is there anything else for me here?
@php4umagento just that we do not encourage protected visibility could we make that a change?
build is running now with "private" scope
@dmanners done
Thanks @php4umagento I will get this PR into the process loop now and see if we can sort some QA out for this task.
@php4umagento thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository.
Hi @php4umagento during testing the QA team validated the customer section is fixed but would it also be possible to fix the product filter part as well?
Steps to reproduce
1) Create product attribute with type multiselect and add to attribute set.
2) Create several product with created attribute.
3) Go to Export and select Products
4) Observe that multiselect attributes are presented as multiselect form field
5) Select multiselect attribute and any option in filter
6) Press "Continue" button
Expected result
- Data is filtered by selected option.
Actual result
- Exported all products, not looking at the filter.
@php4umagento
Preconditions
There is a custom attribute created with input type Multiple Select, 3 options enabled ("Option 1", "Option 2", "Option 3").
There is a customer in the system created with custom attribute value = "Option 1" and second customer in the system created with custom attribute value = "Option 2".
Steps to reproduce
Go to System-> Import/Export-> Export at backend.
Select "Customers" entity type.
In the grid for attribute from precondition select "Option 1" and "Option 2" filter.
Press "Continue" button.
Expected Result
The file was exported with customers from precondition.
Actual Result
File was not exported.
The error message appears "There is no data for export".
The same behavior with products.
…Type Multiple Select
Description
Fixed Issues (if relevant)
Manual testing scenarios
Contribution checklist