angular-ui/ui-grid

Invalid arguments provided in click handler of selectionSelectAllButtons & treeBaseExpandAllButtons

Open

#4551 aperta il 20 ott 2015

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)JavaScript (2496 fork)batch import
good first issuegrid-selection

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hi,

there is a bug in the ui-grid/selectionSelectAllButtons template or its directive: the click handler in directive uiGridSelectionSelectAllButtons is expecting 2 arguments:

$scope.headerButtonClick = function(row, evt) { }

but only the $event argument is provided, resulting in a wrongly assigned argument (i.e. event is passed as the row argument). You should either fix the template or modify the directive. The same goes for the uiGridTreeBaseExpandAllButtons directive btw.

NB I suggest you modify the templates to

ng-click="headerButtonClick(null, $event)"

because then you keep your code in line with the other templates (e.g. selectionRowHeaderButtons.html)

Guida contributor

Direzione di ricerca
Esamina i file di template per selectionSelectAllButtons e treeBaseExpandAllButtons e aggiorna il gestore di clic per passare gli argomenti corretti.
Tech stack
javascriptangular
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitAngular
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90