treveradams / openerp-freeswitch-connector

A connector to integrate FreeSWITCH and OpenERP/Odoo
GNU Affero General Public License v3.0
13 stars 11 forks source link

Odoo v8-9 version #4

Open topecz opened 8 years ago

topecz commented 8 years ago

Trever, Thanks for your work. Do you plan to share a v8 version one, I work with that version and I can't install the actual version under Odoo v8. How can I help you? Thanks a lot Regards

treveradams commented 8 years ago

https://github.com/treveradams/connector-telephony/branches Is where you should be looking right now. I am trying to get more of my changes merged back into the upstream (several features/fixes were merged by someone else a while back). There is an 8.0 branch. I currently have not tested it as I only have an OpenERP 7 install. I will be happy to work with you if you are willing to test it.

Thank you.

treveradams commented 8 years ago

Make sure you also look at the fixed-recruitment branch(es). These give you the name of the applicant as Caller ID instead of the position.

topecz commented 8 years ago

Trever, thanks for the answer, after solving some errors on the freeswitch connector, I attach the remarks, I am upgrading the debian vm to debian 8 as I found another error on the freeswitch ESL.py: ldd /usr/local/lib/python2.7/dist-packages/_ESL.so /usr/local/lib/python2.7/dist-packages/_ESL.so: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.14' not found (required by /usr/local/lib/python2.7/dist-packages/_ESL.so)

Under your module: /freeswitch_click2dial/freeswitch_click2dial.py", line 234 def _get_calling_number(self, cr, uid, context=None): ^ IndentationError: unexpected unindent

It happens because we need a pair try/except e.g: else: try: if fs_manager: fs_manager.disconnect() raise orm.exceptorm( ("Connection Test Successfull!"), _("OpenERP can successfully login to the FreeSWITCH Event " "Socket.")) except Exception, e: raise orm.exceptorm( ("Connection Test Failed!"), _("Here is the error message: %s" % e))

def _get_calling_number(self, cr, uid, context=None):

Tell me if I'm wrong. I have several debian 7/8 to test the modules. Thanks a lot R

treveradams commented 8 years ago

Unfortunately the formatting was lost. Can you upload a diff as part of a post?

treveradams commented 8 years ago

Ok. I think I understood what you were suggesting. I have tried to clean up the code. Please test with valid and invalid username and/or password. Thank you.

treveradams commented 8 years ago

If you wouldn't mind, also check where you cannot connect to FreeSWITCH (due to firewall rules, or unplugged network cable, etc.).

topecz commented 8 years ago

It's not a FS issue. After upgrading the whole vps to debian 8 and verify the odoo installation on the DB, working properly with all modules. I listed the freeswitch_click2dial on the addons, given me a css issue and leting me without access to any interface. Once remove the modules and restarted the service it is working properly once again. It is very important to me that the module works because I only use Freeswitch but now I have to build a cdr module to charge the customers the phone expenses and I could not find any. Following your answers I use the https://github.com/treveradams/connector-telephony/branches, 8 branch.

treveradams commented 8 years ago

I understand it isn't a FS issue. That code does catch errors including those with problems connecting to FS.

I updated the branch, I didn't bump the version so you have to remove and readd the modules. Doing that, does it fix things?

topecz commented 8 years ago

OK! I will try to pull the branch and tell you the results ;) Thnaks

treveradams commented 8 years ago

Can you tell me if this worked for you? If not does the current version work (I have been cleaning things up)?

treveradams commented 8 years ago

As another answer, I will be working on a version 9.0 compatible one once connector-telephony finishes their port of the other modules.

topecz commented 8 years ago

No, it does not work for me. I am in a new job and I have no time left for this project. I will do the debug on my spare time. Thanks a lot

treveradams commented 8 years ago

If you do a fresh pull it may work. I still don't have an 8.0 setup. I am preparing to work on the v9 port as soon as this is finished upstream. I did find that there was work done after I forked from the main project for 8.0 compatibility and that some of that may solve your problem. I have now made similar changes in my tree (https://github.com/treveradams/connector-telephony/tree/freeswitch-8.0). This may solve all of your problems. If it fixes them, let me know.