BNHM / berkeleymapper

Map specimens using simple XML configuration file and Google Maps API
GNU General Public License v3.0
8 stars 2 forks source link

Fix linkback method #47

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
-----Original Message-----
From: Martin Pullan [mailto:M.Pullan@rbge.ac.uk]
Sent: Friday, September 21, 2012 7:18 AM
To: Les, Donald
Subject: RE: Mapper again...

Hi Les

It is a bit ugly bit it works - the issue was that the linkback was forming the 
query part of the link back url by putting an & after the url in the xml config 
file then putting in the value of the specified field in order to parameterise 
the query.

The way we had it set up was with the url

http://rbg-web2.rbge.org.uk/cgi-bin/nph-readbtree.pl/allspecs

which they transfomed into

http://rbg-web2.rbge.org.uk/cgi-bin/nph-readbtree.pl/allspecs&ETI={{BARCODE}}

which would not work because the & should be a ?

So I changed the config to

http://rbg-web2.rbge.org.uk/cgi-bin/nph-readbtree.pl/allspecs?

Which they transform into

http://rbg-web2.rbge.org.uk/cgi-bin/nph-readbtree.pl/allspecs?&ETI={{BARCODE}}

which is ugly but it works and should be

http://rbg-web2.rbge.org.uk/cgi-bin/nph-readbtree.pl/allspecs?ETI={{BARCODE}}

This may be because of the way the config file is set up but the documentation 
is so limited that it is hard to tell if that is the case.

You may want to pass this back to the Berkely mappper people because I think it 
may be a bug in their code whcih should but does not check to see if the url 
already contains a query string - if not they should construct the link back 
differently.

Original issue reported on code.google.com by jdec...@gmail.com on 23 Sep 2012 at 1:46

GoogleCodeExporter commented 9 years ago
jdeck fixed this

Original comment by jdec...@gmail.com on 25 Sep 2012 at 11:20