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
Make player walkthrough optional #1301
Conversation
@@ -825,7 +825,7 @@ bool Player::canWalkthrough(const Creature* creature) const | |||
} | |||
|
|||
const Player* player = creature->getPlayer(); | |||
if (!player) { | |||
if (!player || g_config.getBoolean(ConfigManager::ALLOW_WALKTHROUGH)) { | |||
return false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't be !g_config.getBoolean(ConfigManager::ALLOW_WALKTHROUGH)?
True that. Fixed. |
I feel like this should wait until move events are implemented. |
I agree with @EvanMC. |
I agree with @marksamman that agrees with @EvanMC |
No problem in leaving this PR open? :) |
@@ -24,6 +24,7 @@ maxPlayers = 0 | |||
motd = "Welcome to The Forgotten Server!" | |||
onePlayerOnlinePerAccount = true | |||
allowClones = false | |||
allowWalkthrough = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be false by default? Sefaults should resemble vanilla in my opinion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, in Tibia players are walkthroughable by default. This is not a TFS-only feature, maybe you didn't know it has been implemented in Tibia :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WHAT? Oh hell no, sorry I got stuck in 7.4 apparently.
Sorry man
7a004ab
to
223fd3a
Compare
Fix inverted condition
223fd3a
to
5ff1c2f
Compare
I have just stumbled across this limitation, that the walkthrough could be optionally enabled at config.lua, and done the patch :)