From 60709393692d3e50630639ad391e4c37dbeca006 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ian@liberator.relativity.greenend.org.uk>
Date: Thu, 22 Oct 2009 00:05:18 +0100
Subject: [PATCH] If routetrade query is arbitrage search only, mention this
 fact

---
 yarrg/TODO           | 4 ----
 yarrg/web/docs       | 2 +-
 yarrg/web/routetrade | 9 +++++++++
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/yarrg/TODO b/yarrg/TODO
index 101af03..8e85c93 100644
--- a/yarrg/TODO
+++ b/yarrg/TODO
@@ -22,10 +22,6 @@ support Opal and Jade (currently there are some unicode problems)
 WEBSITE
 -------
 
-trades for route with an archipelago should produce
-  a message saying `searched for arbitrage trades only in
-  Ruby' or some such.
-
 allow unticking based on minimum margin or minimum profit
 
 initial/final stocks feature
diff --git a/yarrg/web/docs b/yarrg/web/docs
index c211683..9feaa6b 100755
--- a/yarrg/web/docs
+++ b/yarrg/web/docs
@@ -89,7 +89,7 @@ potentially profitable trades.  If the route is suitable for the trade
 route optimiser, it will generate a complete voyage plan, telling you
 which goods to buy and sell where at which stalls and prices.
 
-<p>
+<p><a name="arbitrage"></a>
 
 If you specify only one island or one archipelago, the site shows only
 arbitrage trades.  If you want single-hop trades within an
diff --git a/yarrg/web/routetrade b/yarrg/web/routetrade
index 68c727c..68f9d27 100644
--- a/yarrg/web/routetrade
+++ b/yarrg/web/routetrade
@@ -261,6 +261,15 @@ foreach my $v (qw(MaxMass MaxVolume)) {
 <& dumptable:end, qa => $qa &>
 % }
 
+% if (@islandids==1) {
+% 	if (defined $islandids[0]) {
+Searched for arbitrage trades only.
+%	} else {
+Searched for arbitrage trades only, in <% $archipelagoes[0] |h %>
+<a href="docs#arbitrage">[?]</a>.
+%	}
+% }
+
 <%perl>
 
 my @sail_total;
-- 
2.30.2