Skip to content
Snippets Groups Projects
Verified Commit 6b2ecdbb authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3437178 by immaculatexavier, quietone, smustgrave, alexpott: Simplify...

Issue #3437178 by immaculatexavier, quietone, smustgrave, alexpott: Simplify getLegacyDrupalVersion()

(cherry picked from commit 48814794)
parent 19ae323f
Branches
Tags
24 merge requests!11958Issue #3490507 by alexpott, smustgrave: Fix bogus mocking in...,!11769Issue #3517987: Add option to contextual filters to encode slashes in query parameter.,!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4,!10602Issue #3438769 by vinmayiswamy, antonnavi, michelle, amateescu: Sub workspace does not clear,!10301Issue #3469309 by mstrelan, smustgrave, moshe weitzman: Use one-time login...,!10187Issue #3487488 by dakwamine: ExtensionMimeTypeGuesser::guessMimeType must support file names with "0" (zero) like foo.0.zip,!9944Issue #3483353: Consider making the createCopy config action optionally fail...,!9929Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9787Resolve issue 3479427 - bootstrap barrio issue under Windows,!9742Issue #3463908 by catch, quietone: Split OptionsFieldUiTest into two,!9526Issue #3458177 by mondrake, catch, quietone, godotislate, longwave, larowlan,...,!8738Issue #3424162 by camilledavis, dineshkumarbollu, smustgrave: Claro...,!8704Make greek characters available in ckeditor5,!8597Draft: Issue #3442259 by catch, quietone, dww: Reduce time of Migrate Upgrade tests...,!8533Issue #3446962 by kim.pepper: Remove incorrectly added...,!8517Issue #3443748 by NexusNovaz, smustgrave: Testcase creates false positive,!8325Update file Sort.php,!8095Expose document root on install,!7930Resolve #3427374 "Taxonomytid viewsargumentdefault plugin",!7627Issue #3439440 by nicxvan, Binoli Lalani, longwave: Remove country support from DateFormatter,!7445Issue #3440169: When using drupalGet(), provide an associative array for $headers,!6502Draft: Resolve #2938524 "Plach testing issue",!38582585169-10.1.x,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key
Pipeline #146013 passed with warnings
Pipeline: drupal

#146029

    Pipeline: drupal

    #146015

      ......@@ -206,27 +206,23 @@ public static function getLegacyDrupalVersion(Connection $connection) {
      // we're querying. Catch exceptions and report that the source database is
      // not Drupal.
      // Drupal 5/6/7 can be detected by the schema_version in the system table.
      $version_string = FALSE;
      if ($connection->schema()->tableExists('system')) {
      try {
      $legacy_version_string = $connection
      $version_string = $connection
      ->query('SELECT [schema_version] FROM {system} WHERE [name] = :module', [':module' => 'system'])
      ->fetchField();
      if ($legacy_version_string && $legacy_version_string[0] == '1') {
      if ((int) $legacy_version_string >= 1000) {
      $version_string = '5';
      }
      }
      else {
      $version_string = substr($legacy_version_string, 0, 1);
      }
      }
      catch (DatabaseExceptionWrapper $e) {
      // All database errors return FALSE.
      }
      }
      return $version_string;
      return match (TRUE) {
      !isset($version_string) => FALSE,
      (int) $version_string >= 6000 => substr($version_string, 0, 1),
      (int) $version_string >= 1000 => '5',
      default => FALSE,
      };
      }
      /**
      ......
      0% Loading or .
      You are about to add 0 people to the discussion. Proceed with caution.
      Please register or to comment