Skip to content

Commit

Permalink
Add update v2.1.1
Browse files Browse the repository at this point in the history
- Controlling Northern Grey Mountains is no longer required by Karak Kadrin to achieve a Short Victory
- Fixed bugs / Minor changes
    - Lyonesse Ultimate Victory conditions remained vanilla
    - Clan Rictus victory conditions were a mix of vanilla and modded
    - Destroying Borderleaux was missing from The Dreadfleet Short Victory conditions
    - Fall of Man Beastmen quest battle popped randomly and multiple times
- Sad reports
    - It is not possible to turn instances of Sack/Raze specific locations into Occupy/Loot/Raze/Sack for non horde 
    factions in order to prevent weird situations where you had to attack your own settlements. The key 
    OCCUPY_LOOT_RAZE_OR_SACK_X_SETTLEMENTS_INCLUDING does not exist.
  • Loading branch information
msolefonte authored Oct 11, 2020
1 parent 3f9bead commit 20b392e
Show file tree
Hide file tree
Showing 5 changed files with 36 additions and 17 deletions.
15 changes: 14 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Controlling Black Arks objectives have been removed
- Redone of The Bloody Handz
- Razing or looting Sartosa is not required to achieve a victory
- Razing or looting Copher, Fyrus and Al Haikk is no longer required to achive a Short Victory
- Razing or looting Copher, Fyrus and Al Haikk is no longer required to achieve a Short Victory
- Having at least 80% Untained in Southern Badlands and Western Badlands is now required to achieve a Short Victory
- Having at least 80% Untained in Easter Badlands, Southern Badlands and Western Badlands is now required to achieve a Long Victory
- Redone of Vor Carstein's Long Victory conditions
Expand All @@ -74,3 +74,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Solved a bug where Itza long victory conditions remained vanilla
- Fixed a bug where Beastmen final battle never happened
- Solved a bug where some unit requirements numbers where weird and different from the docs

## [2.1.1] Minor Update - 10.11.2020

- Controlling Northern Grey Mountains is no longer required by Karak Kadrin to achieve a Short Victory
- Fixed bugs / Minor changes
- Lyonesse Ultimate Victory conditions remained vanilla
- Clan Rictus victory conditions were a mix of vanilla and modded
- Destroying Borderleaux was missing from The Dreadfleet Short Victory conditions
- Fall of Man Beastmen quest battle popped randomly and multiple times
- Sad reports
- It is not possible to turn instances of Sack/Raze specific locations into Occupy/Loot/Raze/Sack for non horde
factions in order to prevent weird situations where you had to attack your own settlements. The key
OCCUPY_LOOT_RAZE_OR_SACK_X_SETTLEMENTS_INCLUDING does not exist.
31 changes: 18 additions & 13 deletions src/script/campaign/mod/vco2_main_listeners.lua
Original file line number Diff line number Diff line change
Expand Up @@ -95,21 +95,26 @@ function add_listeners()
out("#### Adding Victory Conditions Overhaul Listeners ####");
if cm:is_multiplayer() == false then
out("#### Adding Beastmen Victory Conditions Overhaul Listeners ####");
if not cm:get_saved_value("bst_final_battle_quest") then
core:add_listener(
"vco_beastmen_turn_50_start",
"FactionTurnStart",
function(context)
local faction = context:faction();
return faction:is_human() and faction:name() == "wh_dlc03_bst_beastmen" and cm:turn_number() >= 50;
end,
function()
cm:set_saved_value("vco_bst_final_battle_quest", false);
core:add_listener(
"vco_beastmen_turn_50_start",
"FactionTurnStart",
function(context)
local faction = context:faction();
return faction:is_human() and faction:name() == "wh_dlc03_bst_beastmen" and cm:turn_number() == 50;
end,
function()
-- Duplicated because I do not exactly know how this work:
-- - Key bst_final_battle_quest is vanilla and should be false by default
-- - Key vco_bst_final_battle_quest is custom, but I do not know if value is correctly saved
if not (cm:get_saved_value("bst_final_battle_quest") and cm:get_saved_value("vco_bst_final_battle_quest")) then
cm:trigger_mission("wh_dlc03_bst_beastmen", "wh_dlc03_qb_bst_the_final_battle", true);
cm:set_saved_value("bst_final_battle_quest", true);
end,
false
);
end;
cm:set_saved_value("vco_bst_final_battle_quest", true);
end
end,
false
);

out("#### Adding Bretonnia Victory Conditions Overhaul Listeners ####");
core:add_listener(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ local missions = {
type DESTROY_FACTION;
faction wh2_main_lzd_hexoatl;
province wh2_main_nagarythe;
faction wh2_main_hef_nagarythe;
confederation_valid;
}
objective
Expand Down Expand Up @@ -66,7 +66,7 @@ local missions = {
faction wh2_main_lzd_hexoatl;
faction wh2_main_hef_eataine;
province wh2_main_nagarythe;
faction wh2_main_hef_nagarythe;
confederation_valid;
}
objective
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ local missions = {
type DESTROY_FACTION;
faction wh2_dlc11_def_the_blessed_dread;
faction wh_main_brt_bordeleaux;
faction wh2_dlc11_cst_pirates_of_sartosa;
faction wh2_dlc11_cst_vampire_coast;
faction wh2_dlc11_cst_the_drowned;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ local missions = {
province wh_main_zhufbar;
province wh_main_rib_peaks;
province wh_main_gianthome_mountains;
province wh_main_northern_grey_mountains;
}
objective
{
Expand Down

0 comments on commit 20b392e

Please sign in to comment.