Update of patch #2050 (project freeciv):
Status: In Progress => Ready For Test
_______________________________________________________
Follow-up Comment #1:
Design decisions:
* It turns out that we only currently need a plural name for specialists --
we don't use a singular anywhere -- so I've co-opted the existing _name_ for
that.
** I've renamed "name" to "plural" in various places to make that clear.
** Adding the plural names fixes an ugliness I introduced in 2.2.x in bug
#15710, in the city report ("Specialists: elvis, scientist, taxman" is now
"Specialists: Entertainers, Scientists, Taxmen").
* Text describing the effects of specialists is hand-written. While it would
probably be fairly easy to autogenerate text describing the effects of
specialists in standard rulesets, it would quickly fail for more exotic
effects such as conditional effects; doing this properly (as with so many
other things) requires something of an overhaul of how auto-generated help
for requirements/effects is written. So I've opted not to try.
* Requirements for specialists to exist at all, however, are autogenerated,
since we can do a fairly good job of that. (Examples can be seen in the
civ1/civ2 rulesets.)
Incidental stuff:
* Since I touched the Citizen Governor help, I've split it up into multiple
strings as is done elsewhere, to reduce the translation burden of future
changes.
As well as making it easier to add new specialists in custom rulesets, all
this fixes a long-standing lie in the built-in help about the effects of
specialists; the description in helpdata.txt wasn't true for the civ1 and
civ2 rulesets.
Not done:
* Sorting out the remaining hardcoding of E/S/T in city options and the
client UI for that. This looks like a biggish job that's best handled by a
separate patch; it will probably need a further network capability change.
(file #10968)
_______________________________________________________
Additional Item Attachment:
File name: trunk-regicide.diff Size:36 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?2050>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
