nextcloud/server

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

Aberta

#24.300 aberto em 23 de nov. de 2020

Ver no GitHubGuia do colaborador
 (3 comentários) (1 reação) (0 responsável)PHP (4.865 forks)batch import
1. to developenhancementfeature: occgood first issue

Métricas do repositório

Stars
 (34.953 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Localize o manipulador de comando para `config:system:set` no código e adicione uma verificação comparando valores antigos e novos. Se inalterados, exiba 'Already set' em vez de repetir a mensagem de sucesso.
Pilha de tecnologia
php
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHP
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70