nextcloud/server

occ config:system:set should return a different output when unchanged

Open

#24.300 aperta il 23 nov 2020

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)PHP (4865 fork)batch import
1. to developenhancementfeature: occgood first issue

Metriche repository

Star
 (34.953 star)
Metriche merge PR
 (Merge medio 20g) (627 PR mergiate in 30 g)

Descrizione

Steps

  1. Run occ config:system:set mail_smtpmode --value mail_smtpmode
  2. Run it again

Expected result

An output with "Already set"

Actual result

Exact same output.

Version

Nextcloud 20.0.2

Context

When using tools like Ansible, one might want to know whether a value was changed or not to be able to set the changed_when clause there based stdout result.

Also needs to be applied to the other occ config:*:set commands.

Guida contributor

Direzione di ricerca
Trova l'handler del comando `config:system:set` nel codice e aggiungi un controllo per confrontare i vecchi e nuovi valori. Se invariati, restituisci 'Already set' invece di ripetere il messaggio di successo.
Tech stack
php
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70