Skip to content

[BUG]: The force parameter of the SetDestination() method for the Elevator LabAPI wrapper is reversed #319

@KindaGoodCoder

Description

@KindaGoodCoder

Before You Report

  • I have searched for existing reports of this bug, and this bug report is unique.

Version

1.1.4

Description

The LabAPI Wrapper for SetDestination() has the parameter force which is documented as choosing whether the destination should be changed even if the elevator is not ready. However, the base ElevatorChamber method has the parameter allowQueuing which is the opposite of forcing the elevator.
The LabAPI wrapper passes the value of force directly into the allowQueuing parameter, resulting in the opposite of the desired behaviour.

To Reproduce

  1. Elevator.SetDestination(Elevator.NextDestinationLevel, true)

Result:
Elevator will be queued instead of immediately changing directions

Expected Behavior

The elevator will immediately switch directions when the method is called when the force parameter is equal to true.

Additional Information

N/A

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions