Skip to content

Commit 38c9db4

Browse files
committed
Remove unecessaries ternary conditions
1 parent db0dce6 commit 38c9db4

File tree

11 files changed

+11
-11
lines changed

11 files changed

+11
-11
lines changed

dev/tests/api-functional/testsuite/Magento/GraphQl/Catalog/MediaGalleryTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,6 @@ private function checkImageExists(string $url): bool
198198
curl_exec($connection);
199199
$responseStatus = curl_getinfo($connection, CURLINFO_HTTP_CODE);
200200
// phpcs:enable Magento2.Functions.DiscouragedFunction
201-
return $responseStatus === 200 ? true : false;
201+
return $responseStatus === 200;
202202
}
203203
}

dev/tests/api-functional/testsuite/Magento/GraphQl/Catalog/ProductImageTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,6 @@ private function checkImageExists(string $url): bool
144144
curl_exec($connection);
145145
$responseStatus = curl_getinfo($connection, CURLINFO_HTTP_CODE);
146146

147-
return $responseStatus === 200 ? true : false;
147+
return $responseStatus === 200;
148148
}
149149
}

dev/tests/functional/lib/Magento/Mtf/Client/Element/LiselectstoreElement.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ public function getValues()
133133
*/
134134
protected function isSubstring($haystack, $pattern)
135135
{
136-
return preg_match("/$pattern/", $haystack) != 0 ? true : false;
136+
return preg_match("/$pattern/", $haystack) != 0;
137137
}
138138

139139
/**

dev/tests/functional/tests/app/Magento/Bundle/Test/Block/Catalog/Product/View/Type/Bundle.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ public function fillBundleOptions($bundleOptions)
311311
{
312312
foreach ($bundleOptions as $option) {
313313
$selector = sprintf($this->bundleOptionBlock, $option['title']);
314-
$useDefault = isset($option['use_default']) && strtolower($option['use_default']) == 'true' ? true : false;
314+
$useDefault = isset($option['use_default']) && strtolower($option['use_default']) == 'true';
315315
if (!$useDefault) {
316316
/** @var Option $optionBlock */
317317
$optionBlock = $this->blockFactory->create(

dev/tests/functional/tests/app/Magento/Review/Test/TestCase/MassActionsProductReviewEntityTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ public function test($gridActions, $gridStatus)
102102
$this->reviewIndex->getReviewGrid()->massaction(
103103
[['title' => $this->review->getTitle()]],
104104
[$gridActions => $gridStatus],
105-
($gridActions == 'Delete' ? true : false)
105+
($gridActions == 'Delete')
106106
);
107107
}
108108

lib/internal/Magento/Framework/Acl/AclResource/Config/Converter/Dom.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ protected function _convertResourceNode(\DOMNode $resourceNode)
5353
$sortOrderNode = $resourceAttributes->getNamedItem('sortOrder');
5454
$resourceData['sortOrder'] = $sortOrderNode !== null ? (int)$sortOrderNode->nodeValue : 0;
5555
$disabledNode = $resourceAttributes->getNamedItem('disabled');
56-
$resourceData['disabled'] = $disabledNode !== null && $disabledNode->nodeValue == 'true' ? true : false;
56+
$resourceData['disabled'] = $disabledNode !== null && $disabledNode->nodeValue == 'true';
5757
// convert child resource nodes if needed
5858
$resourceData['children'] = [];
5959
/** @var $childNode \DOMNode */

lib/internal/Magento/Framework/App/MaintenanceMode.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ public function setAddresses($addresses)
110110
throw new \InvalidArgumentException("One or more IP-addresses is expected (comma-separated)\n");
111111
}
112112
$result = $this->flagDir->writeFile(self::IP_FILENAME, $addresses);
113-
return false !== $result ? true : false;
113+
return false !== $result;
114114
}
115115

116116
/**

lib/internal/Magento/Framework/Code/Reader/NamespaceResolver.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public function resolveNamespace($type, array $availableNamespaces)
5050
) {
5151
$name = explode(self::NS_SEPARATOR, $type);
5252
$unqualifiedName = $name[0];
53-
$isQualifiedName = count($name) > 1 ? true : false;
53+
$isQualifiedName = count($name) > 1;
5454
if (isset($availableNamespaces[$unqualifiedName])) {
5555
$namespace = $availableNamespaces[$unqualifiedName];
5656
if ($isQualifiedName) {

lib/internal/Magento/Framework/GraphQl/Schema/Type/ScalarTypes.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ class ScalarTypes
2121
public function isScalarType(string $typeName) : bool
2222
{
2323
$standardTypes = \GraphQL\Type\Definition\Type::getStandardTypes();
24-
return isset($standardTypes[$typeName]) ? true : false;
24+
return isset($standardTypes[$typeName]);
2525
}
2626

2727
/**

lib/internal/Magento/Framework/HTTP/PhpEnvironment/Response.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ public function setHttpResponseCode($code)
130130
throw new \InvalidArgumentException('Invalid HTTP response code');
131131
}
132132

133-
$this->isRedirect = (300 <= $code && 307 >= $code) ? true : false;
133+
$this->isRedirect = (300 <= $code && 307 >= $code);
134134

135135
$this->setStatusCode($code);
136136
return $this;

0 commit comments

Comments
 (0)