real_tabasco_sauce Posted August 17 Report Share Posted August 17 (edited) I am trying to get a technology to enable an aura in the project @wowgetoffyourcellphone and I are working on. I see in (https://code.wildfiregames.com/D962?id=3889#change-h6zNMxpe2m5x) that a technology could enable an aura. However it looks like this no longer works: https://code.wildfiregames.com/source/0ad/browse/ps/trunk/binaries/data/mods/public/globalscripts/Templates.js#:~:text=rP24290-,/**,},-rP18733 https://code.wildfiregames.com/D4093#:~:text=// Auras don't have,.reqs) Am I missing something, or was this capability lost in the efforts to reorganize aura and technology simulation? Any ideas on what it would take to bring it back? At the same time, we have got this which seems to indicate auras do support required technologies: https://code.wildfiregames.com/source/0ad/browse/ps/trunk/binaries/data/mods/public/simulation/components/Auras.js#:~:text=Auras.prototype.CanApply,}%3B Edited August 17 by real_tabasco_sauce Quote Link to comment Share on other sites More sharing options...
Stan` Posted August 17 Report Share Posted August 17 When you say it does not work, do you get errors, or does CanApply ignore the requiredtech ? From this line of code, it seems it should still be supported https://code.wildfiregames.com/source/0ad/browse/ps/trunk/binaries/data/mods/public/simulation/components/Auras.js$144 1 Quote Link to comment Share on other sites More sharing options...
real_tabasco_sauce Posted August 17 Author Report Share Posted August 17 "affects": [ "Human" ], "affectedPlayers": [ "Enemy" ], "requiredTechnology": "healer_aura", "modifications": [ { letting the aura require 'healer_aura', a tech for this particular unit, does not work. No errors are thrown. It just fails to limit the aura until after researching the tech. Quote Link to comment Share on other sites More sharing options...
Stan` Posted August 17 Report Share Posted August 17 Can you add some logs in the can apply method of aura.js ? 1 Quote Link to comment Share on other sites More sharing options...
real_tabasco_sauce Posted August 17 Author Report Share Posted August 17 <p class="warning">WARNING: there is a required technology</p> <p class="warning">WARNING: there is a required technology</p> <p class="warning">WARNING: false</p> <p class="warning">WARNING: there is a required technology</p> <p class="warning">WARNING: true</p> Ok, so it does work, its just the tooltips that are not accounted for. So I think the solution at least for now is to just use some text to indicate that a tech is needed for the aura to take effect. 1 Quote Link to comment Share on other sites More sharing options...
Stan` Posted August 18 Report Share Posted August 18 Also if you can fix it for SVn rather than DE both can benefit. Quote Link to comment Share on other sites More sharing options...
real_tabasco_sauce Posted August 18 Author Report Share Posted August 18 Well, I added the text just to this particular aura description since its the only one that requires a tech in the game. I tried for a bit to come up with a lasting solution to no avail. Quote Link to comment Share on other sites More sharing options...
Stan` Posted August 18 Report Share Posted August 18 Well then maybe create a ticket Quote Link to comment Share on other sites More sharing options...
wowgetoffyourcellphone Posted August 19 Report Share Posted August 19 So, adding a description makes it work? Quote Link to comment Share on other sites More sharing options...
real_tabasco_sauce Posted August 19 Author Report Share Posted August 19 On 19/08/2024 at 1:51 AM, wowgetoffyourcellphone said: So, adding a description makes it work? Expand Haha no, I'd like to think that. It was actually working after all, its just the Aura tooltip is there whether or not the aura is actually in effect. Quote Link to comment Share on other sites More sharing options...
Stan` Posted August 19 Report Share Posted August 19 It might just be missing a if check 1 Quote Link to comment Share on other sites More sharing options...
wowgetoffyourcellphone Posted August 19 Report Share Posted August 19 On 19/08/2024 at 2:34 AM, real_tabasco_sauce said: Haha no, I'd like to think that. It was actually working after all, its just the Aura tooltip is there whether or not the aura is actually in effect. Expand Right, the aura is already there in the unit's UI, it's just activated by the tech. So, there's no "bug" here per se, the tooltip code needs extended or something. 1 Quote Link to comment Share on other sites More sharing options...
real_tabasco_sauce Posted August 19 Author Report Share Posted August 19 Yeah I think its just a matter of where to place said if check. I'll raise an 'issue' or however it will be done on the git instance. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.