Backport MissingProjectInfo 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.

Comments

heddn created an issue. See original summary.

heddn’s picture

Component: Code » Checkers
heddn’s picture

Issue summary: View changes
Status: Postponed » Active
Issue tags: +Novice
heddn’s picture

Status: Active » Needs review
StatusFileSize
new4.88 KB
heddn’s picture

StatusFileSize
new6.26 KB
new2.42 KB
heddn’s picture

StatusFileSize
new991 bytes
new7.23 KB
tatarbj’s picture

StatusFileSize
new7.26 KB
heddn’s picture

interdiff 3063131-6.patch  3063131-7.patch
diff -u b/tests/automatic_updates.test b/tests/automatic_updates.test
--- b/tests/automatic_updates.test
+++ b/tests/automatic_updates.test
@@ -83,7 +83,7 @@
     $this->assertText('The project "drupal_autoload_test" can not be updated because its version is either undefined or a dev release');

     // Ignore certain folders for readiness checks.
-    variable_set('automatic_updates_ignored_paths', "modules/*\nthemes/*");
+    variable_set('automatic_updates_ignored_paths', "sites/all/modules/custom/*\nsites/all/themes/custom/*");
     $this->drupalGet($this->getAbsoluteUrl('admin/config/system/automatic_updates'));
     $this->clickLink('run the readiness checks');
     $this->assertText('No issues found. Your site is completely ready for automatic updates.');
heddn’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me. Thanks for catching the errant config.

  • heddn committed 9713a82 on 7.x-1.x authored by tatarbj
    Issue #3063131 by heddn, tatarbj: Backport MissingProjectInfo
    
heddn’s picture

Thanks for your patch and reviews.

heddn’s picture

#8, I actually ended up reversing this. We need it for that specific test case because we need to ignore the cloned drupal core modules.

heddn’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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