I will add the same files than in PaypalExpressCheckoutBundle.

Then, I will update composer.json regarding to @mmoreram suggest on how we should deal with minimal requirements.

PR to come, stay tuned

I will add the same files than in PaypalExpressCheckoutBundle

make authorship on class header and not in file header: file header authorship is always the same on all the PaymentSuite project

There is a lack of information of how to use this bundle. We need to create some basic usage documentation here and/or create a full documentation in Read the Docs.

ping @mmoreram @arkaitzgarro is this a good idea ? I'm not sure at this point.

To be more consistent

The actual tests require the use of WebTestCase utility class.

Since Symfony 2.5, we need to declare kernel app dir of Symfony application and - for bundle testing purpose - create this Symfony application.

See http://symfony.com/doc/current/book/testing.html#your-first-functional-test

The name format is so weird: paypal-web_checkout-bundle

Please, change it to paypal-web-checkout-bundle, and then , we will add this bundle into the main package.

Thanks.

Controllers are defined dynamically by the Router, so no annotations are read.

Take a look here. You could be able to remove this annotations, and ensure that, if you really need that controller action to respond only to GET requests, define it into the Router class.

An example here

I'm trying to install the Bundle, but a ServiceNotFoundException appears in my Symfony Bundle (src\Symfony\Component\DependencyInjection\Compiler\CheckExceptionOnInvalidReferenceBehaviorPass line 58 ).

The service "paypal_web_checkout.manager" has a dependency on a non-existent service "payment.event.dispatcher"

Any idea? I'm using Symfony v.2.6.3

Thanks for your time.

we can take a look at Paymill Bundle

The idea here is to allow templates to be override

In most of the projects, when the items are shipped to the client, we need to know the costs of this shipment.

I've seen than @mmoreram and @alch are working in a shipping component, but it seems that is not included into the order yet. Are you ok if we include a shipping cost method into PaymentBridge?

/**
 * Get shipping amount
 *
 * @return integer
 */
 public function getShippingAmount() {
     // Calculate the costs for this order
     // 0 if it is free
 }

And then create the form field if applicable:

/*
 * PaymentBridge stores shipping amount in cents
 */
$shippingAmount = $this
        ->paymentBridge
        ->getShippingAmount();

if ($shippingAmount > 0) {
    $formBuilder
        ->add('shipping_1', 'hidden', array(
            'data' => $shippingAmount,
        ))
    ;
}