angular-ui/ui-grid

Since that the treeView module uses treeBase module, saveState module is broken

Open

#4.742 aberto em 20 de nov. de 2015

Ver no GitHub
 (10 comments) (0 reactions) (1 assignee)JavaScript (2.496 forks)batch import
good first issuegrid-groupinggrid-tree-viewtype: bug

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Hi,

I wanted to implement treeView on my grid, and i wanted to restore tree state after refreshing data.

In the saveState module, in saveTreeView and restoreTreeView functions, there is this condition : if ( !grid.api.treeView || !grid.options.saveTreeView ){ return {}; }

It seems obsolete since the treeView module uses treeBase module , isn't it ? If i remove the condition, of course i get this error log : TypeError: Cannot read property 'getTreeView' of undefined

I missed something or i'm true ?

Guia do colaborador

Direção de pesquisa
Investigue por que salvar o estado da visualização em árvore falha. A condição que verifica grid.api.treeView está obsoleta porque o módulo treeView usa treeBase, mas getTreeView é indefinido. Determine se treeBase fornece uma API diferente ou se a condição deve ser atualizada para verificar treeBase.
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
JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65