Bug: set_parameter will ignore everything but the first parameter

[mojomex]

Some callbacks for parameter changes (registered via rclcpp::Node::add_on_set_parameters_callback) use short-circuit || operators to chain multiple get_param calls, e.g.:

get_param(p, "sensor_model", sensor_model_str) ||
get_param(p, "return_mode", return_mode_str) || 
get_param(p, "host_ip", new_param.host_ip)

As soon as one of those parameters is found, get_param returns true, thus stopping execution early (short-circuit operator).

The correct operator to use would be the | operator, which does not short-circuit and thus all get_param statements are executed:

get_param(p, "sensor_model", sensor_model_str) |
get_param(p, "return_mode", return_mode_str) |
get_param(p, "host_ip", new_param.host_ip)

Functions exhibiting this behavior:

• HesaiHwInterfaceRosWrapper::paramCallback
• VelodyneHwInterfaceRosWrapper::paramCallback

[mojomex]

Will be fixed once PRs

• refactor(hesai)!: combine Hesai ROS wrappers into a single node #127
• refactor(velodyne)!: combine Velodyne ROS wrappers into a single node #148
• refactor(robosense)!: combine Robosense ROS wrappers into a single node #147
• feat(continental_ars548): combined continental ars548 ros wrappers into a single node  #151

are merged into main.

[mojomex]

Fixed by #127, #148, #147, #151.