Skip to content

Fix AbstractAirspace setDragEnabled honoring input#288

Open
arpitjain099 wants to merge 1 commit into
NASAWorldWind:developfrom
arpitjain099:chore/fix-abstractairspace-drag-toggle
Open

Fix AbstractAirspace setDragEnabled honoring input#288
arpitjain099 wants to merge 1 commit into
NASAWorldWind:developfrom
arpitjain099:chore/fix-abstractairspace-drag-toggle

Conversation

@arpitjain099

Copy link
Copy Markdown

This fixes a logic bug in AbstractAirspace where setDragEnabled(boolean enabled) always set the internal field to true and ignored the method argument.

Changes:

  • set dragEnabled to the passed enabled value in AbstractAirspace
  • add a focused JUnit regression test class that verifies both false and true paths

Files:

  • src/gov/nasa/worldwind/render/airspaces/AbstractAirspace.java
  • test/gov/nasa/worldwind/render/airspaces/AbstractAirspaceTest.java

Fixes #240

Verification:

  • I validated the logic change by adding JUnit coverage for both branches.
  • I could not run the test suite in this environment because Java runtime/compiler tools are not installed (java and javac are unavailable).

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

AbstractAirspace setDragEnabled always set to true

1 participant