Added Extra transformer feature and fixed tests

kayue / KayueWordpressBundle

A Symfony 2 bundle for providing WordPress repositories and authenticating users (login).

101 stars 43 forks source link

Added Extra transformer feature and fixed tests #43

Closed Nek- closed 9 years ago

Nek- commented 9 years ago

Hello,

I fixed the tests on the bundle but I also add a new way to add basic transformers on text. That's because Wordpress parse freely anywere in his code the content of a blog post.

For example, Wordpress parse the content to add videos as iframe instead of links.

So I:

Added a transformer registry
Added new configuration to be able to disable non desired transformers
Added new tests related to the new feature
Fixed old tests (that should be improved)
Fixed some CS

kayue commented 9 years ago

@Nek- First of all, thank you for your pull request.

Regarding video embeding, I think https://github.com/kayue/KayueEssenceBundle can do just want you want.

{{ post.content|essence_replace|wp_autop }}

What do you think?

Nek- commented 9 years ago

Aw indeed.

I'm going to remove parts of my work and add a notice in the documentation about the second bundle.

kayue commented 9 years ago

Yes, please remove the transformer part as it is handled by oEmbed.

It is nice that you improved and fixed the tests. Thanks a lot. :+1: