Modificare lo stato dell'argomento via API a pinned-global non funziona

marcozambi · 18 Novembre 2019, 4:43pm

Sto lavorando a uno script Python che utilizza l’API di Discourse per impostare lo stato di un post su ‘pinned-globally’, seguendo la documentazione API.

La richiesta per ‘pinned-globally’ fallisce con il codice di ritorno 400, mentre la richiesta con il parametro status impostato su ‘pinned’ funziona perfettamente (cioè posso vedere il post fissato localmente).
Ecco un frammento del mio codice:

apiWebsite = '<my_website_URL>'
apiKey = '<secret>'
apiUsername = '<secret>'

headers = {
    'Api-Key': apiKey,
    'Api-Username': apiUsername
}


# Questa richiesta funziona correttamente
# Fissa lo stesso post localmente
payload = {
    'status': 'pinned',
    'enabled': 'true',
    'until': '2019-11-25T08:00:00'
}
u = requests.put(apiWebsite + '/t/29082/status', headers=headers, data=payload)
print(u.status_code)

Dopo aver reimpostato lo stato fissato su falso, provo a fissarlo globalmente in questo modo.

# Questa richiesta NON funziona
# Fissa lo stesso argomento globalmente
payload = {
    'status': 'pinned-globally',
    'enabled': 'true',
    'until': '2019-11-25T08:00:00'
}
u = requests.put(apiWebsite + '/t/29082/status', headers=headers, data=payload)
print(u.status_code)

Sto trascurando qualcosa di ovvio o si tratta di un vero bug?

P.S. L’utente ha privilegi sufficienti.

david · 18 Novembre 2019, 4:50pm

Non è possibile fissare lo stesso argomento a livello globale e locale contemporaneamente. Puoi scegliere l’uno o l’altro.

marcozambi · 18 Novembre 2019, 4:51pm

Ciao David,

Certo, ho scritto il codice in quel modo solo per mostrare come le due richieste differiscano.

Modificherò il primo post per renderlo più chiaro.

david · 18 Novembre 2019, 4:54pm

Ah, bene . Dando un’occhiata al codice, vedo questo che causerebbe un errore 400

github.com/discourse/discourse

app/controllers/topics_controller.rb

6e1fe22a9


      
              args[:archetype] = params[:archetype]
              args[:participants] = params[:participants] if params[:participants].present? && params[:archetype] == "private_message"
            else
              args[:category_id] = params[:category_id].to_i if params[:category_id].present?
            end
          
            topic.move_posts(current_user, post_ids_including_replies, args)
          end
          
          def check_for_status_presence(key, attr)
            invalid_param(key) unless %w(pinned pinned_globally visible closed archived).include?(attr)
          end
          
          def invalid_param(key)
            raise Discourse::InvalidParameters.new(key.to_sym)
          end
          
          def fetch_username
            params.require(:user)
            params[:user]
          end

Penso che tu debba usare pinned_globally invece di pinned-globally. Puoi provarlo e vedere se funziona? Sembra che ci sia un errore di battitura nella nostra documentazione dell’API - provvederò a correggerlo.

Modifica: corretto in Correct pinned_globally in topic status documentation · discourse/discourse_api_docs@fcf0d5f · GitHub

marcozambi · 18 Novembre 2019, 5:04pm

Confermato!
Grazie mille!

Argomento		Risposte	Visualizzazioni
"Pin globally" does nothing Bug	6	1318	Maggio 14, 2015
Broken „Pin Globally” feature? Bug	2	788	Agosto 14, 2015
Unpin a topic with API? Development rest-api , pinned-topics	2	246	Febbraio 20, 2024
Staff that is not TL4 is unable to pin topics globally anymore Bug	12	996	Marzo 18, 2021
Pin more than one topic globally Feature pinned-topics	6	205	Maggio 30, 2025

Modificare lo stato dell'argomento via API a pinned-global non funziona

Argomenti correlati