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

"Blocked by" vehicle setting does nothing #764

Closed
ohrrpgce-bugbot opened this issue Oct 17, 2009 · 8 comments
Closed

"Blocked by" vehicle setting does nothing #764

ohrrpgce-bugbot opened this issue Oct 17, 2009 · 8 comments
Labels
needs improvement It ain't completely broken... but it ain't quite the way it should be yet rel: zenzizenzic Present in zenzizenzic 2011-05-06 resolved: invalid Not a real bug report/feature request vehicles

Comments

@ohrrpgce-bugbot
Copy link

[bz#764]

The "blocked by" setting should prevent a vehicle from passing into a tile, but it apparently does nothing.

I have only tested this with the "Pass through walls" bitset enabled. It might work with that bitset disabled, but it is obviously intended to work with that bitset enabled.

We should really have a "vehicles" component in Bugzilla.

From: @arperry
Reported version: 20110506 Zenzizenzic
Operating system: Windows Vista
Severity: major

@ohrrpgce-bugbot
Copy link
Author

Comment author: @bob-the-hamster

I can't completely blame this one on the cat. I intentionally designed the "Pass through walls" bitset to ignore ALL passability. I don't want to change it, for fear of breaking edge-cases in existing games.

But this is a bug.

I propose renaming the "Pass through walls" bitset to something like "Always ignore all passaibility" or "Never blocked by walls or tiles". Something that really accurately describes the way it works now.

Then we add a new "Pass through walls" bitset that ignores regular walls, but does NOT ignore "blocked by"

@ohrrpgce-bugbot
Copy link
Author

Comment author: @bob-the-hamster

It is likely that the same problem that caused bug bz⁠#770 would have also messed up vehicle's "blocked by" checking

@ohrrpgce-bugbot
Copy link
Author

Comment author: @rversteegen

I must have just fixed this (the passmap probe in want_to_check_for_walls), however I can't seem manage to reproduce the bug in older builds anyway.

Is this a two part bug report?

@ohrrpgce-bugbot
Copy link
Author

Comment author: msw188

What is the status on this one? Is it safe to call this one fixed, or should we leave it open as a feature request for a "pass through walls but not through blocking tiles"?

@ohrrpgce-bugbot
Copy link
Author

Comment author: @rversteegen

I must have just fixed this (the passmap probe in want_to_check_for_walls),

(I had spotted some typo in the vehicle passability code that was disabling some check.) I'm confused by James saying the behaviour was intentional, and then contradicting himself in comment 2, followed by the discovery of the typo. I think we ought to do some testing.

@ohrrpgce-bugbot
Copy link
Author

Comment author: @bob-the-hamster

I'm confused by James saying the behaviour was intentional, and
then contradicting himself in comment 2, followed by the discovery of the typo.
I think we ought to do some testing.

It is entirely possible that I am confused, yes :)

@ohrrpgce-bugbot
Copy link
Author

Comment author: @rversteegen

      What    |Removed                     |Added

----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |TOOCONFUSING

@rversteegen
Copy link
Contributor

Note: my comment above was a joke. Also, this bug report was a followup to an email Adam had sent to the mailing list titled "Unintuitive "override walls" setting". to which James responded

I can explain my rationale for how vehicle blocking work: I left my
computer on and fell asleep, and the cat implemented it by walking on
the keyboard.

This report complains that the vehicle "blocked by" setting is overridden by "Pass through walls". We agreed (and I still agree) that's not a bug, and the solution to the mess is something else, so I'm closing this. I filed #1144 for getting rid of the "blocked by" setting.

@rversteegen rversteegen added needs improvement It ain't completely broken... but it ain't quite the way it should be yet resolved: invalid Not a real bug report/feature request and removed bug Yeah... that's broken labels Aug 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs improvement It ain't completely broken... but it ain't quite the way it should be yet rel: zenzizenzic Present in zenzizenzic 2011-05-06 resolved: invalid Not a real bug report/feature request vehicles
Projects
None yet
Development

No branches or pull requests

2 participants