Syntaxe pour les paramètres de Data Explorer dans Automation

Arkshine · Mars 5, 2025, 10:46

Je peux reproduire le problème également.

Dans votre exemple, disons que vous voulez vérifier les groupes de modérateurs et d’administrateurs, vous êtes censé avoir :

clé : group_name
valeur : moderators,admins

Trucs techniques pour aider, ne faites pas attention à moi.
J’ai essayé de comprendre en regardant le code, et je pense qu’il y a un problème ici :

github.com/discourse/discourse-data-explorer

lib/discourse_data_explorer/report_generator.rb

main


      
          
            result = DataExplorer.run_query(query, params)
            query.update!(last_run_at: Time.now)
          
            return [] if opts[:skip_empty] && result[:pg_result].values.empty?
            table =
              ResultToMarkdown.convert(result[:pg_result], render_url_columns: opts[:render_url_columns])
          
            build_report_pms(query, table, recipients, attach_csv: opts[:attach_csv], result:)
          end
          
          def self.generate_post(query_id, query_params, opts = {})
            query = DiscourseDataExplorer::Query.find(query_id)
            return {} if !query
          
            params = params_to_hash(query_params)
          
            result = DataExplorer.run_query(query, params)
            query.update!(last_run_at: Time.now)
          
            return {} if opts[:skip_empty] && result[:pg_result].values.empty?

github.com/discourse/discourse-data-explorer

lib/discourse_data_explorer/report_generator.rb

main


      
            build_report_post(query, table, attach_csv: opts[:attach_csv], result:)
          end
          
          def self.params_to_hash(query_params)
            params = JSON.parse(query_params)
          
            params.map { |p| p.is_a?(Hash) ? [p["key"], p["value"]] : p }.to_h
          end
          
          def self.build_report_pms(query, table = "", targets = [], attach_csv: false, result: nil)
            pms = []
            upload = create_csv_upload(query, result) if attach_csv
          
            targets.each do |target|
              name = target[0]
              pm_type = "target_#{target[1]}s"
          
              pm = {}
              pm["title"] = I18n.t(

Les paramètres sont transformés avant que run_query ne soit appelé.
Disons que vous avez cette valeur d’origine :
[{\"key\":\"group_names\",\"value\":\"admins,moderators\"}]
La valeur transformée sera :
{{\"key\"=>\"group_names\", \"value\"=>\"admins,moderators\"}=>nil}

github.com/discourse/discourse-data-explorer

lib/discourse_data_explorer/data_explorer.rb

main


      
          def self.run_query(query, req_params = {}, opts = {})
            # Safety checks
            # see test 'doesn't allow you to modify the database #2'
            if query.sql =~ /;/
              err = ValidationError.new(I18n.t("js.errors.explorer.no_semicolons"))
              return { error: err, duration_nanos: 0 }
            end
          
            query_args = {}
            begin
              query_args = query.cast_params req_params
            rescue ValidationError => e
              return { error: e, duration_nanos: 0 }
            end
          
            time_start, time_end, explain, err, result = nil
            begin
              ActiveRecord::Base.connection.transaction do
                # Setting transaction to read only prevents shoot-in-foot actions like SELECT FOR UPDATE
                # see test 'doesn't allow you to modify the database #1'
                DB.exec "SET TRANSACTION READ ONLY"

github.com/discourse/discourse-data-explorer

app/models/discourse_data_explorer/query.rb

main


      
          def cast_params(input_params)
            result = {}.with_indifferent_access
            self.params.each do |pobj|
              result[pobj.identifier] = pobj.cast_to_ruby input_params[pobj.identifier]
            end
            result
          end

Cependant, cast_params semble s’attendre à {\"group_names\"=>\"admins,moderators\"}

J’ai essayé de tester ce changement naïf, et les paramètres ont fonctionné.

def self.params_to_hash(query_params)
  params = JSON.parse(query_params)
  params_hash = {}

  params.each do |param|
    key = param["key"]
    value = param["value"]
    params_hash[key] = value
  end

  params_hash
end

Sujet		Réponses	Vues
`DataExplorer::ValidationError: Missing parameter` when running Data Explorer queries with [params] via API Data & reporting rest-api , sql-query	8	1349	Avril 10, 2022
How to add query parameters to Data Explorer? Development	7	1191	Novembre 25, 2020
Populate Data Explorer params with URL params Feature data-explorer , completed	12	1780	Septembre 3, 2025
Param dropdown for group_id in data explorer query Feature data-explorer , completed	17	1435	Juin 5, 2025
Using Parameters in Data Explorer Queries Data & reporting data-explorer , sql-tutorial	6	2010	Février 10, 2026

Syntaxe pour les paramètres de Data Explorer dans Automation

Sujets connexes