Problems with category item import

Components

Labels

Description

Here several problem with category item import I've found recently:

  1. When in "Popup Mode", then pressing "Cancel" button on last import step will replace left (menu) frame (and not right frame) import template.

  2. No matter what category is specified in CSV file, but any created product will have target import category set as primary and a category from CSV file as secondary (problem, since "Parent Section" field appeared on an editing form).

  3. Columns "Category1", "Category2", ..., "CategoryN" from CSV file are ignored during import, when they are manually specified in import column list (in automatic column detection mode they work).

Context Information

None

Additional information (do not use)

Solution to 1st problem could me showing import dialog in right frame directly and not in popup/modal window as we do right now.

Solution to 2nd problem could be setting CategoryId virtual field of a category item to a category, found in CSV file (if it was found there) before creating it (category item) in database.

Patch "wrong_category_import_logic.patch" contains a fix for a 3rd problem.

Attachments

7

Activity

Show:

Alex July 25, 2012 at 10:33 AM

Since 5.2.0 version was released.

Alex October 19, 2011 at 10:57 AM

Fix committed to [b]5.2.x branch/b. Commit Message:

Fixes #0001077: Problems with category item import
Commit on behalf of Erik

Alex October 19, 2011 at 10:54 AM

Major:
======
1. Import wasn't working when CSV file to be imported haven't CreatedById column at all - fixed

Minor:
======
1. Records in Skins table wasn't updated in core/install/install_data.sql

2. File "core/install/upgrades.css" changes were made for 5.1.3-B1 and not for 5.2.0 version

Erik S. October 14, 2011 at 5:19 PM

1. Initially I had same effect, but later this effect evaporated - import never opens as Modal Window - no dependance of UsePopups configuration value. Maybe, initially something was cached in the browser.

2. Status field was disabled because CreatedById field is required and not present in the import file - such case is fixed in the new patch.

3. Yes, CreatedById field is required for category items. This is defined in particular category items unit config files, as rule.

In the 2nd version of the patch additionally fixed behavior on pressing Cancel button - ensured redirect to the catalog.

new patch failes attached:
1) categoryitem_import_problems_core.2.patch
2) categoryitem_import_problems_modules.2.patch

Dmitry Andrejev September 7, 2011 at 10:48 PM

Moving to 5.1.3.

[B]Tested okay with fewer Import related issues that needs to be fixed before committing:/B

1. In Modal Window mode after the Import is successful window shows the catalog while in normal conditions should close up and refresh the Catalog. This works before the Patch.

2. For some reason Status field value is always Disabled. I tried putting "1" or "Active" with no luck. I know this worked before.

3. For some reason CreatedById field is required during the Import why it hasn't been before (at least some versions ago).

NOTE: See products Export file used as example for tests

As to the rest it's working good.

Fixed

Details

Priority

Assignee

Reporter

Developer

Change Log Message

Patch Instructions

Patches must be submitted through Phabricator.

External issue ID

Story Points

Fix versions

Affects versions

Created July 5, 2011 at 8:40 AM
Updated December 30, 2024 at 2:02 AM
Resolved July 25, 2012 at 10:33 AM