Since #1180112: Move drupal_*_schema_*() functions into one place, drupal_get_installed_schema_version() was moved to core/includes/schema.inc, causing this error when running drush updb.

CommentFileSizeAuthor
#8 drupal-1618346-8.patch495 bytestim.plunkett
#6 drupal-1618346-6.patch534 bytestim.plunkett
#5 1618346-5-drupal_get_installed_schema_version-in-schema.patch559 bytesalberto56
#3 1618346-3-drupal_get_installed_schema_version-in-schema.patch625 bytesalberto56
#1 1618346-1-drupal_get_installed_schema_version-in-schema.patch625 bytesalberto56

Comments

alberto56’s picture

alberto56 commented
Status: Active » Needs review
StatusFileSize
new 1618346-1-drupal_get_installed_schema_version-in-schema.patch625 bytes

This fixes that error.

Status: Needs review » Needs work

The last submitted patch, 1618346-1-drupal_get_installed_schema_version-in-schema.patch, failed testing.

alberto56’s picture

alberto56 commented
Status: Needs work » Needs review
StatusFileSize
new 1618346-3-drupal_get_installed_schema_version-in-schema.patch625 bytes

Slightly different version

Status: Needs review » Needs work

The last submitted patch, 1618346-3-drupal_get_installed_schema_version-in-schema.patch, failed testing.

alberto56’s picture

alberto56 commented
Status: Needs work » Needs review
StatusFileSize
new 1618346-5-drupal_get_installed_schema_version-in-schema.patch559 bytes

OK, trying again.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new drupal-1618346-6.patch534 bytes
donquixote’s picture

donquixote commented

Why does D8 always break when I pull?
Patch #6 works for me though :)

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new drupal-1618346-8.patch495 bytes

Rerolled.

Status: Needs review » Needs work

The last submitted patch, drupal-1618346-8.patch, failed testing.

pdrake’s picture

pdrake commented
Status: Needs work » Needs review

So, the patch in #8 appears to pass, not fail. Is it appropriate to simply re-test?

pdrake’s picture

pdrake commented

#8: drupal-1618346-8.patch queued for re-testing.

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

Sounds reasonable

pdrake’s picture

pdrake commented

+1 RTBC

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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