Skip to content

Commit 52b184e

Browse files
author
Andrew Chorniy
committed
Fixed issues after code sniffer
1 parent 57cf6ea commit 52b184e

File tree

2 files changed

+17
-7
lines changed

2 files changed

+17
-7
lines changed

app/code/Magento/Customer/Model/Data/Customer.php

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@
99
use \Magento\Framework\Api\AttributeValueFactory;
1010

1111
/**
12-
* Class Customer
12+
* Customer data model
13+
*
1314
* @SuppressWarnings(PHPMD.ExcessivePublicCount)
1415
*/
1516
class Customer extends \Magento\Framework\Api\AbstractExtensibleObject implements
@@ -39,7 +40,7 @@ public function __construct(
3940
}
4041

4142
/**
42-
* {@inheritdoc}
43+
* @inheritdoc
4344
*/
4445
protected function getCustomAttributesCodes()
4546
{
@@ -50,6 +51,8 @@ protected function getCustomAttributesCodes()
5051
}
5152

5253
/**
54+
* Get default billing address id
55+
*
5356
* @return string|null
5457
*/
5558
public function getDefaultBilling()
@@ -489,7 +492,7 @@ public function setDisableAutoGroupChange($disableAutoGroupChange)
489492
}
490493

491494
/**
492-
* {@inheritdoc}
495+
* @inheritdoc
493496
*
494497
* @return \Magento\Customer\Api\Data\CustomerExtensionInterface|null
495498
*/
@@ -499,7 +502,7 @@ public function getExtensionAttributes()
499502
}
500503

501504
/**
502-
* {@inheritdoc}
505+
* @inheritdoc
503506
*
504507
* @param \Magento\Customer\Api\Data\CustomerExtensionInterface $extensionAttributes
505508
* @return $this

app/code/Magento/Customer/Test/Unit/Model/Data/CustomerTest.php

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
11
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
26

37
namespace Magento\Customer\Test\Unit\Model\Data;
48

@@ -45,12 +49,15 @@ public function testGetGroupId()
4549
* @dataProvider getCreatedInDataProvider
4650
*
4751
* @return void
48-
*/
52+
*/
4953
public function testGetCreatedIn($options, $expectedResult)
5054
{
51-
for ($i = 0; $i < count($options); $i++) {
55+
$optionsCount = count($options);
56+
$expectedCount = count($expectedResult);
57+
58+
for ($i = 0; $i < $optionsCount; $i++) {
5259
$this->model->setCreatedIn($options[$i]);
53-
for ($j = $i; $j < count($expectedResult); $j++) {
60+
for ($j = $i; $j < $expectedCount; $j++) {
5461
$this->assertEquals($expectedResult[$j], $this->model->getCreatedIn());
5562
break;
5663
}

0 commit comments

Comments
 (0)