Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-81891

Standardise/improve the naming of autogenerated report element names

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • 4.5
    • Report builder
    • MOODLE_405_STABLE

      Noted by davidcarrillo in discussion of MDL-81434 - however that's a bug fix, so proposing the following as a separate improvement (main-only)

      1. Column field alias ("c");
      2. Column param names ("p");
      3. Condition param names ("c");
      4. Filter param names ("f")

      The purpose of each of these is to ensure uniqueness within a report, for parameter names when used in conjunction with the following helpers:

      1. generate_alias[es](...)
      2. generate_param_name[s](...)

      Uniqueness still being the primary purpose of all these methods/values, it's also useful when using the report SQL debug tool to have helpfully named values too - which we can evaluate here (David has some suggestions) and improve

      My suggestion:

      Params: p => col; c => con; f => fil
      Columns: c => col
      Alias: rbalias => rba
      Param: rbparam => rbp
      

            Unassigned Unassigned
            pholden Paul Holden
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.