Pattern at line #120 catches and redirects any request containing taxonomy/term/* instead of handling only requests fully matching taxonomy/term/*.
This may break CKEditor Link and other modules that handle paths like ckeditor_link/revert/taxonomy/term/10

CommentFileSizeAuthor
#2 fix_taxonomy_term_pattern-1908466-2.patch840 bytesanrikun
#1 fix_taxonomy_term_pattern-1908466-1.patch832 bytesanrikun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anrikun’s picture

anrikun CreditAttribution: anrikun commented
Priority: Normal » Major
Status: Active » Needs review
FileSize
fix_taxonomy_term_pattern-1908466-1.patch832 bytes

Here is a fix.

anrikun’s picture

anrikun CreditAttribution: anrikun commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
FileSize
fix_taxonomy_term_pattern-1908466-2.patch840 bytes

This same bug is present in D7 as well.

anrikun’s picture

anrikun CreditAttribution: anrikun commented
Issue summary: View changes

Added a reference to CKEditor Link.

fangel’s picture

fangel CreditAttribution: fangel commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

The D7 patch works as intended.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Bump. +1.

BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented

Makes a lot of sense. Thanks!

  • BarisW committed 9c099e1 on 7.x-1.x authored by anrikun 
    Issue #1908466 by anrikun, fangel, DamienMcKenna, BarisW: Wrong taxonomy...
BarisW’s picture

BarisW CreditAttribution: BarisW at LimoenGroen commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x-dev

Status: Fixed » Closed (fixed)

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