Re: [Plastic-devs] VOQuest + PLASTIC
Brought to you by:
johndavidtaylor,
thomasboch
|
From: Mark T. <m.b...@br...> - 2006-05-09 12:56:20
|
On Tue, 9 May 2006, Mark Taylor wrote: > On Tue, 9 May 2006, Thomas Boch wrote: > > > > > > > With hindsight I'd say that the way the change was made was suboptimal - > > > if the parameter list change had been > > > > > > (String votable) -> (String votable, String id) > > > > > > rather than > > > > > > (String votable) -> (String id, String votable) > > > > Note that the current version of Aladin incorrectly expects that the > > parameter list is (String votable, String id) ... > > I didn't realise that - in that case since we (well, I) agree it's > better that way, and that's what Aladin does, it might be best > to declare that the bug is in the specification and just change that. > Is there code anywhere which uses this method in the declared > (String id, String votable) form? ... I'll take that as a no for now. If nobody contradicts, then John can you modify the ivo://votech.org/votable/load message arguments in coremessages.html so that they read (votable:String, id:String) rather than the other way around. TOPCAT now tolerates an optional second argument for this message - pre-release version at ftp://andromeda.star.bris.ac.uk/pub/star/topcat/topcat-full.jar Mark -- Mark Taylor Astronomical Programmer Physics, Bristol University, UK m.b...@br... +44-117-928-8776 http://www.star.bris.ac.uk/~mbt/ |