Midburn/spark

Clean global scope in all controllers.

Aberta

#980 aberto em 29 de nov. de 2018

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (1 responsável)JavaScript (44 forks)auto 404
Cleanupgood first issue

Métricas do repositório

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

Description

Overview of the problem

We have several functions declared globally in controller files. This can create issues hard to anticipate and solve.

Description

  1. We need to move all functions into scope. image image
  2. We need to make sure the functions aren't used in any other file (if so, we should create an AngularJS service to be shared between the controllers)
  3. Sanity after the cleanup - check most screens and buttons don't raise exceptions related to missing functions.

Expected behavior

Functions should be scoped

Actual behavior

Functions are declared outside of AngularJS's controllers call back

Guia do colaborador

Direção de pesquisa
Procure por funções declaradas fora dos callbacks dos controladores em todos os arquivos de controladores, verifique seu uso em toda a base de código (por exemplo, usando grep ou pesquisa IDE) e, em seguida, mova as para o escopo do controlador ou extraia para serviços AngularJS se forem compartilhadas.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Refatoração
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.
Meio dia
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
JavaScriptAngularJS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70