Update of bug #23775 (project freeciv):

                  Status:                    None => Ready For Test         
         Planned Release:                         => 2.6.0, 3.0.0           

    _______________________________________________________

Follow-up Comment #1:

That's because get_infrastructure_text() lists only "Infrastructure" extras.

The whole use of get_infrastructure_text() here seems pointless when we have
extras API (2.6 and up). The only advantages of get_infrastructure_text() over
direct extra_name_translation() is that it can format text for multiple extras
(here we always use just one), it checks for hidden_by functionality (we don't
need/want that here), and it filters input to get Infrastructure extras only
(we don't want it here - it's the cause of this bug)

Fix attached



(file #26045, file #26046)
    _______________________________________________________

Additional Item Attachment:

File name: PillageTargetName.patch        Size:2 KB
File name: PillageTargetName-S2_6.patch   Size:3 KB


    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?23775>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to