-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Closed
Labels
CDIssue recommended for the contribution dayIssue recommended for the contribution dayComponent: CatalogComponent: MediaStorageIssue: Clear DescriptionGate 2 Passed. Manual verification of the issue description passedGate 2 Passed. Manual verification of the issue description passedIssue: ConfirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedIssue: Format is validGate 1 Passed. Automatic verification of issue format passedGate 1 Passed. Automatic verification of issue format passedIssue: Ready for WorkGate 4. Acknowledged. Issue is added to backlog and ready for developmentGate 4. Acknowledged. Issue is added to backlog and ready for developmentPriority: P2A defect with this priority could have functionality issues which are not to expectations.A defect with this priority could have functionality issues which are not to expectations.Progress: doneReproduced on 2.1.xThe issue has been reproduced on latest 2.1 releaseThe issue has been reproduced on latest 2.1 releaseReproduced on 2.2.xThe issue has been reproduced on latest 2.2 releaseThe issue has been reproduced on latest 2.2 releaseReproduced on 2.3.xThe issue has been reproduced on latest 2.3 releaseThe issue has been reproduced on latest 2.3 releaseReproduced on 2.4.xThe issue has been reproduced on latest 2.4-develop branchThe issue has been reproduced on latest 2.4-develop branchSeverity: S2Major restrictions or short-term circumventions are required until a fix is available.Major restrictions or short-term circumventions are required until a fix is available.
Description
Preconditions
- Magento 2.4-develop
Steps to reproduce
- Set media storage to database
- Upload category image in admin section
- Save the category
Expected result
- The image row in media_storage_file_storage table should have directory
catalog/category
and correctdirectory_id
Actual result
- The image row in media_storage_file_storage table has directory
catalog/tmp/category
I believe the problem lies with the directory_id
not being changed in method Magento\MediaStorage\Model\ResourceModel\File\Storage\Database::copyFile
When I manually change the directory_id
in $data
from the catalog/tmp/category
directory_id to the catalog/category
directory_id before running the $connection->insertOnDuplicate
method the file is correctly copied to another row with the correct data.
As another note, should it not convert the existing row to simply change the directory_id
and directory
rather than copying to a new row...
minlare, davidwindell and alinolandry
Metadata
Metadata
Assignees
Labels
CDIssue recommended for the contribution dayIssue recommended for the contribution dayComponent: CatalogComponent: MediaStorageIssue: Clear DescriptionGate 2 Passed. Manual verification of the issue description passedGate 2 Passed. Manual verification of the issue description passedIssue: ConfirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedIssue: Format is validGate 1 Passed. Automatic verification of issue format passedGate 1 Passed. Automatic verification of issue format passedIssue: Ready for WorkGate 4. Acknowledged. Issue is added to backlog and ready for developmentGate 4. Acknowledged. Issue is added to backlog and ready for developmentPriority: P2A defect with this priority could have functionality issues which are not to expectations.A defect with this priority could have functionality issues which are not to expectations.Progress: doneReproduced on 2.1.xThe issue has been reproduced on latest 2.1 releaseThe issue has been reproduced on latest 2.1 releaseReproduced on 2.2.xThe issue has been reproduced on latest 2.2 releaseThe issue has been reproduced on latest 2.2 releaseReproduced on 2.3.xThe issue has been reproduced on latest 2.3 releaseThe issue has been reproduced on latest 2.3 releaseReproduced on 2.4.xThe issue has been reproduced on latest 2.4-develop branchThe issue has been reproduced on latest 2.4-develop branchSeverity: S2Major restrictions or short-term circumventions are required until a fix is available.Major restrictions or short-term circumventions are required until a fix is available.
Type
Projects
Status
Done