[OpenSER-Devel] [ openser-Bugs-1803444 ] Carrier-Route: DB-Schema and cr_tree_rewrite_uri (Fix)

[SourceForge.net]

Bugs item #1803444, was opened at 2007-09-27 12:27
Message generated for change (Comment added) made by carstenbock
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1803444&group_id=139143

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: ver devel
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Carsten Bock (carstenbock)
Assigned to: Henning Westerholt (henningw)
Summary: Carrier-Route: DB-Schema and cr_tree_rewrite_uri (Fix)

Initial Comment:
Hi,

after playing a little bit around with the carrier-route module, i found two little bugs:
1) in the DB-Schema, the strip-parameter was defined as an String, not as an int. The module retrieves the int-value (although it is a string) and this results into a strange behaviour; at least in some cases (my setup: mysql-client 4.1.x and Server 5.x).
See attached patch: carrierroute_db_schema.patch
2) when playing around with "cr_tree_rewrite_uri", i've noticed that
a) it uses the default tree, not the tree provided as parameter.
b) the fixup function return the number in the tree, not the id of the carrier.
The attached patch "carrierroute_tree_route_uri.patch" fixes these issues.

Carsten

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

>Comment By: Carsten Bock (carstenbock)
Date: 2007-09-27 15:39

Message:
Logged In: YES 
user_id=1488991
Originator: YES

Actually more or less not only an fix, but also a feature extension:
the cr_tree_rewrite_uri-method now also accepts pseudo-variables for the
carrier-name.
Tha patch is against the latest SVN-Version.
File Added: carrierroute_tree_route_uri.patch

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

Comment By: Carsten Bock (carstenbock)
Date: 2007-09-27 14:31

Message:
Logged In: YES 
user_id=1488991
Originator: YES

added the fix (as just applied to user_route_fixup in SVN-Tree)
File Added: carrierroute_tree_route_uri.patch

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

Comment By: Carsten Bock (carstenbock)
Date: 2007-09-27 12:27

Message:
Logged In: YES 
user_id=1488991
Originator: YES

File Added: carrierroute_tree_route_uri.patch

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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1803444&group_id=139143