Fix flag unsetEmptyFields #228
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The flag shouldn't be reversed
|
+1 |
Collaborator
|
@mbijnsdorp Can someone fix the config, the method names and the system.xml labels, this seems to be confusing everyone :p |
Owner
|
Merged manually into develop branch. Thanks! |
hnsr
pushed a commit
to ho-nl-fork/AvS_FastSimpleImport
that referenced
this pull request
May 8, 2017
* develop: Fixed last missing strtolower replacement by mb_strtolower wrapper Add multibyte aware strtolower to attribute-validation Bump version to 0.8.0 Bump version to 0.8.0 Revert changes in test.php Merge branch 'issue154' of git://github.com/barbazul/AvS_FastSimpleImport into barbazul-issue154 Replace all strtolower occurences by helper method which takes care of multibyte variant (fixes avstudnitz#271) Remove unnecessary logs. Importing attribute options price (_super_attribute_price_corr) for multiple stores/websites. Defined additional storeCodeToId array, it will take the id from our store code in order to get the website id. We do not have this information in dataRow. Adds the missing shell command to modman [SKU-Type] SKU-Type Attribute is now inserted [Grouped] Fix for undefined variable Warning Use pages in _loadSkuSuperAttributeValues() to reduce memory usage Override _loadSkuSuperAttributeValues() Properly set rowNum during saveProducts avstudnitz#327 Load Backend Config when Importing via Backend [possible hack] Fix multiple stores [FIX] copy magento way to delete old scope specific attribute value [FIX] keep category position if set [FIX] delete old product attribute if import is not append mode Fix category get's emptied when there are more rows in the csv file than category associations. Fixes avstudnitz#311 Fixed Issue That Attribute Options Were Created Twice Don't throw exception when external image cannot be downloaded Use suggest rather than require [fix] add support for PHP 5.3 Add getNewCustomers, getOldCustomers, getNewCategory and getOldCategory getters use multibyte string functions if available case-insensitive comparison of dropdown/mutliselect attribute options [FIX] delete not all category assignments during catalog_category_product replace New version 0.7.0 Add support for _use_config columns for available_sort_by, default_sort_by and filter_price_range Fixes avstudnitz#154 Save _bundle_option_title translations. Pull request avstudnitz#262 Pull request avstudnitz#228 Pull request avstudnitz#227 Fix getAffectedEntityIds() for category (missing root) Fixes possible security issues with excel formula injection in csv data, see APPSEC-1110 (SUPEE-7405). Fix getAffectedEntityIds() for category (missing root) fixes avstudnitz#154 - Don't validate "required" catelog attributes
hnsr
pushed a commit
to ho-nl-fork/AvS_FastSimpleImport
that referenced
this pull request
Jul 26, 2017
…into only-exisiting-entities * 'develop' of github.com:avstudnitz/AvS_FastSimpleImport: create downloadable/files/links directory if it does not exist yet Fixed multi-store downloadable product links import, when "downloadable_links_file" is empty Fixed last missing strtolower replacement by mb_strtolower wrapper Add multibyte aware strtolower to attribute-validation Remove debugging code Bump version to 0.8.2 Only delete category product associtions during product import if at least one category is assigned in 'replace' mode Add possibility to import sku_type of bundle products (solves avstudnitz#355) Revert 953a998 as it broke the integration tests Revert 9ffbeb1 as it made the sku_type field a required value Bump version to 0.8.0 Bump version to 0.8.0 Revert changes in test.php Merge branch 'issue154' of git://github.com/barbazul/AvS_FastSimpleImport into barbazul-issue154 Replace all strtolower occurences by helper method which takes care of multibyte variant (fixes avstudnitz#271) Remove unnecessary logs. Importing attribute options price (_super_attribute_price_corr) for multiple stores/websites. Defined additional storeCodeToId array, it will take the id from our store code in order to get the website id. We do not have this information in dataRow. Update Import.php Adds the missing shell command to modman [Downloadable] Links Purchase Attribute is now inserted [SKU-Type] SKU-Type Attribute is now inserted Added ability to import downloadable products with downloadable links [Grouped] Fix for undefined variable Warning Use pages in _loadSkuSuperAttributeValues() to reduce memory usage Override _loadSkuSuperAttributeValues() Properly set rowNum during saveProducts avstudnitz#327 Load Backend Config when Importing via Backend [possible hack] Fix multiple stores [FIX] copy magento way to delete old scope specific attribute value [FIX] keep category position if set [FIX] delete old product attribute if import is not append mode Fix category get's emptied when there are more rows in the csv file than category associations. Fixes avstudnitz#311 Fixed Issue That Attribute Options Were Created Twice Don't throw exception when external image cannot be downloaded Use suggest rather than require [fix] add support for PHP 5.3 Add getNewCustomers, getOldCustomers, getNewCategory and getOldCategory getters use multibyte string functions if available case-insensitive comparison of dropdown/mutliselect attribute options [FIX] delete not all category assignments during catalog_category_product replace New version 0.7.0 Add support for _use_config columns for available_sort_by, default_sort_by and filter_price_range Fixes avstudnitz#154 Save _bundle_option_title translations. Pull request avstudnitz#262 Pull request avstudnitz#228 Pull request avstudnitz#227 Fix getAffectedEntityIds() for category (missing root) Fixes possible security issues with excel formula injection in csv data, see APPSEC-1110 (SUPEE-7405). Fix getAffectedEntityIds() for category (missing root) fixes avstudnitz#154 - Don't validate "required" catelog attributes
hnsr
pushed a commit
to ho-nl-fork/AvS_FastSimpleImport
that referenced
this pull request
Jul 26, 2017
…into qj * 'develop' of github.com:avstudnitz/AvS_FastSimpleImport: create downloadable/files/links directory if it does not exist yet Fixed multi-store downloadable product links import, when "downloadable_links_file" is empty Fixed last missing strtolower replacement by mb_strtolower wrapper Add multibyte aware strtolower to attribute-validation Remove debugging code Bump version to 0.8.2 Only delete category product associtions during product import if at least one category is assigned in 'replace' mode Add possibility to import sku_type of bundle products (solves avstudnitz#355) Revert 953a998 as it broke the integration tests Revert 9ffbeb1 as it made the sku_type field a required value Bump version to 0.8.0 Bump version to 0.8.0 Revert changes in test.php Merge branch 'issue154' of git://github.com/barbazul/AvS_FastSimpleImport into barbazul-issue154 Replace all strtolower occurences by helper method which takes care of multibyte variant (fixes avstudnitz#271) Remove unnecessary logs. Importing attribute options price (_super_attribute_price_corr) for multiple stores/websites. Defined additional storeCodeToId array, it will take the id from our store code in order to get the website id. We do not have this information in dataRow. Update Import.php Adds the missing shell command to modman [Downloadable] Links Purchase Attribute is now inserted [SKU-Type] SKU-Type Attribute is now inserted Added ability to import downloadable products with downloadable links [Grouped] Fix for undefined variable Warning Use pages in _loadSkuSuperAttributeValues() to reduce memory usage Override _loadSkuSuperAttributeValues() Properly set rowNum during saveProducts avstudnitz#327 Load Backend Config when Importing via Backend [possible hack] Fix multiple stores [FIX] copy magento way to delete old scope specific attribute value [FIX] keep category position if set [FIX] delete old product attribute if import is not append mode Fix category get's emptied when there are more rows in the csv file than category associations. Fixes avstudnitz#311 Fixed Issue That Attribute Options Were Created Twice Don't throw exception when external image cannot be downloaded Use suggest rather than require [fix] add support for PHP 5.3 Add getNewCustomers, getOldCustomers, getNewCategory and getOldCategory getters use multibyte string functions if available case-insensitive comparison of dropdown/mutliselect attribute options [FIX] delete not all category assignments during catalog_category_product replace New version 0.7.0 Add support for _use_config columns for available_sort_by, default_sort_by and filter_price_range Fixes avstudnitz#154 Save _bundle_option_title translations. Pull request avstudnitz#262 Pull request avstudnitz#228 Pull request avstudnitz#227 Fix getAffectedEntityIds() for category (missing root) Fixes possible security issues with excel formula injection in csv data, see APPSEC-1110 (SUPEE-7405). Fix getAffectedEntityIds() for category (missing root) fixes avstudnitz#154 - Don't validate "required" catelog attributes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The flag shouldn't be reversed. At the moment, when you set "Unset Field on Empty String" to "no", it will unset empty fields and visa versa.