Merge pull request thephpleague#107 from matchory/radiergummi/feature-support-lazy-root-creation

Added support for lazy root creation

Author: tgalopin

src/Adapter/Builder/LocalAdapterDefinitionBuilder.php

@@ -61,6 +61,9 @@ protected function configureOptions(OptionsResolver $resolver)
                 $permsResolver->setAllowedTypes('private', 'scalar');
             });
         });
+
+        $resolver->setDefault('lazy_root_creation', false);
+        $resolver->setAllowedTypes('lazy_root_creation', 'scalar');
     }
 
     protected function configureDefinition(Definition $definition, array $options)
@@ -84,5 +87,7 @@ protected function configureDefinition(Definition $definition, array $options)
         );
         $definition->setArgument(2, $options['lock']);
         $definition->setArgument(3, $options['skip_links'] ? LocalFilesystemAdapter::SKIP_LINKS : LocalFilesystemAdapter::DISALLOW_LINKS);
+        $definition->setArgument(4, null);
+        $definition->setArgument(5, $options['lazy_root_creation']);
     }
 }

tests/Adapter/Builder/LocalAdapterDefinitionBuilderTest.php

@@ -56,6 +56,16 @@ public function provideValidOptions()
             ],
         ]];
 
+        yield 'lazy_root_creation_enabled' => [[
+            'directory' => __DIR__,
+            'lazy_root_creation' => true,
+        ]];
+
+        yield 'lazy_root_creation_disabled' => [[
+            'directory' => __DIR__,
+            'lazy_root_creation' => false,
+        ]];
+
         yield 'full' => [[
             'directory' => __DIR__,
             'lock' => 0,
@@ -70,6 +80,7 @@ public function provideValidOptions()
                     'private' => 0755,
                 ],
             ],
+            'lazy_root_creation' => true,
         ]];
     }
 
@@ -99,12 +110,14 @@ public function testOptionsBehavior()
             'lock' => LOCK_EX,
             'skip_links' => true,
             'permissions' => $permissions,
+            'lazy_root_creation' => true,
         ]);
 
         $this->assertSame(LocalFilesystemAdapter::class, $definition->getClass());
         $this->assertSame(__DIR__, $definition->getArgument(0));
         $this->assertSame($permissions, $definition->getArgument(1)->getArgument(0));
         $this->assertSame(LOCK_EX, $definition->getArgument(2));
         $this->assertSame(LocalFilesystemAdapter::SKIP_LINKS, $definition->getArgument(3));
+        $this->assertSame(true, $definition->getArgument(5));
     }
 }

tests/Adapter/options.yaml

@@ -38,6 +38,7 @@ fs_local:
         directory: '%kernel.project_dir%/storage'
         lock: 0
         skip_links: false
+        lazy_root_creation: false
         permissions:
             file:
                 public: '0744'