Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2015-11-12 Thread Ionut Ionita
@mishehu would you need both string and int values for the db id in a variable? I mean it's enough to have something like ``` $var(id)=1; avp_db_query(, "$var(id)"); ``` or you would also need something like ``` $var(id) = "1"; avp_db_query(, "$var(id)"); ``` --- Reply to this email d

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2015-11-12 Thread Yossi
@ionutrazvanionita - As far as how I'm going to use it, I will be using only an integer value. But to be honest, maybe other people will find it useful to accept a string as well? I'm not sure. Integer is fine for me, but I'll defer to your better judgment. :-) --- Reply to this email direc

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2015-11-13 Thread Ionut Ionita
That's why i asked, because i didn't know how you will use it. Anyhow, i made it work both use. Please check if everything is ok https://github.com/OpenSIPS/opensips/commit/8a512731df99621025744aaa25b17931759b91d4 --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2015-11-20 Thread Daniel
This works great --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-158557673___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2015-11-20 Thread Yossi
Yep, it's working great for us, thanks for making the change! --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-158572753___ Devel mailing list Devel@lists.opensips.org http://lists.opensi

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2015-11-20 Thread Yossi
Closed #694. --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#event-471046158___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2016-01-18 Thread Yossi
Reopened #694. --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#event-518247989___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2016-01-18 Thread Yossi
@ionutrazvanionita : There seems to be a problem at the current time. We're using this patch on the 2.1 branch, and it doesn't look like it was backported to 2.1. Can you please verify? The patch you previously provided works for 4 out of 5 hunks. There is an avpops.c.rej that I'm attaching

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2016-01-19 Thread Ionut Ionita
@mishehu done https://github.com/OpenSIPS/opensips/commit/d0f5c3d48d26aa9f9b50ac15905a959b6058a913 --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-172776445___ Devel mailing list Devel@

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2016-02-26 Thread Yossi
Closed #694. --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#event-568044243___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)

2016-02-26 Thread Yossi
Sorry, missed closing this out for you. :-) --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-189438297___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/ma