Fix PlanarJointModel::satisfiesPositionBounds

[rcywongaa]

Description

Fixes PlanarJointModel::satisfiesPositionBounds to correctly check for all 3 positions instead of the 0-th position 3 times.

Replaces #1327

Checklist

• Required by CI: Code is auto formatted using clang-format
• Extend the tutorials / documentation reference
• Document API changes relevant to the user in the MIGRATION.md notes
• Create tests, which fail without this PR reference
• Include a screenshot if changing a GUI
• While waiting for someone to review your request, please help review another open pull request to support the maintainers

[JafarAbdi]

Thanks for the fix

[JafarAbdi]

@rcywongaa Do you mind backporting it to https://github.com/ros-planning/moveit?

[rcywongaa]

@JafarAbdi Done 😄

[codecov]

Codecov Report

Merging #1353 (d06d0cb) into main (990925e) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1353      +/-   ##
==========================================
+ Coverage   61.56%   61.57%   +0.01%     
==========================================
  Files         274      274              
  Lines       24966    24966              
==========================================
+ Hits        15369    15370       +1     
+ Misses       9597     9596       -1     

Impacted Files	Coverage Δ
moveit_core/robot_model/src/planar_joint_model.cpp	84.12% <100.00%> (ø)
...ma_kinematics_plugin/src/lma_kinematics_plugin.cpp	73.46% <0.00%> (-3.70%)	⬇️
...nning_scene_monitor/src/planning_scene_monitor.cpp	45.73% <0.00%> (+0.44%)	⬆️
...dl_kinematics_plugin/src/kdl_kinematics_plugin.cpp	76.43% <0.00%> (+1.08%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 990925e...d06d0cb. Read the comment docs.