Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Updated: Comment #N
Problem/Motivation
Attempting to see if /forum can be built with views.
Added new view of taxonomy terms in vocabulary 'forum'.
Added new relationship to 'parent term'.
As soon as I add it, get 'Broken missing handler'.
Proposed resolution
Tests
Fix
Reviews
Remaining tasks
Tests
Fix
Reviews
User interface changes
None
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#14 | interdiff.txt | 841 bytes | jibran |
#14 | 2207229-14.patch | 6.12 KB | jibran |
#4 | taxonomy-parent-relationship-2207229.pass_.patch | 6.16 KB | larowlan |
#4 | taxonomy-parent-relationship.2207229.fail_.patch | 5.81 KB | larowlan |
Comments
Comment #1
larowlanHere's the fix but nfi how to write a test to verify this.
Comment #2
larowlanComment #3
larowlanPointer/guide for tests: Drupal\user\Tests\Views\AccessRoleUITest and the accompanying views.view.test_access_role.yml
Comment #4
larowlanShould be red/green
Comment #7
larowlan4: taxonomy-parent-relationship.2207229.fail_.patch queued for re-testing.
Comment #8
larowlan4: taxonomy-parent-relationship-2207229.pass_.patch queued for re-testing.
Comment #10
tim.plunkettI love the fix-to-test lines of code ratio here.
We're fixing the exporting in another issue, but this should be position: 0
Are we putting these in? I can never remember.
Comment #11
jibranFor 2 in #10 no not anymore.
Comment #12
dawehnerLet's just fix those.
Comment #13
jibrangoing to fix 10.
Comment #14
jibranFixes for #10.
Comment #15
tim.plunkettLooks good to me, thanks!
Comment #16
alexpottCommitted 4b6ba9f and pushed to 8.x. Thanks!