bugFreeciv - Bugs: bug #23361, civ2civ3: sea unit help talks...

 
 
Show feedback again

bug #23361: civ2civ3: sea unit help talks about ZOC

Submitted by:  Jacob Nevins <jtn>
Submitted on:  Sat Mar 7 22:10:37 2015  
 
Category: rulesetsSeverity: 2 - Minor
Priority: 5 - NormalStatus: None
Assigned to: NoneOpen/Closed: Open
Release: Operating System: Any
Planned Release: 2.6.0, 3.0.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon Mar 9 22:28:46 2015, comment #2:

>...where sea units were given ZOC so that a single-line ruleset change could enable naval ZOC in deep ocean.

Right, my plan was to enable zoc in deep ocean, but I have not played v2.6 yet, and I'm still not sure if it is a good idea or not.

Feel free to remove all the ZoC stuff related to sea units.

David Fernandez <bardo>
Sat Mar 7 22:26:16 2015, comment #1:

> the simpler thing would be to comment out all the sea unit ZOC
> stuff


Doesn't improve support for custom rulesets, though.

> something complicated to suppress the ZOC words for units only
> native to NoZoc terrain,


We already do checking of native terrains in some situations, i.e., we have some tools for this, and there will be more in the future, i.e., additional tools made for this will be useful for other things in the future.

Marko Lindqvist <cazfi>
Project Administrator
Sat Mar 7 22:10:37 2015, original submission:

e.g., Frigate:

However, I don't think ZOC are in practice ever relevant for such units, due to water terrains having the NoZoc flag. (The only case where boats can be on land tiles, moving in and out of cities, is a special case for ZOC anyway.)

I think this is from patch #4818, where sea units were given ZOC so that a single-line ruleset change could enable naval ZOC in deep ocean.

While we could do something complicated to suppress the ZOC words for units only native to NoZoc terrain, the simpler thing would be to comment out all the sea unit ZOC stuff (with a greppable tag like NAVAL_ZOC so it can easily be added back in).

(Triremes have similar words, but they are genuinely relevant to Triremes due to river movement.)

Jacob Nevins <jtn>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Digest:
   patch dependencies.

 

Carbon-Copy List
  • -unavailable- added by bardo (Posted a comment)
  • -unavailable- added by cazfi (Posted a comment)
  • -unavailable- added by jtn (ruleset author)
  • -unavailable- added by jtn (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Error: not logged in

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat Mar 7 22:10:54 2015jtnCarbon-Copy-=>Added bardo
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup