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

predictable_random_int() should include script name #20

Closed
emteknetnz opened this issue Sep 7, 2023 · 1 comment
Closed

predictable_random_int() should include script name #20

emteknetnz opened this issue Sep 7, 2023 · 1 comment
Assignees

Comments

@emteknetnz
Copy link
Member

emteknetnz commented Sep 7, 2023

predictable_random_int() currently randomises based on $MODULE_DIR - it should also be random based on the script that calls it. Could use debug_backtrace() to get the script name that called it. This allows the random ints to be random between scripts within a module

Not that updating this will update all cron times, so it would require a mass update

Do at same time as #21

PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants