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

"override class restriction" argument for ForceRTD does not seem to provide much value #90

Closed
6 of 9 tasks
Tiagoquix opened this issue Dec 22, 2023 · 2 comments
Closed
6 of 9 tasks
Labels
bug Something isn't working
Milestone

Comments

@Tiagoquix
Copy link
Contributor

Bug description

When using sm_forcertd, whetever I specify "override class restriction" or not the effect will always be applied regardless of my class. From my point of view, there is no need to have it in sm_forcertd — you would know if you applied a non-intended effect for your class via admin commands.

This is always reproducible, so this is systematic.

RTD version: 2.4.0

Report checks

Please describe the bug in the section above, then post it and tick the correct boxes below.

Required

  • The bug does not have an open issue.
  • I have described the bug.
  • I mentioned whether the bug is sporadic or systematic.

Optional

  • Steps to reproduce the bug are known and have been described.
  • I have access to server error logs and have provided them, if any.
  • I am able to check the RTD version (sm_cvar sm_rtd2_version) and have specified it.
  • I have access to RTD default configuration file (rtd2_perks.default.cfg) and can confirm it's the one bundled with plugin release archive.
  • I have access to RTD custom configuration file (rtd2_perks.custom.cfg) and have attached it to this issue. -- NO CUSTOM CONFIGS PRESENT
  • The issue is related to a particular perk, which has been specified by a label.
@Tiagoquix Tiagoquix added the bug Something isn't working label Dec 22, 2023
@Phil25
Copy link
Owner

Phil25 commented Dec 22, 2023

I agree, even forgot this existed lol

@Phil25
Copy link
Owner

Phil25 commented Dec 23, 2023

I was randomly browsing old changelog and it looks like this argument has been removed all the way back in 2018. The command description has been outdated since then.

@Phil25 Phil25 added this to the 2.5.0 milestone Dec 27, 2023
@Phil25 Phil25 closed this as completed in 3828da8 Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants