Fix tbbc_money.pair_history_manager service definition

[hhamon]

No description provided.

[johanwilfer]

@hhamon Was this broken by the last PR (to make it optional with Doctorine if you use only MongoDB) or what is the background for this PR?

[hhamon]

@johanwilfer after upgrading from 6.0.0 to 6.0.1, my CI fails:

> @php bin/console lint:container


 [ERROR] Invalid definition for service "tbbc_money.pair_history_manager":
         "Tbbc\MoneyBundle\PairHistory\PairHistoryManager::__construct()" requires 2 arguments, 1 passed.

[johanwilfer]

Yeah, this basically reverts #180

@mustanggb What do you suggest? Any way to make it optional for MongoDb users still or do we have to revert your change?

[mustanggb]

I don't know how to denote that a service is optional, perhaps someone cleverer than me does.

However as this was a new addition anyway, we can just remove it, as per #182.

[hhamon]

I think it must be resolved with a DI compiler pass

[johanwilfer]

For now I will merge this, to revert this until we resolved it properly.