nextcloud/server

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

Open

#24 300 ouverte le 23 nov. 2020

Voir sur GitHub
 (3 commentaires) (1 réaction) (0 assignés)PHP (4 865 forks)batch import
1. to developenhancementfeature: occgood first issue

Métriques du dépôt

Stars
 (34 953 stars)
Métriques de merge PR
 (Merge moyen 20j) (627 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Localisez le gestionnaire de commande pour `config:system:set` dans le code et ajoutez une vérification comparant les anciennes et nouvelles valeurs. Si inchangées, affichez 'Already set' au lieu de répéter le message de succès.
Stack technique
php
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70