[Fixed] Sight upgrade won't upgrade detection radius

[lazyanttu]

Well, hello again . I encountered a new (perhaps bug, or I'm doing something wrong) problem with cloak/detection.

One of my factions' main base has detection ability. The detection = unit sight, I think. Well, there was an upgrade which greatly increase main buildings' sight range. But when the bot attacked me with cloaked units, they weren't detected until they get very close (which seemed to be the old detection radius).

Well, I tested it out and gave the faction 10 cloaked units at start and caused the sight upgrade to be so great that it would turn the whole map visible. But I couldn't see the cloaked units. Well, I attacked with my own cloak units and noticed that they could detect them only near their main base (=old range).

So, upgrading sight didn't upgrade detection radius. Is there a way to upgrade detection radius, as it was the upgrades main point?

[silnarm]

Yep, that's a bug alright  

You assumed correctly, it is meant to be tied to sight.

Thanks for the top-notch bug report

Edit: Fixed with commit d59c4b4..

[lazyanttu]

How can I get the fixed version? I think that I read about some kind of version control system.

EDIT: Sorry, I didn't noticed 0.4 beta2 (https://forum.megaglest.org/index.php?topic=7278.0). Is it correct? And what means commit d59c4b4?
Well, I'm not quite professional with coding, sorry for bothering .

[silnarm]

Yes, the new beta has the fix

The commit hash (d59c4b4..) identifies which code revision the fix was in, you don't really need to worry about it, its more for future reference for us, if it appears the fix is not working correctly then we can go straight to the revision and double-check the changes (or just revert them and try again ).