aimeos / aimeos-symfony

DependencyInjection/AimeosShopExtension.php

Spacing +13 added lines, -13 removed lines

@@ -31,21 +31,21 @@ 
 	/**
 	 * {@inheritDoc}
 	 */
-	public function load( array $configs, ContainerBuilder $container )
+	public function load(array $configs, ContainerBuilder $container)
 	{
 		$configuration = new Configuration();
-		$config = $this->processConfiguration( $configuration, $configs );
+		$config = $this->processConfiguration($configuration, $configs);
 
-		foreach( $configs as $list ) {
-			$config = array_replace_recursive( $config, $list );
+		foreach ($configs as $list) {
+			$config = array_replace_recursive($config, $list);
 		}
 
-		foreach( $config as $key => $value ) {
-			$container->setParameter( 'aimeos_shop.' . $key, $value );
+		foreach ($config as $key => $value) {
+			$container->setParameter('aimeos_shop.'.$key, $value);
 		}
 
-		$loader = new Loader\YamlFileLoader( $container, new FileLocator( dirname( __DIR__ ) . '/Resources/config' ) );
-		$loader->load( 'services.yml' );
+		$loader = new Loader\YamlFileLoader($container, new FileLocator(dirname(__DIR__).'/Resources/config'));
+		$loader->load('services.yml');
 	}
 
 
@@ -54,12 +54,12 @@ 
 	 *
 	 * @param ContainerBuilder $container ContainerBuilder object
 	 */
-	public function prepend( ContainerBuilder $container )
+	public function prepend(ContainerBuilder $container)
 	{
-		$configFile = dirname( __DIR__ ) . '/Resources/config/aimeos_shop.yml';
-		$config = Yaml::parse( file_get_contents( $configFile ) );
+		$configFile = dirname(__DIR__).'/Resources/config/aimeos_shop.yml';
+		$config = Yaml::parse(file_get_contents($configFile));
 
-		$container->prependExtensionConfig( 'aimeos_shop', $config );
-		$container->addResource( new FileResource( $configFile ) );
+		$container->prependExtensionConfig('aimeos_shop', $config);
+		$container->addResource(new FileResource($configFile));
 	}
 }

Tests/bootstrap.php

Spacing +7 added lines, -7 removed lines

@@ -3,24 +3,24 @@ 
 use Doctrine\Common\Annotations\AnnotationRegistry;
 
 
-setlocale( LC_ALL, 'en_US.UTF-8' );
+setlocale(LC_ALL, 'en_US.UTF-8');
 
 // if the bundle is within a symfony project, try to reuse the project's autoload
 $files = array(
-	__DIR__ . '/../vendor/autoload.php',
-	__DIR__ . '/../../vendor/autoload.php',
+	__DIR__.'/../vendor/autoload.php',
+	__DIR__.'/../../vendor/autoload.php',
 );
 
 $autoload = false;
-foreach( $files as $file )
+foreach ($files as $file)
 {
-	if( is_file( $file ) ) {
+	if (is_file($file)) {
 		$autoload = require $file;
 		break;
 	}
 }
 
-if( !$autoload )
+if (!$autoload)
 {
 	exit(
 	"Unable to find autoload.php file, please use composer to load dependencies:
@@ -30,4 +30,4 @@ 
 	);
 }
 
-AnnotationRegistry::registerLoader( array( $autoload, 'loadClass' ) );
+AnnotationRegistry::registerLoader(array($autoload, 'loadClass'));

Controller/ExtadmController.php

Spacing +67 added lines, -67 removed lines

@@ -30,64 +30,64 @@ 
 	 * @param Request $request Symfony request object
 	 * @return Response Generated output for the admin interface
 	 */
-	public function indexAction( Request $request )
+	public function indexAction(Request $request)
 	{
-		$site = $request->attributes->get( 'site', $request->query->get( 'site', 'default' ) );
-		$lang = $request->attributes->get( 'lang', $request->query->get( 'lang', 'en' ) );
-		$tab = $request->attributes->get( 'tab', $request->query->get( 'tab', 0 ) );
+		$site = $request->attributes->get('site', $request->query->get('site', 'default'));
+		$lang = $request->attributes->get('lang', $request->query->get('lang', 'en'));
+		$tab = $request->attributes->get('tab', $request->query->get('tab', 0));
 
-		$context = $this->get( 'aimeos_context' )->get( false, 'backend' );
-		$context->setLocale( $this->get( 'aimeos_locale' )->getBackend( $context, $site ) );
+		$context = $this->get('aimeos_context')->get(false, 'backend');
+		$context->setLocale($this->get('aimeos_locale')->getBackend($context, $site));
 
-		$aimeos = $this->get( 'aimeos' );
+		$aimeos = $this->get('aimeos');
 		$bootstrap = $aimeos->get();
 
-		$cntlPaths = $bootstrap->getCustomPaths( 'controller/extjs' );
-		$controller = new \Aimeos\Controller\ExtJS\JsonRpc( $context, $cntlPaths );
+		$cntlPaths = $bootstrap->getCustomPaths('controller/extjs');
+		$controller = new \Aimeos\Controller\ExtJS\JsonRpc($context, $cntlPaths);
 		$cssFiles = array();
 
-		foreach( $bootstrap->getCustomPaths( 'admin/extjs' ) as $base => $paths )
+		foreach ($bootstrap->getCustomPaths('admin/extjs') as $base => $paths)
 		{
-			foreach( $paths as $path )
+			foreach ($paths as $path)
 			{
-				$jsbAbsPath = $base . '/' . $path;
+				$jsbAbsPath = $base.'/'.$path;
 
-				if( !is_file( $jsbAbsPath ) ) {
-					throw new \Exception( sprintf( 'JSB2 file "%1$s" not found', $jsbAbsPath ) );
+				if (!is_file($jsbAbsPath)) {
+					throw new \Exception(sprintf('JSB2 file "%1$s" not found', $jsbAbsPath));
 				}
 
-				$jsb2 = new \Aimeos\MW\Jsb2\Standard( $jsbAbsPath, dirname( $path ) );
-				$cssFiles = array_merge( $cssFiles, $jsb2->getUrls( 'css' ) );
+				$jsb2 = new \Aimeos\MW\Jsb2\Standard($jsbAbsPath, dirname($path));
+				$cssFiles = array_merge($cssFiles, $jsb2->getUrls('css'));
 			}
 		}
 
-		$params = array( 'site' => '{site}', 'lang' => '{lang}', 'tab' => '{tab}' );
-		$adminUrl = $this->generateUrl( 'aimeos_shop_extadm', $params );
+		$params = array('site' => '{site}', 'lang' => '{lang}', 'tab' => '{tab}');
+		$adminUrl = $this->generateUrl('aimeos_shop_extadm', $params);
 
-		$token = $this->get( 'security.csrf.token_manager' )->getToken( 'aimeos_admin_token' )->getValue();
-		$jsonUrl = $this->generateUrl( 'aimeos_shop_extadm_json', array( '_token' => $token, 'site' => $site ) );
+		$token = $this->get('security.csrf.token_manager')->getToken('aimeos_admin_token')->getValue();
+		$jsonUrl = $this->generateUrl('aimeos_shop_extadm_json', array('_token' => $token, 'site' => $site));
 
-		$jqadmUrl = $this->generateUrl( 'aimeos_shop_jqadm_search', array( 'site' => $site, 'resource' => 'product' ) );
+		$jqadmUrl = $this->generateUrl('aimeos_shop_jqadm_search', array('site' => $site, 'resource' => 'product'));
 
 		$vars = array(
 			'lang' => $lang,
 			'cssFiles' => $cssFiles,
 			'languages' => $this->getJsonLanguages(),
-			'config' => $this->getJsonClientConfig( $context ),
-			'site' => $this->getJsonSiteItem( $context, $site ),
-			'i18nContent' => $this->getJsonClientI18n( $bootstrap->getI18nPaths(), $lang ),
+			'config' => $this->getJsonClientConfig($context),
+			'site' => $this->getJsonSiteItem($context, $site),
+			'i18nContent' => $this->getJsonClientI18n($bootstrap->getI18nPaths(), $lang),
 			'searchSchemas' => $controller->getJsonSearchSchemas(),
 			'itemSchemas' => $controller->getJsonItemSchemas(),
-			'smd' => $controller->getJsonSmd( $jsonUrl ),
-			'urlTemplate' => urldecode( $adminUrl ),
-			'uploaddir' => $this->container->getParameter( 'aimeos_shop.uploaddir' ),
-			'extensions' => implode( ',', $bootstrap->getExtensions() ),
+			'smd' => $controller->getJsonSmd($jsonUrl),
+			'urlTemplate' => urldecode($adminUrl),
+			'uploaddir' => $this->container->getParameter('aimeos_shop.uploaddir'),
+			'extensions' => implode(',', $bootstrap->getExtensions()),
 			'version' => $aimeos->getVersion(),
 			'jqadmurl' => $jqadmUrl,
 			'activeTab' => $tab,
 		);
 
-		return $this->render( 'AimeosShopBundle:Extadm:index.html.twig', $vars );
+		return $this->render('AimeosShopBundle:Extadm:index.html.twig', $vars);
 	}
 
 
@@ -97,23 +97,23 @@ 
 	 * @param Request $request Symfony request object
 	 * @return Response JSON RPC message response
 	 */
-	public function doAction( Request $request )
+	public function doAction(Request $request)
 	{
 		$csrfProvider = $this->get('security.csrf.token_manager');
 
-		if( $csrfProvider->isTokenValid( new CsrfToken( 'aimeos_admin_token',  $request->query->get( '_token' ) ) ) !== true ) {
-			throw new \Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException( 'CSRF token is invalid' );
+		if ($csrfProvider->isTokenValid(new CsrfToken('aimeos_admin_token', $request->query->get('_token'))) !== true) {
+			throw new \Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException('CSRF token is invalid');
 		}
 
-		$cntlPaths = $this->get( 'aimeos' )->get()->getCustomPaths( 'controller/extjs' );
-		$context = $this->get( 'aimeos_context' )->get( false, 'backend' );
-		$context->setView( $this->get( 'aimeos_view' )->create( $context, array() ) );
-		$context->setLocale( $this->get( 'aimeos_locale' )->getBackend( $context, 'default' ) );
+		$cntlPaths = $this->get('aimeos')->get()->getCustomPaths('controller/extjs');
+		$context = $this->get('aimeos_context')->get(false, 'backend');
+		$context->setView($this->get('aimeos_view')->create($context, array()));
+		$context->setLocale($this->get('aimeos_locale')->getBackend($context, 'default'));
 
-		$controller = new \Aimeos\Controller\ExtJS\JsonRpc( $context, $cntlPaths );
+		$controller = new \Aimeos\Controller\ExtJS\JsonRpc($context, $cntlPaths);
 
-		$response = $controller->process( $request->request->all(), $request->getContent() );
-		return $this->render( 'AimeosShopBundle:Extadm:do.html.twig', array( 'output' => $response ) );
+		$response = $controller->process($request->request->all(), $request->getContent());
+		return $this->render('AimeosShopBundle:Extadm:do.html.twig', array('output' => $response));
 	}
 
 
@@ -126,27 +126,27 @@ 
 	{
 		$contents = '';
 		$jsFiles = array();
-		$aimeos = $this->get( 'aimeos' )->get();
+		$aimeos = $this->get('aimeos')->get();
 
-		foreach( $aimeos->getCustomPaths( 'admin/extjs' ) as $base => $paths )
+		foreach ($aimeos->getCustomPaths('admin/extjs') as $base => $paths)
 		{
-			foreach( $paths as $path )
+			foreach ($paths as $path)
 			{
-				$jsbAbsPath = $base . '/' . $path;
-				$jsb2 = new \Aimeos\MW\Jsb2\Standard( $jsbAbsPath, dirname( $jsbAbsPath ) );
-				$jsFiles = array_merge( $jsFiles, $jsb2->getFiles( 'js' ) );
+				$jsbAbsPath = $base.'/'.$path;
+				$jsb2 = new \Aimeos\MW\Jsb2\Standard($jsbAbsPath, dirname($jsbAbsPath));
+				$jsFiles = array_merge($jsFiles, $jsb2->getFiles('js'));
 			}
 		}
 
-		foreach( $jsFiles as $file )
+		foreach ($jsFiles as $file)
 		{
-			if( ( $content = file_get_contents( $file ) ) !== false ) {
+			if (($content = file_get_contents($file)) !== false) {
 				$contents .= $content;
 			}
 		}
 
-		$response = new Response( $contents );
-		$response->headers->set( 'Content-Type', 'application/javascript' );
+		$response = new Response($contents);
+		$response->headers->set('Content-Type', 'application/javascript');
 
 		return $response;
 	}
@@ -161,11 +161,11 @@ 
 	{
 		$result = array();
 
-		foreach( $this->get( 'aimeos' )->get()->getI18nList( 'admin' ) as $id ) {
-			$result[] = array( 'id' => $id, 'label' => $id );
+		foreach ($this->get('aimeos')->get()->getI18nList('admin') as $id) {
+			$result[] = array('id' => $id, 'label' => $id);
 		}
 
-		return json_encode( $result );
+		return json_encode($result);
 	}
 
 
@@ -175,10 +175,10 @@ 
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context item object
 	 * @return string JSON encoded configuration object
 	 */
-	protected function getJsonClientConfig( \Aimeos\MShop\Context\Item\Iface $context )
+	protected function getJsonClientConfig(\Aimeos\MShop\Context\Item\Iface $context)
 	{
-		$config = $context->getConfig()->get( 'admin/extjs', array() );
-		return json_encode( array( 'admin' => array( 'extjs' => $config ) ), JSON_FORCE_OBJECT );
+		$config = $context->getConfig()->get('admin/extjs', array());
+		return json_encode(array('admin' => array('extjs' => $config)), JSON_FORCE_OBJECT);
 	}
 
 
@@ -189,16 +189,16 @@ 
 	 * @param string $lang ISO language code like "en" or "en_GB"
 	 * @return string JSON encoded translation object
 	 */
-	protected function getJsonClientI18n( array $i18nPaths, $lang )
+	protected function getJsonClientI18n(array $i18nPaths, $lang)
 	{
-		$i18n = new \Aimeos\MW\Translation\Gettext( $i18nPaths, $lang );
+		$i18n = new \Aimeos\MW\Translation\Gettext($i18nPaths, $lang);
 
 		$content = array(
-			'admin' => $i18n->getAll( 'admin' ),
-			'admin/ext' => $i18n->getAll( 'admin/ext' ),
+			'admin' => $i18n->getAll('admin'),
+			'admin/ext' => $i18n->getAll('admin/ext'),
 		);
 
-		return json_encode( $content, JSON_FORCE_OBJECT );
+		return json_encode($content, JSON_FORCE_OBJECT);
 	}
 
 
@@ -210,18 +210,18 @@ 
 	 * @return string JSON encoded site item object
 	 * @throws Exception If no site item was found for the code
 	 */
-	protected function getJsonSiteItem( \Aimeos\MShop\Context\Item\Iface $context, $site )
+	protected function getJsonSiteItem(\Aimeos\MShop\Context\Item\Iface $context, $site)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'locale/site' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'locale/site');
 
 		$criteria = $manager->createSearch();
-		$criteria->setConditions( $criteria->compare( '==', 'locale.site.code', $site ) );
-		$items = $manager->searchItems( $criteria );
+		$criteria->setConditions($criteria->compare('==', 'locale.site.code', $site));
+		$items = $manager->searchItems($criteria);
 
-		if( ( $item = reset( $items ) ) === false ) {
-			throw new \Exception( sprintf( 'No site found for code "%1$s"', $site ) );
+		if (($item = reset($items)) === false) {
+			throw new \Exception(sprintf('No site found for code "%1$s"', $site));
 		}
 
-		return json_encode( $item->toArray() );
+		return json_encode($item->toArray());
 	}
 }

Controller/JqadmController.php

Spacing +47 added lines, -47 removed lines

@@ -29,35 +29,35 @@ 
 	 * @param $type File type, i.e. "css" or "js"
 	 * @return Response Response object
 	 */
-	public function fileAction( $type )
+	public function fileAction($type)
 	{
 		$contents = '';
 		$files = array();
-		$aimeos = $this->get( 'aimeos' )->get();
+		$aimeos = $this->get('aimeos')->get();
 
-		foreach( $aimeos->getCustomPaths( 'admin/jqadm' ) as $base => $paths )
+		foreach ($aimeos->getCustomPaths('admin/jqadm') as $base => $paths)
 		{
-			foreach( $paths as $path )
+			foreach ($paths as $path)
 			{
-				$jsbAbsPath = $base . '/' . $path;
-				$jsb2 = new \Aimeos\MW\Jsb2\Standard( $jsbAbsPath, dirname( $jsbAbsPath ) );
-				$files = array_merge( $files, $jsb2->getFiles( $type ) );
+				$jsbAbsPath = $base.'/'.$path;
+				$jsb2 = new \Aimeos\MW\Jsb2\Standard($jsbAbsPath, dirname($jsbAbsPath));
+				$files = array_merge($files, $jsb2->getFiles($type));
 			}
 		}
 
-		foreach( $files as $file )
+		foreach ($files as $file)
 		{
-			if( ( $content = file_get_contents( $file ) ) !== false ) {
+			if (($content = file_get_contents($file)) !== false) {
 				$contents .= $content;
 			}
 		}
 
-		$response = new Response( $contents );
+		$response = new Response($contents);
 
-		if( $type === 'js' ) {
-			$response->headers->set( 'Content-Type', 'application/javascript' );
-		} elseif( $type === 'css' ) {
-			$response->headers->set( 'Content-Type', 'text/css' );
+		if ($type === 'js') {
+			$response->headers->set('Content-Type', 'application/javascript');
+		} elseif ($type === 'css') {
+			$response->headers->set('Content-Type', 'text/css');
 		}
 
 		return $response;
@@ -72,10 +72,10 @@ 
 	 * @param string $site Unique site code
 	 * @return Response Generated output
 	 */
-	public function copyAction( Request $request, $resource, $site = 'default' )
+	public function copyAction(Request $request, $resource, $site = 'default')
 	{
-		$cntl = $this->createClient( $request, $site, $resource );
-		return $this->getHtml( $cntl->copy() );
+		$cntl = $this->createClient($request, $site, $resource);
+		return $this->getHtml($cntl->copy());
 	}
 
 
@@ -87,10 +87,10 @@ 
 	 * @param string $site Unique site code
 	 * @return Response Generated output
 	 */
-	public function createAction( Request $request, $resource, $site = 'default' )
+	public function createAction(Request $request, $resource, $site = 'default')
 	{
-		$cntl = $this->createClient( $request, $site, $resource );
-		return $this->getHtml( $cntl->create() );
+		$cntl = $this->createClient($request, $site, $resource);
+		return $this->getHtml($cntl->create());
 	}
 
 
@@ -102,10 +102,10 @@ 
 	 * @param string $site Unique site code
 	 * @return Response Generated output
 	 */
-	public function deleteAction( Request $request, $resource, $site = 'default' )
+	public function deleteAction(Request $request, $resource, $site = 'default')
 	{
-		$cntl = $this->createClient( $request, $site, $resource );
-		return $this->getHtml( $cntl->delete() . $cntl->search() );
+		$cntl = $this->createClient($request, $site, $resource);
+		return $this->getHtml($cntl->delete().$cntl->search());
 	}
 
 
@@ -117,10 +117,10 @@ 
 	 * @param string $site Unique site code
 	 * @return Response Generated output
 	 */
-	public function getAction( Request $request, $resource, $site = 'default' )
+	public function getAction(Request $request, $resource, $site = 'default')
 	{
-		$cntl = $this->createClient( $request, $site, $resource );
-		return $this->getHtml( $cntl->get() );
+		$cntl = $this->createClient($request, $site, $resource);
+		return $this->getHtml($cntl->get());
 	}
 
 
@@ -132,10 +132,10 @@ 
 	 * @param string $site Unique site code
 	 * @return Response Generated output
 	 */
-	public function saveAction( Request $request, $resource, $site = 'default' )
+	public function saveAction(Request $request, $resource, $site = 'default')
 	{
-		$cntl = $this->createClient( $request, $site, $resource );
-		return $this->getHtml( ( $cntl->save() ? : $cntl->search() ) );
+		$cntl = $this->createClient($request, $site, $resource);
+		return $this->getHtml(($cntl->save() ?: $cntl->search()));
 	}
 
 
@@ -147,10 +147,10 @@ 
 	 * @param string $site Unique site code
 	 * @return Response Generated output
 	 */
-	public function searchAction( Request $request, $resource, $site = 'default' )
+	public function searchAction(Request $request, $resource, $site = 'default')
 	{
-		$cntl = $this->createClient( $request, $site, $resource );
-		return $this->getHtml( $cntl->search() );
+		$cntl = $this->createClient($request, $site, $resource);
+		return $this->getHtml($cntl->search());
 	}
 
 
@@ -162,21 +162,21 @@ 
 	 * @param string $resource Resource location, e.g. "product"
 	 * @return \Aimeos\Admin\JQAdm\Iface Context item
 	 */
-	protected function createClient( Request $request, $site, $resource )
+	protected function createClient(Request $request, $site, $resource)
 	{
-		$lang = $request->get( 'lang', 'en' );
+		$lang = $request->get('lang', 'en');
 
-		$aimeos = $this->get( 'aimeos' )->get();
-		$templatePaths = $aimeos->getCustomPaths( 'admin/jqadm/templates' );
+		$aimeos = $this->get('aimeos')->get();
+		$templatePaths = $aimeos->getCustomPaths('admin/jqadm/templates');
 
-		$context = $this->get( 'aimeos_context' )->get( false, 'backend' );
-		$context->setI18n( $this->get( 'aimeos_i18n' )->get( array( $lang, 'en' ) ) );
-		$context->setLocale( $this->get( 'aimeos_locale' )->getBackend( $context, $site ) );
+		$context = $this->get('aimeos_context')->get(false, 'backend');
+		$context->setI18n($this->get('aimeos_i18n')->get(array($lang, 'en')));
+		$context->setLocale($this->get('aimeos_locale')->getBackend($context, $site));
 
-		$view = $this->get( 'aimeos_view' )->create( $context, $templatePaths, $lang );
-		$context->setView( $view );
+		$view = $this->get('aimeos_view')->create($context, $templatePaths, $lang);
+		$context->setView($view);
 
-		return \Aimeos\Admin\JQAdm\Factory::createClient( $context, $templatePaths, $resource );
+		return \Aimeos\Admin\JQAdm\Factory::createClient($context, $templatePaths, $resource);
 	}
 
 
@@ -186,12 +186,12 @@ 
 	 * @param string $content Content from admin client
 	 * @return Response View for rendering the output
 	 */
-	protected function getHtml( $content )
+	protected function getHtml($content)
 	{
-		$version = $this->get( 'aimeos' )->getVersion();
-		$extnames = implode( ',', $this->get( 'aimeos' )->get()->getExtensions() );
-		$content = str_replace( ['{type}', '{version}', '{extensions}'], ['Symfony', $version, $extnames], $content );
+		$version = $this->get('aimeos')->getVersion();
+		$extnames = implode(',', $this->get('aimeos')->get()->getExtensions());
+		$content = str_replace(['{type}', '{version}', '{extensions}'], ['Symfony', $version, $extnames], $content);
 
-		return $this->render( 'AimeosShopBundle:Jqadm:index.html.twig', array( 'content' => $content ) );
+		return $this->render('AimeosShopBundle:Jqadm:index.html.twig', array('content' => $content));
 	}
 }

Controller/JsonapiController.php

Spacing +18 added lines, -18 removed lines

@@ -31,9 +31,9 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Psr\Http\Message\ResponseInterface Response object containing the generated output
 	 */
-	public function deleteAction( ServerRequestInterface $request, $resource, $related = '' )
+	public function deleteAction(ServerRequestInterface $request, $resource, $related = '')
 	{
-		return $this->createClient( $resource, $related )->delete( $request, new Response() );
+		return $this->createClient($resource, $related)->delete($request, new Response());
 	}
 
 
@@ -45,9 +45,9 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Psr\Http\Message\ResponseInterface Response object containing the generated output
 	 */
-	public function getAction( ServerRequestInterface $request, $resource, $related = '' )
+	public function getAction(ServerRequestInterface $request, $resource, $related = '')
 	{
-		return $this->createClient( $resource, $related )->get( $request, new Response() );
+		return $this->createClient($resource, $related)->get($request, new Response());
 	}
 
 
@@ -59,9 +59,9 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Psr\Http\Message\ResponseInterface Response object containing the generated output
 	 */
-	public function patchAction( ServerRequestInterface $request, $resource, $related = '' )
+	public function patchAction(ServerRequestInterface $request, $resource, $related = '')
 	{
-		return $this->createClient( $resource, $related )->patch( $request, new Response() );
+		return $this->createClient($resource, $related)->patch($request, new Response());
 	}
 
 
@@ -73,9 +73,9 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Psr\Http\Message\ResponseInterface Response object containing the generated output
 	 */
-	public function postAction( ServerRequestInterface $request, $resource, $related = '' )
+	public function postAction(ServerRequestInterface $request, $resource, $related = '')
 	{
-		return $this->createClient( $resource, $related )->post( $request, new Response() );
+		return $this->createClient($resource, $related)->post($request, new Response());
 	}
 
 
@@ -87,9 +87,9 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Psr\Http\Message\ResponseInterface Response object containing the generated output
 	 */
-	public function putAction( ServerRequestInterface $request, $resource, $related = '' )
+	public function putAction(ServerRequestInterface $request, $resource, $related = '')
 	{
-		return $this->createClient( $resource, $related )->put( $request, new Response() );
+		return $this->createClient($resource, $related)->put($request, new Response());
 	}
 
 
@@ -101,9 +101,9 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Psr\Http\Message\ResponseInterface Response object containing the generated output
 	 */
-	public function optionsAction( ServerRequestInterface $request, $resource = '', $related = '' )
+	public function optionsAction(ServerRequestInterface $request, $resource = '', $related = '')
 	{
-		return $this->createClient( $resource, $related )->options( $request, new Response() );
+		return $this->createClient($resource, $related)->options($request, new Response());
 	}
 
 
@@ -114,15 +114,15 @@ 
 	 * @param string Related resource location, e.g. "address"
 	 * @return \Aimeos\Client\JsonApi\Iface JSON API client
 	 */
-	protected function createClient( $resource, $related )
+	protected function createClient($resource, $related)
 	{
-		$tmplPaths = $this->container->get( 'aimeos' )->get()->getCustomPaths( 'client/jsonapi/templates' );
-		$context = $this->container->get( 'aimeos_context' )->get();
+		$tmplPaths = $this->container->get('aimeos')->get()->getCustomPaths('client/jsonapi/templates');
+		$context = $this->container->get('aimeos_context')->get();
 		$langid = $context->getLocale()->getLanguageId();
 
-		$view = $this->container->get( 'aimeos_view' )->create( $context, $tmplPaths, $langid );
-		$context->setView( $view );
+		$view = $this->container->get('aimeos_view')->create($context, $tmplPaths, $langid);
+		$context->setView($view);
 
-		return \Aimeos\Client\JsonApi\Factory::createClient( $context, $tmplPaths, $resource . '/' . $related );
+		return \Aimeos\Client\JsonApi\Factory::createClient($context, $tmplPaths, $resource.'/'.$related);
 	}
 }

Tests/Controller/JsonapiControllerTest.php

Spacing +197 added lines, -197 removed lines

@@ -10,60 +10,60 @@ 
 	public function testOptionsAction()
 	{
 		$client = static::createClient();
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertArrayHasKey( 'resources', $json['meta'] );
-		$this->assertGreaterThan( 1, count( $json['meta']['resources'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertArrayHasKey('resources', $json['meta']);
+		$this->assertGreaterThan(1, count($json['meta']['resources']));
 	}
 
 
 	public function testGetAttributeAction()
 	{
 		$client = static::createClient();
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/attribute', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/attribute', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 24, $json['meta']['total'] );
-		$this->assertEquals( 24, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(24, $json['meta']['total']);
+		$this->assertEquals(24, count($json['data']));
 	}
 
 
 	public function testGetCatalogAction()
 	{
 		$client = static::createClient();
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/catalog', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/catalog', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 1, $json['meta']['total'] );
-		$this->assertEquals( 4, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(1, $json['meta']['total']);
+		$this->assertEquals(4, count($json['data']));
 	}
 
 
 	public function testGetLocaleAction()
 	{
 		$client = static::createClient();
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/locale', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/locale', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 1, $json['meta']['total'] );
-		$this->assertEquals( 1, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(1, $json['meta']['total']);
+		$this->assertEquals(1, count($json['data']));
 	}
 
 
@@ -72,60 +72,60 @@ 
 		$client = static::createClient();
 
 		$params = ['filter' => ['f_search' => 'Cafe Noire Cap', 'f_listtype' => 'unittype19']];
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/product', $params );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/product', $params);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 1, $json['meta']['total'] );
-		$this->assertEquals( 1, count( $json['data'] ) );
-		$this->assertArrayHasKey( 'id', $json['data'][0] );
-		$this->assertEquals( 'CNC', $json['data'][0]['attributes']['product.code'] );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(1, $json['meta']['total']);
+		$this->assertEquals(1, count($json['data']));
+		$this->assertArrayHasKey('id', $json['data'][0]);
+		$this->assertEquals('CNC', $json['data'][0]['attributes']['product.code']);
 
 		$id = $json['data'][0]['id'];
 
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/product/' . $id );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/product/'.$id);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 1, $json['meta']['total'] );
-		$this->assertArrayHasKey( 'id', $json['data'] );
-		$this->assertEquals( 'CNC', $json['data']['attributes']['product.code'] );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(1, $json['meta']['total']);
+		$this->assertArrayHasKey('id', $json['data']);
+		$this->assertEquals('CNC', $json['data']['attributes']['product.code']);
 	}
 
 
 	public function testGetServiceAction()
 	{
 		$client = static::createClient();
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/service', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/service', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 4, $json['meta']['total'] );
-		$this->assertEquals( 4, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(4, $json['meta']['total']);
+		$this->assertEquals(4, count($json['data']));
 	}
 
 
 	public function testGetStockAction()
 	{
 		$client = static::createClient();
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/stock', ['filter' => ['s_prodcode' => ['CNC', 'CNE']]] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/stock', ['filter' => ['s_prodcode' => ['CNC', 'CNE']]]);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 2, $json['meta']['total'] );
-		$this->assertEquals( 2, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(2, $json['meta']['total']);
+		$this->assertEquals(2, count($json['data']));
 	}
 
 
@@ -133,24 +133,24 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $json['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($json['meta']['resources']));
 
 		// catalog root
-		$client->request( 'GET', $json['meta']['resources']['catalog'], ['include' => 'catalog'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'categories', $json['included'][0]['attributes']['catalog.code'] );
+		$client->request('GET', $json['meta']['resources']['catalog'], ['include' => 'catalog']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('categories', $json['included'][0]['attributes']['catalog.code']);
 
 		// "categories" category
-		$client->request( 'GET', $json['included'][0]['links']['self']['href'], ['include' => 'catalog'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'cafe', $json['included'][0]['attributes']['catalog.code'] );
+		$client->request('GET', $json['included'][0]['links']['self']['href'], ['include' => 'catalog']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('cafe', $json['included'][0]['attributes']['catalog.code']);
 
 		// product list for "cafe" category
-		$client->request( 'GET', $json['included'][0]['links']['product']['href'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'CNE', $json['data'][0]['attributes']['product.code'] );
+		$client->request('GET', $json['included'][0]['links']['product']['href']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('CNE', $json['data'][0]['attributes']['product.code']);
 	}
 
 
@@ -158,26 +158,26 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$options = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $options['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$options = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($options['meta']['resources']));
 
 		// all available attrbutes
-		$client->request( 'GET', $options['meta']['resources']['attribute'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
+		$client->request('GET', $options['meta']['resources']['attribute']);
+		$json = json_decode($client->getResponse()->getContent(), true);
 
-		foreach( $json['data'] as $entry )
+		foreach ($json['data'] as $entry)
 		{
-			if( $entry['attributes']['attribute.code'] === 'xl' )
+			if ($entry['attributes']['attribute.code'] === 'xl')
 			{
 				// products with attrbute "xl"
-				$client->request( 'GET', $options['meta']['resources']['product'], ['filter' => ['f_attrid' => $entry['id']]] );
+				$client->request('GET', $options['meta']['resources']['product'], ['filter' => ['f_attrid' => $entry['id']]]);
 				break;
 			}
 		}
 
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 2, $json['meta']['total'] );
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(2, $json['meta']['total']);
 	}
 
 
@@ -185,14 +185,14 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $json['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($json['meta']['resources']));
 
 		// product list for full text search
-		$client->request( 'GET', $json['meta']['resources']['product'], ['filter' => ['f_search' => 'selection']] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 3, count( $json['data'] ) );
+		$client->request('GET', $json['meta']['resources']['product'], ['filter' => ['f_search' => 'selection']]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(3, count($json['data']));
 	}
 
 
@@ -200,23 +200,23 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $json['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($json['meta']['resources']));
 
 		// get empty basket
-		$client->request( 'GET', $json['meta']['resources']['basket'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket', $json['data']['type'] );
+		$client->request('GET', $json['meta']['resources']['basket']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket', $json['data']['type']);
 
 		$content = '{"data": {"id": "delivery", "attributes": {"order.base.address.firstname": "test"}}}';
-		$client->request( 'POST', $json['links']['basket/address']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/address', $json['included'][0]['type'] );
+		$client->request('POST', $json['links']['basket/address']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/address', $json['included'][0]['type']);
 
-		$client->request( 'DELETE', $json['included'][0]['links']['self']['href'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 0, count( $json['included'] ) );
+		$client->request('DELETE', $json['included'][0]['links']['self']['href']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(0, count($json['included']));
 	}
 
 
@@ -224,31 +224,31 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $json['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($json['meta']['resources']));
 
 		// product for code "CNC"
-		$client->request( 'GET', $json['meta']['resources']['product'], ['filter' => ['==' => ['product.code' => 'CNC']]] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 1, count( $json['data'] ) );
+		$client->request('GET', $json['meta']['resources']['product'], ['filter' => ['==' => ['product.code' => 'CNC']]]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(1, count($json['data']));
 
 		// add product "CNC" as prerequisite
-		$content = '{"data": {"attributes": {"product.id": ' . $json['data'][0]['id'] . '}}}';
-		$client->request( 'POST', $json['data'][0]['links']['basket/product']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/product', $json['included'][0]['type'] );
+		$content = '{"data": {"attributes": {"product.id": '.$json['data'][0]['id'].'}}}';
+		$client->request('POST', $json['data'][0]['links']['basket/product']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/product', $json['included'][0]['type']);
 
 		// add coupon "GHIJ"
 		$content = '{"data": {"id": "GHIJ"}}';
-		$client->request( 'POST', $json['links']['basket/coupon']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/coupon', $json['included'][2]['type'] );
+		$client->request('POST', $json['links']['basket/coupon']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/coupon', $json['included'][2]['type']);
 
 		// remove coupon "GHIJ" again
-		$client->request( 'DELETE', $json['included'][2]['links']['self']['href'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 1, count( $json['included'] ) );
+		$client->request('DELETE', $json['included'][2]['links']['self']['href']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(1, count($json['included']));
 	}
 
 
@@ -256,28 +256,28 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $json['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($json['meta']['resources']));
 
 		// product for code "CNC"
-		$client->request( 'GET', $json['meta']['resources']['product'], ['filter' => ['f_search' => 'ABCD']] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 1, count( $json['data'] ) );
+		$client->request('GET', $json['meta']['resources']['product'], ['filter' => ['f_search' => 'ABCD']]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(1, count($json['data']));
 
-		$content = '{"data": {"attributes": {"product.id": ' . $json['data'][0]['id'] . '}}}';
-		$client->request( 'POST', $json['data'][0]['links']['basket/product']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/product', $json['included'][0]['type'] );
+		$content = '{"data": {"attributes": {"product.id": '.$json['data'][0]['id'].'}}}';
+		$client->request('POST', $json['data'][0]['links']['basket/product']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/product', $json['included'][0]['type']);
 
 		$content = '{"data": {"attributes": {"quantity": 2}}}';
-		$client->request( 'PATCH', $json['included'][0]['links']['self']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 2, $json['included'][0]['attributes']['order.base.product.quantity'] );
+		$client->request('PATCH', $json['included'][0]['links']['self']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(2, $json['included'][0]['attributes']['order.base.product.quantity']);
 
-		$client->request( 'DELETE', $json['included'][0]['links']['self']['href'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 0, count( $json['included'] ) );
+		$client->request('DELETE', $json['included'][0]['links']['self']['href']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(0, count($json['included']));
 	}
 
 
@@ -285,14 +285,14 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $json['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($json['meta']['resources']));
 
 		// payment services
-		$client->request( 'GET', $json['meta']['resources']['service'], ['filter' => ['cs_type' => 'payment']] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 3, count( $json['data'] ) );
+		$client->request('GET', $json['meta']['resources']['service'], ['filter' => ['cs_type' => 'payment']]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(3, count($json['data']));
 
 		$content = ['data' => ['id' => 'payment', 'attributes' => [
 			'service.id' => $json['data'][1]['id'],
@@ -301,15 +301,15 @@ 
 			'directdebit.bankcode' => 'ABCDEFGH',
 			'directdebit.bankname' => 'test bank',
 		]]];
-		$client->request( 'POST', $json['data'][1]['links']['basket/service']['href'], [], [], [], json_encode( $content ) );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/service', $json['included'][0]['type'] );
-		$this->assertEquals( 'directdebit-test', $json['included'][0]['attributes']['order.base.service.code'] );
-		$this->assertEquals( 5, count( $json['included'][0]['attributes']['attribute'] ) );
-
-		$client->request( 'DELETE', $json['included'][0]['links']['self']['href'] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 0, count( $json['included'] ) );
+		$client->request('POST', $json['data'][1]['links']['basket/service']['href'], [], [], [], json_encode($content));
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/service', $json['included'][0]['type']);
+		$this->assertEquals('directdebit-test', $json['included'][0]['attributes']['order.base.service.code']);
+		$this->assertEquals(5, count($json['included'][0]['attributes']['attribute']));
+
+		$client->request('DELETE', $json['included'][0]['links']['self']['href']);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(0, count($json['included']));
 	}
 
 
@@ -318,17 +318,17 @@ 
 		$client = static::createClient(array(), array(
 			'PHP_AUTH_USER' => 'UTC001',
 			'PHP_AUTH_PW'   => 'unittest',
-		) );
+		));
 
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/customer', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/customer', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 1, $json['meta']['total'] );
-		$this->assertEquals( 4, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(1, $json['meta']['total']);
+		$this->assertEquals(4, count($json['data']));
 	}
 
 
@@ -337,22 +337,22 @@ 
 		$client = static::createClient(array(), array(
 			'PHP_AUTH_USER' => 'UTC001',
 			'PHP_AUTH_PW'   => 'unittest',
-		) );
+		));
 
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/customer', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/customer', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$client->request( 'GET', $json['links']['customer/address']['href'], [] );
+		$client->request('GET', $json['links']['customer/address']['href'], []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 1, $json['meta']['total'] );
-		$this->assertEquals( 1, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(1, $json['meta']['total']);
+		$this->assertEquals(1, count($json['data']));
 	}
 
 
@@ -361,17 +361,17 @@ 
 		$client = static::createClient(array(), array(
 			'PHP_AUTH_USER' => 'UTC001',
 			'PHP_AUTH_PW'   => 'unittest',
-		) );
+		));
 
-		$client->request( 'GET', '/unittest/de/EUR/jsonapi/order', [] );
+		$client->request('GET', '/unittest/de/EUR/jsonapi/order', []);
 		$response = $client->getResponse();
 
-		$json = json_decode( $response->getContent(), true );
+		$json = json_decode($response->getContent(), true);
 
-		$this->assertNotNull( $json );
-		$this->assertEquals( 200, $response->getStatusCode() );
-		$this->assertEquals( 4, $json['meta']['total'] );
-		$this->assertEquals( 4, count( $json['data'] ) );
+		$this->assertNotNull($json);
+		$this->assertEquals(200, $response->getStatusCode());
+		$this->assertEquals(4, $json['meta']['total']);
+		$this->assertEquals(4, count($json['data']));
 	}
 
 
@@ -379,63 +379,63 @@ 
 	{
 		$client = static::createClient();
 
-		$client->request( 'OPTIONS', '/unittest/de/EUR/jsonapi' );
-		$optJson = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertGreaterThan( 8, count( $optJson['meta']['resources'] ) );
+		$client->request('OPTIONS', '/unittest/de/EUR/jsonapi');
+		$optJson = json_decode($client->getResponse()->getContent(), true);
+		$this->assertGreaterThan(8, count($optJson['meta']['resources']));
 
 		// product for code "CNC"
-		$client->request( 'GET', $optJson['meta']['resources']['product'], ['filter' => ['==' => ['product.code' => 'CNC']]] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 1, count( $json['data'] ) );
+		$client->request('GET', $optJson['meta']['resources']['product'], ['filter' => ['==' => ['product.code' => 'CNC']]]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(1, count($json['data']));
 
 		// add product "CNC"
-		$content = '{"data": {"attributes": {"product.id": ' . $json['data'][0]['id'] . '}}}';
-		$client->request( 'POST', $json['data'][0]['links']['basket/product']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/product', $json['included'][0]['type'] );
+		$content = '{"data": {"attributes": {"product.id": '.$json['data'][0]['id'].'}}}';
+		$client->request('POST', $json['data'][0]['links']['basket/product']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/product', $json['included'][0]['type']);
 
 		// delivery services
-		$client->request( 'GET', $optJson['meta']['resources']['service'], ['filter' => ['cs_type' => 'delivery']] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 1, count( $json['data'] ) );
+		$client->request('GET', $optJson['meta']['resources']['service'], ['filter' => ['cs_type' => 'delivery']]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(1, count($json['data']));
 
 		// add delivery service
-		$content = '{"data": {"id": "delivery", "attributes": {"service.id": ' . $json['data'][0]['id'] . '}}}';
-		$client->request( 'POST', $json['data'][0]['links']['basket/service']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/service', $json['included'][1]['type'] );
+		$content = '{"data": {"id": "delivery", "attributes": {"service.id": '.$json['data'][0]['id'].'}}}';
+		$client->request('POST', $json['data'][0]['links']['basket/service']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/service', $json['included'][1]['type']);
 
 		// payment services
-		$client->request( 'GET', $optJson['meta']['resources']['service'], ['filter' => ['cs_type' => 'payment']] );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 3, count( $json['data'] ) );
+		$client->request('GET', $optJson['meta']['resources']['service'], ['filter' => ['cs_type' => 'payment']]);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(3, count($json['data']));
 
 		// add payment service
-		$content = '{"data": {"id": "payment", "attributes": {"service.id": ' . $json['data'][0]['id'] . '}}}';
-		$client->request( 'POST', $json['data'][0]['links']['basket/service']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/service', $json['included'][2]['type'] );
+		$content = '{"data": {"id": "payment", "attributes": {"service.id": '.$json['data'][0]['id'].'}}}';
+		$client->request('POST', $json['data'][0]['links']['basket/service']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/service', $json['included'][2]['type']);
 
 		// add address
 		$content = '{"data": {"id": "payment", "attributes": {"order.base.address.firstname": "test"}}}';
-		$client->request( 'POST', $json['links']['basket/address']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( 'basket/address', $json['included'][3]['type'] );
+		$client->request('POST', $json['links']['basket/address']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals('basket/address', $json['included'][3]['type']);
 
 		// store basket
-		$client->request( 'POST', $json['links']['self']['href'] );
-		$basketJson = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( true, ctype_digit( $basketJson['data']['id'] ) );
+		$client->request('POST', $json['links']['self']['href']);
+		$basketJson = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(true, ctype_digit($basketJson['data']['id']));
 
 		// add order
-		$content = '{"data": {"attributes": {"order.baseid": ' . $basketJson['data']['id'] . '}}}';
-		$client->request( 'POST', $basketJson['links']['order']['href'], [], [], [], $content );
-		$json = json_decode( $client->getResponse()->getContent(), true );
-		$this->assertEquals( true, ctype_digit( $json['data']['id'] ) );
+		$content = '{"data": {"attributes": {"order.baseid": '.$basketJson['data']['id'].'}}}';
+		$client->request('POST', $basketJson['links']['order']['href'], [], [], [], $content);
+		$json = json_decode($client->getResponse()->getContent(), true);
+		$this->assertEquals(true, ctype_digit($json['data']['id']));
 
 
 		// delete created order
-		$context = static::$kernel->getContainer()->get( 'aimeos_context' )->get();
-		\Aimeos\MShop\Factory::createManager( $context, 'order/base' )->deleteItem( $basketJson['data']['id'] );
+		$context = static::$kernel->getContainer()->get('aimeos_context')->get();
+		\Aimeos\MShop\Factory::createManager($context, 'order/base')->deleteItem($basketJson['data']['id']);
 	}
 }

Tests/Fixtures/app/AppKernel.php

Spacing +4 added lines, -4 removed lines

@@ -28,9 +28,9 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function registerContainerConfiguration( LoaderInterface $loader )
+	public function registerContainerConfiguration(LoaderInterface $loader)
 	{
-		$loader->load( __DIR__ . '/config/config.yml' );
+		$loader->load(__DIR__.'/config/config.yml');
 	}
 
 
@@ -39,7 +39,7 @@ 
 	 */
 	public function getCacheDir()
 	{
-		return sys_get_temp_dir() . '/aimeos-symfony/cache';
+		return sys_get_temp_dir().'/aimeos-symfony/cache';
 	}
 
 
@@ -48,6 +48,6 @@ 
 	 */
 	public function getLogDir()
 	{
-		return sys_get_temp_dir() . '/aimeos-symfony/logs';
+		return sys_get_temp_dir().'/aimeos-symfony/logs';
 	}
 }

Service/I18n.php

Spacing +13 added lines, -13 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @param Container $container Container object to access parameters
 	 */
-	public function __construct( Container $container )
+	public function __construct(Container $container)
 	{
 		$this->container = $container;
 	}
@@ -41,27 +41,27 @@ 
 	 * @param array $languageIds List of two letter ISO language IDs
 	 * @return \Aimeos\MW\Translation\Interface[] List of translation objects
 	 */
-	public function get( array $languageIds )
+	public function get(array $languageIds)
 	{
-		$i18nPaths = $this->container->get( 'aimeos' )->get()->getI18nPaths();
+		$i18nPaths = $this->container->get('aimeos')->get()->getI18nPaths();
 
-		foreach( $languageIds as $langid )
+		foreach ($languageIds as $langid)
 		{
-			if( !isset( $this->i18n[$langid] ) )
+			if (!isset($this->i18n[$langid]))
 			{
-				$i18n = new \Aimeos\MW\Translation\Gettext( $i18nPaths, $langid );
+				$i18n = new \Aimeos\MW\Translation\Gettext($i18nPaths, $langid);
 
-				$apc = (bool) $this->container->getParameter( 'aimeos_shop.apc_enable' );
-				$prefix = $this->container->getParameter( 'aimeos_shop.apc_prefix' );
+				$apc = (bool) $this->container->getParameter('aimeos_shop.apc_enable');
+				$prefix = $this->container->getParameter('aimeos_shop.apc_prefix');
 
-				if( function_exists( 'apcu_store' ) === true && $apc === true ) {
-					$i18n = new \Aimeos\MW\Translation\Decorator\APC( $i18n, $prefix );
+				if (function_exists('apcu_store') === true && $apc === true) {
+					$i18n = new \Aimeos\MW\Translation\Decorator\APC($i18n, $prefix);
 				}
 
-				$translations = $this->container->getParameter( 'aimeos_shop.i18n' );
+				$translations = $this->container->getParameter('aimeos_shop.i18n');
 
-				if( isset( $translations[$langid] ) ) {
-					$i18n = new \Aimeos\MW\Translation\Decorator\Memory( $i18n, $translations[$langid] );
+				if (isset($translations[$langid])) {
+					$i18n = new \Aimeos\MW\Translation\Decorator\Memory($i18n, $translations[$langid]);
 				}
 
 				$this->i18n[$langid] = $i18n;

Service/Config.php

Spacing +17 added lines, -17 removed lines

@@ -28,7 +28,7 @@ 
 	 *
 	 * @param Container $container Container object to access parameters
 	 */
-	public function __construct( Container $container )
+	public function __construct(Container $container)
 	{
 		$this->container = $container;
 	}
@@ -40,33 +40,33 @@ 
 	 * @param string $type Configuration type ("frontend" or "backend")
 	 * @return \Aimeos\MW\Config\Iface Config object
 	 */
-	public function get( $type = 'frontend' )
+	public function get($type = 'frontend')
 	{
 		$configPaths = $this->container->get('aimeos')->get()->getConfigPaths();
 
-		$conf = new \Aimeos\MW\Config\PHPArray( array(), $configPaths );
+		$conf = new \Aimeos\MW\Config\PHPArray(array(), $configPaths);
 
-		$apc = (bool) $this->container->getParameter( 'aimeos_shop.apc_enable' );
-		$prefix = $this->container->getParameter( 'aimeos_shop.apc_prefix' );
+		$apc = (bool) $this->container->getParameter('aimeos_shop.apc_enable');
+		$prefix = $this->container->getParameter('aimeos_shop.apc_prefix');
 
-		if( function_exists( 'apcu_store' ) === true && $apc === true ) {
-			$conf = new \Aimeos\MW\Config\Decorator\APC( $conf, $prefix );
+		if (function_exists('apcu_store') === true && $apc === true) {
+			$conf = new \Aimeos\MW\Config\Decorator\APC($conf, $prefix);
 		}
 
 		$local = array(
-			'admin' => $this->container->getParameter( 'aimeos_shop.admin' ),
-			'client' => $this->container->getParameter( 'aimeos_shop.client' ),
-			'controller' => $this->container->getParameter( 'aimeos_shop.controller' ),
-			'madmin' => $this->container->getParameter( 'aimeos_shop.madmin' ),
-			'mshop' => $this->container->getParameter( 'aimeos_shop.mshop' ),
-			'resource' => $this->container->getParameter( 'aimeos_shop.resource' ),
+			'admin' => $this->container->getParameter('aimeos_shop.admin'),
+			'client' => $this->container->getParameter('aimeos_shop.client'),
+			'controller' => $this->container->getParameter('aimeos_shop.controller'),
+			'madmin' => $this->container->getParameter('aimeos_shop.madmin'),
+			'mshop' => $this->container->getParameter('aimeos_shop.mshop'),
+			'resource' => $this->container->getParameter('aimeos_shop.resource'),
 		);
 
-		$config = new \Aimeos\MW\Config\Decorator\Memory( $conf, $local );
-		$settings = $this->container->getParameter( 'aimeos_shop.' . $type );
+		$config = new \Aimeos\MW\Config\Decorator\Memory($conf, $local);
+		$settings = $this->container->getParameter('aimeos_shop.'.$type);
 
-		if( $settings !== array() ) {
-			$config = new \Aimeos\MW\Config\Decorator\Memory( $config, $settings );
+		if ($settings !== array()) {
+			$config = new \Aimeos\MW\Config\Decorator\Memory($config, $settings);
 		}
 
 		return $config;