Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 7897 #15399

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
51000b9
Add 'const' type support to layout arguments
IgorVitol May 8, 2018
825832b
ENGCOM-1473: Add 'const' type support to layout arguments #15058
magento-engcom-team May 8, 2018
6a7d40a
Sync braintree config with customer's data.
vovayatsyuk May 10, 2018
8712f3e
JSDoc fix
vovayatsyuk May 11, 2018
d6ba03c
Fixed failed test
vovayatsyuk May 11, 2018
ebd8af4
ESLint and JSCS fixes
vovayatsyuk May 11, 2018
0b786fe
ENGCOM-1513: Fix outdated address data when using Braintree's 'Pay wi…
magento-engcom-team May 14, 2018
9d64743
Fix typo in Image::open exception message
t-richards May 17, 2018
ef68020
[fix] typo in method name _getCharg[e]ableOptionPrice
May 17, 2018
d1953f1
ENGCOM-1582: Fix typo in Image::open exception message #15269
magento-engcom-team May 17, 2018
541fdf6
[fix] typo in private method name getUniq[ue]ImageIndex
May 17, 2018
7dbc62a
ENGCOM-1584: [fix] typo in private method name getUniq[ue]ImageIndex …
magento-engcom-team May 17, 2018
1d518fd
Fix typo in database column comment
VitaliyBoyko May 17, 2018
22c058a
ENGCOM-1586: [Backport] Fix typo in database column comment #15291
magento-engcom-team May 17, 2018
f665db5
Fix typo in property name
dmytro-ch May 17, 2018
076e60a
Merge branch '2.2-develop' into fix/2.2-fix-typos-in-property-name
dmytro-ch May 17, 2018
08575f3
Fix typos in PHPDocs and comments
dmytro-ch May 17, 2018
4f85f87
Fix typos in variable names
dmytro-ch May 17, 2018
ad501a9
Fix typo in DeploymentConfig class
dmytro-ch May 18, 2018
cbe91ae
ENGCOM-1587: Fix typo in property name #15292
magento-engcom-team May 18, 2018
8ea661c
[task] add deprecated notice to misspelled method
May 18, 2018
48a8b1d
Fixed typo mistake in function comment
sanganinamrata May 18, 2018
1e1ceb9
Removed extra added space in function comment and fixed typo mistakes.
sanganinamrata May 18, 2018
298e392
Revert back file changes
sanganinamrata May 18, 2018
44031e0
Revert back file changes.
sanganinamrata May 18, 2018
c4db071
Merge branch 'ns-patch-8' of https://github.com/NamrataChangani/magen…
sanganinamrata May 18, 2018
630faa1
ENGCOM-1587: Fix typo in property name #15292
VladimirZaets May 18, 2018
96d31f5
ENGCOM-1586: [Backport] Fix typo in database column comment #15291
VladimirZaets May 18, 2018
807ab61
ENGCOM-1584: [fix] typo in private method name getUniq[ue]ImageIndex …
VladimirZaets May 18, 2018
c920e3e
ENGCOM-1582: Fix typo in Image::open exception message #15269
VladimirZaets May 18, 2018
94759da
ENGCOM-1513: Fix outdated address data when using Braintree's "Pay wi…
VladimirZaets May 18, 2018
819c65e
ENGCOM-1473: Add 'const' type support to layout arguments #15058
VladimirZaets May 18, 2018
b40aae2
Unused variable removed
VitaliyBoyko May 19, 2018
9a28908
:arrows_clockwise: [EngCom] Public Pull Requests - 2.2-develop
May 20, 2018
1da6d36
ENGCOM-1593: [Backport-2.2] Unused variable removed #15386
magento-engcom-team May 20, 2018
54ffb8c
ENGCOM-1599: Fixed typo mistake in function comment #15302
magento-engcom-team May 20, 2018
288c1b6
ENGCOM-1602: Fix typos in variable names #15294
magento-engcom-team May 20, 2018
62050b8
ENGCOM-1580: [fix] typo in method name _getCharg[e]ableOptionPrice #1…
magento-engcom-team May 20, 2018
b295a28
ENGCOM-1588: Fix typos in PHPDocs and comments #15293
magento-engcom-team May 20, 2018
a3f8bd5
ENGCOM-1602: Fix typos in variable names #15294
May 20, 2018
bd027a7
ENGCOM-1599: Fixed typo mistake in function comment #15302
May 20, 2018
ae634b4
ENGCOM-1593: [Backport-2.2] Unused variable removed #15386
May 20, 2018
a6c4865
ENGCOM-1588: Fix typos in PHPDocs and comments #15293
May 20, 2018
2bd2994
ENGCOM-1580: [fix] typo in method name _getCharg[e]ableOptionPrice #1…
May 20, 2018
32892a4
:arrows_clockwise: [EngCom] Public Pull Requests - 2.2-develop
May 20, 2018
48b9597
issue:7897 fixed - Menu widget submenu alignment
Abdulrahimbadiyal May 21, 2018
f2ce324
issue:7897 fixed - Menu widget submenu alignment
Abdulrahimbadiyal 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
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
define([
'jquery',
'underscore',
'mage/utils/wrapper',
'Magento_Checkout/js/view/payment/default',
'Magento_Braintree/js/view/payment/adapter',
'Magento_Checkout/js/model/quote',
Expand All @@ -18,6 +19,7 @@ define([
], function (
$,
_,
wrapper,
Component,
Braintree,
quote,
Expand Down Expand Up @@ -218,8 +220,9 @@ define([

/**
* Re-init PayPal Auth Flow
* @param {Function} callback - Optional callback
*/
reInitPayPal: function () {
reInitPayPal: function (callback) {
if (Braintree.checkout) {
Braintree.checkout.teardown(function () {
Braintree.checkout = null;
Expand All @@ -228,6 +231,18 @@ define([

this.disableButton();
this.clientConfig.paypal.amount = this.grandTotalAmount;
this.clientConfig.paypal.shippingAddressOverride = this.getShippingAddress();

if (callback) {
this.clientConfig.onReady = wrapper.wrap(
this.clientConfig.onReady,
function (original, checkout) {
this.clientConfig.onReady = original;
original(checkout);
callback();
}.bind(this)
);
}

Braintree.setConfig(this.clientConfig);
Braintree.setup();
Expand Down Expand Up @@ -404,15 +419,19 @@ define([
* Triggers when customer click "Continue to PayPal" button
*/
payWithPayPal: function () {
if (additionalValidators.validate()) {
this.reInitPayPal(function () {
if (!additionalValidators.validate()) {
return;
}

try {
Braintree.checkout.paypal.initAuthFlow();
} catch (e) {
this.messageContainer.addErrorMessage({
message: $t('Payment ' + this.getTitle() + ' can\'t be initialized.')
});
}
}
}.bind(this));
},

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@ private function getViewImages(array $themes): array
]);
$images = $config->getMediaEntities('Magento_Catalog', ImageHelper::MEDIA_TYPE_CONFIG_NODE);
foreach ($images as $imageId => $imageData) {
$uniqIndex = $this->getUniqImageIndex($imageData);
$uniqIndex = $this->getUniqueImageIndex($imageData);
$imageData['id'] = $imageId;
$viewImages[$uniqIndex] = $imageData;
}
Expand All @@ -225,11 +225,11 @@ private function getViewImages(array $themes): array
}

/**
* Get uniq image index
* Get unique image index
* @param array $imageData
* @return string
*/
private function getUniqImageIndex(array $imageData): string
private function getUniqueImageIndex(array $imageData): string
{
ksort($imageData);
unset($imageData['type']);
Expand Down
19 changes: 16 additions & 3 deletions app/code/Magento/Catalog/Model/Product/Option/Type/DefaultType.php
Original file line number Diff line number Diff line change
Expand Up @@ -338,7 +338,7 @@ public function getOptionPrice($optionValue, $basePrice)
{
$option = $this->getOption();

return $this->_getChargableOptionPrice($option->getPrice(), $option->getPriceType() == 'percent', $basePrice);
return $this->_getChargeableOptionPrice($option->getPrice(), $option->getPriceType() == 'percent', $basePrice);
}

/**
Expand Down Expand Up @@ -392,14 +392,27 @@ public function getProductOptions()
}

/**
* Return final chargable price for option
*
* @param float $price Price of option
* @param boolean $isPercent Price type - percent or fixed
* @param float $basePrice For percent price type
* @return float
* @deprecated 102.0.4 typo in method name
* @see _getChargeableOptionPrice
*/
protected function _getChargableOptionPrice($price, $isPercent, $basePrice)
{
return $this->_getChargeableOptionPrice($price, $isPercent, $basePrice);
}

/**
* Return final chargeable price for option
*
* @param float $price Price of option
* @param boolean $isPercent Price type - percent or fixed
* @param float $basePrice For percent price type
* @return float
*/
protected function _getChargeableOptionPrice($price, $isPercent, $basePrice)
{
if ($isPercent) {
return $basePrice * $price / 100;
Expand Down
4 changes: 2 additions & 2 deletions app/code/Magento/Catalog/Model/Product/Option/Type/Select.php
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ public function getOptionPrice($optionValue, $basePrice)
foreach (explode(',', $optionValue) as $value) {
$_result = $option->getValueById($value);
if ($_result) {
$result += $this->_getChargableOptionPrice(
$result += $this->_getChargeableOptionPrice(
$_result->getPrice(),
$_result->getPriceType() == 'percent',
$basePrice
Expand All @@ -237,7 +237,7 @@ public function getOptionPrice($optionValue, $basePrice)
} elseif ($this->_isSingleSelection()) {
$_result = $option->getValueById($optionValue);
if ($_result) {
$result = $this->_getChargableOptionPrice(
$result = $this->_getChargeableOptionPrice(
$_result->getPrice(),
$_result->getPriceType() == 'percent',
$basePrice
Expand Down
2 changes: 1 addition & 1 deletion app/code/Magento/Catalog/Setup/InstallSchema.php
Original file line number Diff line number Diff line change
Expand Up @@ -674,7 +674,7 @@ public function install(SchemaSetupInterface $setup, ModuleContextInterface $con
\Magento\Framework\DB\Ddl\Table::TYPE_SMALLINT,
null,
['unsigned' => true, 'nullable' => false, 'default' => '0'],
'Attriute Set ID'
'Attribute Set ID'
)
->addColumn(
'parent_id',
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 @@ -1427,15 +1427,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
1 change: 1 addition & 0 deletions app/code/Magento/Ui/etc/di.xml
Original file line number Diff line number Diff line change
Expand Up @@ -251,6 +251,7 @@
<type name="Magento\Framework\Data\Argument\Interpreter\Composite">
<arguments>
<argument name="interpreters" xsi:type="array">
<item name="const" xsi:type="object">Magento\Framework\Data\Argument\Interpreter\Constant</item>
<item name="object" xsi:type="object">configurableObjectArgumentInterpreterProxy</item>
<item name="configurableObject" xsi:type="object">configurableObjectArgumentInterpreterProxy</item>
<item name="array" xsi:type="object">arrayArgumentInterpreterProxy</item>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -312,9 +312,9 @@
}

.navigation{
ul, ol{
> li{
position: relative;
ul, ol{
>li{
position:relative;
}
}
}
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 @@ -20,7 +20,7 @@
* 3. Open the created order.
* 4. Create partial invoice
* 4. Do cancel Order.
* 5. Perform all assetions.
* 5. Perform all assertions.
*
* @group Order_Management
* @ZephyrId MAGETWO-67787
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 @@ -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 @@ -19,7 +19,7 @@
*
* 1. Login to backend.
* 2. Create a product with invalid from and To dates
* 3. Save the product which generates an error messsage
* 3. Save the product which generates an error message
* 4. Modify the dates to valid values
* 5. Save the product again
* 6. Product is saved successfully
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 @@ -13,7 +13,7 @@
* Steps:
* 1. Log in to Admin.
* 2. Go to Stores>Configuration>Advanced>admin>Security.
* 3. * 7. Verify admin Acoount Sharing option availability.
* 3. Verify admin Account Sharing option availability.
*
* @group Config_(PS)
* @ZephyrId MAGETWO-47822
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 @@ -38,7 +38,7 @@ class WebsiteId 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 @@ -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
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ class ReAuthorizeTokensIntegrationEntityTest extends Injectable
*
* @param FixtureFactory $fixtureFactory
* @param IntegrationIndex $integrationIndex
* @retun void
* @return void
*/
public function __inject(IntegrationIndex $integrationIndex, FixtureFactory $fixtureFactory)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ abstract class Sidebar extends Block
protected $addToOrderProductName = './/tr/td[.="%s"]';

/**
* Add productz to order.
* Add products to order.
*
* @param array $products
* @return void
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* 2. Sales > Orders.
* 3. Open the created order.
* 4. Do cancel Order.
* 5. Perform all assetions.
* 5. Perform all assertions.
*
* @group Order_Management
* @ZephyrId MAGETWO-28191
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class CreateCustomOrderStatusEntityTest extends Injectable
/* end tags */

/**
* Order staus page.
* Order status page.
*
* @var OrderStatusIndex
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
use Magento\Mtf\Constraint\AbstractConstraint;

/**
* Assert sales rule delte message.
* Assert sales rule delete message.
*/
class AssertCartPriceRuleSuccessDeleteMessage extends AbstractConstraint
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -84,15 +84,15 @@ public function __inject(
/**
* Run Lock user when creating new integration test.
*
* @param Integration $initintegration
* @param Integration $initIntegration
* @param Integration $integration
* @param int $attempts
* @param User $customAdmin
* @param string $configData
* @return void
*/
public function test(
Integration $initintegration,
Integration $initIntegration,
Integration $integration,
$attempts,
User $customAdmin,
Expand All @@ -106,15 +106,15 @@ public function test(
['configData' => $this->configData]
)->run();
$customAdmin->persist();
$initintegration->persist();
$initIntegration->persist();

// login to backend with new user
$this->adminAuthLogin->open();
$this->adminAuthLogin->getLoginBlock()->fill($customAdmin);
$this->adminAuthLogin->getLoginBlock()->submit();

// Steps
$filter = ['name' => $initintegration->getName()];
$filter = ['name' => $initIntegration->getName()];
$this->integrationIndexPage->open();
$this->integrationIndexPage->getIntegrationGrid()->searchAndOpen($filter);
for ($i = 0; $i < $attempts; $i++) {
Expand Down
Loading