Solved small issue with a message not being logged due to early return

[FlorisE]

Description

Tiny change. Due to wrong ordering of two lines in MoveItCpp::loadPlanningPipelines the function will return false when no planning pipelines are present, without logging this specific message.

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

[codecov]

Codecov Report

Merging #368 (b1ee767) into main (20c7d33) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #368      +/-   ##
==========================================
+ Coverage   53.24%   53.26%   +0.02%     
==========================================
  Files         209      209              
  Lines       18856    18856              
==========================================
+ Hits        10038    10041       +3     
+ Misses       8818     8815       -3     

Impacted Files	Coverage Δ
...nning_scene_monitor/src/planning_scene_monitor.cpp	56.82% <0.00%> (+0.46%)	⬆️

---

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 20c7d33...b1ee767. Read the comment docs.

[brookdpg]

Ideally your IDE should tell you when code is unreachable.