Project

General

Profile

Bug #23618

JS function show_stats() in show_campaign_stages template generating URLs missing role

Added by Kevin Retzke 21 days ago. Updated 17 days ago.

Status:
Closed
Priority:
Immediate
Assignee:
Target version:
-
Start date:
11/18/2019
Due date:
% Done:

100%

Estimated time:
First Occurred:
Scope:
Internal
Experiment:
-
Stakeholders:
Duration:

Description

This will take someone from a URL like
https://pomsgpvm01.fnal.gov/poms/show_campaign_stages/dune/production?campaign_name=audit-protodune-run-website&view_active=view_active&view_mine=96&view_others=96&view_production=view_production&update_view=update_view
to one like
https://pomsgpvm01.fnal.gov/poms/show_campaign_stages/dune/show_campaign_stages?cl=4299
thus wiping out their role.

    function show_stats() {
        // If any boxes are checked only use those, otherwise take all visible.
        var max_rows = $("#c_table_max_rows").val();
        var csv = [];
        var csv_checked = [];
        for (var i = 1; i <= max_rows; i++) {
            if ( $("#c_tr_"+i).is(":visible") ) {
                csv.push($("#c_id_"+i).html());
            }
            if ( $("#c_cbox_"+i).is(":checked") ) {
                csv_checked.push($("#c_id_"+i).html());
            }
        }
        if (csv_checked.length == 0) {
            location.href = "./show_campaign_stages?cl=" + csv.join(',');
        } else {
            location.href = "./show_campaign_stages?cl=" + csv_checked.join(',');
        }
    }

Associated revisions

Revision 65f1515f (diff)
Added by Marc Mengel 20 days ago

issue #23618 fix

History

#1 Updated by Marc Mengel 20 days ago

Patched the url generation, also fixed the parameter name from cl to campaign_ids

c125d24effdf37b72f5bd90b3e73b3f6ac92f8d6

65f1515ffed1bb6b21a2ff0fb53955056218a9ad

#2 Updated by Marc Mengel 17 days ago

  • % Done changed from 0 to 100
  • Status changed from New to Closed


Also available in: Atom PDF