znuny / Znuny4OTRS-CTI

URL-CTI Support for incoming calls.
https://www.znuny.com/en/add-ons
13 stars 6 forks source link

CustomerUser config should not get overwritten #4

Closed thorsteneckel closed 5 years ago

thorsteneckel commented 8 years ago

Currently the File Znuny4OTRSCTI.pm overwrites the CustomerUser config. This can have different implications which can cause errors like wrong/not loaded CustomerUser configs, wrong CustomerUser BackendModules etc.

We should replace the file by a 1.1 config file that adds the UserPhone attribute (if given) to the CustomerUser CustomerUserSearchFields configurations.

thorsteneckel commented 8 years ago

A workaround would be to use the CustomerUser config from the Defauls.pm and add it to the Config.pm if no custom CustomerUser config present yet. After that the 'phone' (or matching storage field) string have to be added to the 'CustomerUserSearchFields' list like in: https://github.com/znuny/Znuny4OTRS-CTI/blob/master/Kernel/Config/Files/Znuny4OTRSCTI.pm#L41

rkaldung commented 8 years ago

I'd like to talk about this :-)

rkaldung commented 5 years ago

Yes