#41120: Counter requirement type
Open Date: 2021-01-07 23:24
Last Update: 2022-02-22 10:32
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/41120
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=41120
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2022-02-22 10:32 Updated by: cazfi
Comment:
- You should not set checkpoint to 5 for ALL counters in counters_init(), but set for "Owned" counter only when initializing 'static struct counter counters'
- Document the new requirement type in README.effects ("Requirement types and supported ranges" section)
- Use uppercase boolean macro, instead of assuming actual boolean type, in C-code: false -> FALSE
- req_text_insert(): This should use translated name, not rule name. Same in universal_name_translation(). Unfortunately we don't have translated name yet -> I'll open a new ticket about that in a moment
- Add empty line between variable declarations and code, i.e., after "struct counter *count = req->source.value.counter;"
- No space after ! (not): "if (! target_city) {" -> "if (!target_city) {"
- Indentation wrong in worklist_item_post()
---------------------------------------------------------------------
Ticket Status:
Reporter: cazfi
Owner: (None)
Type: Patches
Status: Open
Priority: 5 - Medium
MileStone: 3.2.0
Component: General
Severity: 5 - Medium
Resolution: None
---------------------------------------------------------------------
Ticket details:
Add a new requirement type 'Counter'. Name of the requirement is name of the counter. Requirement works only at City range for now.
Add new 'checkpoint' int field to struct counter, and set it e.g. 5 for the City Owned counter. Requirement is fulfilled if the value of the requirement is at least checkpoint.
Test for example by introducing a ruleset rule that when city has been owned less than those 5 turns, it suffers extra unhappiness.
--
Ticket information of Freeciv project
Freeciv Project is hosted on OSDN
Project URL: https://osdn.net/projects/freeciv/
OSDN: https://osdn.net
URL for this Ticket:
https://osdn.net/projects/freeciv/ticket/41120
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=41120