Skip to content
This repository was archived by the owner on Apr 29, 2019. It is now read-only.

Added $connection property to class #141

Merged
merged 4 commits into from
Dec 6, 2018
Merged

Added $connection property to class #141

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ac90f7a
Added $connection property to class
maxalmonte14 Oct 26, 2018
6789eb4
Removed unused connection property, added PHPMD.UnusedFormalParameter…
maxalmonte14 Oct 30, 2018
7f3808c
Added public connection property
maxalmonte14 Nov 10, 2018
5ff88c0
Removed PHPMD suppress warning directive
maxalmonte14 Nov 16, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions app/code/Magento/DownloadableImportExport/Helper/Uploader.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,11 @@ class Uploader extends \Magento\Framework\App\Helper\AbstractHelper
*/
protected $parameters = [];

/**
* @var \Magento\Framework\DB\Adapter\AdapterInterface
*/
public $connection;

/**
* Construct
*
Expand All @@ -46,6 +51,8 @@ class Uploader extends \Magento\Framework\App\Helper\AbstractHelper
* @param \Magento\CatalogImportExport\Model\Import\UploaderFactory $uploaderFactory
* @param \Magento\Framework\App\ResourceConnection $resource
* @param \Magento\Framework\Filesystem $filesystem
*
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess now we do not need this suppress warning since the connection is being set.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

*/
public function __construct(
\Magento\Framework\App\Helper\Context $context,
Expand Down