Adds ip-traceable listener

[alterphp]

Adds support for IpTraceable Gedmo extension

[dev-ish]

+1

[lopsided]

+1

[alterphp]

@stof Any opinion about this PR ?

[winzou]

:+1: but it needs some cleaning, see comments.

[fotomerchant]

+1

[dirkluijk]

+1

[al3d]

+1

[oscargala]

+1

[alterphp]

@stof Can you review this PR ?

[dante2008]

:+1:

[alterphp]

Hello @stof

I know you're working on many other projects, I heard about on php-cs-fixer and symfony... But can you review this one please ?

[alterphp]

Hey @stof : any review ?

[Axinet]

+1

[alterphp]

UP

[lemoinem]

:+1: Please merge this ASAP...

[jrmyio]

+1

[jillro]

+1

[stof]

@alterphp please rebase your branch to fix conflict, and fix the few things I asked

[alterphp]

@stof Done

[jewome62]

+1

[conradfr]

+1

[jrmyio]

+1

[Invis1ble]

+1

[QuentinPi]

So how could we activate the ip_traceable extension with the StofDoctrineExtensionsBundle ? With my test configuration, all the IPs fields are NULL.

[Invis1ble]

@QuozPowa

stof_doctrine_extensions:
    orm:
        default:
            ip_traceable: true

[QuentinPi]

I tried but I have this error : Unrecognized option "ip_traceable" under "stof_doctrine_extensions.orm.default"

I use the dev-master version of stof/doctrine-extensions-bundle in composer.

[alterphp]

@QuozPowa As long as @stof does not merge this PR, you can't use it from this repository... Or you have to use my fork (I created for this PR) : [email protected]:alterphp/StofDoctrineExtensionsBundle.git on branch iptraceable-listener

If you personnaly know Stof, you can tell him to come back over here ;-)

[jdemangeon]

+1

[Pierstoval]

:+1: too, I think this can be merged

[ctaverne]

+1 :)