1. OSBot Version (do NOT put "current version", be specific): 2.4.107

 

2. A description of the issue. Include relevant logs.: If you define an Area with a Position array, and you were to call area#getPlane, it would return a plane of 0 regardless of what the actual plane is.

 

3. Are you receiving any errors in the client canvas or the logger?: No.

 

4. How can you replicate the issue?: The code below would return a plane of 0 instead of 1.

    private Area test_area = new Area(new Position[]{

            new Position(-1, -1, 1),

            new Position(-1, -1, 1),

            new Position(-1, -1, 1),

            new Position(-1, -1, 1)

    });

test_area.getPlane();

5. Has this issue persisted through multiple versions? If so, how far back?: Undetermined

Referencing the API docs here states

 

Note: By default the plane is set to 0

Though as you've probably noticed, you can use setPlane() to fix the plane.

 

I would not be opposed to adding checks to the constructors and only allowing successful creation if all the Positions were on the same plane. Would a change like this introduce bugs to existing scripts? If so, then instead of throwing an error, we can potentially force set the plane to 0.

 

Thoughts anyone?

The method which you are using does not set plane by default, however Area(Position, Position) does. I only see four points in your polygon, so perhaps you can use that constructor? Otherwise, setPlane() is the correct method (returns a new Area instance). I'll run my tools to see which scripts will be affected and make the change if it's relatively safe.