Backport PendingDbUpdates to D7. The basic building blocks landed with #3059742: Backport base readiness checker logic to 7.x, now we need to copy/paste and D7ify things.
Don't use base Filesystem class. See #3063128: Backport FileOwnership and Filesystem as an example.

CommentFileSizeAuthor
#6 3063134-6.patch2.15 KBheddn
#4 3063134-4.patch1.96 KBtatarbj
#3 3063134.patch1.97 KBheddn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Issue summary: View changes
Status: Postponed » Active
Issue tags: +Novice
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Status: Active » Needs review
FileSize
3063134.patch1.97 KB
tatarbj’s picture

tatarbj
Primary language Hungarian
Location Brussels
CreditAttribution: tatarbj at Petend commented
FileSize
3063134-4.patch1.96 KB

It looks removing the error checker type of PhpSapi is a mistake and it should stay (such as it's reported under #3063129: Backport DiskSpace)

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented

Removal was intentional. I had only placeholdered php sapi in there until we got a proper error. This is a proper error.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
3063134-6.patch2.15 KB

re-roll

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs review » Reviewed & tested by the community

This is a simple backport. Landing.

  • heddn committed 7793959 on 7.x-1.x 
    Issue #3063134 by heddn, tatarbj: Backport PendingDbUpdates
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Status: Reviewed & tested by the community » Fixed

Thanks for your review @tatarbj.

Status: Fixed » Closed (fixed)

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