Question - upload file - controler example

[thim81]

Hi Mathielen,

In your controller example

public function importAction(\Symfony\Component\HttpFoundation\Request $request)
    {
        //create the request for the import-engine
        $importRequest = new \Mathielen\ImportEngine\ValueObject\ImportRequest($request->files->getIterator()->current());

        /** @var \Mathielen\ImportEngine\Import\ImportBuilder $importBuilder */
        $importBuilder = $this->container->get('mathielen_importengine.import.builder');
        $import = $importBuilder->build($importRequest);

        /** @var \Mathielen\ImportEngine\Import\Run\ImportRunner $importRunner */
        $importRunner = $this->container->get('mathielen_importengine.import.runner');
        $importRun = $importRunner->run($import);

        return $this->render('default/import.html.twig', $importRun->getStatistics());
    }

What is not clear from the example is that how you define which import to run?

I looked at the functions and

$importBuilder->build(ImportConfiguration $importConfiguration, StorageInterface $sourceStorage = null, $createdBy = null)

Expects a ImportConfiguration while your ImportRequest seems to process the files?
Or does the ImportRequest detect which file and importer to use?

This is how I currently upload a file via a form POST, how could I use this with your example above?

// CSV import (from "file" upload via a form)
            /** @var UploadedFile $importFile */
            $importFile = $request->files->get('importFile');
            $fileName = md5(uniqid()).'.'.$importFile->guessClientExtension();

            // Move the file to the directory where brochures are stored
            $importDir = $this->container->getParameter('kernel.root_dir').'/../web/imports/parcel';
            $importFile->move($importDir, $fileName);

            // Read file
            $file = new \SplFileObject($importDir.'/'.$fileName);

[mathielen]

Ok so if you are using the auto-detection feature (i.e. preconditions), then the example above should automatically figure out which importer to use based on the fields/format/etc you have set as preconditions.
If you want to force an import with the file, just use the additional parameters of ImportRequest.

$importRequest = new ImportRequest($request->files->get('importFile'), 'default', 'myImporter');

Does that work for you?

[thim81]

This is again crystal clear.

I figured out that it's possible to manually set the configuration but I was not sure how the magic of the auto-detection would work.

[thim81]

Hi Markus,

I have tried your

new ImportRequest($request->files->get('importFile'), 'default', 'myImporter');

But then I get this error, seems you expect an ImportConfiguration and not an ImportRequest?

Catchable Fatal Error: Argument 1 passed to Mathielen\ImportEngine\Import\ImportBuilder::build() must be an instance of Mathielen\ImportEngine\ValueObject\ImportConfiguration, instance of Mathielen\ImportEngine\ValueObject\ImportRequest given

[thim81]

Hi Markus,

Can you have a quick look at what I'm doing wrong, because the class expect a "ImportConfiguration" instead of "ImportRequest" and It's not yet clear how to use the magic of the bundle to get an ImportConfiguration based on a file.

Regards,
Tim

[Notmarrco]

Hi @thim81 ,

I may be too late, but I think you should use $importBuilder->buildFromRequest($importRequest); instead of $importBuilder->build(

:)