Commit 7bca01f0 authored by Gábor Hojtsy's avatar Gábor Hojtsy
Browse files

Issue #3178526 by gaards: Fix spelling mistakes in code comments

parent 3cc9096b
......@@ -84,7 +84,7 @@ final class LibraryDeprecationAnalyzer {
* Analyzes usages of deprecated libraries in an extension.
*
* @param \Drupal\Core\Extension\Extension $extension
* The extensiion to be analyzed.
* The extension to be analyzed.
*
* @return \Drupal\upgrade_status\DeprecationMessage[]
* A list of deprecation messages.
......@@ -114,7 +114,7 @@ final class LibraryDeprecationAnalyzer {
*/
private function analyzeLibraryDependencies(Extension $extension): array {
// Drupal\Core\Asset\LibraryDiscoveryParser::buildByExtension() assumes a
// disaled module is a theme and fails not finding it then, so check if
// disabled module is a theme and fails not finding it then, so check if
// the extension identified he is an installed module or theme. The library
// info will not be available otherwise.
$installed_modules = array_keys($this->moduleExtensionList->getAllInstalledInfo());
......@@ -414,7 +414,7 @@ final class LibraryDeprecationAnalyzer {
list($extension_name, $library_name) = explode('/', $library, 2);
// Drupal\Core\Asset\LibraryDiscoveryParser::buildByExtension() assumes a
// disaled module is a theme and fails not finding it then, so check if
// disabled module is a theme and fails not finding it then, so check if
// the extension identified he is an installed module or theme. The library
// info will not be available otherwise.
if ($extension_name != 'core') {
......
......@@ -223,7 +223,7 @@ class ProjectCollector {
continue;
}
// Attempt to identfy if the project was contrib based on the directory
// Attempt to identify if the project was contrib based on the directory
// structure it is in. Extension placement is not a mandatory requirement
// and theoretically this could lead to false positives, but if
// composer_deploy or git_deploy is not available (and/or did not
......@@ -311,7 +311,7 @@ class ProjectCollector {
$extension->info['upgrade_status_next'] = self::NEXT_REMOVE;
}
elseif (isset($extension->info['upgrade_status_update']) && $extension->info['upgrade_status_update'] == self::UPDATE_AVAILABLE) {
// If there was a Drupal 9 compatible update or even a yet incompatble
// If there was a Drupal 9 compatible update or even a yet incompatible
// update to this project, the best course of action is to update to
// that, since that should move closer to Drupal 9 compatibility.
$extension->info['upgrade_status_next'] = self::NEXT_UPDATE;
......@@ -449,7 +449,7 @@ class ProjectCollector {
* Return list of possible next steps and their labels and descriptions.
*
* @return array
* Associative array keyes by next step identifier. Values are arrays
* Associative array keys by next step identifier. Values are arrays
* where the first item is a label an the second is a description.
*/
public function getNextStepInfo() {
......
......@@ -60,7 +60,7 @@ class UpgradeStatusUiTest extends UpgradeStatusTestBase {
// Contrib test modules should show with results.
$this->assertSame('2 problems', strip_tags($page->find('css', 'tr.project-upgrade_status_test_contrib_error td.scan-result')->getHtml()));
$this->assertSame('No problems found', strip_tags($page->find('css', 'tr.project-upgrade_status_test_contrib_no_error td.scan-result')->getHtml()));
// This contrib module has a diferrent project name. Ensure the drupal.org link used that.
// This contrib module has a different project name. Ensure the drupal.org link used that.
$this->assertSession()->linkByHrefExists('https://drupal.org/project/issues/upgrade_status_test_contributed_no_error?text=Drupal+9&status=All');
// Click the first '4 problems' link. Should be the contrib project.
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment