Complete.Org: Mailing Lists: Archives: freeciv-dev: July 2004:
[Freeciv-Dev] Re: (PR#8754) effects patch
Home

[Freeciv-Dev] Re: (PR#8754) effects patch

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: vasc@xxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#8754) effects patch
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 21 Jul 2004 13:31:18 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=8754 >

This code:

+       notify_player_ex(pplayer, -1, -1, E_TECH_GAIN,
+           _("Game: %s acquired from The Great Library!"),
+           advances[i].name);
+       gamelog(GAMELOG_TECH, _("%s discover %s (Library)"),
+           get_nation_name_plural(pplayer->nation), advances[i].name);
+       notify_embassies(pplayer, NULL,
+           _("Game: The %s have acquired %s"
+             " from the Great Library."),
+           get_nation_name_plural(pplayer->nation),
+           advances[i].name);

should not hard-code "Great Library" or "(Library)".  Somehow we need to 
find the source of the effect and tell that instead.

This would help with aqueduct/sewer system as well.  You changed "Needs 
aqueduct" to "Needs a building".  Better would be to find the next 
available source and name it.

jason




[Prev in Thread] Current Thread [Next in Thread]