Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Grid view tag filtering bug #4801

Closed
rajgoesout opened this issue May 30, 2019 · 1 comment · Fixed by #5039
Closed

Grid view tag filtering bug #4801

rajgoesout opened this issue May 30, 2019 · 1 comment · Fixed by #5039

Comments

@rajgoesout
Copy link
Collaborator

rajgoesout commented May 30, 2019

Builders with selected tags are highlighted, but all the builders are displayed on the grid.

Error: "branch" is read-only
refresh@http://localhost:8010/grid_view/scripts.js:676:5
toggleTag@http://localhost:8010/grid_view/scripts.js:667:17
fn@http://localhost:8010/scripts.js?_1559194180589 line 40425 > Function:4:346
expensiveCheckFn@http://localhost:8010/scripts.js?_1559194180589:41438:18
callback@http://localhost:8010/scripts.js?_1559194180589:50965:17
$eval@http://localhost:8010/scripts.js?_1559194180589:43182:28
$apply@http://localhost:8010/scripts.js?_1559194180589:43282:25
ngEventHandler/<@http://localhost:8010/scripts.js?_1559194180589:50970:23
dispatch@http://localhost:8010/scripts.js?_1559194180589:149296:27
add/elemData.handle@http://localhost:8010/scripts.js?_1559194180589:149103:28
@rajgoesout rajgoesout changed the title Grid view tag filtering broken Grid view tag filtering bug May 30, 2019
@rajgoesout rajgoesout added the bug label May 30, 2019
@rjarry
Copy link
Contributor

rjarry commented Oct 17, 2019

I have the same problem, I have a more exploitable backtrace:

angular.js:15567 TypeError: Cannot assign to read only property 'branch' of object '#<Object>'
    at Grid.refresh (grid.controller.js:258)
    at Grid.changeResult (grid.controller.js:239)
    at fn (eval at compile (angular.js:16418), <anonymous>:4:255)
    at l.$eval (angular.js:19393)
    at angular.js:27827
    at Object.<anonymous> (angular.js:30812)
    at k (angular.js:387)
    at Object.$$writeModelToScope (angular.js:30810)
    at o (angular.js:30803)
    at angular.js:30797

The problem occurs here: https://github.com/buildbot/buildbot/blob/v2.4.1/www/grid_view/src/module/grid.controller.js#L258

rjarry added a commit to rjarry/buildbot that referenced this issue Oct 17, 2019
Fix the following error when filtering by branch/results and/or builder
tags:

angular.js:15567 TypeError: Cannot assign to read only property 'branch' of object '#<Object>'
    at Grid.refresh (grid.controller.js:258)
    at Grid.changeResult (grid.controller.js:239)
    at fn (eval at compile (angular.js:16418), <anonymous>:4:255)
    at l.$eval (angular.js:19393)
    at angular.js:27827
    at Object.<anonymous> (angular.js:30812)
    at k (angular.js:387)
    at Object.$$writeModelToScope (angular.js:30810)
    at o (angular.js:30803)
    at angular.js:30797

There is no need to modify the $stateParams object anyway. Affect the
values in the local params object directly.

Fixes: buildbot#4801
Fixes: f098bdd ("www: add grid_view plugin")
Signed-off-by: Robin Jarry <robin@jarry.cc>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants