Add aura/di basics

[harikt]

No description provided.

[harikt]

Fixes #109

[samsonasik]

missing , before next line

[harikt]

good catch.

[samsonasik]

yup, new line for end of file, a blank line after }

[weierophinney]

@samsonasik No blank line at the end of the file, please!!!!

[weierophinney]

My principal suggestion is that you follow the same patterns used for the ZF2 service manager and Pimple: in other words, have a config/services.php file that returns the fully configured container. We plan to use this pattern in the skeleton application, with example files for each container we document; this allows keeping the bootstrap file the same across implementations.

[harikt]

@weierophinney question :

Do you think changing to config/services.php

use Aura\Di\ContainerBuilder;

$container_builder = new ContainerBuilder();

// use the builder to create and configure a container
// using an array of ContainerConfig classes
$di = $container_builder->newConfiguredInstance([
    'Application\_Config\Common',
]);
return $di;

and having a seprate class is acceptable? The things are you can load other config classes if they are already provided by the package and don't need to add the config our own.

<?php
chdir(dirname(__DIR__));
$container = require 'config/services.php';
$app = $container->get('Zend\Expressive\Application');
$app->run();

If this is acceptable let me know.

[weierophinney]

Absolutely; that's exactly how we handled the other two, and why I
suggested it. :-)
On Aug 28, 2015 11:20 AM, "Hari K T" [email protected] wrote:

@weierophinney https://github.com/weierophinney question :

Do you think changing to config/services.php

use Aura\Di\ContainerBuilder;$container_builder = new ContainerBuilder();// use the builder to create and configure a container// using an array of ContainerConfig classes$di = $container_builder->newConfiguredInstance([ 'Application_Config\Common',]);return $di;

and having a seprate class is acceptable? The things are you can load
other config classes if they are already provided by the package and don't
need to add the config our own.

get('Zend\Expressive\Application'); $app->run(); If this is acceptable let me know. — Reply to this email directly or view it on GitHub https://github.com//pull/112#issuecomment-135823413 .

[harikt]

Have made the changes @weierophinney . Hope you like. Feel free to make the necessary changes.

Thank you.

[weierophinney]

Where should this class file be placed?

[weierophinney]

I'm assuming src/_Config/Common.php?

[harikt]

Normally in config , and add autoload to composer.json . See skelton https://github.com/harikt/expressive-aura-skelton/blob/bbbbf233170ee4b76877860d676116f44f1d75d2/composer.json#L11 .