[OpenSIPS-Devel] [ opensips-Bugs-2187089 ] drouting dr_rule table column groupid wrong type?

[SourceForge.net]

Bugs item #2187089, was opened at 2008-10-22 18:49
Message generated for change (Comment added) made by bogdan_iancu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=2187089&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: trunk
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Lucian Marginean (voicemailtel)
Assigned to: Bogdan-Andrei Iancu (bogdan_iancu)
Summary: drouting dr_rule table column groupid  wrong type?

Initial Comment:
Hi, 
on module documentation http://www.opensips.org/html/docs/modules/devel/drouting.html page I see that on Table 1.5. "Sample dr_rules records" that on column group is used a text "postpaid" or "prepaid". 
 
I try to setup like in example but I get this error when I put any text on groupid column 
 
Oct 22 07:31:21 vmbox1 /sbin/opensips[17385]: ERROR:drouting:add_rule: bad grp id 't' (0)[test] 
Oct 22 07:31:21 vmbox1 /sbin/opensips[17385]: ERROR:drouting:dr_load_routing_info: failed to add rule -> skipping 
 
Where is the problem? Wrong documentation or module is not well implemented? 
 
With numberic values is working fine all the time. 

----------------------------------------------------------------------

>Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2008-10-24 10:30

Message:
The definition is correct, but the description is not - the groupid field
is a string and contains a comma separated list of numerical ids
(1,5,200,3). I updated the docs to be more clear on this.

The DB schema doc contains the correct definition of the table:
   
http://www.opensips.org/html/docs/db/db-schema-devel.html#GEN-DB-DR-RULES


Regards,
Bogdan

----------------------------------------------------------------------

Comment By: Lucian Marginean (voicemailtel)
Date: 2008-10-22 22:35

Message:
I think also the column type should be changed to some numeric type, now is
varchar 255.

----------------------------------------------------------------------

Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2008-10-22 19:18

Message:
The docs are wrong - I will update them asap.

Thanks for the report,
Bogdan

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=2187089&group_id=232389