Skip to content
Snippets Groups Projects
Verified Commit 48814794 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()
parent c5365987
Branches
Tags
28 merge requests!11131[10.4.x-only-DO-NOT-MERGE]: Issue ##2842525 Ajax attached to Views exposed filter form does not trigger callbacks,!9470[10.3.x-only-DO-NOT-MERGE]: #3331771 Fix file_get_contents(): Passing null to parameter,!8540Issue #3457061: Bootstrap Modal dialog Not closing after 10.3.0 Update,!8528Issue #3456871 by Tim Bozeman: Support NULL services,!8373Issue #3427374 by danflanagan8, Vighneshh: taxonomy_tid ViewsArgumentDefault...,!7526Expose roles in response,!3878Removed unused condition head title for views,!3818Issue #2140179: $entity->original gets stale between updates,!3742Issue #3328429: Create item list field formatter for displaying ordered and unordered lists,!3731Claro: role=button on status report items,!3651Issue #3347736: Create new SDC component for Olivero (header-search),!3531Issue #3336994: StringFormatter always displays links to entity even if the user in context does not have access,!3355Issue #3209129: Scrolling problems when adding a block via layout builder,!3154Fixes #2987987 - CSRF token validation broken on routes with optional parameters.,!3133core/modules/system/css/components/hidden.module.css,!2964Issue #2865710 : Dependencies from only one instance of a widget are used in display modes,!2812Issue #3312049: [Followup] Fix Drupal.Commenting.FunctionComment.MissingReturnType returns for NULL,!2794Issue #3100732: Allow specifying `meta` data on JSON:API objects,!2378Issue #2875033: Optimize joins and table selection in SQL entity query implementation,!2062Issue #3246454: Add weekly granularity to views date sort,!1105Issue #3025039: New non translatable field on translatable content throws error,!1073issue #3191727: Focus states on mobile second level navigation items fixed,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!877Issue #2708101: Default value for link text is not saved,!617Issue #3043725: Provide a Entity Handler for user cancelation,!579Issue #2230909: Simple decimals fail to pass validation,!560Move callback classRemove outside of the loop,!555Issue #3202493
Pipeline #146018 passed with warnings
Pipeline: drupal

#146025

    Pipeline: drupal

    #146024

      ......@@ -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