JMSSerializer namespace change

xamado commented 11 years ago

Updated the namespace changes in the latest JMSSerializerBundle (Serializer was moved to a separate library).

thewilkybarkid commented 11 years ago

There's a couple of other things to do:

In Tests/boostrap.php change the annotation autoloading namespace/path (to AnnotationRegistry::registerAutoloadNamespaces(array('JMS\\Serializer' => __DIR__ . '/../vendor/jms/serializer/src/'));).
composer.json also needs "minimum-stability":"dev" to load the Serializer correctly, and the required bundle version should become "jms/serializer-bundle":"1.0.*". When it's stable I'll remove the dev requirement.

Once you can confirm that the tests are passing, you could also squash your commits?

Thanks!

xamado commented 11 years ago

I was working on getting the tests running last night, I'm having trouble with the annotations. I did change the autoload in bootstrap just like you said, but the tests still complain it can't find or autoload the XmlRoot annotation. I'll post the exact error in a bit.

xamado commented 11 years ago

Ok it works now, and I squashed the commits.

thewilkybarkid commented 11 years ago

Thanks. :thumbsup:

misd-service-development / guzzle-bundle

JMSSerializer namespace change #8