Merge pull request #26 from gabrielmscampos/master

For completion and future reference, this endpoint (joint_lumisection_ranges) is an aggregate of oms_lumisection_ranges and rr_lumisection_ranges.

Those are used mostly in the frontend, in the modal that appears when clicking LS (next to Manage) for a specific run.

The OMS LUMISECTIONS button uses oms_lumisection_ranges, the RR LUMISECTIONS uses the rr_lumisection_ranges and the BOTH button uses the joint_lumisection_ranges. All calls to each endpoint from the frontend are passed the dataset argument, but the joint_lumisection_ranges was ignoring it, always using online instead. This was probably not the intended functionality.

runregistry_backend/controllers/lumisection.js

@@ -779,7 +779,7 @@ exports.get_rr_and_oms_lumisection_ranges = async (req, res) => {
   );
   const oms_lumisections = await exports.get_oms_lumisections_for_dataset(
     run_number,
-    'online'
+    dataset_name || 'online'
   );
   let joint_lumisections;
   if (rr_lumisections.length >= oms_lumisections.length) {