Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transitional animal randomization fix for 1.48/1.50. Fixes #114. #118

Merged
merged 1 commit into from
Aug 15, 2015

Conversation

robalo
Copy link
Contributor

@robalo robalo commented Aug 15, 2015

No description provided.

class Snake_random_F /* : Animal_Base_F */ {
SLX_BIS = "_FRnd = compile loadFile ""\A3\animals_f\Snakes\scripts\randomize.sqf"";(_this select 0) call _FRnd;";
class Cock_random_F /* : Fowl_Base_F */ {
SLX_BIS = "_FRnd = compile preprocessFile ""\A3\animals_f_beta\Chicken\scripts\randomize.sqf"";(_this select 0) call _FRnd;";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cock_random_F is not randomized in 1.48 (it does not have an init event handler). We can still add it into a pre-A3 1.50 XEH release, but with a forward-looking XEH init event handler - the one that the animal will have in 1.50: (_this select 0) call bis_fnc_animalRandomization;

That way it will do nothing in 1.48 and work as intended in 1.50.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does actually work with 1.48, looks like it was missed by BIS.

Killswitch00 added a commit that referenced this pull request Aug 15, 2015
Transitional animal randomization fix for 1.48/1.50. Fixes #114.

We will keep the Cock_random_F randomization for 1.48.
@Killswitch00 Killswitch00 merged commit f53b5f6 into master Aug 15, 2015
@Killswitch00 Killswitch00 deleted the interim_animal_fix branch August 17, 2015 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants