Skip to content

Commit f9e6a50

Browse files
committed
MAGETWO-31977: [Remove TODOs] Integration
- Removed TODOs from Magento\Integration module
1 parent ae9c6a0 commit f9e6a50

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

app/code/Magento/Integration/Controller/Adminhtml/Integration/Save.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ protected function _redirectOnSaveError()
2929
* Save integration action.
3030
*
3131
* @return void
32-
* @todo: Fix cyclomatic complexity.
3332
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
3433
*/
3534
public function execute()
@@ -55,7 +54,6 @@ public function execute()
5554
/** @var array $data */
5655
$data = $this->getRequest()->getPost();
5756
if (!empty($data)) {
58-
// TODO: Move out work with API permissions to Web API module
5957
if (!isset($data['resource'])) {
6058
$integrationData['resource'] = [];
6159
}

app/code/Magento/Integration/Service/V1/Integration.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ public function create(array $integrationData)
4444
{
4545
$this->_checkIntegrationByName($integrationData['name']);
4646
$integration = $this->_integrationFactory->create($integrationData);
47-
// TODO: Think about double save issue
4847
$integration->save();
4948
$consumerName = 'Integration' . $integration->getId();
5049
$consumer = $this->_oauthService->createConsumer(['name' => $consumerName]);

0 commit comments

Comments
 (0)