Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
a5e014a
[Backport] Fix typos in variable names
dmytro-ch May 18, 2018
ee90e54
[Backport] Fix typos in PHPDocs and comments
dmytro-ch May 18, 2018
b13d09a
Eliminate usage of "else" statements. Code improvements
rogyar May 18, 2018
1b2d726
Fixed readme minor typo
rogyar May 18, 2018
11c9081
Alignment Array assignment
VitaliyBoyko May 19, 2018
27e74b9
Fix typo in securityCheckers array
VitaliyBoyko May 19, 2018
e74bea1
Typo in SSL port number
VitaliyBoyko May 19, 2018
d8fd4ee
Fix incorrect phpdoc return type
VitaliyBoyko May 19, 2018
38c44cb
ENGCOM-1590: [Backport] Added hyphenation cutting edge to cutting-edg…
magento-engcom-team May 19, 2018
2a547a2
Unused variable removed
VitaliyBoyko May 19, 2018
638f7b0
ENGCOM-1594: [Backport] Unused variable removed #15385
magento-engcom-team May 20, 2018
221a75c
ENGCOM-1595: [Backport] Fix incorrect phpdoc return type #15380
magento-engcom-team May 20, 2018
95e5a31
ENGCOM-1596: [Backport] Typo in SSL port number #15377
magento-engcom-team May 20, 2018
0882ff9
ENGCOM-1597: [Backport] Fix typo in securityCheckers array #15375
magento-engcom-team May 20, 2018
8b7dbb6
ENGCOM-1598: [Backport] Alignment Array assignment #15363
magento-engcom-team May 20, 2018
2582a6b
ENGCOM-1600: [Backport] Fix typos in PHPDocs and comments #15299
magento-engcom-team May 20, 2018
02240c3
ENGCOM-1601: [Backport] Fix typos in variable names #15298
magento-engcom-team May 20, 2018
3f85adf
ENGCOM-1601: [Backport] Fix typos in variable names #15298
May 20, 2018
31ebdf5
ENGCOM-1600: [Backport] Fix typos in PHPDocs and comments #15299
May 20, 2018
344f8cc
ENGCOM-1598: [Backport] Alignment Array assignment #15363
May 20, 2018
a1087dd
ENGCOM-1597: [Backport] Fix typo in securityCheckers array #15375
May 20, 2018
df4f4dc
ENGCOM-1596: [Backport] Typo in SSL port number #15377
May 20, 2018
d950ba0
ENGCOM-1595: [Backport] Fix incorrect phpdoc return type #15380
May 20, 2018
17a46af
ENGCOM-1594: [Backport] Unused variable removed #15385
May 20, 2018
a74e6c4
ENGCOM-1590: [Backport] Added hyphenation cutting edge to cutting-edg…
May 20, 2018
5e6d1b2
Fix typos in PHPDocs and comments
May 20, 2018
cee0c40
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
May 20, 2018
3c89d45
[Backport] Remove non-existing argument
dmytro-ch May 20, 2018
2c31e4d
Fixed typo mistake in function comment
sanganinamrata May 18, 2018
32c4a33
Removed extra added space in function comment and fixed typo mistakes.
sanganinamrata May 18, 2018
c13e1fd
Revert back file changes
sanganinamrata May 18, 2018
07e3a8f
Revert back file changes.
sanganinamrata May 18, 2018
c8a39e1
ENGCOM-1612: [Backport] Remove non-existing argument #15394
magento-engcom-team May 21, 2018
b0ca9a8
ENGCOM-1614: [Backport] Fixed typo mistake in function comment #15396
magento-engcom-team May 21, 2018
23e5158
ENGCOM-1614: [Backport] Fixed typo mistake in function comment #15396
May 21, 2018
a03e205
ENGCOM-1612: [Backport] Remove non-existing argument #15394
May 21, 2018
d1c4cfc
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
magento-engcom-team May 21, 2018
5365c12
ENGCOM-1629: [Backport] Eliminate usage of 'else' statements #15309
magento-engcom-team May 21, 2018
47dc210
Fixed typo in MagentoUi abstract.js
VitaliyBoyko May 21, 2018
1a814fb
ENGCOM-1638: [Backport-2.1] Fixed typo in MagentoUi abstract.js #15412
magento-engcom-team May 21, 2018
a07e620
ENGCOM-1638: [Backport-2.1] Fixed typo in MagentoUi abstract.js #15412
May 22, 2018
e73f29c
ENGCOM-1629: [Backport] Eliminate usage of "else" statements #15309
May 22, 2018
3085bda
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
magento-engcom-team May 22, 2018
e47349a
Fixed Purchased Order Form button should visible properly
gelanivishal May 19, 2018
c5d2d77
Minor styles nesting fix
rogyar May 21, 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
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
[![Open Source Helpers](https://www.codetriage.com/magento/magento2/badges/users.svg)](https://www.codetriage.com/magento/magento2)
[![Gitter](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/magento/magento2?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge)
<h2>Welcome</h2>
Welcome to Magento 2 installation! We're glad you chose to install Magento 2, a cutting edge, feature-rich eCommerce solution that gets results.
Welcome to Magento 2 installation! We're glad you chose to install Magento 2, a cutting-edge, feature-rich eCommerce solution that gets results.

The installation instructions that used to be here are now published on our GitHub site. Use the information on this page to get started or go directly to the <a href="http://devdocs.magento.com/guides/v2.0/install-gde/bk-install-guide.html" target="_blank">guide</a>.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,7 @@ public function getCondition()
{
if ($this->getValue()) {
return $this->getColumn()->getValue();
} else {
return [['neq' => $this->getColumn()->getValue()], ['is' => new \Zend_Db_Expr('NULL')]];
}
return [['neq' => $this->getColumn()->getValue()], ['is' => new \Zend_Db_Expr('NULL')]];
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -205,9 +205,8 @@ public function getSelectedJson()
if ($selected = $this->getRequest()->getParam($this->getFormFieldNameInternal())) {
$selected = explode(',', $selected);
return join(',', $selected);
} else {
return '';
}
return '';
}

/**
Expand All @@ -220,9 +219,8 @@ public function getSelected()
if ($selected = $this->getRequest()->getParam($this->getFormFieldNameInternal())) {
$selected = explode(',', $selected);
return $selected;
} else {
return [];
}
return [];
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -215,9 +215,8 @@ public function getSelectedJson()
if ($selected = $this->getRequest()->getParam($this->getFormFieldNameInternal())) {
$selected = explode(',', $selected);
return join(',', $selected);
} else {
return '';
}
return '';
}

/**
Expand All @@ -230,9 +229,8 @@ public function getSelected()
if ($selected = $this->getRequest()->getParam($this->getFormFieldNameInternal())) {
$selected = explode(',', $selected);
return $selected;
} else {
return [];
}
return [];
}

/**
Expand Down
3 changes: 1 addition & 2 deletions app/code/Magento/Backend/Controller/Adminhtml/Auth/Login.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,8 @@ public function execute()
// redirect according to rewrite rule
if ($requestUrl != $backendUrl) {
return $this->getRedirect($backendUrl);
} else {
return $this->resultPageFactory->create();
}
return $this->resultPageFactory->create();
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,8 @@ private function getSecurityCookie()
{
if (!($this->securityCookie instanceof SecurityCookie)) {
return \Magento\Framework\App\ObjectManager::getInstance()->get(SecurityCookie::class);
} else {
return $this->securityCookie;
}
return $this->securityCookie;
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,9 @@ public function __get($name)
{
if (array_key_exists($name, $this->_attributes)) {
return $this->_attributes[$name];
} else {
trigger_error('Undefined property on ' . get_class($this) . ': ' . $name, E_USER_NOTICE);
return null;
}
trigger_error('Undefined property on ' . get_class($this) . ': ' . $name, E_USER_NOTICE);
return null;
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,8 @@ public function getChildren($item)

if (isset($itemsArray[$item->getOrderItem()->getId()])) {
return $itemsArray[$item->getOrderItem()->getId()];
} else {
return null;
}
return null;
}

/**
Expand Down Expand Up @@ -186,9 +185,8 @@ public function getOrderItem()
{
if ($this->getItem() instanceof \Magento\Sales\Model\Order\Item) {
return $this->getItem();
} else {
return $this->getItem()->getOrderItem();
}
return $this->getItem()->getOrderItem();
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -174,9 +174,8 @@ public function isSelected($selection)
return in_array($selection->getSelectionId(), $selectedOptions);
} elseif ($selectedOptions == 'None') {
return false;
} else {
return $selection->getIsDefault() && $selection->isSaleable();
}
return $selection->getIsDefault() && $selection->isSaleable();
}

/**
Expand Down
6 changes: 2 additions & 4 deletions app/code/Magento/Bundle/Block/Sales/Order/Items/Renderer.php
Original file line number Diff line number Diff line change
Expand Up @@ -114,9 +114,8 @@ public function getValueHtml($item)
if ($attributes = $this->getSelectionAttributes($item)) {
return sprintf('%d', $attributes['qty']) . ' x ' . $this->escapeHtml($item->getName()) . " "
. $this->getOrder()->formatPrice($attributes['price']);
} else {
return $this->escapeHtml($item->getName());
}
return $this->escapeHtml($item->getName());
}

/**
Expand Down Expand Up @@ -151,9 +150,8 @@ public function getChildren($item)

if (isset($itemsArray[$item->getOrderItem()->getId()])) {
return $itemsArray[$item->getOrderItem()->getId()];
} else {
return null;
}
return null;
}

/**
Expand Down
3 changes: 1 addition & 2 deletions app/code/Magento/Bundle/Model/Plugin/PriceBackend.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,7 @@ public function aroundValidate(
&& $object->getPriceType() == \Magento\Bundle\Model\Product\Price::PRICE_TYPE_DYNAMIC
) {
return true;
} else {
return $proceed($object);
}
return $proceed($object);
}
}
3 changes: 1 addition & 2 deletions app/code/Magento/Bundle/Model/Product/Type.php
Original file line number Diff line number Diff line change
Expand Up @@ -948,9 +948,8 @@ public function shakeSelections($firstItem, $secondItem)
];
if ($aPosition == $bPosition) {
return 0;
} else {
return $aPosition < $bPosition ? -1 : 1;
}
return $aPosition < $bPosition ? -1 : 1;
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,8 @@ public function getChildren($item)

if (isset($itemsArray[$item->getOrderItem()->getId()])) {
return $itemsArray[$item->getOrderItem()->getId()];
} else {
return null;
}
return null;
}

/**
Expand Down Expand Up @@ -196,9 +195,8 @@ public function getOrderItem()
{
if ($this->getItem() instanceof \Magento\Sales\Model\Order\Item) {
return $this->getItem();
} else {
return $this->getItem()->getOrderItem();
}
return $this->getItem()->getOrderItem();
}

/**
Expand Down
3 changes: 1 addition & 2 deletions app/code/Magento/Bundle/Pricing/Adjustment/Calculator.php
Original file line number Diff line number Diff line change
Expand Up @@ -267,9 +267,8 @@ public function calculateBundleAmount($basePriceValue, $bundleProduct, $selectio
{
if ($bundleProduct->getPriceType() == Price::PRICE_TYPE_FIXED) {
return $this->calculateFixedBundleAmount($basePriceValue, $bundleProduct, $selectionPriceList, $exclude);
} else {
return $this->calculateDynamicBundleAmount($basePriceValue, $bundleProduct, $selectionPriceList, $exclude);
}
return $this->calculateDynamicBundleAmount($basePriceValue, $bundleProduct, $selectionPriceList, $exclude);
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -163,8 +163,7 @@ public function getProduct()
{
if ($this->bundleProduct->getPriceType() == Price::PRICE_TYPE_DYNAMIC) {
return parent::getProduct();
} else {
return $this->bundleProduct;
}
return $this->bundleProduct;
}
}
3 changes: 1 addition & 2 deletions app/code/Magento/Bundle/Pricing/Price/ConfiguredPrice.php
Original file line number Diff line number Diff line change
Expand Up @@ -124,9 +124,8 @@ public function getValue()
$this->priceInfo
->getPrice(BundleDiscountPrice::PRICE_CODE)
->calculateDiscount($configuredOptionsAmount);
} else {
return parent::getValue();
}
return parent::getValue();
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ public function getDefaultStoreId()
*
* @param string $table
* @param array|int $attributeIds
* @return \Magento\Eav\Model\Entity\Collection\AbstractCollection
* @return \Magento\Framework\DB\Select
*/
protected function _getLoadAttributesSelect($table, $attributeIds = [])
{
Expand Down
6 changes: 3 additions & 3 deletions app/code/Magento/Paypal/Model/Api/Nvp.php
Original file line number Diff line number Diff line change
Expand Up @@ -1027,7 +1027,7 @@ public function callGetPalDetails()
}

/**
* Set Customer BillingA greement call
* Set Customer BillingAgreement call
*
* @return void
* @link https://cms.paypal.com/us/cgi-bin/?&cmd=_render-content&content_ID=developer/e_howto_api_nvp_r_SetCustomerBillingAgreement
Expand Down Expand Up @@ -1418,15 +1418,15 @@ protected function _deformatNVP($nvpstr)
$nvpstr = strpos($nvpstr, "\r\n\r\n") !== false ? substr($nvpstr, strpos($nvpstr, "\r\n\r\n") + 4) : $nvpstr;

while (strlen($nvpstr)) {
//postion of Key
//position of Key
$keypos = strpos($nvpstr, '=');
//position of value
$valuepos = strpos($nvpstr, '&') ? strpos($nvpstr, '&') : strlen($nvpstr);

/*getting the Key and Value values and storing in a Associative Array*/
$keyval = substr($nvpstr, $intial, $keypos);
$valval = substr($nvpstr, $keypos + 1, $valuepos - $keypos - 1);
//decoding the respose
//decoding the response
$nvpArray[urldecode($keyval)] = urldecode($valval);
$nvpstr = substr($nvpstr, $valuepos + 1, strlen($nvpstr));
}
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Security/etc/adminhtml/di.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
<arguments>
<argument name="securityCheckers" xsi:type="array">
<item name="frequency" xsi:type="object">Magento\Security\Model\SecurityChecker\Frequency</item>
<item name="wuantity" xsi:type="object">Magento\Security\Model\SecurityChecker\Quantity</item>
<item name="quantity" xsi:type="object">Magento\Security\Model\SecurityChecker\Quantity</item>
</argument>
</arguments>
</type>
Expand Down
8 changes: 7 additions & 1 deletion app/code/Magento/Store/Setup/InstallSchema.php
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,13 @@ public function install(SchemaSetupInterface $setup, ModuleContextInterface $con
*/
$connection->insertForce(
$installer->getTable('store_group'),
['group_id' => 0, 'website_id' => 0, 'name' => 'Default', 'root_category_id' => 0, 'default_store_id' => 0]
[
'group_id' => 0,
'website_id' => 0,
'name' => 'Default',
'root_category_id' => 0,
'default_store_id' => 0
]
);
$connection->insertForce(
$installer->getTable('store_group'),
Expand Down
1 change: 0 additions & 1 deletion app/code/Magento/Translation/Block/Html/Head/Config.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ class Config extends \Magento\Framework\View\Element\AbstractBlock

/**
* @param \Magento\Framework\View\Element\Context $context
* @param RequireJsConfig $config
* @param \Magento\Framework\View\Page\Config $pageConfig
* @param \Magento\Translation\Model\FileManager $fileManager
* @param Inline $inline
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ define([
},

/**
* Sets 'value' as 'hidden' propertie's value, triggers 'toggle' event,
* Sets 'value' as 'hidden' property's value, triggers 'toggle' event,
* sets instance's hidden identifier in params storage based on
* 'value'.
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@
}
}
}

#po_number {
margin-bottom: 20px;
}
}

.payment-method-title {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ class SimplifiedselectElement extends SelectElement
protected $optionGroupValue = ".//*[@data-title='%s' or contains(normalize-space(.), %s)]";

/**
* Select value in ropdown which has option groups.
* Select value in dropdown which has option groups.
*
* @param string $value
* @return void
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ class SwitcherElement extends SimpleElement
protected $parentContainer = 'parent::div[@data-role="switcher"]';

/**
* XPath selector for label text on swticher element.
* XPath selector for label text on switcher element.
*
* @var string
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ class Search extends SuggestElement
protected $actionToggle = '.action-toggle';

/**
* Saerch result dropdown.
* Search result dropdown.
*
* @var string
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ class AssertProductAttributeIsHtmlAllowed extends AbstractConstraint

/**
* Check whether html tags are using in attribute value.
* Checked tag structure <b><i>atttribute_default_value</p></i></b>
* Checked tag structure <b><i>attribute_default_value</p></i></b>
*
* @param InjectableFixture $product
* @param CatalogProductAttribute $attribute
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ class TierPrice extends DataSource
private $fixtureFactory;

/**
* Rought fixture field data.
* Rough fixture field data.
*
* @var array
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ class WebsiteIds extends DataSource
private $fixtureFactory;

/**
* Rought fixture field data.
* Rough fixture field data.
*
* @var array
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public function processAssert(CheckoutOnepage $checkoutOnepage, $billingCheckbox
}

/**
* Returns a string representation of successfull assertion.
* Returns a string representation of successful assertion.
*
* @return string
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
use Magento\ConfigurableProduct\Test\Block\Product\View\ConfigurableOptions;

/**
* Open created configurble product on frontend and choose variation with tier price.
* Open created configurable product on frontend and choose variation with tier price.
*/
class AssertProductTierPriceOnProductPage extends AssertProductPage
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ public function __prepare(FixtureFactory $fixtureFactory)
}

/**
* Prepare VAT ID confguration.
* Prepare VAT ID configuration.
*
* @param ConfigData $vatConfig
* @param string $customerGroup
Expand Down
Loading