Kaiserreich / Kaiserreich-4-Bug-Reports

Issue tracker for Kaiserreich for Hearts of Iron 4
110 stars 40 forks source link

Unconverted Encryption and Decryption Bonuses #6901

Closed Myrten closed 4 years ago

Myrten commented 4 years ago

Explanation of the issue: There are quite a lot of national spirits with bonuses to encryption/decryption or encryption/decryption techs. I'm pretty sure they no longer do anything for people with LaR so they need to be replaced with other effects.

My quick search found them in these idea files: image

In vanilla US idea office_of_strategic_services got two separate versions for players with and without LaR so I guess that's how it should be changed.

Checklist

Zankoas commented 4 years ago

Should be 110% they do nothing with LaR, but if that is the case those should be changed. The vast vast majority of our players play with the DLC and so that is who we should target

Vidyaorszag commented 4 years ago

Fixed FRA and WHR

Alpinia commented 4 years ago

Fixed SZC, RUS, USA, TEX, CSA.

Alpinia commented 4 years ago

Fixed SWE, FIN, DEI, BRA, URG, MON.

Zankoas commented 4 years ago

Note that @BugBot3000 has found it still does have some effect, it's just a bit odd:

image

Doesn't seem it is hooked in too well, so I think we are still go with conversation

Myrten commented 4 years ago

Note that @BugBot3000 has found it still does have some effect, it's just a bit odd:

image

Doesn't seem it is hooked in too well, so I think we are still go with conversation

Are you sure that's not from decrypting cypher in Cryptology department?

BugBot3000 commented 4 years ago

@Myrten nah I didn't even take the thing to be able to decrypt anything, setting aside I'm the Zhili Clique and wouldn't decipher Germany. I could see all focus trees with my decryption +1 where the country did not have a minister or any other buff towards decryption. My guess is that this is how vanilla without La Resistance works and all PDX did was hide the techs if you have La Resistance.

Pietrus69 commented 4 years ago

If non-dlc owners can get access to the new modifiers, I would rather just keep it how it is for the DR.

PPsyrius commented 4 years ago

If the older bonus still affects stuff then there's no need to change out the minister effects I guess. Making two separate version of certain ministers would be an ass to maintain too.

Zankoas commented 4 years ago

Wait wait nono; not having a system for all nations apart from DR, that's not how standards work.

If you want to code in DLC checks for DR, that's up to you @Pietrus69, but if not you'll need to change the effects to be something else.

Zankoas commented 4 years ago

@PPsyrius Totally agree, so I think the solution there is to change the minister effects to be something that works fine with or without the DLC. I'll take a look and see if anything fits well

Zankoas commented 4 years ago

And ministers sorted