Title: fix(filterFilter) deeply-nested predicate objects + multiple conditions were not giving expected results by albertboada · Pull Request #7329 · angular/angular.js · GitHub
Open Graph Title: fix(filterFilter) deeply-nested predicate objects + multiple conditions were not giving expected results by albertboada · Pull Request #7329 · angular/angular.js
X Title: fix(filterFilter) deeply-nested predicate objects + multiple conditions were not giving expected results by albertboada · Pull Request #7329 · angular/angular.js
Description: Request Type: bug How to reproduce: http://plnkr.co/edit/Yd705mYbqKJ3n79TXLmx?p=preview Component(s): misc core Impact: medium Complexity: small Detailed Description: Fixes current filterFilter's implementation (introduced in 1.2.13 as a result of #6215) for deeply-nested predicate objects + multiple conditions scenarios. Problem: when using deeply nested predicate objects, filterFilter is currently behaving as if it was using OR operators (i.e. it doesn't matter if some conditions don't match as long as one does), and should be behaving as if it used AND operators in order to be consistent with the original implementation for non-deeply-nested predicate objects.
Open Graph Description: Request Type: bug How to reproduce: http://plnkr.co/edit/Yd705mYbqKJ3n79TXLmx?p=preview Component(s): misc core Impact: medium Complexity: small Detailed Description: Fixes current filterFilter'...
X Description: Request Type: bug How to reproduce: http://plnkr.co/edit/Yd705mYbqKJ3n79TXLmx?p=preview Component(s): misc core Impact: medium Complexity: small Detailed Description: Fixes current filterFilter&...
Opengraph URL: https://github.com/angular/angular.js/pull/7329
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:0ef35326-25e4-8ab2-f1ee-2aac757d384e |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9558:DC5C6:205B51:2E51B1:6A58AAA1 |
| html-safe-nonce | 0539453c7b6374b0f2215663f2c8aff8be662c8dba1ddee03e0d0ade786981cf |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5NTU4OkRDNUM2OjIwNUI1MToyRTUxQjE6NkE1OEFBQTEiLCJ2aXNpdG9yX2lkIjoiODQ0ODc2NTY0NTg5OTIxMzQ3MyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 196e224f22a505b287da2b677a44ad3498503df235485065b550548e4d0937cd |
| hovercard-subject-tag | pull_request:15418516 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/angular/angular.js/pull/7329/files |
| twitter:image | https://avatars.githubusercontent.com/u/1840284?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1840284?s=400&v=4 |
| og:image:alt | Request Type: bug How to reproduce: http://plnkr.co/edit/Yd705mYbqKJ3n79TXLmx?p=preview Component(s): misc core Impact: medium Complexity: small Detailed Description: Fixes current filterFilter'... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f2a0c7865178af3d91dd9f13b0cdfc3c73a2529c873d2780bb4c01271a57ec6 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/angular/angular.js git https://github.com/angular/angular.js.git |
| octolytics-dimension-user_id | 139426 |
| octolytics-dimension-user_login | angular |
| octolytics-dimension-repository_id | 460078 |
| octolytics-dimension-repository_nwo | angular/angular.js |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 460078 |
| octolytics-dimension-repository_network_root_nwo | angular/angular.js |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 8aae7b8d6caacacf5c66eaeb2702d8dc88d85b4a |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width