angular-ui/ui-grid

Memory leak on ui-grid-menu showMenu

Fechada

#3.972 aberto em 13 de jul. de 2015

Ver no GitHub
 (1 comentário) (0 reação) (1 responsável)JavaScript (2.496 forks)batch import
help wantedseverity: memory leaktype: bug

Métricas do repositório

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

Description

Hi,

On ui-grid-menu.js, there are 2 methods showMenu and hideMenu that are added to scope but also are added to some self variable that happens to be the ``window` object.

The problem is that in a spa app, navigating away from the list, on the window object that is not going to be destroyed, the 2 functions will remain with the scope as context variable, so all the grid objects, columns, rows will be carried through the rest of the journey.

The 2 functions seem to be used only through scope, so no need for them on the ``window` ... at least to the point I got now.

Guia do colaborador

Direção de pesquisa
Inspecione as funções showMenu e hideMenu no ui grid menu.js. Confirme que elas estão desnecessariamente anexadas ao objeto window e remova a atribuição ao window, mantendo a anexação ao scope.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
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
JavaScriptAngularJS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60