Line 242 of OverviewTerms.php results in 'taxonomy/term/()' always being returned.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nick_schuch’s picture

I have tried to find an active issue but cannot. Currently writing tests.

nick_schuch’s picture

I have supplied 2 patches. One to confirm the fail and another to fix it.

nick_schuch’s picture

Status: Active » Needs review
Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Yeah, I broke that :(

looks good.

Berdir’s picture

Status: Reviewed & tested by the community » Needs work

Oh, not quite yet.

+++ b/core/modules/taxonomy/lib/Drupal/taxonomy/Tests/TermTest.phpundefined
@@ -323,6 +323,10 @@ function testTermInterface() {
+    $this->assertResponse(200, 'Term page can be accessessed via the listing link.');

a few s and e's too much :)

nick_schuch’s picture

Status: Needs work » Needs review
FileSize
1.57 KB

Oh! Woops :)

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Now we're good. Let's get this in.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Nice catch and nice to have a test...

Committed 68ee461 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.