Hello everyone.
I played with Claude 4 to update this old module. 
It was fun and cool.
Check out the screenshots.
7 Likes
Looks great! Maybe it can be incorporated into TangoPBX as the default CDR module.
3 Likes
Thank you sir. You are nice.
1 Like
@franck.danard, I really appreciate what you bring to the community.
1 Like
Bonjour Franck! Is this up somewhere where we can give it test run?
1 Like
Send me an email to gmail, I forgot your gmail address sorry.
2 Likes
Check your email pro. I sent a tar file but just uncompress it directly on the /modules path.
Ha it’s not the good one.
It’s time to sleep. I waiting your email address and I will send you a tar file tomorrow.
I raised a PR on the FreePBX Github.
Torpedo launched.
2 Likes
Torpedo deflected. The PR is currently being denied due to the AI based code and SQL injections it created.
Turns out some of the SQL issues already exist in the current version and Franck just used them. So, while AI was used the possible injection issues don’t seem to come from there.
Yes, I will review the code.
Some reviewer looks hating the fact to use AI. But whatever. I don’t care. 
I trusted Claude, that’s my bad, but I’m not used to work with A.I. I need to learn what is right and wrong with the AI.
The +.
It’s usefull when we are in this situation modernizeing this kind of code.It’s quick,
The -
On another hand, need to spend a max of time to review everything the AI coded, fixed and modified.
With Claude, we can spend much dollars to use it. It’s pretty expensive…
My last post was providing you with a bit of absolution on this. The SQL issues, that I found, exist in the current code. Claude didn’t touch that.
1 Like
Who say modernization, say review as well. It makes sense too. So, I will take a free time to improve the code.
Why not add a unit test, I don’t know. I will see if my mood is good or not. 
I need to test it on 17 by the way.
I appreciate, you are nice.
That works on 17 too.
So one of my back burner thoughts was to do a reporting project for cdr/cel and had Gemini do a research .
2 Likes
@franck.danard consider adding something that can handle a retention policy within the module
Hi @ChrisD, How are you?
What do you mean exactly? I don’t understand.