List Info

Thread: Carrier-Route: DB-Schema and cr_tree_rewrite_uri (Fix)




Carrier-Route: DB-Schema and cr_tree_rewrite_uri (Fix)
country flaguser name
United States
2007-09-27 07:27:06
Bugs item #1803444, was opened at 2007-09-27 12:27
Message generated for change (Tracker Item Submitted) made
by Item Submitter
You can respond by visiting: 
https://sourcefo
rge.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: Nobody/Anonymous (nobody)
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

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

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

_______________________________________________
Devel mailing list
Developenser.org
htt
p://openser.org/cgi-bin/mailman/listinfo/devel

[1]

about | contact  Other archives ( Real Estate discussion Medical topics )