Re: [OpenSIPS-Devel] [opensips] Add ability to provide db_id to avp_db_query() as either $var or $avp (#694)
@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 directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-156063028___ 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)
@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 directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-156181647___ 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)
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/opensips/issues/694#issuecomment-156427898___ 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)
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)
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.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)
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)
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)
@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. This wa done against commit 0a642c5e8ba315453c064c174f7ae86b8ecdb1a2 . Thanks! [avpops.c.rej.txt](https://github.com/OpenSIPS/opensips/files/94835/avpops.c.rej.txt) --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/issues/694#issuecomment-172619133___ 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)
@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@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)
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)
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/mailman/listinfo/devel