Skip to content

Commit dd77f7e

Browse files
committed
ACPT-493: Upload csv with request parameter
1 parent 601ccea commit dd77f7e

File tree

4 files changed

+3
-15
lines changed

4 files changed

+3
-15
lines changed

app/code/Magento/ImportExport/Model/Import.php

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,6 @@ class Import extends AbstractModel
220220
* @param DateTime $localeDate
221221
* @param array $data
222222
* @param ManagerInterface|null $messageManager
223-
* @param SourceFactory $sourceFactory
224223
* @param Upload|null $upload
225224
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
226225
*/
@@ -241,7 +240,6 @@ public function __construct(
241240
DateTime $localeDate,
242241
array $data = [],
243242
ManagerInterface $messageManager = null,
244-
SourceFactory $sourceFactory = null,
245243
Upload $upload = null
246244
) {
247245
$this->_importExportData = $importExportData;
@@ -259,8 +257,6 @@ public function __construct(
259257
$this->localeDate = $localeDate;
260258
$this->messageManager = $messageManager ?: ObjectManager::getInstance()
261259
->get(ManagerInterface::class);
262-
$this->sourceFactory = $sourceFactory?? ObjectManager::getInstance()
263-
->get(SourceFactory::class);
264260
$this->upload = $upload ?: ObjectManager::getInstance()
265261
->get(Upload::class);
266262
parent::__construct($logger, $filesystem, $data);
@@ -313,8 +309,7 @@ protected function _getEntityAdapter()
313309

314310
/**
315311
* Returns source adapter object.
316-
* @Deprecated
317-
* @see \Magento\ImportExport\Model\Import\Source\Factory::create()
312+
*
318313
* @param string $sourceFile Full path to source file
319314
* @return AbstractSource
320315
* @throws FileSystemException
@@ -584,11 +579,7 @@ public function uploadFileAndGetSource()
584579
{
585580
$sourceFile = $this->uploadSource();
586581
try {
587-
$source = $this->sourceFactory->create(
588-
$sourceFile,
589-
$this->_filesystem->getDirectoryWrite(DirectoryList::ROOT),
590-
$this->getData(self::FIELD_FIELD_SEPARATOR)
591-
);
582+
$source = $this->_getSourceAdapter($sourceFile);
592583
} catch (\Exception $e) {
593584
$this->_varDirectory->delete($this->_varDirectory->getRelativePath($sourceFile));
594585
throw new LocalizedException(__($e->getMessage()));

app/code/Magento/ImportExport/Model/Import/Source/Base64EncodedCsvData.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ class Base64EncodedCsvData extends AbstractSource
3535
*/
3636
public function __construct(string $source)
3737
{
38+
// phpcs:ignore Magento2.Functions.DiscouragedFunction
3839
$source = trim(base64_decode($source));
3940
$rowsData = preg_split("/\r\n|\n|\r/", $source);
4041
$colNames = explode(',', $rowsData[0]);

app/code/Magento/ImportExport/Test/Unit/Helper/ReportTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,6 @@ public function testGetSummaryStats()
204204
$importHistoryModel = $this->createMock(History::class);
205205
$localeDate = $this->createMock(\Magento\Framework\Stdlib\DateTime\DateTime::class);
206206
$upload = $this->createMock(Upload::class);
207-
$sourceFactoryMock = $this->createMock(\Magento\ImportExport\Model\Import\Source\Factory::class);
208207
$import = new Import(
209208
$logger,
210209
$filesystem,
@@ -222,7 +221,6 @@ public function testGetSummaryStats()
222221
$localeDate,
223222
[],
224223
null,
225-
$sourceFactoryMock,
226224
$upload
227225
);
228226
$import->setData('entity', 'catalog_product');

app/code/Magento/ImportExport/Test/Unit/Model/ImportTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,6 @@ protected function setUp(): void
231231
->expects($this->any())
232232
->method('getDriver')
233233
->willReturn($this->_driver);
234-
$sourceFactoryMock = $this->createMock(\Magento\ImportExport\Model\Import\Source\Factory::class);
235234
$this->upload = $this->createMock(Upload::class);
236235
$this->import = $this->getMockBuilder(Import::class)
237236
->setConstructorArgs(
@@ -252,7 +251,6 @@ protected function setUp(): void
252251
$this->dateTime,
253252
[],
254253
null,
255-
$sourceFactoryMock,
256254
$this->upload
257255
]
258256
)

0 commit comments

Comments
 (0)