Issue #3363938: Package Manager should ignore default.settings.php and default.services.yml

package_manager/src/PathExcluder/SiteConfigurationExcluder.php

@@ -69,11 +69,26 @@ class SiteConfigurationExcluder implements EventSubscriberInterface {
   /**
    * Makes the staged `sites/default` directory world-writable.
    *
+   * This is done to allow the core scaffold plugin to make changes in
+   * `sites/default`, if necessary, without breaking if `sites/default` is not
+   * writable (this can happen because rsync preserves directory permissions,
+   * and Drupal will try to harden the site directory's permissions as much as
+   * possible). We specifically exclude the `default.settings.php` and
+   * `default.services.yml` files from Package Manager operations, so we want to
+   * allow the scaffold plugin to make whatever changes it wants to those files
+   * in the stage directory.
+   *
    * @param \Drupal\package_manager\Event\PostCreateEvent $event
    *   The event being handled.
+   *
+   * @see ::excludeSiteConfiguration()
    */
   public function makeDefaultSiteDirectoryWritable(PostCreateEvent $event): void {
     $dir = $this->getDefaultSiteDirectoryPath($event->stage->getStageDirectory());
+    // If the directory doesn't even exist, there's nothing to do here.
+    if (!is_dir($dir)) {
+      return;
+    }
     if (!$this->fileSystem->chmod($dir, 0777)) {
       throw new FileException("Could not change permissions on '$dir'.");
     }
@@ -86,21 +101,23 @@ class SiteConfigurationExcluder implements EventSubscriberInterface {
    *   The event being handled.
    *
    * @throws \Drupal\Core\File\Exception\FileException
-   *   If the permissions of the live `sites/default` cannot be determined, or
-   *   changed on the staged `sites/default`.
+   *   If the permissions of either the live or staged `sites/default` cannot
+   *   be determined, or cannot be changed on the staged `sites/default`.
    */
   public function syncDefaultSiteDirectoryPermissions(PreApplyEvent $event): void {
-    $live_dir = $this->getDefaultSiteDirectoryPath($this->pathLocator->getProjectRoot());
     $staged_dir = $this->getDefaultSiteDirectoryPath($event->stage->getStageDirectory());
+    // If the directory doesn't even exist, there's nothing to do here.
+    if (!is_dir($staged_dir)) {
+      return;
+    }
+    $live_dir = $this->getDefaultSiteDirectoryPath($this->pathLocator->getProjectRoot());
 
-    $original_permissions = fileperms($live_dir);
-    if ($original_permissions === FALSE) {
+    $permissions = fileperms($live_dir);
+    if ($permissions === FALSE) {
       throw new FileException("Could not determine permissions for '$live_dir'.");
     }
 
-    // @see \Symfony\Component\Filesystem\Filesystem::copy() for another use
-    // of the 0111 bitmask.
-    if (!$this->fileSystem->chmod($staged_dir, $original_permissions & 0111)) {
+    if (!$this->fileSystem->chmod($staged_dir, $permissions)) {
       throw new FileException("Could not change permissions on '$staged_dir'.");
     }
   }
@@ -115,12 +132,12 @@ class SiteConfigurationExcluder implements EventSubscriberInterface {
    *   The full path to `sites/default` within the given root directory.
    */
   private function getDefaultSiteDirectoryPath(string $root_dir): string {
-    $dir = [$root_dir, 'sites', 'default'];
+    $dir = [$root_dir];
     $web_root = $this->pathLocator->getWebRoot();
     if ($web_root) {
-      array_splice($dir, 1, 0, $web_root);
+      $dir[] = $web_root;
     }
-    return implode(DIRECTORY_SEPARATOR, $dir);
+    return implode(DIRECTORY_SEPARATOR, [...$dir, 'sites', 'default']);
   }
 
   /**