Comments

borisson_ created an issue. See original summary.

borisson_’s picture

Status: Needs review » Fixed

Tests are green, thanks for the patch. Great work Jose!

  • borisson_ committed 5e361b5 on 8.x-1.x authored by marthinal
    Issue #2693861 by marthinal: Fix core search tests with postgres
    
borisson_’s picture

Looks like the tests still fail but I don't think we can fix that very easy: https://www.drupal.org/pift-ci-job/255333

See the error message:

exception: [Uncaught exception] Line 486 of core/includes/install.core.inc:
Drupal\Core\Installer\Exception\AlreadyInstalledException:
To start over, you must empty your existing database and copy default.settings.php over settings.php.
To upgrade an existing installation, proceed to the update script.

I think we can leave this as fixed and we should open an issue for the bots?

marthinal’s picture

Yep I agree.

marthinal’s picture

Hmmm looks that the problem is only for PHP 7

marthinal’s picture

opps sorry the patch is already commited :(

Status: Fixed » Needs work

The last submitted patch, 2: 2693861-2.patch, failed testing.

marthinal’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.